NXP LPSPI: Add support for Peripheral (slave) mode #87144
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
zephyrbot
added
platform: NXP
zephyrbot
requested review from
butok,
dbaluta,
dleach02,
EmilioCBen,
manuargue,
mmahadevan108,
Raymond0225,
tbursztyka and
teburd
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
decsny
force-pushed
the
feature/lpspi_slave
branch
from
67f58c1 to
8ee79a0
Compare
decsny
force-pushed
the
feature/lpspi_slave
branch
from
8ee79a0 to
513dc46
Compare
EmilioCBen
reviewed
Mar 20, 2025
EmilioCBen
previously approved these changes
Mar 20, 2025
|
put DNM because there are some timing issues with CPHA=0, need to debug |
| peripheral-cs = <1>; | ||
| }; | ||
| }; | ||
|
|
There was a problem hiding this comment.
It would be useful for testing if you could include the connection setup in a comment here.
There was a problem hiding this comment.
its in the board level dts , also its just the only two standard spi headers on the board which are labelled on silk screen, mikrobus and arduino
decsny
force-pushed
the
feature/lpspi_slave
branch
from
513dc46 to
205c571
Compare
|
update: rebased on top of #90182 . Still WIP, so moved to draft |
decsny
force-pushed
the
feature/lpspi_slave
branch
from
205c571 to
d10716a
Compare
decsny
force-pushed
the
feature/lpspi_slave
branch
from
d10716a to
decb325
Compare
|
command : connecting the lpspis used in the overlay, result: |
bjarki-andreasen
approved these changes
Jun 25, 2025
Add rudimentary slave mode support to the interrupt based LPSPI driver. Signed-off-by: Declan Snyder <declan.snyder@nxp.com>
Add overlay for a reworked board that breaks out LPSPI1 to the arduino header. Signed-off-by: Declan Snyder <declan.snyder@nxp.com>
Add an overlay to the spi_controller_peripheral test for the frdm_mcxa156. Since this IP has multiple CS, need to introduce a way to configure the correct CS to the test; just going to use zephyr,user for now instead of refactoring to be less nordic-specific, keeping the usage optional. Signed-off-by: Declan Snyder <declan.snyder@nxp.com>
decsny
force-pushed
the
feature/lpspi_slave
branch
from
decb325 to
057a46e
Compare
mmahadevan108
approved these changes
Jun 26, 2025
dleach02
approved these changes
Jun 26, 2025
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds support for Peripheral mode into the NXP LPSPI interrupt-based driver.
So far, it is only tested on MCXA156, but this is a great start because due to the small fifo, it is probably the most difficult to enable on.
Fixes #57747