Fix Siwx91x SPI - #26
Merged
Aksel Mellbye (asmellby) merged 3 commits intoJul 7, 2026
Merged
Aksel Mellbye (asmellby) merged 3 commits into
Aksel Mellbye (asmellby) merged 3 commits into
Conversation
Jérôme Pouiller (jerome-pouiller)
force-pushed
the
fix-spi-2-downstream
branch
2 times, most recently
from
July 7, 2026 09:11
e7f8ca4 to
14e03a0
Compare
When the SPI hardware block is use to send data (without receiving any data), the DMA callback is called when the data are transmitted to the SPI controller, but before the data are actually sent over the wire. Then, the chip-select signal could be released while the Tx was still in progress. Upstream-status: pr Signed-off-by: Jérôme Pouiller <jerome.pouiller@silabs.com> (cherry picked from commit 79d5276)
The siwx91x SPI driver required to wait for a very small amouth of time (100ns to 50µs). When CONFIG_PM is enabled, the granularity of k_busy_wait() is not sufficient (~32µs). So, the driver implement a raw busy loop. This busy loop depends on the frequency of the CPU. This was wrongly retrieved from CONFIG_SYS_CLOCK_HW_CYCLES_PER_SEC. DT_PROP(DT_NODELABEL(cpu0), clock_frequency) is more accurate. Upstream-status: pr Signed-off-by: Jérôme Pouiller <jerome.pouiller@silabs.com> (cherry picked from commit 8db4d92)
With CONFIG_PM=y, the duration of the SPI transfer is slightly longer. Upstream-status: pr Signed-off-by: Jérôme Pouiller <jerome.pouiller@silabs.com> (cherry picked from commit 7b1c961)
Jérôme Pouiller (jerome-pouiller)
force-pushed
the
fix-spi-2-downstream
branch
from
July 7, 2026 09:13
14e03a0 to
8efef79
Compare
Aksel Mellbye (asmellby)
approved these changes
Jul 7, 2026
Jérôme Pouiller (jerome-pouiller)
changed the title
Aksel Mellbye (asmellby)
merged commit Jul 7, 2026
f60fcfc
into
SiliconLabsSoftware:silabs/dev
3 checks passed
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Cherry-pick the upstream patches.