[PATCH v2 08/22] board: st: stm32mp2: add user button support

[Patrice CHOTARD]

On 4/25/25 15:15, Patrice Chotard wrote:
> From: Patrick Delaunay <patrick.delaunay at foss.st.com>
> 
> Handle user button 2 to force boot with STM32CubeProgrammer.
> 
> Signed-off-by: Patrick Delaunay <patrick.delaunay at foss.st.com>
> Signed-off-by: Patrice Chotard <patrice.chotard at foss.st.com>
> ---
> 
> (no changes since v1)
> 
>  board/st/stm32mp2/stm32mp2.c | 26 ++++++++++++++++++++++++++
>  configs/stm32mp25_defconfig  |  4 +++-
>  2 files changed, 29 insertions(+), 1 deletion(-)
> 
> diff --git a/board/st/stm32mp2/stm32mp2.c b/board/st/stm32mp2/stm32mp2.c
> index 576a958c783..6e4f19b1664 100644
> --- a/board/st/stm32mp2/stm32mp2.c
> +++ b/board/st/stm32mp2/stm32mp2.c
> @@ -5,6 +5,7 @@
>  
>  #define LOG_CATEGORY LOGC_BOARD
>  
> +#include <button.h>
>  #include <config.h>
>  #include <env_internal.h>
>  #include <fdt_support.h>
> @@ -13,10 +14,12 @@
>  #include <misc.h>
>  #include <mmc.h>
>  #include <asm/global_data.h>
> +#include <asm/io.h>
>  #include <asm/arch/sys_proto.h>
>  #include <dm/device.h>
>  #include <dm/ofnode.h>
>  #include <dm/uclass.h>
> +#include <linux/delay.h>
>  
>  /*
>   * Get a global data pointer
> @@ -90,10 +93,33 @@ static int setup_led(enum led_state_t cmd)
>  	return ret;
>  }
>  
> +static void check_user_button(void)
> +{
> +	struct udevice *button;
> +	int i;
> +
> +	if (!IS_ENABLED(CONFIG_CMD_STM32PROG) || !IS_ENABLED(CONFIG_BUTTON))
> +		return;
> +
> +	if (button_get_by_label("User-2", &button))
> +		return;
> +
> +	for (i = 0; i < 21; ++i) {
> +		if (button_get_state(button) != BUTTON_ON)
> +			return;
> +		if (i < 20)
> +			mdelay(50);
> +	}
> +
> +	log_notice("entering download mode...\n");
> +	clrsetbits_le32(TAMP_BOOT_CONTEXT, TAMP_BOOT_FORCED_MASK, BOOT_STM32PROG);
> +}
> +
>  /* board dependent setup after realloc */
>  int board_init(void)
>  {
>  	setup_led(LEDST_ON);
> +	check_user_button();
>  
>  	return 0;
>  }
> diff --git a/configs/stm32mp25_defconfig b/configs/stm32mp25_defconfig
> index 3d6665335f5..3f765346904 100644
> --- a/configs/stm32mp25_defconfig
> +++ b/configs/stm32mp25_defconfig
> @@ -39,12 +39,14 @@ CONFIG_ENV_IS_IN_MMC=y
>  CONFIG_SYS_MMC_ENV_DEV=-1
>  CONFIG_NO_NET=y
>  CONFIG_SYS_64BIT_LBA=y
> +CONFIG_BUTTON=y
> +CONFIG_BUTTON_GPIO=y
>  CONFIG_GPIO_HOG=y
>  CONFIG_DM_I2C=y
>  CONFIG_SYS_I2C_STM32F7=y
> -CONFIG_SUPPORT_EMMC_BOOT=y
>  CONFIG_LED=y
>  CONFIG_LED_GPIO=y
> +CONFIG_SUPPORT_EMMC_BOOT=y
>  CONFIG_STM32_SDMMC2=y
>  CONFIG_MTD=y
>  CONFIG_USE_SYS_MAX_FLASH_BANKS=y
Applied to u-boot-stm32/master

Thanks
Patrice