-
-
Notifications
You must be signed in to change notification settings - Fork 8.2k
NUCLEO-H7A3ZI-Q #11398
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
base: master
Are you sure you want to change the base?
NUCLEO-H7A3ZI-Q #11398
Conversation
…rogrammed after loading this firmware
Thanks for the contribution. How did you generate the |
//#define MICROPY_HW_CLK_PLLR (2) | ||
//#define MICROPY_HW_CLK_PLLVCI (RCC_PLL1VCIRANGE_1) | ||
//#define MICROPY_HW_CLK_PLLVCO (RCC_PLL1VCOWIDE) | ||
//#define MICROPY_HW_CLK_PLLFRAC (0) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove commented-out code (unless it's for some reason useful, then wrap it in #if 0
/#endif
and add a comment about it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed commented code
#define MICROPY_HW_CLK_PLLN (70) | ||
#define MICROPY_HW_CLK_PLLP (2) | ||
#define MICROPY_HW_CLK_PLLQ (14) | ||
#define MICROPY_HW_CLK_PLLR (14) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please indent/format like the other lines.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be ok now
//#define MICROPY_HW_UART4_TX (pin_H13) // Arduino Connector CN11-Pin1 | ||
//#define MICROPY_HW_UART4_RX (pin_H14) // Arduino Connector CN11-Pin2 | ||
//#define MICROPY_HW_UART_REPL PYB_UART_1 | ||
//#define MICROPY_HW_UART_REPL_BAUD 115200 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove or un-comment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed
#include "boards/stm32h7xx_hal_conf_base.h" | ||
|
||
// Oscillator values in Hz | ||
//#define HSE_VALUE (24000000) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove commented-out lines.
What is the HSE value?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is 8MHz. Should be ok now.
ports/stm32/boards/stm32h7a3.ld
Outdated
_heap_start = _ebss; /* heap starts just after statically allocated memory */ | ||
_heap_end = _sstack; | ||
|
||
/* not truncated */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What does this comment mean?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed this comment
ports/stm32/boards/stm32h7a3.ld
Outdated
{ | ||
FLASH (rx) : ORIGIN = 0x08000000, LENGTH = 2048K /* 8K sector size */ | ||
FLASH_ISR (rx) : ORIGIN = 0x08000000, LENGTH = 32K /* bootloader */ | ||
FLASH_FS (r) : ORIGIN = 0x08008000, LENGTH = 96K /* minimal file system */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The flash page erase size is 128k, so the bootloader must take that much space, and the minimum filesystem must also be 128k.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is probably related to #11138 for the adapted flash block size on the H7A and H7B devices
This works on my NUCLEO-H7A3ZI-Q boards. It was mainly adapted from the STM32H7B3I_DK board files.
There are probably a few peripherals activated that are not available at the NUCLEO board.
The PLL settings are taken from the suggestion at: https://forum.micropython.org/viewtopic.php?t=7930