Skip to content

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

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

NUCLEO-H7A3ZI-Q #11398

wants to merge 7 commits into from

Conversation

pschindler
Copy link

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

@dpgeorge
Copy link
Member

dpgeorge commented May 9, 2023

Thanks for the contribution.

How did you generate the stm32h7a3_af.csv file?

//#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)
Copy link
Member

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.

Copy link
Author

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)
Copy link
Member

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.

Copy link
Author

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
Copy link
Member

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.

Copy link
Author

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)
Copy link
Member

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?

Copy link
Author

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.

_heap_start = _ebss; /* heap starts just after statically allocated memory */
_heap_end = _sstack;

/* not truncated */
Copy link
Member

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?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed this comment

{
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 */
Copy link
Member

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.

Copy link
Author

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

@mattytrentini mattytrentini added the board-definition New or updated board definition files. Combine with a port- label. label Oct 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
board-definition New or updated board definition files. Combine with a port- label. port-stm32
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants