-
-
Notifications
You must be signed in to change notification settings - Fork 8.6k
STM32G4 series added #6808
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
STM32G4 series added #6808
Conversation
* usbd_def.h: add ifndef around 2 defines (defined in stm32lib)
|
I did not enable the continuous integration build of the added Nucleo board in de ci.sh script, because I don't know if that should be done for all pull requests, and because I changed the build optimization setting for the STM32 boards in the Makefile during debug. Otherwise the code wouldn't fit during my debug sessions. |
|
Thanks for this, it's definitely a great addition! I will get to reviewing this as soon as I can. |
|
@dpgeorge Would now be a good time to review #5396 - support for LPUART1 as standard UART? It would be good to have REPL on STLINK without having to modify the NUCLEO G474RE hardware. I can rebase and retest it. I would also like to discuss the port name ('LP1' or next available UART number) as using LP1 added some complexity. Due to how the UART driver works the next available UART number has to be used internally and can easliy be explained in mpconfigboard.h which is where everyone has to look to see what UARTs are available for a port. |
… micopython specifies MICROPY_HW_MAX_UART in mpconfigboard_common.h (is max also max number of uarts, or is it related to internal UART numbering (e.g. [UART1, UART2, UART3, UART6] -> max = 6?) All credits go to @christmas9 for pull request micropython#5396
|
Integrated pull request #5396 and tested it on Nucleo board G474RE. Now we don't need additional wires for REPL uart |
…nd, and had to be fixed the first time.
|
I'v renamed the branch in our repo to STMG4, which breaks the link to this pull request. I will re-create the pull request. |
I've added the STM32G4 series to the micropython project. I did test the implementation on the Nucleo board that is also included here (G474RE), and I also tested the FDCAN in pyb_can.c. But probably seeing what is happening to other pyb_* files this one needs to be transfered later to a machine_can.c.
Only thing with the Nucleo board is that it uses the LPUART as connection to the USB serial on the integrated programmer (ST-Link v3), so to use MicroPython I needed to re-wire the uart. A readme is supplied within the board folder with the 2 wires I added.
The G4 series is also added to the stm32lib, which I also just sent in a pull request.