Skip to content
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

Merged

Conversation

grevaillot
Copy link
Collaborator

  • Add id and infos for stm32g47/48 chips.
  • Update comments regarding g43/g44

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 ?

@Nightwalker-87
Copy link
Member

Thank you for keeping this up to date.
We shall have some additional testing here...
As I can't help out here due to missing hardware, we should ask the community for assistance.
Maybe someone also has an idea regarding your note.

@Nightwalker-87
Copy link
Member

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 ?

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.

@Nightwalker-87
Copy link
Member

@grevaillot How do you think about this?

@Nightwalker-87
Copy link
Member

ping @grevaillot

@grevaillot
Copy link
Collaborator Author

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.

@Nightwalker-87 Nightwalker-87 merged commit 89fa734 into stlink-org:develop Apr 6, 2020
@Nightwalker-87
Copy link
Member

Thanks!

@grevaillot grevaillot deleted the to_merge/stm32g4_update branch April 11, 2020 16:05
@stlink-org stlink-org locked as resolved and limited conversation to collaborators May 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants