-
Notifications
You must be signed in to change notification settings - Fork 1.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
stm32g4 chips update #901
stm32g4 chips update #901
Conversation
Thank you for keeping this up to date. |
Looking at this again and the respective datasheet (page 19) this would likely make use of the MCU's S-bus. If this should require option bytes to be set, we don't need to address this with this PR necessarily. I'm not familiar with this device though. |
@grevaillot How do you think about this? |
ping @grevaillot |
i think that PR is okay as it. i'll push some enhanced support for Gx devices with option read/write and dynamic dual bank flash handling soom, in a new pr. |
Thanks! |
I followed current g43x scheme of considering sram1/sram2/ccm sram as contiguous, but documented it.
Note that I've only worked with the first of the two banks. stm32g474 can also be configured as a single 512kb bank, with 4k pages via option bytes. but i think there's no logic to handle that properly ?