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

[Feature Request] Add more STM32F303 variants #10836

Closed
2 of 4 tasks
yiancar opened this issue Nov 2, 2020 · 7 comments
Closed
2 of 4 tasks

[Feature Request] Add more STM32F303 variants #10836

yiancar opened this issue Nov 2, 2020 · 7 comments
Labels
enhancement help wanted stale Issues or pull requests that have become inactive without resolution.

Comments

@yiancar
Copy link
Contributor

yiancar commented Nov 2, 2020

STM32F303 comes in multiple variants.

STM32F303xC and STM32F303xB are essentially the same except for the flash size and RAM.

Currently when MCU = STM32F303 it is assumed that its a STM32F303xC.
This can become more fine grain.

According to ST the die for F303xB and F303xC are the same but verified only up to the corresponding Flash and RAM sizes. This means that the ld scripts for F303xC are valid but when used for F303xB the user should be aware of the limitations. Everything else compiler side is the same.

As a first start, to add the F303xB variant, the only core change would be to change where in flash the eeprom is stored. This is trivial. If a more correct implementation is needed, the ld script should also be changed.

Currently chibios only has ld scripts for F303xC (same die as xB) and F303x8 (same die as x6). I would personally opt out of supporting x8 and x6 chips because of the 16kb RAM available compared to 40+kb

Feature Request Type

  • Core functionality
  • Add-on hardware support (eg. audio, RGB, OLED screen, etc.)
  • Alteration (enhancement/optimization) of existing feature(s)
  • New behavior

Description

@tzarc
Copy link
Member

tzarc commented Nov 5, 2020

Some variants don't have USB device support, anyway. F303K8T6, for example.
There's no reason why ldscripts can't be provided by keyboards in the short term -- anything with a user bootloader likely does so already. Realistically, this would just require dropping in a few more files into the common ldscripts folder that'll turn up with the new ChibiOS upgrade, feel free to @ me when it drops and we can see what we can sort out. Should be relatively early in the next develop cycle.

@yiancar
Copy link
Contributor Author

yiancar commented Nov 5, 2020

That sounds good,
Ill keep an eye out lets see who remembers first:P
So you think going with a MCU=STM2F303xC and MCU=STM2F303xB

Is the way to go? That to me seems best as it allows the ld file to be picked up as well. Cheeky K devices in the summary sheet it says they do have USB but not in the datasheet..

@stale
Copy link

stale bot commented Feb 4, 2021

This issue has been automatically marked as stale because it has not had activity in the last 90 days. It will be closed in the next 30 days unless it is tagged properly or other activity occurs.
For maintainers: Please label with bug, in progress, on hold, discussion or to do to prevent the issue from being re-flagged.

@stale stale bot added the stale Issues or pull requests that have become inactive without resolution. label Feb 4, 2021
@yiancar
Copy link
Contributor Author

yiancar commented Feb 4, 2021

any comments @tzarc :)

@stale stale bot removed the stale Issues or pull requests that have become inactive without resolution. label Feb 4, 2021
@fauxpark
Copy link
Member

fauxpark commented Feb 4, 2021

See also #8179

@stale
Copy link

stale bot commented Jun 2, 2021

This issue has been automatically marked as stale because it has not had activity in the last 90 days. It will be closed in the next 30 days unless it is tagged properly or other activity occurs.
For maintainers: Please label with bug, in progress, on hold, discussion or to do to prevent the issue from being re-flagged.

@stale stale bot added the stale Issues or pull requests that have become inactive without resolution. label Jun 2, 2021
@stale
Copy link

stale bot commented Jul 8, 2021

This issue has been automatically closed because it has not had activity in the last 30 days. If this issue is still valid, re-open the issue and let us know.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement help wanted stale Issues or pull requests that have become inactive without resolution.
Projects
None yet
Development

No branches or pull requests

3 participants