Repository navigation
stm32/mpu: Enable STM32WB mpu use to support qspi flash. - #12512
Merged
Merged
Conversation
Member
|
This is very similar to #11932. |
Contributor
Author
|
Ah yes, interesting discussion over there. I should have submitted these two commits here as two separate pr's sorry, they really are independent. I'll take a closer look at that other one to do the spiflash change cleaner. |
andrewleech
force-pushed
the
stm32wb55_qspi_flash
branch
from
October 12, 2023 02:52
3adf26f to
4db71e1
Compare
Contributor
Author
|
@dpgeorge I've split this MR out, keeping this one as just the initial stm32WB55 support for mpu needed for qspi use. I'll re-evaluate the other commit in conjunction with #11932 and raise a new PR as needed. |
andrewleech
force-pushed
the
stm32wb55_qspi_flash
branch
from
October 18, 2023 00:39
4db71e1 to
4c0f705
Compare
Signed-off-by: Andrew Leech <[email protected]>
dpgeorge
force-pushed
the
stm32wb55_qspi_flash
branch
from
November 17, 2023 02:18
4c0f705 to
37c1c5f
Compare
Member
|
Thanks for simplifying this. Now merged. |
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
3 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.
This MR has been tested on custom hardware that includes a stm32WB55 and a qspi flash chip. Tested on both:
The Renesas QSPI Flash chip also required a change to the spiflash driver which is included in this MR:
The Macronix, and winbond chips I've used on previous projects, all seem to support writing to both status and config registers with the one command, sending two bytes of data.
The Renesas chip on the other hand only supports writing one byte of data on the WRSR command, instead opting for a separate command to write the the CR register.