-
Notifications
You must be signed in to change notification settings - Fork 18.6k
AP_HAL_ChibiOS: add hwdef files for NarinFC-H5 #29753
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
Conversation
Co-authored-by: Henry Wurzburg <hwurzburg@yahoo.com>
Co-authored-by: Henry Wurzburg <hwurzburg@yahoo.com>
@vololand since this is just a minor change to the H7, it should be done via includes...which I have provided new hwdefs...I built a new bootloader with the new board id (you had the H7 one), changed the bd id, since it was already taken.... as to the wiki page, it should just be included on the H7 page as a variant. the big problem is the name...STM32-Hxxx is a valid processor series...naming it an -H5 is misleading and will cause confusion.... |
- ICM45686 SPI MAX SPEED UPDATE.(8*MHZ -> 16*MHZ)
@Hwurzburg |
@vololand are you absolutely sure that you want to call this a "H5"? This might turn consumers off as they want a H7 processor and would consider a "H5" to be a "lesser device" than a H7. The fact it's running a H7 processor will be completely lost! Please confirm you really want to use "H5" - everyone on the DevCall I'm currently on thinks its a bad name! |
We decided to hold off on merging this for another week in order to revisit the name again. |
@peterbarker |
@rmackay9 |
Merged it! |
AP_HAL_ChibiOS: add hwdef files for NarinFC-H5