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

Wrong RAM size in linker scripts #544

Closed
abrandemuehl opened this issue Jan 30, 2020 · 2 comments
Closed

Wrong RAM size in linker scripts #544

abrandemuehl opened this issue Jan 30, 2020 · 2 comments

Comments

@abrandemuehl
Copy link

Looks like the RAM size is incorrect in the linker scripts for the STM32F405. The one on the CF2.0 has 192kB of RAM, but is declared as 128kB. Obviously the limit is not being hit in the main firmware, but in a project I'm doing right now I've run into it.

I'm also submitting a PR to fix it

https://github.com/bitcraze/crazyflie-firmware/blob/master/tools/make/F405/linker/DEF_CLOAD.ld#L13
https://github.com/bitcraze/crazyflie-firmware/blob/master/tools/make/F405/linker/DEF.ld#L13

@knmcguire
Copy link
Member

Hi! Thanks for pointing this out. However, the full 192 kb RAM does not all contain the same RAM. 128 kb is 'normal' RAM but the remaining 64 kb RAM is special CCM RAM. So unfortunately they can not be handled in the same way and your pull request will unfortunately not work...

But we do agree that we should work on getting the CCR ram support on the firmware, so that it is more accessible to users. I made a separate issue about this so we can implement it soon. #546

Would you mind closing this issue and your pullrequest?

@abrandemuehl
Copy link
Author

Great! Thanks for the clarification!

cafeciaojoe pushed a commit to cafeciaojoe/crazyflie-firmware that referenced this issue Sep 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants