On Mon, 9 Jun 2025 10:34:39 +0200 Patrice CHOTARD <patrice.chot...@foss.st.com> wrote:
> On 6/9/25 10:07, Lukasz Majewski wrote: > > Hi Patrice, > > > >> On 6/7/25 11:37, Dario Binacchi wrote: > >>> The board includes an STM32H747XI SoC with the following > >>> resources: > >>> - 2 Mbytes Flash > >>> - 1 Mbyte SRAM > >>> - LCD-TFT controller > >>> - MIPI-DSI interface > >>> - FD-CAN > >>> - USB 2.0 high-speed/full-speed > >>> - Ethernet MAC > >>> - camera interface > >>> > >>> Detailed information can be found at: > >>> https://www.st.com/en/evaluation-tools/stm32h747i-disco.html > >>> > >>> Signed-off-by: Dario Binacchi > >>> <dario.binac...@amarulasolutions.com> > >>> > >>> --- > >>> > >>> arch/arm/mach-stm32/stm32h7/Kconfig | 4 +++ > >>> board/st/stm32h747-disco/Kconfig | 15 ++++++++ > >>> board/st/stm32h747-disco/MAINTAINERS | 7 ++++ > >>> board/st/stm32h747-disco/Makefile | 6 ++++ > >>> board/st/stm32h747-disco/stm32h747-disco.c | 42 > >>> ++++++++++++++++++++++ configs/stm32h747-disco_defconfig > >>> | 35 ++++++++++++++++++ drivers/clk/stm32/clk-stm32h7.c > >>> | 5 +++ include/configs/stm32h747-disco.h | 32 > >>> +++++++++++++++++ 8 files changed, 146 insertions(+) > >>> create mode 100644 board/st/stm32h747-disco/Kconfig > >>> create mode 100644 board/st/stm32h747-disco/MAINTAINERS > >>> create mode 100644 board/st/stm32h747-disco/Makefile > >>> create mode 100644 board/st/stm32h747-disco/stm32h747-disco.c > >>> create mode 100644 configs/stm32h747-disco_defconfig > >>> create mode 100644 include/configs/stm32h747-disco.h > >>> > >>> diff --git a/arch/arm/mach-stm32/stm32h7/Kconfig > >>> b/arch/arm/mach-stm32/stm32h7/Kconfig index > >>> 70233a4b23cd..72f20c477d04 100644 --- > >>> a/arch/arm/mach-stm32/stm32h7/Kconfig +++ > >>> b/arch/arm/mach-stm32/stm32h7/Kconfig @@ -6,11 +6,15 @@ config > >>> TARGET_STM32H743_DISCO config TARGET_STM32H743_EVAL > >>> bool "STM32H743 Evaluation board" > >>> > >>> +config TARGET_STM32H747_DISCO > >>> + bool "STM32H747 Discovery board" > >>> + > >>> config TARGET_STM32H750_ART_PI > >>> bool "STM32H750 ART Pi board" > >>> > >>> source "board/st/stm32h743-eval/Kconfig" > >>> source "board/st/stm32h743-disco/Kconfig" > >>> +source "board/st/stm32h747-disco/Kconfig" > >>> source "board/st/stm32h750-art-pi/Kconfig" > >>> > >>> endif > >>> diff --git a/board/st/stm32h747-disco/Kconfig > >>> b/board/st/stm32h747-disco/Kconfig new file mode 100644 > >>> index 000000000000..a7b2c09a327f > >>> --- /dev/null > >>> +++ b/board/st/stm32h747-disco/Kconfig > >>> @@ -0,0 +1,15 @@ > >>> +if TARGET_STM32H747_DISCO > >>> + > >>> +config SYS_BOARD > >>> + default "stm32h747-disco" > >>> + > >>> +config SYS_VENDOR > >>> + default "st" > >>> + > >>> +config SYS_SOC > >>> + default "stm32h7" > >>> + > >>> +config SYS_CONFIG_NAME > >>> + default "stm32h747-disco" > >>> + > >>> +endif > >>> diff --git a/board/st/stm32h747-disco/MAINTAINERS > >>> b/board/st/stm32h747-disco/MAINTAINERS new file mode 100644 > >>> index 000000000000..d48649f773f3 > >>> --- /dev/null > >>> +++ b/board/st/stm32h747-disco/MAINTAINERS > >>> @@ -0,0 +1,7 @@ > >>> +STM32H747 DISCOVERY BOARD > >>> +M: Dario Binacchi <dario.binac...@amarulasolutions.com> > >>> +S: Maintained > >>> +F: board/st/stm32h747-disco > >>> +F: include/configs/stm32h747-disco.h > >>> +F: configs/stm32h747-disco_defconfig > >>> +F: arch/arm/dts/stm32h747* > >>> diff --git a/board/st/stm32h747-disco/Makefile > >>> b/board/st/stm32h747-disco/Makefile new file mode 100644 > >>> index 000000000000..e11f052cc88f > >>> --- /dev/null > >>> +++ b/board/st/stm32h747-disco/Makefile > >>> @@ -0,0 +1,6 @@ > >>> +# SPDX-License-Identifier: GPL-2.0+ > >>> +# > >>> +# Copyright (c) 2025 Dario Binacchi > >>> <dario.binac...@amarulasolutions.com> +# > >>> + > >>> +obj-y := stm32h747-disco.o > >>> diff --git a/board/st/stm32h747-disco/stm32h747-disco.c > >>> b/board/st/stm32h747-disco/stm32h747-disco.c new file mode 100644 > >>> index 000000000000..be0884bdeb4d > >>> --- /dev/null > >>> +++ b/board/st/stm32h747-disco/stm32h747-disco.c > >>> @@ -0,0 +1,42 @@ > >>> +// SPDX-License-Identifier: GPL-2.0+ > >>> +/* > >>> + * stm32h747i-disco support > >>> + * > >>> + * Copyright (C) 2025 Dario Binacchi > >>> <dario.binac...@amarulasolutions.com> > >>> + */ > >>> + > >>> +#include <dm.h> > >>> +#include <init.h> > >>> +#include <log.h> > >>> +#include <asm/global_data.h> > >>> + > >>> +DECLARE_GLOBAL_DATA_PTR; > >>> + > >>> +int dram_init(void) > >>> +{ > >>> + struct udevice *dev; > >>> + int ret; > >>> + > >>> + ret = uclass_get_device(UCLASS_RAM, 0, &dev); > >>> + if (ret) { > >>> + debug("DRAM init failed: %d\n", ret); > >>> + return ret; > >>> + } > >>> + > >>> + if (fdtdec_setup_mem_size_base() != 0) > >>> + ret = -EINVAL; > >>> + > >>> + return ret; > >>> +} > >>> + > >>> +int dram_init_banksize(void) > >>> +{ > >>> + fdtdec_setup_memory_banksize(); > >>> + > >>> + return 0; > >>> +} > >>> + > >>> +int board_init(void) > >>> +{ > >>> + return 0; > >>> +} > >>> diff --git a/configs/stm32h747-disco_defconfig > >>> b/configs/stm32h747-disco_defconfig new file mode 100644 > >>> index 000000000000..8a0c72450d1e > >>> --- /dev/null > >>> +++ b/configs/stm32h747-disco_defconfig > >>> @@ -0,0 +1,35 @@ > >>> +CONFIG_ARM=y > >>> +CONFIG_ARCH_STM32=y > >>> +CONFIG_TEXT_BASE=0x08000000 > >>> +CONFIG_SYS_MALLOC_LEN=0x100000 > >>> +CONFIG_NR_DRAM_BANKS=1 > >>> +CONFIG_HAS_CUSTOM_SYS_INIT_SP_ADDR=y > >>> +CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0x24040000 > >>> +CONFIG_ENV_SIZE=0x2000 > >>> +CONFIG_DEFAULT_DEVICE_TREE="st/stm32h747i-disco" > >>> +CONFIG_OF_LIBFDT_OVERLAY=y > >>> +CONFIG_SYS_LOAD_ADDR=0xd0400000 > >>> +CONFIG_STM32H7=y > >>> +CONFIG_TARGET_STM32H747_DISCO=y > >>> +CONFIG_DISTRO_DEFAULTS=y > > > > I'm just wondering if there is any plan to move forward to use the > > new approach? > > > > As fair as I remember this is the "deprecated" option... > > Hi Lukasz > > I was not aware of deprecation of CONFIG_DISTRO_DEFAULTS . > Can you point to me the other alternative ? > I think it is: CONFIG_BOOTSTD The bootstd command and friends (I did some research on STM32MP157C board and it can be enabled ... ). > Thanks > Patrice > > > > >>> +CONFIG_BOOTDELAY=3 > >>> +CONFIG_AUTOBOOT_KEYED=y > >>> +CONFIG_AUTOBOOT_PROMPT="Hit SPACE in %d seconds to stop > >>> autoboot.\n" +CONFIG_AUTOBOOT_STOP_STR=" " > >>> +CONFIG_DEFAULT_FDT_FILE="stm32h747i-disco" > >>> +CONFIG_SYS_CBSIZE=256 > >>> +CONFIG_SYS_PBSIZE=282 > >>> +# CONFIG_DISPLAY_CPUINFO is not set > >>> +CONFIG_SYS_PROMPT="U-Boot > " > >>> +CONFIG_CMD_GPT=y > >>> +CONFIG_CMD_MMC=y > >>> +# CONFIG_CMD_SETEXPR is not set > >>> +CONFIG_CMD_CACHE=y > >>> +CONFIG_CMD_TIMER=y > >>> +CONFIG_CMD_EXT4_WRITE=y > >>> +# CONFIG_ISO_PARTITION is not set > >>> +CONFIG_OF_CONTROL=y > >>> +CONFIG_SYS_RELOC_GD_ENV_ADDR=y > >>> +CONFIG_NO_NET=y > >>> +CONFIG_STM32_SDMMC2=y > >>> +# CONFIG_PINCTRL_FULL is not set > >>> diff --git a/drivers/clk/stm32/clk-stm32h7.c > >>> b/drivers/clk/stm32/clk-stm32h7.c index 6acf2ff0a8fb..aa3be414a29f > >>> 100644 --- a/drivers/clk/stm32/clk-stm32h7.c > >>> +++ b/drivers/clk/stm32/clk-stm32h7.c > >>> @@ -114,6 +114,7 @@ > >>> #define QSPISRC_PER_CK 3 > >>> > >>> #define PWR_CR3 0x0c > >>> +#define PWR_CR3_LDOEN BIT(1) > >>> #define PWR_CR3_SCUEN BIT(2) > >>> #define PWR_D3CR 0x18 > >>> #define PWR_D3CR_VOS_MASK GENMASK(15, 14) > >>> @@ -375,7 +376,11 @@ int configure_clocks(struct udevice *dev) > >>> clrsetbits_le32(pwr_base + PWR_D3CR, PWR_D3CR_VOS_MASK, > >>> VOS_SCALE_1 << PWR_D3CR_VOS_SHIFT); > >>> /* Lock supply configuration update */ > >>> +#if IS_ENABLED(CONFIG_TARGET_STM32H747_DISCO) > >>> + clrbits_le32(pwr_base + PWR_CR3, PWR_CR3_LDOEN); > >>> +#else > >>> clrbits_le32(pwr_base + PWR_CR3, PWR_CR3_SCUEN); > >>> +#endif > >>> while (!(readl(pwr_base + PWR_D3CR) & PWR_D3CR_VOSREADY)) > >>> ; > >>> > >>> diff --git a/include/configs/stm32h747-disco.h > >>> b/include/configs/stm32h747-disco.h new file mode 100644 > >>> index 000000000000..393445a8ae1f > >>> --- /dev/null > >>> +++ b/include/configs/stm32h747-disco.h > >>> @@ -0,0 +1,32 @@ > >>> +/* SPDX-License-Identifier: GPL-2.0+ */ > >>> +/* > >>> + * Copyright (C) 2025 Dario Binacchi > >>> <dario.binac...@amarulasolutions.com> > >>> + */ > >>> + > >>> +#ifndef __CONFIG_H > >>> +#define __CONFIG_H > >>> + > >>> +#include <config.h> > >>> +#include <linux/sizes.h> > >>> + > >>> +/* For booting Linux, use the first 16MB of memory */ > >>> +#define CFG_SYS_BOOTMAPSZ SZ_16M > >>> + > >>> +#define CFG_SYS_FLASH_BASE 0x08000000 > >>> + > >>> +#define CFG_SYS_HZ_CLOCK 1000000 > >>> + > >>> +#define BOOT_TARGET_DEVICES(func) \ > >>> + func(MMC, mmc, 0) > >>> + > >>> +#include <config_distro_bootcmd.h> > >>> +#define CFG_EXTRA_ENV_SETTINGS \ > >>> + "kernel_addr_r=0xD0008000\0" > >>> \ > >>> + "fdtfile=stm32h747i-disco.dtb\0" \ > >>> + "fdt_addr_r=0xD0408000\0" > >>> \ > >>> + "scriptaddr=0xD0418000\0" > >>> \ > >>> + "pxefile_addr_r=0xD0428000\0" \ > >>> + "ramdisk_addr_r=0xD0438000\0" > >>> \ > >>> + BOOTENV > >>> + > >>> +#endif /* __CONFIG_H */ > >> Reviewed-by: Patrice Chotard <patrice.chot...@foss.st.com> > >> > >> Thanks > >> Patrice > > > > > > > > > > Best regards, > > > > Lukasz Majewski > > > > -- > > > > DENX Software Engineering GmbH, Managing Director: Erika Unter > > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, > > Germany Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: > > lu...@denx.de Best regards, Lukasz Majewski -- DENX Software Engineering GmbH, Managing Director: Erika Unter HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lu...@denx.de
pgpibbQNdKdSS.pgp
Description: OpenPGP digital signature
