Integration of Nucleo f411re

AdaCore / bb-runtimes

Source repository for the GNAT Bare Metal BSPs

Other

65 stars 51 forks source link

Integration of Nucleo f411re #47

Closed FredPraca closed 3 years ago

Fabien-Chouteau commented 3 years ago

Hello @FredPraca,

There is not enough difference between the 401 and the 411 to justify to sets of files. The only significant difference is that the memory sizes are different.

I will close this PR and focus on the 401, once the 401 is merged you can make a new PR with just the memory sizes using templates.

FredPraca commented 3 years ago

I checked and I also found:

SPI4 clock on APB2 bus in i-stm32-rcc.ads
the SPI5_Base address in i-stm32.ads

I'm not sure how to deal with these using templates only, can I ?

Fabien-Chouteau commented 3 years ago

As far as the run-time is concerned, we don't care about SPI so don't mind those.

FredPraca commented 3 years ago

Ok, so I'll see the impact on Ada_Drivers_Library before creating a pull request on it.

Thanks

FredPraca commented 3 years ago

Is it better to provide a runtime for 4x1re boards or provide 401re and 411re based on templating ?

Fabien-Chouteau commented 3 years ago

I depends on the differences between the two BSP. I guess it will only be a question of memory map and clock speed here, so indeed templates should be a good solution.