-
Notifications
You must be signed in to change notification settings - Fork 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
STM32 SPI: Update and improvement #15028
Conversation
df41a15
to
ef87978
Compare
@jeromecoutant, thank you for your changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the change includes a move of pullnone to pulldown that is not related to SPI V2 I think, better to split it
ef87978
to
083d24a
Compare
This is waiting for #14981 as it contains the same commits here now, I'll set as preceding PR required |
This PR cannot be merged due to conflicts. Please rebase to resolve them. |
083d24a
to
525ed63
Compare
Rebase done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1 simple question check, if probably ok so approved :-)
Let's start CI ? |
525ed63
to
066c07b
Compare
Pull request has been modified.
CI restarted |
Jenkins CI Test : ✔️ SUCCESSBuild Number: 1 | 🔒 Jenkins CI Job | 🌐 Logs & ArtifactsCLICK for Detailed Summary
|
Summary of changes
Few updates for STM32 SPI:
STM32H7 is the first family using the new ST IP driver. Coming families will use the same IP.
So many
#if H7
are replaced by#if SPI_V2
init procedure is updated following discussion in STM: Clear stale data from STM SPI peripheral before starting data transfer #14534
@LMESTM
@rvasquez6089 @vznncv
Impact of changes
Migration actions required
Documentation
Pull request type
Test results
FPGA tests OK
Reviewers