-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
drivers/spi: stm32: Revert while loop on tx only #35661
drivers/spi: stm32: Revert while loop on tx only #35661
Conversation
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.
Commit message should contains your signed-off
Otherwise LGTM
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.
Please fix commit title. For instance:
"drivers/spi: stm32: Revert while loop on tx only"
Also please don't abuse of github references:
"This commit will solve issue created after #34731.
This commit removes all th regression that caused due to commit #34731."
It's good to add links but it should be done in addition to explicit comments.
okay. |
e3a7aa2
to
74ccbc7
Compare
This commit reverted while loop on tx only. This commit solves SPI loopback failure and SPI wrong behaviour on RX. fix: * zephyrproject-rtos#35297 * zephyrproject-rtos#35539 Revert "drivers/spi: STM32: This solves SPI infinite loop on Tranceive" This reverts commit 50c2acb. Signed-off-by: Affrin Pinhero <affrin.pinhero@hcl.com>
74ccbc7
to
0ccf4a3
Compare
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.
LGTM
This commit reverted while loop on tx only.
This commit solves SPI loopback failure and SPI wrong behaviour on RX.
fix:
Revert "drivers/spi: STM32: This solves SPI infinite loop on Tranceive"
This reverts commit 50c2acb.
Signed-off-by: Affrin Pinhero affrin.pinhero@hcl.com