stm32/mpu: Enable STM32WB mpu use to support qspi flash. #12512
Merged
Conversation
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
|
This is very similar to #11932. |
|
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. |
3adf26f to
4db71e1
Compare
|
@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. |
4db71e1 to
4c0f705
Compare
Signed-off-by: Andrew Leech <[email protected]>
4c0f705 to
37c1c5f
Compare
|
Thanks for simplifying this. Now merged. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.