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

stm32f723 related issues #1171

Open
jeandet opened this issue Jun 25, 2024 · 4 comments
Open

stm32f723 related issues #1171

jeandet opened this issue Jun 25, 2024 · 4 comments

Comments

@jeandet
Copy link

jeandet commented Jun 25, 2024

Few issues related to stm32f723 (maybe others f7)

  • it has a single precision FPU while modm builds with -mfpu=fpv5-d16 instead of -mfpu=fpv5-sp-d16
  • the integrated USB HS phy seems not totally supported/initialized
  • USB D+ and D- in USB HS are initialized with Gpio::OutputSpeed::High instead of Gpio::OutputSpeed::VeryHigh

I'll try to PR fixes soon.

@salkinium
Copy link
Member

  1. Single vs dual precision in F7 is generated here originally.
  2. Yup, I didn't find a F723 device with the built-in USB HS PHY in my possession, so couldn't test it.
  3. Huh, it's probably also wrong for USB FS?

@jeandet
Copy link
Author

jeandet commented Jun 26, 2024

@salkinium, thx for the information, I'm actually working on a custom board, I just ordered an stm32f723 discovery board so I'll be able to PR some examples too.
For 1, I think I should spend some time checking which models have double precision FPU and see if there is any rule from the reference.
I'm also considering making a PR for adding meson build since I'm the QTC meson plugin dev and an hardcore QTC user, would you be open for this too?

@salkinium
Copy link
Member

I'm also considering making a PR for adding meson build since I'm the QTC meson plugin dev and an hardcore QTC user, would you be open for this too?

YES PLEASE! I like meson, and I'm very happy to help the porting, since I wrote the make and scons one (we don't speak about the cmake one 😒).

@salkinium
Copy link
Member

I've updated the modm-devices submodule to include your changes in this repo, so at least point 1 should be fixed now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

2 participants