-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
applications: sdp: mspi: Improve setting DQ2, DQ3 initial state #20712
applications: sdp: mspi: Improve setting DQ2, DQ3 initial state #20712
Conversation
CI InformationTo view the history of this post, clich the 'edited' button above Inputs:Sources:sdk-nrf: PR head: 263007e8cfd01fa0a657d4af48e814c244fa2992 more detailssdk-nrf:
Github labels
List of changed files detected by CI (2)
Outputs:ToolchainVersion: 4ffa2202d5 Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
You can find the documentation preview for this PR here. |
bfcd065
to
7a30d4a
Compare
1d681c1
to
2f958a2
Compare
2f958a2
to
6d5223b
Compare
|
6d5223b
to
1feff5c
Compare
1feff5c
to
c48d2ec
Compare
Improve setting initial state of DQ2 and DQ3 and clearing when changing mode to QUAD. Signed-off-by: Magdalena Pastula <magdalena.pastula@nordicsemi.no>
Add checking if one data line is not assigned to more than one GPIO pin. Signed-off-by: Magdalena Pastula <magdalena.pastula@nordicsemi.no>
c48d2ec
to
263007e
Compare
Improve setting pins DQ2 and DQ3 to high state. Now it is done only when they are defined.
I left setting them to high by default. It is done the same way for QSPI driver in
pinctrl_nrf.c
, at least for DQ3 pin (RESET).Also, DTS does not support setting default/initial state of a pin.
Based on #20806 , please, review only two last commits.