Skip to content

[HAL][KSDK] Fix issue with KSDK SPI timing #1088

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

Merged
merged 2 commits into from
May 4, 2015
Merged

[HAL][KSDK] Fix issue with KSDK SPI timing #1088

merged 2 commits into from
May 4, 2015

Conversation

Sissors
Copy link
Contributor

@Sissors Sissors commented May 3, 2015

Depending on the format by default the Freescale SPI peripheral does not
give proper delay between SPI transfers. So code has been added which
sets half a period delay between SPI transfers.

See the following picture:
http://static.tweakers.net/ext/f/gAoTfuAGPXuMgtqbCm34UOJ9/full.png

This is related to #840 (you can see the same clock issue in the pic there). This issue: #968 does not seem to be solved by this, in mode 3 the MOSI line goes up stil way too soon.

Sissors added 2 commits May 3, 2015 17:37
Depending on the format by default the Freescale SPI peripheral does not
give proper delay between SPI transfers. So code has been added which
sets half a period delay between SPI transfers.
Even though it does not actually toggle the CS signal, we can also use
the delay to CS toggle to get proper waveforms. This also fixes SPI in
mode 3.
@Sissors
Copy link
Contributor Author

Sissors commented May 3, 2015

Switched delay type, now it should also solve #968

@0xc0170
Copy link
Contributor

0xc0170 commented May 4, 2015

Thanks Minion 😄

0xc0170 added a commit that referenced this pull request May 4, 2015
KSDK - Fix issue with KSDK SPI timing
@0xc0170 0xc0170 merged commit f54dc44 into ARMmbed:master May 4, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants