* [PATCH v1 00/25] stm32mp: Fix keys & leds management
@ 2024-04-09 15:01 Patrice Chotard
2024-04-09 15:01 ` [PATCH v1 01/25] configs: stm32mp13: Enable FASTBOOT Patrice Chotard
` (24 more replies)
0 siblings, 25 replies; 79+ messages in thread
From: Patrice Chotard @ 2024-04-09 15:01 UTC (permalink / raw)
To: u-boot
Cc: Patrice CHOTARD, Patrick DELAUNAY, U-Boot STM32, Fabrice Gasnier,
Gatien Chevallier, Grzegorz Szymaszek, Igor Opaniuk,
Marcel Ziswiler, Masahisa Kojima, Rasmus Villemoes, Sean Anderson,
Simon Glass, Tom Rini
Adding FASTBOOT support for stm32mp13 shows issues when entering
in fastboot mode using a dedicated key.
On several STM32MP boards, same gpio is shared between key and led.
Restore the fastboot/stm32prog activation using a dedicated key.
Restore the led-blue which indicates U-Boot entering / exit.
Make usage of BUTTON-UCLASS for key management on STM32MP1.
Patrice Chotard (25):
configs: stm32mp13: Enable FASTBOOT
configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp15_defconfig
configs: stm32mp1: Enable BUTTON_GPIO flag for
stm32mp15_basic_defconfig
configs: stm32mp1: Enable BUTTON_GPIO flag for
stm32mp15_trusted_defconfig
configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp13_defconfig
board: st: stmp32mp1: Use BUTTON UCLASS in board_key_check()
ARM: dts: stm32: Add gpio-keys for stm32mp135f-dk-u-boot
ARM: dts: stm32: Don't probe led-red/led-blue at boot for
stm32mp135f-dk-u-boot
ARM: dts: stm32: Clean led-red node for stm32mp135f-dk-u-boot
ARM: dts: stm32: Add gpio-keys for stm32mp157a-dk1-scmi-u-boot
ARM: dts: stm32: Don't probe red led at boot for
stm32mp157a-dk1-scmi-u-boot
ARM: dts: stm32: Update red led node for stm32mp157a-dk1-scmi-u-boot
ARM: dts: stm32: Add led-blue for stm32mp157a-dk1-scmi-u-boot
ARM: dts: stm32: Add gpio-keys for stm32mp157a-dk1-u-boot
ARM: dts: stm32: Don't probe red led at boot for
stm32mp157a-dk1-u-boot
ARM: dts: stm32: Update red led node for stm32mp157a-dk1-u-boot
ARM: dts: stm32: Update u-boot,boot-led for stm32mp157a-dk1-u-boot
ARM: dts: stm32: Add gpio-keys for stm32mp157c-ed1-u-boot
ARM: dts: stm32: Don't probe red led at boot for
stm32mp157c-ed1-u-boot
ARM: dts: stm32: Update red led node for stm32mp157c-ed1-u-boot
ARM: dts: stm32: Add led-blue for stm32mp157c-ed1-u-boot
ARM: dts: stm32: Add gpio-keys for stm32mp157c-ed1-scmi-u-boot
ARM: dts: stm32: Don't probe red led at boot for
stm32mp157c-ed1-scmi-u-boot
ARM: dts: stm32: Update red led node for stm32mp157c-ed1-scmi-u-boot
ARM: dts: stm32: Add led-blue for stm32mp157c-ed1-scmi-u-boot
arch/arm/dts/stm32mp135f-dk-u-boot.dtsi | 19 +++++-
arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi | 32 ++++++---
arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi | 32 ++++++---
arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi | 34 +++++++---
arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi | 34 +++++++---
board/st/stm32mp1/stm32mp1.c | 68 +++++++++++--------
configs/stm32mp13_defconfig | 11 ++-
configs/stm32mp15_basic_defconfig | 2 +
configs/stm32mp15_defconfig | 2 +
configs/stm32mp15_trusted_defconfig | 2 +
10 files changed, 173 insertions(+), 63 deletions(-)
--
2.25.1
^ permalink raw reply [flat|nested] 79+ messages in thread
* [PATCH v1 01/25] configs: stm32mp13: Enable FASTBOOT
2024-04-09 15:01 [PATCH v1 00/25] stm32mp: Fix keys & leds management Patrice Chotard
@ 2024-04-09 15:01 ` Patrice Chotard
2024-04-17 9:03 ` Patrick DELAUNAY
2024-04-18 11:48 ` Igor Opaniuk
2024-04-09 15:01 ` [PATCH v1 02/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp15_defconfig Patrice Chotard
` (23 subsequent siblings)
24 siblings, 2 replies; 79+ messages in thread
From: Patrice Chotard @ 2024-04-09 15:01 UTC (permalink / raw)
To: u-boot
Cc: Patrice CHOTARD, Patrick DELAUNAY, U-Boot STM32,
Gatien Chevallier, Igor Opaniuk, Simon Glass, Tom Rini
Enable FASTBOOT relative flags for stm32mp13_defconfig.
Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
---
configs/stm32mp13_defconfig | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/configs/stm32mp13_defconfig b/configs/stm32mp13_defconfig
index c893e272db9..db09e63100e 100644
--- a/configs/stm32mp13_defconfig
+++ b/configs/stm32mp13_defconfig
@@ -15,6 +15,7 @@ CONFIG_CMD_STM32PROG=y
CONFIG_SYS_LOAD_ADDR=0xc2000000
CONFIG_SYS_MEMTEST_START=0xc0000000
CONFIG_SYS_MEMTEST_END=0xc4000000
+# CONFIG_ANDROID_BOOT_IMAGE is not set
CONFIG_FIT=y
CONFIG_SYS_BOOTM_LEN=0x2000000
CONFIG_DISTRO_DEFAULTS=y
@@ -53,6 +54,13 @@ CONFIG_SYS_MMC_ENV_DEV=-1
CONFIG_ENV_MMC_USE_DT=y
CONFIG_CLK_SCMI=y
CONFIG_SET_DFU_ALT_INFO=y
+CONFIG_USB_FUNCTION_FASTBOOT=y
+CONFIG_FASTBOOT_BUF_ADDR=0xc0000000
+CONFIG_FASTBOOT_BUF_SIZE=0x02000000
+CONFIG_FASTBOOT_FLASH=y
+CONFIG_FASTBOOT_FLASH_MMC_DEV=0
+CONFIG_FASTBOOT_CMD_OEM_FORMAT=y
+CONFIG_FASTBOOT_CMD_OEM_PARTCONF=y
CONFIG_GPIO_HOG=y
CONFIG_DM_I2C=y
CONFIG_SYS_I2C_STM32F7=y
@@ -92,7 +100,6 @@ CONFIG_USB_GADGET_MANUFACTURER="STMicroelectronics"
CONFIG_USB_GADGET_VENDOR_NUM=0x0483
CONFIG_USB_GADGET_PRODUCT_NUM=0x5720
CONFIG_USB_GADGET_DWC2_OTG=y
-CONFIG_USB_GADGET_DOWNLOAD=y
CONFIG_ERRNO_STR=y
# CONFIG_LMB_USE_MAX_REGIONS is not set
CONFIG_LMB_MEMORY_REGIONS=2
--
2.25.1
^ permalink raw reply related [flat|nested] 79+ messages in thread
* [PATCH v1 02/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp15_defconfig
2024-04-09 15:01 [PATCH v1 00/25] stm32mp: Fix keys & leds management Patrice Chotard
2024-04-09 15:01 ` [PATCH v1 01/25] configs: stm32mp13: Enable FASTBOOT Patrice Chotard
@ 2024-04-09 15:01 ` Patrice Chotard
2024-04-17 9:03 ` Patrick DELAUNAY
2024-04-09 15:01 ` [PATCH v1 03/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp15_basic_defconfig Patrice Chotard
` (22 subsequent siblings)
24 siblings, 1 reply; 79+ messages in thread
From: Patrice Chotard @ 2024-04-09 15:01 UTC (permalink / raw)
To: u-boot
Cc: Patrice CHOTARD, Patrick DELAUNAY, U-Boot STM32,
Gatien Chevallier, Grzegorz Szymaszek, Simon Glass, Tom Rini
Enable BUTTON_GPIO flag for STM32MP15.
Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
---
configs/stm32mp15_defconfig | 2 ++
1 file changed, 2 insertions(+)
diff --git a/configs/stm32mp15_defconfig b/configs/stm32mp15_defconfig
index 3302b306996..ffe7512650e 100644
--- a/configs/stm32mp15_defconfig
+++ b/configs/stm32mp15_defconfig
@@ -69,6 +69,8 @@ CONFIG_TFTP_TSIZE=y
CONFIG_USE_SERVERIP=y
CONFIG_SERVERIP="192.168.1.1"
CONFIG_STM32_ADC=y
+CONFIG_BUTTON=y
+CONFIG_BUTTON_GPIO=y
CONFIG_CLK_SCMI=y
CONFIG_SET_DFU_ALT_INFO=y
CONFIG_USB_FUNCTION_FASTBOOT=y
--
2.25.1
^ permalink raw reply related [flat|nested] 79+ messages in thread
* [PATCH v1 03/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp15_basic_defconfig
2024-04-09 15:01 [PATCH v1 00/25] stm32mp: Fix keys & leds management Patrice Chotard
2024-04-09 15:01 ` [PATCH v1 01/25] configs: stm32mp13: Enable FASTBOOT Patrice Chotard
2024-04-09 15:01 ` [PATCH v1 02/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp15_defconfig Patrice Chotard
@ 2024-04-09 15:01 ` Patrice Chotard
2024-04-17 9:04 ` Patrick DELAUNAY
2024-04-09 15:01 ` [PATCH v1 04/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp15_trusted_defconfig Patrice Chotard
` (21 subsequent siblings)
24 siblings, 1 reply; 79+ messages in thread
From: Patrice Chotard @ 2024-04-09 15:01 UTC (permalink / raw)
To: u-boot
Cc: Patrice CHOTARD, Patrick DELAUNAY, U-Boot STM32,
Gatien Chevallier, Grzegorz Szymaszek, Marcel Ziswiler,
Sean Anderson, Simon Glass, Tom Rini
Enable BUTTON_GPIO flag for STM32MP15.
Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
---
configs/stm32mp15_basic_defconfig | 2 ++
1 file changed, 2 insertions(+)
diff --git a/configs/stm32mp15_basic_defconfig b/configs/stm32mp15_basic_defconfig
index 005f1d55f80..2e22bf86000 100644
--- a/configs/stm32mp15_basic_defconfig
+++ b/configs/stm32mp15_basic_defconfig
@@ -97,6 +97,8 @@ CONFIG_TFTP_TSIZE=y
CONFIG_USE_SERVERIP=y
CONFIG_SERVERIP="192.168.1.1"
CONFIG_STM32_ADC=y
+CONFIG_BUTTON=y
+CONFIG_BUTTON_GPIO=y
CONFIG_SET_DFU_ALT_INFO=y
CONFIG_USB_FUNCTION_FASTBOOT=y
CONFIG_FASTBOOT_BUF_ADDR=0xC0000000
--
2.25.1
^ permalink raw reply related [flat|nested] 79+ messages in thread
* [PATCH v1 04/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp15_trusted_defconfig
2024-04-09 15:01 [PATCH v1 00/25] stm32mp: Fix keys & leds management Patrice Chotard
` (2 preceding siblings ...)
2024-04-09 15:01 ` [PATCH v1 03/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp15_basic_defconfig Patrice Chotard
@ 2024-04-09 15:01 ` Patrice Chotard
2024-04-17 9:04 ` Patrick DELAUNAY
2024-04-09 15:01 ` [PATCH v1 05/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp13_defconfig Patrice Chotard
` (20 subsequent siblings)
24 siblings, 1 reply; 79+ messages in thread
From: Patrice Chotard @ 2024-04-09 15:01 UTC (permalink / raw)
To: u-boot
Cc: Patrice CHOTARD, Patrick DELAUNAY, U-Boot STM32,
Gatien Chevallier, Grzegorz Szymaszek, Simon Glass, Tom Rini
Enable BUTTON_GPIO flag for STM32MP15.
Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
---
configs/stm32mp15_trusted_defconfig | 2 ++
1 file changed, 2 insertions(+)
diff --git a/configs/stm32mp15_trusted_defconfig b/configs/stm32mp15_trusted_defconfig
index 84b0854b557..74deaaba2e4 100644
--- a/configs/stm32mp15_trusted_defconfig
+++ b/configs/stm32mp15_trusted_defconfig
@@ -70,6 +70,8 @@ CONFIG_TFTP_TSIZE=y
CONFIG_USE_SERVERIP=y
CONFIG_SERVERIP="192.168.1.1"
CONFIG_STM32_ADC=y
+CONFIG_BUTTON=y
+CONFIG_BUTTON_GPIO=y
CONFIG_CLK_SCMI=y
CONFIG_SET_DFU_ALT_INFO=y
CONFIG_USB_FUNCTION_FASTBOOT=y
--
2.25.1
^ permalink raw reply related [flat|nested] 79+ messages in thread
* [PATCH v1 05/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp13_defconfig
2024-04-09 15:01 [PATCH v1 00/25] stm32mp: Fix keys & leds management Patrice Chotard
` (3 preceding siblings ...)
2024-04-09 15:01 ` [PATCH v1 04/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp15_trusted_defconfig Patrice Chotard
@ 2024-04-09 15:01 ` Patrice Chotard
2024-04-17 9:04 ` Patrick DELAUNAY
2024-04-18 11:48 ` Igor Opaniuk
2024-04-09 15:01 ` [PATCH v1 06/25] board: st: stmp32mp1: Use BUTTON UCLASS in board_key_check() Patrice Chotard
` (19 subsequent siblings)
24 siblings, 2 replies; 79+ messages in thread
From: Patrice Chotard @ 2024-04-09 15:01 UTC (permalink / raw)
To: u-boot
Cc: Patrice CHOTARD, Patrick DELAUNAY, U-Boot STM32,
Gatien Chevallier, Igor Opaniuk, Simon Glass, Tom Rini
Enable BUTTON_GPIO flag for STM32MP15.
Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
---
configs/stm32mp13_defconfig | 2 ++
1 file changed, 2 insertions(+)
diff --git a/configs/stm32mp13_defconfig b/configs/stm32mp13_defconfig
index db09e63100e..caaabf39ef3 100644
--- a/configs/stm32mp13_defconfig
+++ b/configs/stm32mp13_defconfig
@@ -52,6 +52,8 @@ CONFIG_SYS_REDUNDAND_ENVIRONMENT=y
CONFIG_SYS_RELOC_GD_ENV_ADDR=y
CONFIG_SYS_MMC_ENV_DEV=-1
CONFIG_ENV_MMC_USE_DT=y
+CONFIG_BUTTON=y
+CONFIG_BUTTON_GPIO=y
CONFIG_CLK_SCMI=y
CONFIG_SET_DFU_ALT_INFO=y
CONFIG_USB_FUNCTION_FASTBOOT=y
--
2.25.1
^ permalink raw reply related [flat|nested] 79+ messages in thread
* [PATCH v1 06/25] board: st: stmp32mp1: Use BUTTON UCLASS in board_key_check()
2024-04-09 15:01 [PATCH v1 00/25] stm32mp: Fix keys & leds management Patrice Chotard
` (4 preceding siblings ...)
2024-04-09 15:01 ` [PATCH v1 05/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp13_defconfig Patrice Chotard
@ 2024-04-09 15:01 ` Patrice Chotard
2024-04-17 9:05 ` Patrick DELAUNAY
2024-04-09 15:01 ` [PATCH v1 07/25] ARM: dts: stm32: Add gpio-keys for stm32mp135f-dk-u-boot Patrice Chotard
` (18 subsequent siblings)
24 siblings, 1 reply; 79+ messages in thread
From: Patrice Chotard @ 2024-04-09 15:01 UTC (permalink / raw)
To: u-boot
Cc: Patrice CHOTARD, Patrick DELAUNAY, U-Boot STM32, Masahisa Kojima,
Rasmus Villemoes, Simon Glass, Tom Rini
Instead of using gpio directly to detect key pressed on button
dedicated for fastboot and stm32mprog, make usage of BUTTON UCLASS.
Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
---
board/st/stm32mp1/stm32mp1.c | 68 +++++++++++++++++++++---------------
1 file changed, 40 insertions(+), 28 deletions(-)
diff --git a/board/st/stm32mp1/stm32mp1.c b/board/st/stm32mp1/stm32mp1.c
index a17c314daeb..82f7d4ff4ea 100644
--- a/board/st/stm32mp1/stm32mp1.c
+++ b/board/st/stm32mp1/stm32mp1.c
@@ -8,6 +8,7 @@
#include <common.h>
#include <adc.h>
#include <bootm.h>
+#include <button.h>
#include <clk.h>
#include <config.h>
#include <dm.h>
@@ -38,6 +39,7 @@
#include <asm/gpio.h>
#include <asm/arch/stm32.h>
#include <asm/arch/sys_proto.h>
+#include <dm/device-internal.h>
#include <dm/ofnode.h>
#include <jffs2/load_kernel.h>
#include <linux/bitops.h>
@@ -150,45 +152,55 @@ int checkboard(void)
static void board_key_check(void)
{
- ofnode node;
- struct gpio_desc gpio;
+ struct udevice *button1 = NULL, *button2 = NULL;
enum forced_boot_mode boot_mode = BOOT_NORMAL;
+ int ret;
+
+ if (!IS_ENABLED(CONFIG_BUTTON))
+ return;
if (!IS_ENABLED(CONFIG_FASTBOOT) && !IS_ENABLED(CONFIG_CMD_STM32PROG))
return;
- node = ofnode_path("/config");
- if (!ofnode_valid(node)) {
- log_debug("no /config node?\n");
+ if (IS_ENABLED(CONFIG_CMD_STM32PROG))
+ button_get_by_label("User-1", &button1);
+
+ if (IS_ENABLED(CONFIG_FASTBOOT))
+ button_get_by_label("User-2", &button2);
+
+ if (!button1 && !button2)
return;
- }
- if (IS_ENABLED(CONFIG_FASTBOOT)) {
- if (gpio_request_by_name_nodev(node, "st,fastboot-gpios", 0,
- &gpio, GPIOD_IS_IN)) {
- log_debug("could not find a /config/st,fastboot-gpios\n");
- } else {
- udelay(20);
- if (dm_gpio_get_value(&gpio)) {
- log_notice("Fastboot key pressed, ");
- boot_mode = BOOT_FASTBOOT;
- }
- dm_gpio_free(NULL, &gpio);
+ if (button2) {
+ if (button_get_state(button2) == BUTTON_ON) {
+ log_notice("Fastboot key pressed, ");
+ boot_mode = BOOT_FASTBOOT;
}
+ /*
+ * On some boards, same gpio is shared betwwen gpio-keys and
+ * leds, remove the button device to free the gpio for led
+ * usage
+ */
+ ret = device_remove(button2, DM_REMOVE_NORMAL);
+ if (ret)
+ log_err("Can't remove button2 (%d)\n", ret);
}
- if (IS_ENABLED(CONFIG_CMD_STM32PROG)) {
- if (gpio_request_by_name_nodev(node, "st,stm32prog-gpios", 0,
- &gpio, GPIOD_IS_IN)) {
- log_debug("could not find a /config/st,stm32prog-gpios\n");
- } else {
- udelay(20);
- if (dm_gpio_get_value(&gpio)) {
- log_notice("STM32Programmer key pressed, ");
- boot_mode = BOOT_STM32PROG;
- }
- dm_gpio_free(NULL, &gpio);
+
+ if (button1) {
+ if (button_get_state(button1) == BUTTON_ON) {
+ log_notice("STM32Programmer key pressed, ");
+ boot_mode = BOOT_STM32PROG;
}
+ /*
+ * On some boards, same gpio is shared betwwen gpio-keys and
+ * leds, remove the button device to free the gpio for led
+ * usage
+ */
+ ret = device_remove(button1, DM_REMOVE_NORMAL);
+ if (ret)
+ log_err("Can't remove button1 (%d)\n", ret);
}
+
if (boot_mode != BOOT_NORMAL) {
log_notice("entering download mode...\n");
clrsetbits_le32(TAMP_BOOT_CONTEXT,
--
2.25.1
^ permalink raw reply related [flat|nested] 79+ messages in thread
* [PATCH v1 07/25] ARM: dts: stm32: Add gpio-keys for stm32mp135f-dk-u-boot
2024-04-09 15:01 [PATCH v1 00/25] stm32mp: Fix keys & leds management Patrice Chotard
` (5 preceding siblings ...)
2024-04-09 15:01 ` [PATCH v1 06/25] board: st: stmp32mp1: Use BUTTON UCLASS in board_key_check() Patrice Chotard
@ 2024-04-09 15:01 ` Patrice Chotard
2024-04-17 9:06 ` Patrick DELAUNAY
2024-04-09 15:01 ` [PATCH v1 08/25] ARM: dts: stm32: Don't probe led-red/led-blue at boot " Patrice Chotard
` (17 subsequent siblings)
24 siblings, 1 reply; 79+ messages in thread
From: Patrice Chotard @ 2024-04-09 15:01 UTC (permalink / raw)
To: u-boot
Cc: Patrice CHOTARD, Patrick DELAUNAY, U-Boot STM32, Fabrice Gasnier,
Tom Rini
Add 2 gpio-keys :
_ button-user-1 for stm32prog mode activation.
_ update button-user's label (defined in kernel DT) to match label
requested in board_key_check() for fastboot mode activation.
Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
---
arch/arm/dts/stm32mp135f-dk-u-boot.dtsi | 13 +++++++++++++
1 file changed, 13 insertions(+)
diff --git a/arch/arm/dts/stm32mp135f-dk-u-boot.dtsi b/arch/arm/dts/stm32mp135f-dk-u-boot.dtsi
index ba0c02489d1..e64ded13462 100644
--- a/arch/arm/dts/stm32mp135f-dk-u-boot.dtsi
+++ b/arch/arm/dts/stm32mp135f-dk-u-boot.dtsi
@@ -16,6 +16,19 @@
u-boot,mmc-env-partition = "u-boot-env";
};
+ gpio-keys {
+ button-user-1 {
+ label = "User-1";
+ linux,code = <BTN_2>;
+ gpios = <&gpioa 14 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>;
+ };
+
+ button-user {
+ /* update label to match the label requested in board_key_check() */
+ label = "User-2";
+ };
+ };
+
leds {
led-red {
color = <LED_COLOR_ID_RED>;
--
2.25.1
^ permalink raw reply related [flat|nested] 79+ messages in thread
* [PATCH v1 08/25] ARM: dts: stm32: Don't probe led-red/led-blue at boot for stm32mp135f-dk-u-boot
2024-04-09 15:01 [PATCH v1 00/25] stm32mp: Fix keys & leds management Patrice Chotard
` (6 preceding siblings ...)
2024-04-09 15:01 ` [PATCH v1 07/25] ARM: dts: stm32: Add gpio-keys for stm32mp135f-dk-u-boot Patrice Chotard
@ 2024-04-09 15:01 ` Patrice Chotard
2024-04-17 9:06 ` Patrick DELAUNAY
2024-04-09 15:01 ` [PATCH v1 09/25] ARM: dts: stm32: Clean led-red node " Patrice Chotard
` (16 subsequent siblings)
24 siblings, 1 reply; 79+ messages in thread
From: Patrice Chotard @ 2024-04-09 15:01 UTC (permalink / raw)
To: u-boot
Cc: Patrice CHOTARD, Patrick DELAUNAY, U-Boot STM32, Fabrice Gasnier,
Tom Rini
led-red and button dedicated to fastboot share the same gpio GPIOA13.
led-blue and button dedicated to stm32prog share the same gpio GPIOA14.
Led driver is probed early so the corresponding gpio is taken and
configured in output which forbid fastboot and stm32prog button usage.
To avoid this, remove the "default-state" property from led-red and
led-blue led's node.
This will avoid to trigger the led driver probe() to configure the led
default state during startup.
Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
---
arch/arm/dts/stm32mp135f-dk-u-boot.dtsi | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/arch/arm/dts/stm32mp135f-dk-u-boot.dtsi b/arch/arm/dts/stm32mp135f-dk-u-boot.dtsi
index e64ded13462..8f42735609a 100644
--- a/arch/arm/dts/stm32mp135f-dk-u-boot.dtsi
+++ b/arch/arm/dts/stm32mp135f-dk-u-boot.dtsi
@@ -30,10 +30,13 @@
};
leds {
+ led-blue {
+ /delete-property/default-state;
+ };
+
led-red {
color = <LED_COLOR_ID_RED>;
gpios = <&gpioa 13 GPIO_ACTIVE_LOW>;
- default-state = "off";
};
};
};
--
2.25.1
^ permalink raw reply related [flat|nested] 79+ messages in thread
* [PATCH v1 09/25] ARM: dts: stm32: Clean led-red node for stm32mp135f-dk-u-boot
2024-04-09 15:01 [PATCH v1 00/25] stm32mp: Fix keys & leds management Patrice Chotard
` (7 preceding siblings ...)
2024-04-09 15:01 ` [PATCH v1 08/25] ARM: dts: stm32: Don't probe led-red/led-blue at boot " Patrice Chotard
@ 2024-04-09 15:01 ` Patrice Chotard
2024-04-17 9:07 ` Patrick DELAUNAY
2024-04-09 15:02 ` [PATCH v1 10/25] ARM: dts: stm32: Add gpio-keys for stm32mp157a-dk1-scmi-u-boot Patrice Chotard
` (15 subsequent siblings)
24 siblings, 1 reply; 79+ messages in thread
From: Patrice Chotard @ 2024-04-09 15:01 UTC (permalink / raw)
To: u-boot
Cc: Patrice CHOTARD, Patrick DELAUNAY, U-Boot STM32, Fabrice Gasnier,
Tom Rini
Remove "color" property from led-red node which is not supported
by U-Boot.
Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
---
arch/arm/dts/stm32mp135f-dk-u-boot.dtsi | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm/dts/stm32mp135f-dk-u-boot.dtsi b/arch/arm/dts/stm32mp135f-dk-u-boot.dtsi
index 8f42735609a..f004e9840a2 100644
--- a/arch/arm/dts/stm32mp135f-dk-u-boot.dtsi
+++ b/arch/arm/dts/stm32mp135f-dk-u-boot.dtsi
@@ -35,7 +35,6 @@
};
led-red {
- color = <LED_COLOR_ID_RED>;
gpios = <&gpioa 13 GPIO_ACTIVE_LOW>;
};
};
--
2.25.1
^ permalink raw reply related [flat|nested] 79+ messages in thread
* [PATCH v1 10/25] ARM: dts: stm32: Add gpio-keys for stm32mp157a-dk1-scmi-u-boot
2024-04-09 15:01 [PATCH v1 00/25] stm32mp: Fix keys & leds management Patrice Chotard
` (8 preceding siblings ...)
2024-04-09 15:01 ` [PATCH v1 09/25] ARM: dts: stm32: Clean led-red node " Patrice Chotard
@ 2024-04-09 15:02 ` Patrice Chotard
2024-04-17 9:07 ` Patrick DELAUNAY
2024-04-09 15:02 ` [PATCH v1 11/25] ARM: dts: stm32: Don't probe red led at boot " Patrice Chotard
` (14 subsequent siblings)
24 siblings, 1 reply; 79+ messages in thread
From: Patrice Chotard @ 2024-04-09 15:02 UTC (permalink / raw)
To: u-boot; +Cc: Patrice CHOTARD, Patrick DELAUNAY, U-Boot STM32, Tom Rini
Instead of using "st,fastboot-gpios" and "st,stm32prog-gpios", declare
2 gpio-keys.
Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
---
arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi | 19 +++++++++++++++++--
1 file changed, 17 insertions(+), 2 deletions(-)
diff --git a/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi b/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
index 20728f27ee1..5d49b09c35d 100644
--- a/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
+++ b/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
@@ -3,6 +3,7 @@
* Copyright : STMicroelectronics 2022
*/
+#include <dt-bindings/input/linux-event-codes.h>
#include "stm32mp15-scmi-u-boot.dtsi"
/ {
@@ -16,8 +17,22 @@
u-boot,error-led = "error";
u-boot,mmc-env-partition = "u-boot-env";
st,adc_usb_pd = <&adc1 18>, <&adc1 19>;
- st,fastboot-gpios = <&gpioa 13 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>;
- st,stm32prog-gpios = <&gpioa 14 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>;
+ };
+
+ gpio-keys {
+ compatible = "gpio-keys";
+
+ button-user-1 {
+ label = "User-1";
+ linux,code = <BTN_1>;
+ gpios = <&gpioa 14 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>;
+ };
+
+ button-user-2 {
+ label = "User-2";
+ linux,code = <BTN_2>;
+ gpios = <&gpioa 13 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>;
+ };
};
led {
--
2.25.1
^ permalink raw reply related [flat|nested] 79+ messages in thread
* [PATCH v1 11/25] ARM: dts: stm32: Don't probe red led at boot for stm32mp157a-dk1-scmi-u-boot
2024-04-09 15:01 [PATCH v1 00/25] stm32mp: Fix keys & leds management Patrice Chotard
` (9 preceding siblings ...)
2024-04-09 15:02 ` [PATCH v1 10/25] ARM: dts: stm32: Add gpio-keys for stm32mp157a-dk1-scmi-u-boot Patrice Chotard
@ 2024-04-09 15:02 ` Patrice Chotard
2024-04-17 9:07 ` Patrick DELAUNAY
2024-04-09 15:02 ` [PATCH v1 12/25] ARM: dts: stm32: Update red led node " Patrice Chotard
` (13 subsequent siblings)
24 siblings, 1 reply; 79+ messages in thread
From: Patrice Chotard @ 2024-04-09 15:02 UTC (permalink / raw)
To: u-boot; +Cc: Patrice CHOTARD, Patrick DELAUNAY, U-Boot STM32, Tom Rini
red led and button dedicated to fastboot share the same gpio GPIOA13.
Led driver is probed early so the corresponding gpio is taken and
configured in output which forbid fastboot and stm32prog button usage.
To avoid this, remove the "default-state" property from red led node.
This will avoid to trigger the led driver probe() to configure the led
default state during startup.
Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
---
arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi b/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
index 5d49b09c35d..8760d6c7d93 100644
--- a/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
+++ b/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
@@ -39,7 +39,6 @@
red {
label = "error";
gpios = <&gpioa 13 GPIO_ACTIVE_LOW>;
- default-state = "off";
status = "okay";
};
};
--
2.25.1
^ permalink raw reply related [flat|nested] 79+ messages in thread
* [PATCH v1 12/25] ARM: dts: stm32: Update red led node for stm32mp157a-dk1-scmi-u-boot
2024-04-09 15:01 [PATCH v1 00/25] stm32mp: Fix keys & leds management Patrice Chotard
` (10 preceding siblings ...)
2024-04-09 15:02 ` [PATCH v1 11/25] ARM: dts: stm32: Don't probe red led at boot " Patrice Chotard
@ 2024-04-09 15:02 ` Patrice Chotard
2024-04-17 9:08 ` Patrick DELAUNAY
2024-04-09 15:02 ` [PATCH v1 13/25] ARM: dts: stm32: Add led-blue " Patrice Chotard
` (12 subsequent siblings)
24 siblings, 1 reply; 79+ messages in thread
From: Patrice Chotard @ 2024-04-09 15:02 UTC (permalink / raw)
To: u-boot; +Cc: Patrice CHOTARD, Patrick DELAUNAY, U-Boot STM32, Tom Rini
As indicated in kernel led dt-bindings, label is a deprecated
property, so remove it and use red led node's name instead
for u-boot,error-led property.
Rename "red" led node's name to "led-red".
Remove status property which is useless.
Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
---
arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi b/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
index 8760d6c7d93..e61814fd66e 100644
--- a/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
+++ b/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
@@ -14,7 +14,7 @@
config {
u-boot,boot-led = "heartbeat";
- u-boot,error-led = "error";
+ u-boot,error-led = "led-red";
u-boot,mmc-env-partition = "u-boot-env";
st,adc_usb_pd = <&adc1 18>, <&adc1 19>;
};
@@ -36,10 +36,8 @@
};
led {
- red {
- label = "error";
+ led-red {
gpios = <&gpioa 13 GPIO_ACTIVE_LOW>;
- status = "okay";
};
};
};
--
2.25.1
^ permalink raw reply related [flat|nested] 79+ messages in thread
* [PATCH v1 13/25] ARM: dts: stm32: Add led-blue for stm32mp157a-dk1-scmi-u-boot
2024-04-09 15:01 [PATCH v1 00/25] stm32mp: Fix keys & leds management Patrice Chotard
` (11 preceding siblings ...)
2024-04-09 15:02 ` [PATCH v1 12/25] ARM: dts: stm32: Update red led node " Patrice Chotard
@ 2024-04-09 15:02 ` Patrice Chotard
2024-04-17 9:09 ` Patrick DELAUNAY
2024-04-09 15:02 ` [PATCH v1 14/25] ARM: dts: stm32: Add gpio-keys for stm32mp157a-dk1-u-boot Patrice Chotard
` (11 subsequent siblings)
24 siblings, 1 reply; 79+ messages in thread
From: Patrice Chotard @ 2024-04-09 15:02 UTC (permalink / raw)
To: u-boot; +Cc: Patrice CHOTARD, Patrick DELAUNAY, U-Boot STM32, Tom Rini
As indicated in kernel led dt-bindings, label is a deprecated
property, so remove it and use blue led node's name instead
for u-boot,boot-led property.
Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
---
arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi b/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
index e61814fd66e..a5158fec7ef 100644
--- a/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
+++ b/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
@@ -13,7 +13,7 @@
};
config {
- u-boot,boot-led = "heartbeat";
+ u-boot,boot-led = "led-blue";
u-boot,error-led = "led-red";
u-boot,mmc-env-partition = "u-boot-env";
st,adc_usb_pd = <&adc1 18>, <&adc1 19>;
@@ -36,6 +36,10 @@
};
led {
+ led-blue {
+ /delete-property/label;
+ };
+
led-red {
gpios = <&gpioa 13 GPIO_ACTIVE_LOW>;
};
--
2.25.1
^ permalink raw reply related [flat|nested] 79+ messages in thread
* [PATCH v1 14/25] ARM: dts: stm32: Add gpio-keys for stm32mp157a-dk1-u-boot
2024-04-09 15:01 [PATCH v1 00/25] stm32mp: Fix keys & leds management Patrice Chotard
` (12 preceding siblings ...)
2024-04-09 15:02 ` [PATCH v1 13/25] ARM: dts: stm32: Add led-blue " Patrice Chotard
@ 2024-04-09 15:02 ` Patrice Chotard
2024-04-17 9:09 ` Patrick DELAUNAY
2024-04-09 15:02 ` [PATCH v1 15/25] ARM: dts: stm32: Don't probe red led at boot " Patrice Chotard
` (10 subsequent siblings)
24 siblings, 1 reply; 79+ messages in thread
From: Patrice Chotard @ 2024-04-09 15:02 UTC (permalink / raw)
To: u-boot
Cc: Patrice CHOTARD, Patrick DELAUNAY, U-Boot STM32, Simon Glass,
Tom Rini
Instead of using "st,fastboot-gpios" and "st,stm32prog-gpios", declare
2 gpio-keys.
Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
---
arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi | 19 +++++++++++++++++--
1 file changed, 17 insertions(+), 2 deletions(-)
diff --git a/arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi b/arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi
index a16358266a2..6bf6136c5fd 100644
--- a/arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi
+++ b/arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi
@@ -4,6 +4,7 @@
*/
#include <dt-bindings/clock/stm32mp1-clksrc.h>
+#include <dt-bindings/input/linux-event-codes.h>
#include "stm32mp15-u-boot.dtsi"
#include "stm32mp15-ddr3-1x4Gb-1066-binG.dtsi"
@@ -18,8 +19,6 @@
u-boot,error-led = "error";
u-boot,mmc-env-partition = "fip";
st,adc_usb_pd = <&adc1 18>, <&adc1 19>;
- st,fastboot-gpios = <&gpioa 13 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>;
- st,stm32prog-gpios = <&gpioa 14 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>;
};
#if defined(CONFIG_STM32MP15X_STM32IMAGE) || defined(CONFIG_SPL)
@@ -48,6 +47,22 @@
};
#endif
+ gpio-keys {
+ compatible = "gpio-keys";
+
+ button-user-1 {
+ label = "User-1";
+ linux,code = <BTN_1>;
+ gpios = <&gpioa 14 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>;
+ };
+
+ button-user-2 {
+ label = "User-2";
+ linux,code = <BTN_2>;
+ gpios = <&gpioa 13 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>;
+ };
+ };
+
led {
red {
label = "error";
--
2.25.1
^ permalink raw reply related [flat|nested] 79+ messages in thread
* [PATCH v1 15/25] ARM: dts: stm32: Don't probe red led at boot for stm32mp157a-dk1-u-boot
2024-04-09 15:01 [PATCH v1 00/25] stm32mp: Fix keys & leds management Patrice Chotard
` (13 preceding siblings ...)
2024-04-09 15:02 ` [PATCH v1 14/25] ARM: dts: stm32: Add gpio-keys for stm32mp157a-dk1-u-boot Patrice Chotard
@ 2024-04-09 15:02 ` Patrice Chotard
2024-04-17 9:09 ` Patrick DELAUNAY
2024-04-09 15:02 ` [PATCH v1 16/25] ARM: dts: stm32: Update red led node " Patrice Chotard
` (9 subsequent siblings)
24 siblings, 1 reply; 79+ messages in thread
From: Patrice Chotard @ 2024-04-09 15:02 UTC (permalink / raw)
To: u-boot
Cc: Patrice CHOTARD, Patrick DELAUNAY, U-Boot STM32, Simon Glass,
Tom Rini
red led and button dedicated to fastboot share the same gpio GPIOA13.
Led driver is probed early so the corresponding gpio is taken and
configured in output which forbid fastboot and stm32prog button usage.
To avoid this, remove the "default-state" property from red led node.
This will avoid to trigger the led driver probe() to configure the led
default state during startup.
Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
---
arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi b/arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi
index 6bf6136c5fd..ee9b51d42b7 100644
--- a/arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi
+++ b/arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi
@@ -67,7 +67,6 @@
red {
label = "error";
gpios = <&gpioa 13 GPIO_ACTIVE_LOW>;
- default-state = "off";
status = "okay";
};
};
--
2.25.1
^ permalink raw reply related [flat|nested] 79+ messages in thread
* [PATCH v1 16/25] ARM: dts: stm32: Update red led node for stm32mp157a-dk1-u-boot
2024-04-09 15:01 [PATCH v1 00/25] stm32mp: Fix keys & leds management Patrice Chotard
` (14 preceding siblings ...)
2024-04-09 15:02 ` [PATCH v1 15/25] ARM: dts: stm32: Don't probe red led at boot " Patrice Chotard
@ 2024-04-09 15:02 ` Patrice Chotard
2024-04-17 9:10 ` Patrick DELAUNAY
2024-04-09 15:02 ` [PATCH v1 17/25] ARM: dts: stm32: Update u-boot, boot-led " Patrice Chotard
` (8 subsequent siblings)
24 siblings, 1 reply; 79+ messages in thread
From: Patrice Chotard @ 2024-04-09 15:02 UTC (permalink / raw)
To: u-boot
Cc: Patrice CHOTARD, Patrick DELAUNAY, U-Boot STM32, Simon Glass,
Tom Rini
As indicated in kernel led dt-bindings, label is a deprecated
property, so remove it and use red led node's name instead
for u-boot,error-led property.
Rename red led node's name to led-red.
Remove status property which is useless.
Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
---
arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi b/arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi
index ee9b51d42b7..92153b2a657 100644
--- a/arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi
+++ b/arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi
@@ -16,7 +16,7 @@
config {
u-boot,boot-led = "heartbeat";
- u-boot,error-led = "error";
+ u-boot,error-led = "led-red";
u-boot,mmc-env-partition = "fip";
st,adc_usb_pd = <&adc1 18>, <&adc1 19>;
};
@@ -64,10 +64,8 @@
};
led {
- red {
- label = "error";
+ led-red {
gpios = <&gpioa 13 GPIO_ACTIVE_LOW>;
- status = "okay";
};
};
};
--
2.25.1
^ permalink raw reply related [flat|nested] 79+ messages in thread
* [PATCH v1 17/25] ARM: dts: stm32: Update u-boot, boot-led for stm32mp157a-dk1-u-boot
2024-04-09 15:01 [PATCH v1 00/25] stm32mp: Fix keys & leds management Patrice Chotard
` (15 preceding siblings ...)
2024-04-09 15:02 ` [PATCH v1 16/25] ARM: dts: stm32: Update red led node " Patrice Chotard
@ 2024-04-09 15:02 ` Patrice Chotard
2024-04-17 9:10 ` [PATCH v1 17/25] ARM: dts: stm32: Update u-boot,boot-led " Patrick DELAUNAY
2024-04-09 15:02 ` [PATCH v1 18/25] ARM: dts: stm32: Add gpio-keys for stm32mp157c-ed1-u-boot Patrice Chotard
` (7 subsequent siblings)
24 siblings, 1 reply; 79+ messages in thread
From: Patrice Chotard @ 2024-04-09 15:02 UTC (permalink / raw)
To: u-boot
Cc: Patrice CHOTARD, Patrick DELAUNAY, U-Boot STM32, Simon Glass,
Tom Rini
As indicated in kernel led dt-bindings, label is a deprecated
property, so remove it and use blue led node's name instead
for u-boot,boot-led property.
Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
---
arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi b/arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi
index 92153b2a657..f97debaa0e4 100644
--- a/arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi
+++ b/arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi
@@ -15,7 +15,7 @@
};
config {
- u-boot,boot-led = "heartbeat";
+ u-boot,boot-led = "led-blue";
u-boot,error-led = "led-red";
u-boot,mmc-env-partition = "fip";
st,adc_usb_pd = <&adc1 18>, <&adc1 19>;
@@ -64,6 +64,10 @@
};
led {
+ led-blue {
+ /delete-property/label;
+ };
+
led-red {
gpios = <&gpioa 13 GPIO_ACTIVE_LOW>;
};
--
2.25.1
^ permalink raw reply related [flat|nested] 79+ messages in thread
* [PATCH v1 18/25] ARM: dts: stm32: Add gpio-keys for stm32mp157c-ed1-u-boot
2024-04-09 15:01 [PATCH v1 00/25] stm32mp: Fix keys & leds management Patrice Chotard
` (16 preceding siblings ...)
2024-04-09 15:02 ` [PATCH v1 17/25] ARM: dts: stm32: Update u-boot, boot-led " Patrice Chotard
@ 2024-04-09 15:02 ` Patrice Chotard
2024-04-17 9:11 ` Patrick DELAUNAY
2024-04-09 15:02 ` [PATCH v1 19/25] ARM: dts: stm32: Don't probe red led at boot " Patrice Chotard
` (6 subsequent siblings)
24 siblings, 1 reply; 79+ messages in thread
From: Patrice Chotard @ 2024-04-09 15:02 UTC (permalink / raw)
To: u-boot
Cc: Patrice CHOTARD, Patrick DELAUNAY, U-Boot STM32, Simon Glass,
Tom Rini
Add 2 gpio-keys :
_ button-user-1 for stm32prog mode activation.
_ button-user-2 for fastboot mode activation.
Remove proprietary st,fastboot-gpios and st,stm32prog-gpios.
Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
---
arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi | 19 +++++++++++++++++--
1 file changed, 17 insertions(+), 2 deletions(-)
diff --git a/arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi b/arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi
index ef91088aa37..e6a452c9d8e 100644
--- a/arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi
+++ b/arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi
@@ -4,6 +4,7 @@
*/
#include <dt-bindings/clock/stm32mp1-clksrc.h>
+#include <dt-bindings/input/linux-event-codes.h>
#include "stm32mp15-u-boot.dtsi"
#include "stm32mp15-ddr3-2x4Gb-1066-binG.dtsi"
@@ -16,8 +17,6 @@
u-boot,boot-led = "heartbeat";
u-boot,error-led = "error";
u-boot,mmc-env-partition = "fip";
- st,fastboot-gpios = <&gpioa 13 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>;
- st,stm32prog-gpios = <&gpioa 14 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>;
};
#if defined(CONFIG_STM32MP15X_STM32IMAGE) || defined(CONFIG_SPL)
@@ -43,6 +42,22 @@
};
#endif
+ gpio-keys {
+ compatible = "gpio-keys";
+
+ button-user-1 {
+ label = "User-1";
+ linux,code = <BTN_1>;
+ gpios = <&gpioa 14 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>;
+ };
+
+ button-user-2 {
+ label = "User-2";
+ linux,code = <BTN_2>;
+ gpios = <&gpioa 13 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>;
+ };
+ };
+
led {
red {
label = "error";
--
2.25.1
^ permalink raw reply related [flat|nested] 79+ messages in thread
* [PATCH v1 19/25] ARM: dts: stm32: Don't probe red led at boot for stm32mp157c-ed1-u-boot
2024-04-09 15:01 [PATCH v1 00/25] stm32mp: Fix keys & leds management Patrice Chotard
` (17 preceding siblings ...)
2024-04-09 15:02 ` [PATCH v1 18/25] ARM: dts: stm32: Add gpio-keys for stm32mp157c-ed1-u-boot Patrice Chotard
@ 2024-04-09 15:02 ` Patrice Chotard
2024-04-17 9:11 ` Patrick DELAUNAY
2024-04-09 15:02 ` [PATCH v1 20/25] ARM: dts: stm32: Update red led node " Patrice Chotard
` (5 subsequent siblings)
24 siblings, 1 reply; 79+ messages in thread
From: Patrice Chotard @ 2024-04-09 15:02 UTC (permalink / raw)
To: u-boot
Cc: Patrice CHOTARD, Patrick DELAUNAY, U-Boot STM32, Simon Glass,
Tom Rini
red led and button dedicated to fastboot share the same gpio GPIOA13.
Led driver is probed early so the corresponding gpio is taken and
configured in output which forbid fastboot and stm32prog button usage.
To avoid this, remove the "default-state" property from red led node.
This will avoid to trigger the led driver probe() to configure the led
default state during startup.
Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
---
arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi b/arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi
index e6a452c9d8e..edcbfee0fc7 100644
--- a/arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi
+++ b/arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi
@@ -62,7 +62,6 @@
red {
label = "error";
gpios = <&gpioa 13 GPIO_ACTIVE_LOW>;
- default-state = "off";
status = "okay";
};
};
--
2.25.1
^ permalink raw reply related [flat|nested] 79+ messages in thread
* [PATCH v1 20/25] ARM: dts: stm32: Update red led node for stm32mp157c-ed1-u-boot
2024-04-09 15:01 [PATCH v1 00/25] stm32mp: Fix keys & leds management Patrice Chotard
` (18 preceding siblings ...)
2024-04-09 15:02 ` [PATCH v1 19/25] ARM: dts: stm32: Don't probe red led at boot " Patrice Chotard
@ 2024-04-09 15:02 ` Patrice Chotard
2024-04-17 9:11 ` Patrick DELAUNAY
2024-04-09 15:02 ` [PATCH v1 21/25] ARM: dts: stm32: Add led-blue " Patrice Chotard
` (4 subsequent siblings)
24 siblings, 1 reply; 79+ messages in thread
From: Patrice Chotard @ 2024-04-09 15:02 UTC (permalink / raw)
To: u-boot
Cc: Patrice CHOTARD, Patrick DELAUNAY, U-Boot STM32, Simon Glass,
Tom Rini
As indicated in kernel led dt-bindings, label is a deprecated
property, so remove it and use led node's name instead for
u-boot,error-led property.
Rename red led node's name to led-red.
Remove status property which is useless.
Add compatible = "gpio-leds" which is not present in kernel DT.
Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
---
arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi b/arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi
index edcbfee0fc7..09cfe9bc246 100644
--- a/arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi
+++ b/arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi
@@ -15,7 +15,7 @@
config {
u-boot,boot-led = "heartbeat";
- u-boot,error-led = "error";
+ u-boot,error-led = "led-red";
u-boot,mmc-env-partition = "fip";
};
@@ -59,10 +59,10 @@
};
led {
- red {
- label = "error";
+ compatible = "gpio-leds";
+
+ led-red {
gpios = <&gpioa 13 GPIO_ACTIVE_LOW>;
- status = "okay";
};
};
};
--
2.25.1
^ permalink raw reply related [flat|nested] 79+ messages in thread
* [PATCH v1 21/25] ARM: dts: stm32: Add led-blue for stm32mp157c-ed1-u-boot
2024-04-09 15:01 [PATCH v1 00/25] stm32mp: Fix keys & leds management Patrice Chotard
` (19 preceding siblings ...)
2024-04-09 15:02 ` [PATCH v1 20/25] ARM: dts: stm32: Update red led node " Patrice Chotard
@ 2024-04-09 15:02 ` Patrice Chotard
2024-04-17 9:11 ` Patrick DELAUNAY
2024-04-09 15:02 ` [PATCH v1 22/25] ARM: dts: stm32: Add gpio-keys for stm32mp157c-ed1-scmi-u-boot Patrice Chotard
` (3 subsequent siblings)
24 siblings, 1 reply; 79+ messages in thread
From: Patrice Chotard @ 2024-04-09 15:02 UTC (permalink / raw)
To: u-boot
Cc: Patrice CHOTARD, Patrick DELAUNAY, U-Boot STM32, Simon Glass,
Tom Rini
The blue led is used to indicate U-Boot entering / exit indication
then Linux heartbeat.
Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
---
arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi b/arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi
index 09cfe9bc246..d93359f967c 100644
--- a/arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi
+++ b/arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi
@@ -14,7 +14,7 @@
};
config {
- u-boot,boot-led = "heartbeat";
+ u-boot,boot-led = "led-blue";
u-boot,error-led = "led-red";
u-boot,mmc-env-partition = "fip";
};
@@ -61,6 +61,10 @@
led {
compatible = "gpio-leds";
+ led-blue {
+ gpios = <&gpiod 9 GPIO_ACTIVE_HIGH>;
+ };
+
led-red {
gpios = <&gpioa 13 GPIO_ACTIVE_LOW>;
};
--
2.25.1
^ permalink raw reply related [flat|nested] 79+ messages in thread
* [PATCH v1 22/25] ARM: dts: stm32: Add gpio-keys for stm32mp157c-ed1-scmi-u-boot
2024-04-09 15:01 [PATCH v1 00/25] stm32mp: Fix keys & leds management Patrice Chotard
` (20 preceding siblings ...)
2024-04-09 15:02 ` [PATCH v1 21/25] ARM: dts: stm32: Add led-blue " Patrice Chotard
@ 2024-04-09 15:02 ` Patrice Chotard
2024-04-17 9:12 ` Patrick DELAUNAY
2024-04-09 15:02 ` [PATCH v1 23/25] ARM: dts: stm32: Don't probe red led at boot " Patrice Chotard
` (2 subsequent siblings)
24 siblings, 1 reply; 79+ messages in thread
From: Patrice Chotard @ 2024-04-09 15:02 UTC (permalink / raw)
To: u-boot; +Cc: Patrice CHOTARD, Patrick DELAUNAY, U-Boot STM32, Tom Rini
Add 2 gpio-keys :
_ button-user-1 for stm32prog mode activation.
_ button-user-2 for fastboot mode activation.
Remove proprietary st,fastboot-gpios and st,stm32prog-gpios.
Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
---
arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi | 19 +++++++++++++++++--
1 file changed, 17 insertions(+), 2 deletions(-)
diff --git a/arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi b/arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi
index 4d763bd3a2c..84920f53496 100644
--- a/arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi
+++ b/arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi
@@ -3,6 +3,7 @@
* Copyright : STMicroelectronics 2022
*/
+#include <dt-bindings/input/linux-event-codes.h>
#include "stm32mp15-scmi-u-boot.dtsi"
/ {
@@ -14,8 +15,22 @@
u-boot,boot-led = "heartbeat";
u-boot,error-led = "error";
u-boot,mmc-env-partition = "u-boot-env";
- st,fastboot-gpios = <&gpioa 13 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>;
- st,stm32prog-gpios = <&gpioa 14 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>;
+ };
+
+ gpio-keys {
+ compatible = "gpio-keys";
+
+ button-user-1 {
+ label = "User-1";
+ linux,code = <BTN_1>;
+ gpios = <&gpioa 14 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>;
+ };
+
+ button-user-2 {
+ label = "User-2";
+ linux,code = <BTN_2>;
+ gpios = <&gpioa 13 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>;
+ };
};
led {
--
2.25.1
^ permalink raw reply related [flat|nested] 79+ messages in thread
* [PATCH v1 23/25] ARM: dts: stm32: Don't probe red led at boot for stm32mp157c-ed1-scmi-u-boot
2024-04-09 15:01 [PATCH v1 00/25] stm32mp: Fix keys & leds management Patrice Chotard
` (21 preceding siblings ...)
2024-04-09 15:02 ` [PATCH v1 22/25] ARM: dts: stm32: Add gpio-keys for stm32mp157c-ed1-scmi-u-boot Patrice Chotard
@ 2024-04-09 15:02 ` Patrice Chotard
2024-04-17 9:13 ` Patrick DELAUNAY
2024-04-19 12:30 ` Patrice CHOTARD
2024-04-09 15:02 ` [PATCH v1 24/25] ARM: dts: stm32: Update red led node " Patrice Chotard
2024-04-09 15:02 ` [PATCH v1 25/25] ARM: dts: stm32: Add led-blue " Patrice Chotard
24 siblings, 2 replies; 79+ messages in thread
From: Patrice Chotard @ 2024-04-09 15:02 UTC (permalink / raw)
To: u-boot; +Cc: Patrice CHOTARD, Patrick DELAUNAY, U-Boot STM32, Tom Rini
red led and button dedicated to fastboot share the same gpio GPIOA13.
Led driver is probed early so the corresponding gpio is taken and
configured in output which forbid fastboot and stm32prog button usage.
To avoid this, remove the "default-state" property from red led node.
This will avoid to trigger the led driver probe() to configure the led
default state during startup.
Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
---
arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi b/arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi
index 84920f53496..2abd512e297 100644
--- a/arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi
+++ b/arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi
@@ -37,7 +37,6 @@
red {
label = "error";
gpios = <&gpioa 13 GPIO_ACTIVE_LOW>;
- default-state = "off";
status = "okay";
};
};
--
2.25.1
^ permalink raw reply related [flat|nested] 79+ messages in thread
* [PATCH v1 24/25] ARM: dts: stm32: Update red led node for stm32mp157c-ed1-scmi-u-boot
2024-04-09 15:01 [PATCH v1 00/25] stm32mp: Fix keys & leds management Patrice Chotard
` (22 preceding siblings ...)
2024-04-09 15:02 ` [PATCH v1 23/25] ARM: dts: stm32: Don't probe red led at boot " Patrice Chotard
@ 2024-04-09 15:02 ` Patrice Chotard
2024-04-17 9:13 ` Patrick DELAUNAY
2024-04-09 15:02 ` [PATCH v1 25/25] ARM: dts: stm32: Add led-blue " Patrice Chotard
24 siblings, 1 reply; 79+ messages in thread
From: Patrice Chotard @ 2024-04-09 15:02 UTC (permalink / raw)
To: u-boot; +Cc: Patrice CHOTARD, Patrick DELAUNAY, U-Boot STM32, Tom Rini
As indicated in kernel led dt-bindings, label is a deprecated
property, so remove it and use led node's name instead for
u-boot,error-led property.
Rename red led node's name to led-red.
Remove status property which is useless.
Add compatible = "gpio-leds"; which is not present in kernel DT.
Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
---
arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi b/arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi
index 2abd512e297..b38e816f888 100644
--- a/arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi
+++ b/arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi
@@ -13,7 +13,7 @@
config {
u-boot,boot-led = "heartbeat";
- u-boot,error-led = "error";
+ u-boot,error-led = "led-red";
u-boot,mmc-env-partition = "u-boot-env";
};
@@ -34,10 +34,10 @@
};
led {
- red {
- label = "error";
+ compatible = "gpio-leds";
+
+ led-red {
gpios = <&gpioa 13 GPIO_ACTIVE_LOW>;
- status = "okay";
};
};
};
--
2.25.1
^ permalink raw reply related [flat|nested] 79+ messages in thread
* [PATCH v1 25/25] ARM: dts: stm32: Add led-blue for stm32mp157c-ed1-scmi-u-boot
2024-04-09 15:01 [PATCH v1 00/25] stm32mp: Fix keys & leds management Patrice Chotard
` (23 preceding siblings ...)
2024-04-09 15:02 ` [PATCH v1 24/25] ARM: dts: stm32: Update red led node " Patrice Chotard
@ 2024-04-09 15:02 ` Patrice Chotard
2024-04-17 9:28 ` Patrick DELAUNAY
24 siblings, 1 reply; 79+ messages in thread
From: Patrice Chotard @ 2024-04-09 15:02 UTC (permalink / raw)
To: u-boot; +Cc: Patrice CHOTARD, Patrick DELAUNAY, U-Boot STM32, Tom Rini
The blue led is used to indicate U-Boot entering / exit indication
then Linux heartbeat.
Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
---
arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi b/arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi
index b38e816f888..7c0d1bab11a 100644
--- a/arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi
+++ b/arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi
@@ -12,7 +12,7 @@
};
config {
- u-boot,boot-led = "heartbeat";
+ u-boot,boot-led = "led-blue";
u-boot,error-led = "led-red";
u-boot,mmc-env-partition = "u-boot-env";
};
@@ -36,6 +36,10 @@
led {
compatible = "gpio-leds";
+ led-blue {
+ gpios = <&gpiod 9 GPIO_ACTIVE_HIGH>;
+ };
+
led-red {
gpios = <&gpioa 13 GPIO_ACTIVE_LOW>;
};
--
2.25.1
^ permalink raw reply related [flat|nested] 79+ messages in thread
* Re: [PATCH v1 01/25] configs: stm32mp13: Enable FASTBOOT
2024-04-09 15:01 ` [PATCH v1 01/25] configs: stm32mp13: Enable FASTBOOT Patrice Chotard
@ 2024-04-17 9:03 ` Patrick DELAUNAY
2024-04-18 11:48 ` Igor Opaniuk
1 sibling, 0 replies; 79+ messages in thread
From: Patrick DELAUNAY @ 2024-04-17 9:03 UTC (permalink / raw)
To: Patrice Chotard, u-boot
Cc: U-Boot STM32, Gatien Chevallier, Igor Opaniuk, Simon Glass,
Tom Rini
Hi,
On 4/9/24 17:01, Patrice Chotard wrote:
> Enable FASTBOOT relative flags for stm32mp13_defconfig.
>
> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
>
> ---
>
> configs/stm32mp13_defconfig | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Thanks
Patrick
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 02/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp15_defconfig
2024-04-09 15:01 ` [PATCH v1 02/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp15_defconfig Patrice Chotard
@ 2024-04-17 9:03 ` Patrick DELAUNAY
2024-04-19 12:16 ` Patrice CHOTARD
0 siblings, 1 reply; 79+ messages in thread
From: Patrick DELAUNAY @ 2024-04-17 9:03 UTC (permalink / raw)
To: Patrice Chotard, u-boot
Cc: U-Boot STM32, Gatien Chevallier, Grzegorz Szymaszek, Simon Glass,
Tom Rini
Hi,
On 4/9/24 17:01, Patrice Chotard wrote:
> Enable BUTTON_GPIO flag for STM32MP15.
>
> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
> ---
>
> configs/stm32mp15_defconfig | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/configs/stm32mp15_defconfig b/configs/stm32mp15_defconfig
> index 3302b306996..ffe7512650e 100644
> --- a/configs/stm32mp15_defconfig
> +++ b/configs/stm32mp15_defconfig
> @@ -69,6 +69,8 @@ CONFIG_TFTP_TSIZE=y
> CONFIG_USE_SERVERIP=y
> CONFIG_SERVERIP="192.168.1.1"
> CONFIG_STM32_ADC=y
> +CONFIG_BUTTON=y
> +CONFIG_BUTTON_GPIO=y
> CONFIG_CLK_SCMI=y
> CONFIG_SET_DFU_ALT_INFO=y
> CONFIG_USB_FUNCTION_FASTBOOT=y
Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Thanks
Patrick
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 03/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp15_basic_defconfig
2024-04-09 15:01 ` [PATCH v1 03/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp15_basic_defconfig Patrice Chotard
@ 2024-04-17 9:04 ` Patrick DELAUNAY
2024-04-19 12:16 ` Patrice CHOTARD
0 siblings, 1 reply; 79+ messages in thread
From: Patrick DELAUNAY @ 2024-04-17 9:04 UTC (permalink / raw)
To: Patrice Chotard, u-boot
Cc: U-Boot STM32, Gatien Chevallier, Grzegorz Szymaszek,
Marcel Ziswiler, Sean Anderson, Simon Glass, Tom Rini
Hi,
On 4/9/24 17:01, Patrice Chotard wrote:
> Enable BUTTON_GPIO flag for STM32MP15.
>
> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
> ---
>
> configs/stm32mp15_basic_defconfig | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/configs/stm32mp15_basic_defconfig b/configs/stm32mp15_basic_defconfig
> index 005f1d55f80..2e22bf86000 100644
> --- a/configs/stm32mp15_basic_defconfig
> +++ b/configs/stm32mp15_basic_defconfig
> @@ -97,6 +97,8 @@ CONFIG_TFTP_TSIZE=y
> CONFIG_USE_SERVERIP=y
> CONFIG_SERVERIP="192.168.1.1"
> CONFIG_STM32_ADC=y
> +CONFIG_BUTTON=y
> +CONFIG_BUTTON_GPIO=y
> CONFIG_SET_DFU_ALT_INFO=y
> CONFIG_USB_FUNCTION_FASTBOOT=y
> CONFIG_FASTBOOT_BUF_ADDR=0xC0000000
Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Thanks
Patrick
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 04/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp15_trusted_defconfig
2024-04-09 15:01 ` [PATCH v1 04/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp15_trusted_defconfig Patrice Chotard
@ 2024-04-17 9:04 ` Patrick DELAUNAY
2024-04-19 12:16 ` Patrice CHOTARD
0 siblings, 1 reply; 79+ messages in thread
From: Patrick DELAUNAY @ 2024-04-17 9:04 UTC (permalink / raw)
To: Patrice Chotard, u-boot
Cc: U-Boot STM32, Gatien Chevallier, Grzegorz Szymaszek, Simon Glass,
Tom Rini
Hi,
On 4/9/24 17:01, Patrice Chotard wrote:
> Enable BUTTON_GPIO flag for STM32MP15.
>
> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
> ---
>
> configs/stm32mp15_trusted_defconfig | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/configs/stm32mp15_trusted_defconfig b/configs/stm32mp15_trusted_defconfig
> index 84b0854b557..74deaaba2e4 100644
> --- a/configs/stm32mp15_trusted_defconfig
> +++ b/configs/stm32mp15_trusted_defconfig
> @@ -70,6 +70,8 @@ CONFIG_TFTP_TSIZE=y
> CONFIG_USE_SERVERIP=y
> CONFIG_SERVERIP="192.168.1.1"
> CONFIG_STM32_ADC=y
> +CONFIG_BUTTON=y
> +CONFIG_BUTTON_GPIO=y
> CONFIG_CLK_SCMI=y
> CONFIG_SET_DFU_ALT_INFO=y
> CONFIG_USB_FUNCTION_FASTBOOT=y
Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Thanks
Patrick
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 05/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp13_defconfig
2024-04-09 15:01 ` [PATCH v1 05/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp13_defconfig Patrice Chotard
@ 2024-04-17 9:04 ` Patrick DELAUNAY
2024-04-18 11:48 ` Igor Opaniuk
1 sibling, 0 replies; 79+ messages in thread
From: Patrick DELAUNAY @ 2024-04-17 9:04 UTC (permalink / raw)
To: Patrice Chotard, u-boot
Cc: U-Boot STM32, Gatien Chevallier, Igor Opaniuk, Simon Glass,
Tom Rini
Hi,
On 4/9/24 17:01, Patrice Chotard wrote:
> Enable BUTTON_GPIO flag for STM32MP15.
>
> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
> ---
>
> configs/stm32mp13_defconfig | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/configs/stm32mp13_defconfig b/configs/stm32mp13_defconfig
> index db09e63100e..caaabf39ef3 100644
> --- a/configs/stm32mp13_defconfig
> +++ b/configs/stm32mp13_defconfig
> @@ -52,6 +52,8 @@ CONFIG_SYS_REDUNDAND_ENVIRONMENT=y
> CONFIG_SYS_RELOC_GD_ENV_ADDR=y
> CONFIG_SYS_MMC_ENV_DEV=-1
> CONFIG_ENV_MMC_USE_DT=y
> +CONFIG_BUTTON=y
> +CONFIG_BUTTON_GPIO=y
> CONFIG_CLK_SCMI=y
> CONFIG_SET_DFU_ALT_INFO=y
> CONFIG_USB_FUNCTION_FASTBOOT=y
Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Thanks
Patrick
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 06/25] board: st: stmp32mp1: Use BUTTON UCLASS in board_key_check()
2024-04-09 15:01 ` [PATCH v1 06/25] board: st: stmp32mp1: Use BUTTON UCLASS in board_key_check() Patrice Chotard
@ 2024-04-17 9:05 ` Patrick DELAUNAY
2024-04-19 12:16 ` Patrice CHOTARD
0 siblings, 1 reply; 79+ messages in thread
From: Patrick DELAUNAY @ 2024-04-17 9:05 UTC (permalink / raw)
To: Patrice Chotard, u-boot
Cc: U-Boot STM32, Masahisa Kojima, Rasmus Villemoes, Simon Glass,
Tom Rini
Hi,
On 4/9/24 17:01, Patrice Chotard wrote:
> Instead of using gpio directly to detect key pressed on button
> dedicated for fastboot and stm32mprog, make usage of BUTTON UCLASS.
>
> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
> ---
>
> board/st/stm32mp1/stm32mp1.c | 68 +++++++++++++++++++++---------------
> 1 file changed, 40 insertions(+), 28 deletions(-)
>
Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Thanks
Patrick
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 07/25] ARM: dts: stm32: Add gpio-keys for stm32mp135f-dk-u-boot
2024-04-09 15:01 ` [PATCH v1 07/25] ARM: dts: stm32: Add gpio-keys for stm32mp135f-dk-u-boot Patrice Chotard
@ 2024-04-17 9:06 ` Patrick DELAUNAY
2024-04-19 12:17 ` Patrice CHOTARD
0 siblings, 1 reply; 79+ messages in thread
From: Patrick DELAUNAY @ 2024-04-17 9:06 UTC (permalink / raw)
To: Patrice Chotard, u-boot; +Cc: U-Boot STM32, Fabrice Gasnier, Tom Rini
Hi
On 4/9/24 17:01, Patrice Chotard wrote:
> Add 2 gpio-keys :
> _ button-user-1 for stm32prog mode activation.
> _ update button-user's label (defined in kernel DT) to match label
> requested in board_key_check() for fastboot mode activation.
>
> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
> ---
>
> arch/arm/dts/stm32mp135f-dk-u-boot.dtsi | 13 +++++++++++++
> 1 file changed, 13 insertions(+)
>
Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Thanks
Patrick
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 08/25] ARM: dts: stm32: Don't probe led-red/led-blue at boot for stm32mp135f-dk-u-boot
2024-04-09 15:01 ` [PATCH v1 08/25] ARM: dts: stm32: Don't probe led-red/led-blue at boot " Patrice Chotard
@ 2024-04-17 9:06 ` Patrick DELAUNAY
2024-04-19 12:17 ` Patrice CHOTARD
0 siblings, 1 reply; 79+ messages in thread
From: Patrick DELAUNAY @ 2024-04-17 9:06 UTC (permalink / raw)
To: Patrice Chotard, u-boot; +Cc: U-Boot STM32, Fabrice Gasnier, Tom Rini
Hi,
On 4/9/24 17:01, Patrice Chotard wrote:
> led-red and button dedicated to fastboot share the same gpio GPIOA13.
> led-blue and button dedicated to stm32prog share the same gpio GPIOA14.
> Led driver is probed early so the corresponding gpio is taken and
> configured in output which forbid fastboot and stm32prog button usage.
>
> To avoid this, remove the "default-state" property from led-red and
> led-blue led's node.
>
> This will avoid to trigger the led driver probe() to configure the led
> default state during startup.
>
> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
> ---
>
> arch/arm/dts/stm32mp135f-dk-u-boot.dtsi | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Thanks
Patrick
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 09/25] ARM: dts: stm32: Clean led-red node for stm32mp135f-dk-u-boot
2024-04-09 15:01 ` [PATCH v1 09/25] ARM: dts: stm32: Clean led-red node " Patrice Chotard
@ 2024-04-17 9:07 ` Patrick DELAUNAY
2024-04-19 12:21 ` Patrice CHOTARD
0 siblings, 1 reply; 79+ messages in thread
From: Patrick DELAUNAY @ 2024-04-17 9:07 UTC (permalink / raw)
To: Patrice Chotard, u-boot; +Cc: U-Boot STM32, Fabrice Gasnier, Tom Rini
Hi,
On 4/9/24 17:01, Patrice Chotard wrote:
> Remove "color" property from led-red node which is not supported
> by U-Boot.
>
> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
> ---
>
> arch/arm/dts/stm32mp135f-dk-u-boot.dtsi | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/arm/dts/stm32mp135f-dk-u-boot.dtsi b/arch/arm/dts/stm32mp135f-dk-u-boot.dtsi
> index 8f42735609a..f004e9840a2 100644
> --- a/arch/arm/dts/stm32mp135f-dk-u-boot.dtsi
> +++ b/arch/arm/dts/stm32mp135f-dk-u-boot.dtsi
> @@ -35,7 +35,6 @@
> };
>
> led-red {
> - color = <LED_COLOR_ID_RED>;
> gpios = <&gpioa 13 GPIO_ACTIVE_LOW>;
> };
> };
Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Thanks
Patrick
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 10/25] ARM: dts: stm32: Add gpio-keys for stm32mp157a-dk1-scmi-u-boot
2024-04-09 15:02 ` [PATCH v1 10/25] ARM: dts: stm32: Add gpio-keys for stm32mp157a-dk1-scmi-u-boot Patrice Chotard
@ 2024-04-17 9:07 ` Patrick DELAUNAY
2024-04-19 12:21 ` Patrice CHOTARD
0 siblings, 1 reply; 79+ messages in thread
From: Patrick DELAUNAY @ 2024-04-17 9:07 UTC (permalink / raw)
To: Patrice Chotard, u-boot; +Cc: U-Boot STM32, Tom Rini
Hi,
On 4/9/24 17:02, Patrice Chotard wrote:
> Instead of using "st,fastboot-gpios" and "st,stm32prog-gpios", declare
> 2 gpio-keys.
>
> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
> ---
>
> arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi | 19 +++++++++++++++++--
> 1 file changed, 17 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi b/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
> index 20728f27ee1..5d49b09c35d 100644
> --- a/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
> +++ b/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
> @@ -3,6 +3,7 @@
> * Copyright : STMicroelectronics 2022
> */
>
> +#include <dt-bindings/input/linux-event-codes.h>
> #include "stm32mp15-scmi-u-boot.dtsi"
>
> / {
> @@ -16,8 +17,22 @@
> u-boot,error-led = "error";
> u-boot,mmc-env-partition = "u-boot-env";
> st,adc_usb_pd = <&adc1 18>, <&adc1 19>;
> - st,fastboot-gpios = <&gpioa 13 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>;
> - st,stm32prog-gpios = <&gpioa 14 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>;
> + };
> +
> + gpio-keys {
> + compatible = "gpio-keys";
> +
> + button-user-1 {
> + label = "User-1";
> + linux,code = <BTN_1>;
> + gpios = <&gpioa 14 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>;
> + };
> +
> + button-user-2 {
> + label = "User-2";
> + linux,code = <BTN_2>;
> + gpios = <&gpioa 13 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>;
> + };
> };
>
> led {
Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Thanks
Patrick
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 11/25] ARM: dts: stm32: Don't probe red led at boot for stm32mp157a-dk1-scmi-u-boot
2024-04-09 15:02 ` [PATCH v1 11/25] ARM: dts: stm32: Don't probe red led at boot " Patrice Chotard
@ 2024-04-17 9:07 ` Patrick DELAUNAY
2024-04-19 12:21 ` Patrice CHOTARD
0 siblings, 1 reply; 79+ messages in thread
From: Patrick DELAUNAY @ 2024-04-17 9:07 UTC (permalink / raw)
To: Patrice Chotard, u-boot; +Cc: U-Boot STM32, Tom Rini
Hi,
On 4/9/24 17:02, Patrice Chotard wrote:
> red led and button dedicated to fastboot share the same gpio GPIOA13.
> Led driver is probed early so the corresponding gpio is taken and
> configured in output which forbid fastboot and stm32prog button usage.
>
> To avoid this, remove the "default-state" property from red led node.
>
> This will avoid to trigger the led driver probe() to configure the led
> default state during startup.
>
> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
> ---
>
> arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi b/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
> index 5d49b09c35d..8760d6c7d93 100644
> --- a/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
> +++ b/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
> @@ -39,7 +39,6 @@
> red {
> label = "error";
> gpios = <&gpioa 13 GPIO_ACTIVE_LOW>;
> - default-state = "off";
> status = "okay";
> };
> };
Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Thanks
Patrick
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 12/25] ARM: dts: stm32: Update red led node for stm32mp157a-dk1-scmi-u-boot
2024-04-09 15:02 ` [PATCH v1 12/25] ARM: dts: stm32: Update red led node " Patrice Chotard
@ 2024-04-17 9:08 ` Patrick DELAUNAY
2024-04-19 12:21 ` Patrice CHOTARD
0 siblings, 1 reply; 79+ messages in thread
From: Patrick DELAUNAY @ 2024-04-17 9:08 UTC (permalink / raw)
To: Patrice Chotard, u-boot; +Cc: U-Boot STM32, Tom Rini
Hi,
On 4/9/24 17:02, Patrice Chotard wrote:
> As indicated in kernel led dt-bindings, label is a deprecated
> property, so remove it and use red led node's name instead
> for u-boot,error-led property.
> Rename "red" led node's name to "led-red".
> Remove status property which is useless.
>
> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
> ---
>
> arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi b/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
> index 8760d6c7d93..e61814fd66e 100644
> --- a/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
> +++ b/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
> @@ -14,7 +14,7 @@
>
> config {
> u-boot,boot-led = "heartbeat";
> - u-boot,error-led = "error";
> + u-boot,error-led = "led-red";
> u-boot,mmc-env-partition = "u-boot-env";
> st,adc_usb_pd = <&adc1 18>, <&adc1 19>;
> };
> @@ -36,10 +36,8 @@
> };
>
> led {
> - red {
> - label = "error";
> + led-red {
> gpios = <&gpioa 13 GPIO_ACTIVE_LOW>;
> - status = "okay";
> };
> };
> };
Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Thanks
Patrick
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 13/25] ARM: dts: stm32: Add led-blue for stm32mp157a-dk1-scmi-u-boot
2024-04-09 15:02 ` [PATCH v1 13/25] ARM: dts: stm32: Add led-blue " Patrice Chotard
@ 2024-04-17 9:09 ` Patrick DELAUNAY
2024-04-19 12:21 ` Patrice CHOTARD
0 siblings, 1 reply; 79+ messages in thread
From: Patrick DELAUNAY @ 2024-04-17 9:09 UTC (permalink / raw)
To: Patrice Chotard, u-boot; +Cc: U-Boot STM32, Tom Rini
Hi,
On 4/9/24 17:02, Patrice Chotard wrote:
> As indicated in kernel led dt-bindings, label is a deprecated
> property, so remove it and use blue led node's name instead
> for u-boot,boot-led property.
>
> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
> ---
>
> arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi b/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
> index e61814fd66e..a5158fec7ef 100644
> --- a/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
> +++ b/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
> @@ -13,7 +13,7 @@
> };
>
> config {
> - u-boot,boot-led = "heartbeat";
> + u-boot,boot-led = "led-blue";
> u-boot,error-led = "led-red";
> u-boot,mmc-env-partition = "u-boot-env";
> st,adc_usb_pd = <&adc1 18>, <&adc1 19>;
> @@ -36,6 +36,10 @@
> };
>
> led {
> + led-blue {
> + /delete-property/label;
> + };
> +
> led-red {
> gpios = <&gpioa 13 GPIO_ACTIVE_LOW>;
> };
Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Thanks
Patrick
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 14/25] ARM: dts: stm32: Add gpio-keys for stm32mp157a-dk1-u-boot
2024-04-09 15:02 ` [PATCH v1 14/25] ARM: dts: stm32: Add gpio-keys for stm32mp157a-dk1-u-boot Patrice Chotard
@ 2024-04-17 9:09 ` Patrick DELAUNAY
2024-04-19 12:26 ` Patrice CHOTARD
0 siblings, 1 reply; 79+ messages in thread
From: Patrick DELAUNAY @ 2024-04-17 9:09 UTC (permalink / raw)
To: Patrice Chotard, u-boot; +Cc: U-Boot STM32, Simon Glass, Tom Rini
Hi,
On 4/9/24 17:02, Patrice Chotard wrote:
> Instead of using "st,fastboot-gpios" and "st,stm32prog-gpios", declare
> 2 gpio-keys.
>
> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
> ---
>
> arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi | 19 +++++++++++++++++--
> 1 file changed, 17 insertions(+), 2 deletions(-)
>
Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Thanks
Patrick
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 15/25] ARM: dts: stm32: Don't probe red led at boot for stm32mp157a-dk1-u-boot
2024-04-09 15:02 ` [PATCH v1 15/25] ARM: dts: stm32: Don't probe red led at boot " Patrice Chotard
@ 2024-04-17 9:09 ` Patrick DELAUNAY
2024-04-19 12:27 ` Patrice CHOTARD
0 siblings, 1 reply; 79+ messages in thread
From: Patrick DELAUNAY @ 2024-04-17 9:09 UTC (permalink / raw)
To: Patrice Chotard, u-boot; +Cc: U-Boot STM32, Simon Glass, Tom Rini
Hi,
On 4/9/24 17:02, Patrice Chotard wrote:
> red led and button dedicated to fastboot share the same gpio GPIOA13.
> Led driver is probed early so the corresponding gpio is taken and
> configured in output which forbid fastboot and stm32prog button usage.
>
> To avoid this, remove the "default-state" property from red led node.
>
> This will avoid to trigger the led driver probe() to configure the led
> default state during startup.
>
> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
> ---
>
> arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi b/arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi
> index 6bf6136c5fd..ee9b51d42b7 100644
> --- a/arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi
> +++ b/arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi
> @@ -67,7 +67,6 @@
> red {
> label = "error";
> gpios = <&gpioa 13 GPIO_ACTIVE_LOW>;
> - default-state = "off";
> status = "okay";
> };
> };
Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Thanks
Patrick
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 16/25] ARM: dts: stm32: Update red led node for stm32mp157a-dk1-u-boot
2024-04-09 15:02 ` [PATCH v1 16/25] ARM: dts: stm32: Update red led node " Patrice Chotard
@ 2024-04-17 9:10 ` Patrick DELAUNAY
2024-04-19 12:27 ` Patrice CHOTARD
0 siblings, 1 reply; 79+ messages in thread
From: Patrick DELAUNAY @ 2024-04-17 9:10 UTC (permalink / raw)
To: Patrice Chotard, u-boot; +Cc: U-Boot STM32, Simon Glass, Tom Rini
Hi,
On 4/9/24 17:02, Patrice Chotard wrote:
> As indicated in kernel led dt-bindings, label is a deprecated
> property, so remove it and use red led node's name instead
> for u-boot,error-led property.
> Rename red led node's name to led-red.
> Remove status property which is useless.
>
> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
> ---
>
> arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Thanks
Patrick
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 17/25] ARM: dts: stm32: Update u-boot,boot-led for stm32mp157a-dk1-u-boot
2024-04-09 15:02 ` [PATCH v1 17/25] ARM: dts: stm32: Update u-boot, boot-led " Patrice Chotard
@ 2024-04-17 9:10 ` Patrick DELAUNAY
2024-04-19 12:27 ` Patrice CHOTARD
0 siblings, 1 reply; 79+ messages in thread
From: Patrick DELAUNAY @ 2024-04-17 9:10 UTC (permalink / raw)
To: Patrice Chotard, u-boot; +Cc: U-Boot STM32, Simon Glass, Tom Rini
Hi,
On 4/9/24 17:02, Patrice Chotard wrote:
> As indicated in kernel led dt-bindings, label is a deprecated
> property, so remove it and use blue led node's name instead
> for u-boot,boot-led property.
>
> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
> ---
>
> arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Thanks
Patrick
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 18/25] ARM: dts: stm32: Add gpio-keys for stm32mp157c-ed1-u-boot
2024-04-09 15:02 ` [PATCH v1 18/25] ARM: dts: stm32: Add gpio-keys for stm32mp157c-ed1-u-boot Patrice Chotard
@ 2024-04-17 9:11 ` Patrick DELAUNAY
2024-04-19 12:27 ` Patrice CHOTARD
0 siblings, 1 reply; 79+ messages in thread
From: Patrick DELAUNAY @ 2024-04-17 9:11 UTC (permalink / raw)
To: Patrice Chotard, u-boot; +Cc: U-Boot STM32, Simon Glass, Tom Rini
Hi,
On 4/9/24 17:02, Patrice Chotard wrote:
> Add 2 gpio-keys :
> _ button-user-1 for stm32prog mode activation.
> _ button-user-2 for fastboot mode activation.
>
> Remove proprietary st,fastboot-gpios and st,stm32prog-gpios.
>
> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
> ---
>
> arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi | 19 +++++++++++++++++--
> 1 file changed, 17 insertions(+), 2 deletions(-)
>
Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Thanks
Patrick
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 19/25] ARM: dts: stm32: Don't probe red led at boot for stm32mp157c-ed1-u-boot
2024-04-09 15:02 ` [PATCH v1 19/25] ARM: dts: stm32: Don't probe red led at boot " Patrice Chotard
@ 2024-04-17 9:11 ` Patrick DELAUNAY
2024-04-19 12:27 ` Patrice CHOTARD
0 siblings, 1 reply; 79+ messages in thread
From: Patrick DELAUNAY @ 2024-04-17 9:11 UTC (permalink / raw)
To: Patrice Chotard, u-boot; +Cc: U-Boot STM32, Simon Glass, Tom Rini
Hi,
On 4/9/24 17:02, Patrice Chotard wrote:
> red led and button dedicated to fastboot share the same gpio GPIOA13.
> Led driver is probed early so the corresponding gpio is taken and
> configured in output which forbid fastboot and stm32prog button usage.
>
> To avoid this, remove the "default-state" property from red led node.
>
> This will avoid to trigger the led driver probe() to configure the led
> default state during startup.
>
> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
> ---
>
> arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi | 1 -
> 1 file changed, 1 deletion(-)
>
Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Thanks
Patrick
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 20/25] ARM: dts: stm32: Update red led node for stm32mp157c-ed1-u-boot
2024-04-09 15:02 ` [PATCH v1 20/25] ARM: dts: stm32: Update red led node " Patrice Chotard
@ 2024-04-17 9:11 ` Patrick DELAUNAY
2024-04-19 12:27 ` Patrice CHOTARD
0 siblings, 1 reply; 79+ messages in thread
From: Patrick DELAUNAY @ 2024-04-17 9:11 UTC (permalink / raw)
To: Patrice Chotard, u-boot; +Cc: U-Boot STM32, Simon Glass, Tom Rini
Hi,
On 4/9/24 17:02, Patrice Chotard wrote:
> As indicated in kernel led dt-bindings, label is a deprecated
> property, so remove it and use led node's name instead for
> u-boot,error-led property.
> Rename red led node's name to led-red.
> Remove status property which is useless.
> Add compatible = "gpio-leds" which is not present in kernel DT.
>
> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
> ---
>
> arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Thanks
Patrick
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 21/25] ARM: dts: stm32: Add led-blue for stm32mp157c-ed1-u-boot
2024-04-09 15:02 ` [PATCH v1 21/25] ARM: dts: stm32: Add led-blue " Patrice Chotard
@ 2024-04-17 9:11 ` Patrick DELAUNAY
2024-04-19 12:30 ` Patrice CHOTARD
0 siblings, 1 reply; 79+ messages in thread
From: Patrick DELAUNAY @ 2024-04-17 9:11 UTC (permalink / raw)
To: Patrice Chotard, u-boot; +Cc: U-Boot STM32, Simon Glass, Tom Rini
Hi,
On 4/9/24 17:02, Patrice Chotard wrote:
> The blue led is used to indicate U-Boot entering / exit indication
> then Linux heartbeat.
>
> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
> ---
>
> arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Thanks
Patrick
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 22/25] ARM: dts: stm32: Add gpio-keys for stm32mp157c-ed1-scmi-u-boot
2024-04-09 15:02 ` [PATCH v1 22/25] ARM: dts: stm32: Add gpio-keys for stm32mp157c-ed1-scmi-u-boot Patrice Chotard
@ 2024-04-17 9:12 ` Patrick DELAUNAY
2024-04-19 12:30 ` Patrice CHOTARD
0 siblings, 1 reply; 79+ messages in thread
From: Patrick DELAUNAY @ 2024-04-17 9:12 UTC (permalink / raw)
To: Patrice Chotard, u-boot; +Cc: U-Boot STM32, Tom Rini
Hi,
On 4/9/24 17:02, Patrice Chotard wrote:
> Add 2 gpio-keys :
> _ button-user-1 for stm32prog mode activation.
> _ button-user-2 for fastboot mode activation.
>
> Remove proprietary st,fastboot-gpios and st,stm32prog-gpios.
>
> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
> ---
>
> arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi | 19 +++++++++++++++++--
> 1 file changed, 17 insertions(+), 2 deletions(-)
>
Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Thanks
Patrick
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 23/25] ARM: dts: stm32: Don't probe red led at boot for stm32mp157c-ed1-scmi-u-boot
2024-04-09 15:02 ` [PATCH v1 23/25] ARM: dts: stm32: Don't probe red led at boot " Patrice Chotard
@ 2024-04-17 9:13 ` Patrick DELAUNAY
2024-04-19 12:30 ` Patrice CHOTARD
2024-04-19 12:30 ` Patrice CHOTARD
1 sibling, 1 reply; 79+ messages in thread
From: Patrick DELAUNAY @ 2024-04-17 9:13 UTC (permalink / raw)
To: Patrice Chotard, u-boot; +Cc: U-Boot STM32, Tom Rini
Hi,
On 4/9/24 17:02, Patrice Chotard wrote:
> red led and button dedicated to fastboot share the same gpio GPIOA13.
> Led driver is probed early so the corresponding gpio is taken and
> configured in output which forbid fastboot and stm32prog button usage.
>
> To avoid this, remove the "default-state" property from red led node.
>
> This will avoid to trigger the led driver probe() to configure the led
> default state during startup.
>
> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
> ---
>
> arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi | 1 -
> 1 file changed, 1 deletion(-)
>
Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Thanks
Patrick
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 24/25] ARM: dts: stm32: Update red led node for stm32mp157c-ed1-scmi-u-boot
2024-04-09 15:02 ` [PATCH v1 24/25] ARM: dts: stm32: Update red led node " Patrice Chotard
@ 2024-04-17 9:13 ` Patrick DELAUNAY
2024-04-19 12:30 ` Patrice CHOTARD
0 siblings, 1 reply; 79+ messages in thread
From: Patrick DELAUNAY @ 2024-04-17 9:13 UTC (permalink / raw)
To: Patrice Chotard, u-boot; +Cc: U-Boot STM32, Tom Rini
Hi,
On 4/9/24 17:02, Patrice Chotard wrote:
> As indicated in kernel led dt-bindings, label is a deprecated
> property, so remove it and use led node's name instead for
> u-boot,error-led property.
> Rename red led node's name to led-red.
> Remove status property which is useless.
> Add compatible = "gpio-leds"; which is not present in kernel DT.
>
> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
> ---
>
> arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Thanks
Patrick
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 25/25] ARM: dts: stm32: Add led-blue for stm32mp157c-ed1-scmi-u-boot
2024-04-09 15:02 ` [PATCH v1 25/25] ARM: dts: stm32: Add led-blue " Patrice Chotard
@ 2024-04-17 9:28 ` Patrick DELAUNAY
2024-04-19 12:32 ` Patrice CHOTARD
0 siblings, 1 reply; 79+ messages in thread
From: Patrick DELAUNAY @ 2024-04-17 9:28 UTC (permalink / raw)
To: Patrice Chotard, u-boot; +Cc: U-Boot STM32, Tom Rini
Hi,
On 4/9/24 17:02, Patrice Chotard wrote:
> The blue led is used to indicate U-Boot entering / exit indication
> then Linux heartbeat.
>
> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
> ---
>
> arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Thanks
Patrick
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 01/25] configs: stm32mp13: Enable FASTBOOT
2024-04-09 15:01 ` [PATCH v1 01/25] configs: stm32mp13: Enable FASTBOOT Patrice Chotard
2024-04-17 9:03 ` Patrick DELAUNAY
@ 2024-04-18 11:48 ` Igor Opaniuk
2024-04-19 12:16 ` Patrice CHOTARD
1 sibling, 1 reply; 79+ messages in thread
From: Igor Opaniuk @ 2024-04-18 11:48 UTC (permalink / raw)
To: Patrice Chotard
Cc: u-boot, Patrick DELAUNAY, U-Boot STM32, Gatien Chevallier,
Igor Opaniuk, Simon Glass, Tom Rini
On Tue, Apr 9, 2024 at 5:19 PM Patrice Chotard
<patrice.chotard@foss.st.com> wrote:
>
> Enable FASTBOOT relative flags for stm32mp13_defconfig.
>
> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
>
> ---
>
> configs/stm32mp13_defconfig | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/configs/stm32mp13_defconfig b/configs/stm32mp13_defconfig
> index c893e272db9..db09e63100e 100644
> --- a/configs/stm32mp13_defconfig
> +++ b/configs/stm32mp13_defconfig
> @@ -15,6 +15,7 @@ CONFIG_CMD_STM32PROG=y
> CONFIG_SYS_LOAD_ADDR=0xc2000000
> CONFIG_SYS_MEMTEST_START=0xc0000000
> CONFIG_SYS_MEMTEST_END=0xc4000000
> +# CONFIG_ANDROID_BOOT_IMAGE is not set
> CONFIG_FIT=y
> CONFIG_SYS_BOOTM_LEN=0x2000000
> CONFIG_DISTRO_DEFAULTS=y
> @@ -53,6 +54,13 @@ CONFIG_SYS_MMC_ENV_DEV=-1
> CONFIG_ENV_MMC_USE_DT=y
> CONFIG_CLK_SCMI=y
> CONFIG_SET_DFU_ALT_INFO=y
> +CONFIG_USB_FUNCTION_FASTBOOT=y
> +CONFIG_FASTBOOT_BUF_ADDR=0xc0000000
> +CONFIG_FASTBOOT_BUF_SIZE=0x02000000
> +CONFIG_FASTBOOT_FLASH=y
> +CONFIG_FASTBOOT_FLASH_MMC_DEV=0
> +CONFIG_FASTBOOT_CMD_OEM_FORMAT=y
> +CONFIG_FASTBOOT_CMD_OEM_PARTCONF=y
> CONFIG_GPIO_HOG=y
> CONFIG_DM_I2C=y
> CONFIG_SYS_I2C_STM32F7=y
> @@ -92,7 +100,6 @@ CONFIG_USB_GADGET_MANUFACTURER="STMicroelectronics"
> CONFIG_USB_GADGET_VENDOR_NUM=0x0483
> CONFIG_USB_GADGET_PRODUCT_NUM=0x5720
> CONFIG_USB_GADGET_DWC2_OTG=y
> -CONFIG_USB_GADGET_DOWNLOAD=y
> CONFIG_ERRNO_STR=y
> # CONFIG_LMB_USE_MAX_REGIONS is not set
> CONFIG_LMB_MEMORY_REGIONS=2
> --
> 2.25.1
>
Reviewed-by: Igor Opaniuk <igor.opaniuk@gmail.com>
--
Best regards - Atentamente - Meilleures salutations
Igor Opaniuk
mailto: igor.opaniuk@gmail.com
skype: igor.opanyuk
https://www.linkedin.com/in/iopaniuk
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 05/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp13_defconfig
2024-04-09 15:01 ` [PATCH v1 05/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp13_defconfig Patrice Chotard
2024-04-17 9:04 ` Patrick DELAUNAY
@ 2024-04-18 11:48 ` Igor Opaniuk
2024-04-19 12:16 ` Patrice CHOTARD
1 sibling, 1 reply; 79+ messages in thread
From: Igor Opaniuk @ 2024-04-18 11:48 UTC (permalink / raw)
To: Patrice Chotard
Cc: u-boot, Patrick DELAUNAY, U-Boot STM32, Gatien Chevallier,
Igor Opaniuk, Simon Glass, Tom Rini
On Tue, Apr 9, 2024 at 5:05 PM Patrice Chotard
<patrice.chotard@foss.st.com> wrote:
>
> Enable BUTTON_GPIO flag for STM32MP15.
>
> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
> ---
>
> configs/stm32mp13_defconfig | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/configs/stm32mp13_defconfig b/configs/stm32mp13_defconfig
> index db09e63100e..caaabf39ef3 100644
> --- a/configs/stm32mp13_defconfig
> +++ b/configs/stm32mp13_defconfig
> @@ -52,6 +52,8 @@ CONFIG_SYS_REDUNDAND_ENVIRONMENT=y
> CONFIG_SYS_RELOC_GD_ENV_ADDR=y
> CONFIG_SYS_MMC_ENV_DEV=-1
> CONFIG_ENV_MMC_USE_DT=y
> +CONFIG_BUTTON=y
> +CONFIG_BUTTON_GPIO=y
> CONFIG_CLK_SCMI=y
> CONFIG_SET_DFU_ALT_INFO=y
> CONFIG_USB_FUNCTION_FASTBOOT=y
> --
> 2.25.1
>
Reviewed-by: Igor Opaniuk <igor.opaniuk@gmail.com>
--
Best regards - Atentamente - Meilleures salutations
Igor Opaniuk
mailto: igor.opaniuk@gmail.com
skype: igor.opanyuk
https://www.linkedin.com/in/iopaniuk
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 01/25] configs: stm32mp13: Enable FASTBOOT
2024-04-18 11:48 ` Igor Opaniuk
@ 2024-04-19 12:16 ` Patrice CHOTARD
0 siblings, 0 replies; 79+ messages in thread
From: Patrice CHOTARD @ 2024-04-19 12:16 UTC (permalink / raw)
To: Igor Opaniuk
Cc: u-boot, Patrick DELAUNAY, U-Boot STM32, Gatien Chevallier,
Igor Opaniuk, Simon Glass, Tom Rini
On 4/18/24 13:48, Igor Opaniuk wrote:
> On Tue, Apr 9, 2024 at 5:19 PM Patrice Chotard
> <patrice.chotard@foss.st.com> wrote:
>>
>> Enable FASTBOOT relative flags for stm32mp13_defconfig.
>>
>> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
>>
>> ---
>>
>> configs/stm32mp13_defconfig | 9 ++++++++-
>> 1 file changed, 8 insertions(+), 1 deletion(-)
>>
>> diff --git a/configs/stm32mp13_defconfig b/configs/stm32mp13_defconfig
>> index c893e272db9..db09e63100e 100644
>> --- a/configs/stm32mp13_defconfig
>> +++ b/configs/stm32mp13_defconfig
>> @@ -15,6 +15,7 @@ CONFIG_CMD_STM32PROG=y
>> CONFIG_SYS_LOAD_ADDR=0xc2000000
>> CONFIG_SYS_MEMTEST_START=0xc0000000
>> CONFIG_SYS_MEMTEST_END=0xc4000000
>> +# CONFIG_ANDROID_BOOT_IMAGE is not set
>> CONFIG_FIT=y
>> CONFIG_SYS_BOOTM_LEN=0x2000000
>> CONFIG_DISTRO_DEFAULTS=y
>> @@ -53,6 +54,13 @@ CONFIG_SYS_MMC_ENV_DEV=-1
>> CONFIG_ENV_MMC_USE_DT=y
>> CONFIG_CLK_SCMI=y
>> CONFIG_SET_DFU_ALT_INFO=y
>> +CONFIG_USB_FUNCTION_FASTBOOT=y
>> +CONFIG_FASTBOOT_BUF_ADDR=0xc0000000
>> +CONFIG_FASTBOOT_BUF_SIZE=0x02000000
>> +CONFIG_FASTBOOT_FLASH=y
>> +CONFIG_FASTBOOT_FLASH_MMC_DEV=0
>> +CONFIG_FASTBOOT_CMD_OEM_FORMAT=y
>> +CONFIG_FASTBOOT_CMD_OEM_PARTCONF=y
>> CONFIG_GPIO_HOG=y
>> CONFIG_DM_I2C=y
>> CONFIG_SYS_I2C_STM32F7=y
>> @@ -92,7 +100,6 @@ CONFIG_USB_GADGET_MANUFACTURER="STMicroelectronics"
>> CONFIG_USB_GADGET_VENDOR_NUM=0x0483
>> CONFIG_USB_GADGET_PRODUCT_NUM=0x5720
>> CONFIG_USB_GADGET_DWC2_OTG=y
>> -CONFIG_USB_GADGET_DOWNLOAD=y
>> CONFIG_ERRNO_STR=y
>> # CONFIG_LMB_USE_MAX_REGIONS is not set
>> CONFIG_LMB_MEMORY_REGIONS=2
>> --
>> 2.25.1
>>
> Reviewed-by: Igor Opaniuk <igor.opaniuk@gmail.com>
>
Applied on u-boot-stm32/master
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 02/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp15_defconfig
2024-04-17 9:03 ` Patrick DELAUNAY
@ 2024-04-19 12:16 ` Patrice CHOTARD
0 siblings, 0 replies; 79+ messages in thread
From: Patrice CHOTARD @ 2024-04-19 12:16 UTC (permalink / raw)
To: Patrick DELAUNAY, u-boot
Cc: U-Boot STM32, Gatien Chevallier, Grzegorz Szymaszek, Simon Glass,
Tom Rini
On 4/17/24 11:03, Patrick DELAUNAY wrote:
> Hi,
>
> On 4/9/24 17:01, Patrice Chotard wrote:
>> Enable BUTTON_GPIO flag for STM32MP15.
>>
>> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
>> ---
>>
>> configs/stm32mp15_defconfig | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/configs/stm32mp15_defconfig b/configs/stm32mp15_defconfig
>> index 3302b306996..ffe7512650e 100644
>> --- a/configs/stm32mp15_defconfig
>> +++ b/configs/stm32mp15_defconfig
>> @@ -69,6 +69,8 @@ CONFIG_TFTP_TSIZE=y
>> CONFIG_USE_SERVERIP=y
>> CONFIG_SERVERIP="192.168.1.1"
>> CONFIG_STM32_ADC=y
>> +CONFIG_BUTTON=y
>> +CONFIG_BUTTON_GPIO=y
>> CONFIG_CLK_SCMI=y
>> CONFIG_SET_DFU_ALT_INFO=y
>> CONFIG_USB_FUNCTION_FASTBOOT=y
>
>
>
> Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
>
> Thanks
> Patrick
>
Applied on u-boot-stm32/master
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 03/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp15_basic_defconfig
2024-04-17 9:04 ` Patrick DELAUNAY
@ 2024-04-19 12:16 ` Patrice CHOTARD
0 siblings, 0 replies; 79+ messages in thread
From: Patrice CHOTARD @ 2024-04-19 12:16 UTC (permalink / raw)
To: Patrick DELAUNAY, u-boot
Cc: U-Boot STM32, Gatien Chevallier, Grzegorz Szymaszek,
Marcel Ziswiler, Sean Anderson, Simon Glass, Tom Rini
On 4/17/24 11:04, Patrick DELAUNAY wrote:
> Hi,
>
> On 4/9/24 17:01, Patrice Chotard wrote:
>> Enable BUTTON_GPIO flag for STM32MP15.
>>
>> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
>> ---
>>
>> configs/stm32mp15_basic_defconfig | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/configs/stm32mp15_basic_defconfig b/configs/stm32mp15_basic_defconfig
>> index 005f1d55f80..2e22bf86000 100644
>> --- a/configs/stm32mp15_basic_defconfig
>> +++ b/configs/stm32mp15_basic_defconfig
>> @@ -97,6 +97,8 @@ CONFIG_TFTP_TSIZE=y
>> CONFIG_USE_SERVERIP=y
>> CONFIG_SERVERIP="192.168.1.1"
>> CONFIG_STM32_ADC=y
>> +CONFIG_BUTTON=y
>> +CONFIG_BUTTON_GPIO=y
>> CONFIG_SET_DFU_ALT_INFO=y
>> CONFIG_USB_FUNCTION_FASTBOOT=y
>> CONFIG_FASTBOOT_BUF_ADDR=0xC0000000
>
>
>
> Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
>
> Thanks
> Patrick
>
Applied on u-boot-stm32/master
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 04/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp15_trusted_defconfig
2024-04-17 9:04 ` Patrick DELAUNAY
@ 2024-04-19 12:16 ` Patrice CHOTARD
0 siblings, 0 replies; 79+ messages in thread
From: Patrice CHOTARD @ 2024-04-19 12:16 UTC (permalink / raw)
To: Patrick DELAUNAY, u-boot
Cc: U-Boot STM32, Gatien Chevallier, Grzegorz Szymaszek, Simon Glass,
Tom Rini
On 4/17/24 11:04, Patrick DELAUNAY wrote:
> Hi,
>
> On 4/9/24 17:01, Patrice Chotard wrote:
>> Enable BUTTON_GPIO flag for STM32MP15.
>>
>> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
>> ---
>>
>> configs/stm32mp15_trusted_defconfig | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/configs/stm32mp15_trusted_defconfig b/configs/stm32mp15_trusted_defconfig
>> index 84b0854b557..74deaaba2e4 100644
>> --- a/configs/stm32mp15_trusted_defconfig
>> +++ b/configs/stm32mp15_trusted_defconfig
>> @@ -70,6 +70,8 @@ CONFIG_TFTP_TSIZE=y
>> CONFIG_USE_SERVERIP=y
>> CONFIG_SERVERIP="192.168.1.1"
>> CONFIG_STM32_ADC=y
>> +CONFIG_BUTTON=y
>> +CONFIG_BUTTON_GPIO=y
>> CONFIG_CLK_SCMI=y
>> CONFIG_SET_DFU_ALT_INFO=y
>> CONFIG_USB_FUNCTION_FASTBOOT=y
>
> Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
>
> Thanks
> Patrick
>
Applied on u-boot-stm32/master
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 05/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp13_defconfig
2024-04-18 11:48 ` Igor Opaniuk
@ 2024-04-19 12:16 ` Patrice CHOTARD
0 siblings, 0 replies; 79+ messages in thread
From: Patrice CHOTARD @ 2024-04-19 12:16 UTC (permalink / raw)
To: Igor Opaniuk
Cc: u-boot, Patrick DELAUNAY, U-Boot STM32, Gatien Chevallier,
Igor Opaniuk, Simon Glass, Tom Rini
On 4/18/24 13:48, Igor Opaniuk wrote:
> On Tue, Apr 9, 2024 at 5:05 PM Patrice Chotard
> <patrice.chotard@foss.st.com> wrote:
>>
>> Enable BUTTON_GPIO flag for STM32MP15.
>>
>> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
>> ---
>>
>> configs/stm32mp13_defconfig | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/configs/stm32mp13_defconfig b/configs/stm32mp13_defconfig
>> index db09e63100e..caaabf39ef3 100644
>> --- a/configs/stm32mp13_defconfig
>> +++ b/configs/stm32mp13_defconfig
>> @@ -52,6 +52,8 @@ CONFIG_SYS_REDUNDAND_ENVIRONMENT=y
>> CONFIG_SYS_RELOC_GD_ENV_ADDR=y
>> CONFIG_SYS_MMC_ENV_DEV=-1
>> CONFIG_ENV_MMC_USE_DT=y
>> +CONFIG_BUTTON=y
>> +CONFIG_BUTTON_GPIO=y
>> CONFIG_CLK_SCMI=y
>> CONFIG_SET_DFU_ALT_INFO=y
>> CONFIG_USB_FUNCTION_FASTBOOT=y
>> --
>> 2.25.1
>>
>
> Reviewed-by: Igor Opaniuk <igor.opaniuk@gmail.com>
Applied on u-boot-stm32/master
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 06/25] board: st: stmp32mp1: Use BUTTON UCLASS in board_key_check()
2024-04-17 9:05 ` Patrick DELAUNAY
@ 2024-04-19 12:16 ` Patrice CHOTARD
0 siblings, 0 replies; 79+ messages in thread
From: Patrice CHOTARD @ 2024-04-19 12:16 UTC (permalink / raw)
To: Patrick DELAUNAY, u-boot
Cc: U-Boot STM32, Masahisa Kojima, Rasmus Villemoes, Simon Glass,
Tom Rini
On 4/17/24 11:05, Patrick DELAUNAY wrote:
> Hi,
>
> On 4/9/24 17:01, Patrice Chotard wrote:
>> Instead of using gpio directly to detect key pressed on button
>> dedicated for fastboot and stm32mprog, make usage of BUTTON UCLASS.
>>
>> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
>> ---
>>
>> board/st/stm32mp1/stm32mp1.c | 68 +++++++++++++++++++++---------------
>> 1 file changed, 40 insertions(+), 28 deletions(-)
>>
>
>
> Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
>
> Thanks
> Patrick
>
Applied on u-boot-stm32/master
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 07/25] ARM: dts: stm32: Add gpio-keys for stm32mp135f-dk-u-boot
2024-04-17 9:06 ` Patrick DELAUNAY
@ 2024-04-19 12:17 ` Patrice CHOTARD
0 siblings, 0 replies; 79+ messages in thread
From: Patrice CHOTARD @ 2024-04-19 12:17 UTC (permalink / raw)
To: Patrick DELAUNAY, u-boot; +Cc: U-Boot STM32, Fabrice Gasnier, Tom Rini
On 4/17/24 11:06, Patrick DELAUNAY wrote:
> Hi
>
> On 4/9/24 17:01, Patrice Chotard wrote:
>> Add 2 gpio-keys :
>> _ button-user-1 for stm32prog mode activation.
>> _ update button-user's label (defined in kernel DT) to match label
>> requested in board_key_check() for fastboot mode activation.
>>
>> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
>> ---
>>
>> arch/arm/dts/stm32mp135f-dk-u-boot.dtsi | 13 +++++++++++++
>> 1 file changed, 13 insertions(+)
>>
>
>
> Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
>
> Thanks
> Patrick
>
Applied on u-boot-stm32/master
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 08/25] ARM: dts: stm32: Don't probe led-red/led-blue at boot for stm32mp135f-dk-u-boot
2024-04-17 9:06 ` Patrick DELAUNAY
@ 2024-04-19 12:17 ` Patrice CHOTARD
0 siblings, 0 replies; 79+ messages in thread
From: Patrice CHOTARD @ 2024-04-19 12:17 UTC (permalink / raw)
To: Patrick DELAUNAY, u-boot; +Cc: U-Boot STM32, Fabrice Gasnier, Tom Rini
On 4/17/24 11:06, Patrick DELAUNAY wrote:
> Hi,
>
> On 4/9/24 17:01, Patrice Chotard wrote:
>> led-red and button dedicated to fastboot share the same gpio GPIOA13.
>> led-blue and button dedicated to stm32prog share the same gpio GPIOA14.
>> Led driver is probed early so the corresponding gpio is taken and
>> configured in output which forbid fastboot and stm32prog button usage.
>>
>> To avoid this, remove the "default-state" property from led-red and
>> led-blue led's node.
>>
>> This will avoid to trigger the led driver probe() to configure the led
>> default state during startup.
>>
>> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
>> ---
>>
>> arch/arm/dts/stm32mp135f-dk-u-boot.dtsi | 5 ++++-
>> 1 file changed, 4 insertions(+), 1 deletion(-)
>>
>
>
> Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
>
> Thanks
> Patrick
>
Applied on u-boot-stm32/master
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 09/25] ARM: dts: stm32: Clean led-red node for stm32mp135f-dk-u-boot
2024-04-17 9:07 ` Patrick DELAUNAY
@ 2024-04-19 12:21 ` Patrice CHOTARD
0 siblings, 0 replies; 79+ messages in thread
From: Patrice CHOTARD @ 2024-04-19 12:21 UTC (permalink / raw)
To: Patrick DELAUNAY, u-boot; +Cc: U-Boot STM32, Fabrice Gasnier, Tom Rini
On 4/17/24 11:07, Patrick DELAUNAY wrote:
> Hi,
>
> On 4/9/24 17:01, Patrice Chotard wrote:
>> Remove "color" property from led-red node which is not supported
>> by U-Boot.
>>
>> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
>> ---
>>
>> arch/arm/dts/stm32mp135f-dk-u-boot.dtsi | 1 -
>> 1 file changed, 1 deletion(-)
>>
>> diff --git a/arch/arm/dts/stm32mp135f-dk-u-boot.dtsi b/arch/arm/dts/stm32mp135f-dk-u-boot.dtsi
>> index 8f42735609a..f004e9840a2 100644
>> --- a/arch/arm/dts/stm32mp135f-dk-u-boot.dtsi
>> +++ b/arch/arm/dts/stm32mp135f-dk-u-boot.dtsi
>> @@ -35,7 +35,6 @@
>> };
>> led-red {
>> - color = <LED_COLOR_ID_RED>;
>> gpios = <&gpioa 13 GPIO_ACTIVE_LOW>;
>> };
>> };
>
>
>
> Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
>
> Thanks
> Patrick
>
Applied on u-boot-stm32/master
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 10/25] ARM: dts: stm32: Add gpio-keys for stm32mp157a-dk1-scmi-u-boot
2024-04-17 9:07 ` Patrick DELAUNAY
@ 2024-04-19 12:21 ` Patrice CHOTARD
0 siblings, 0 replies; 79+ messages in thread
From: Patrice CHOTARD @ 2024-04-19 12:21 UTC (permalink / raw)
To: Patrick DELAUNAY, u-boot; +Cc: U-Boot STM32, Tom Rini
On 4/17/24 11:07, Patrick DELAUNAY wrote:
> Hi,
>
> On 4/9/24 17:02, Patrice Chotard wrote:
>> Instead of using "st,fastboot-gpios" and "st,stm32prog-gpios", declare
>> 2 gpio-keys.
>>
>> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
>> ---
>>
>> arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi | 19 +++++++++++++++++--
>> 1 file changed, 17 insertions(+), 2 deletions(-)
>>
>> diff --git a/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi b/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
>> index 20728f27ee1..5d49b09c35d 100644
>> --- a/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
>> +++ b/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
>> @@ -3,6 +3,7 @@
>> * Copyright : STMicroelectronics 2022
>> */
>> +#include <dt-bindings/input/linux-event-codes.h>
>> #include "stm32mp15-scmi-u-boot.dtsi"
>> / {
>> @@ -16,8 +17,22 @@
>> u-boot,error-led = "error";
>> u-boot,mmc-env-partition = "u-boot-env";
>> st,adc_usb_pd = <&adc1 18>, <&adc1 19>;
>> - st,fastboot-gpios = <&gpioa 13 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>;
>> - st,stm32prog-gpios = <&gpioa 14 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>;
>> + };
>> +
>> + gpio-keys {
>> + compatible = "gpio-keys";
>> +
>> + button-user-1 {
>> + label = "User-1";
>> + linux,code = <BTN_1>;
>> + gpios = <&gpioa 14 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>;
>> + };
>> +
>> + button-user-2 {
>> + label = "User-2";
>> + linux,code = <BTN_2>;
>> + gpios = <&gpioa 13 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>;
>> + };
>> };
>> led {
>
>
>
> Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
>
> Thanks
> Patrick
>
Applied on u-boot-stm32/master
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 11/25] ARM: dts: stm32: Don't probe red led at boot for stm32mp157a-dk1-scmi-u-boot
2024-04-17 9:07 ` Patrick DELAUNAY
@ 2024-04-19 12:21 ` Patrice CHOTARD
0 siblings, 0 replies; 79+ messages in thread
From: Patrice CHOTARD @ 2024-04-19 12:21 UTC (permalink / raw)
To: Patrick DELAUNAY, u-boot; +Cc: U-Boot STM32, Tom Rini
On 4/17/24 11:07, Patrick DELAUNAY wrote:
> Hi,
>
> On 4/9/24 17:02, Patrice Chotard wrote:
>> red led and button dedicated to fastboot share the same gpio GPIOA13.
>> Led driver is probed early so the corresponding gpio is taken and
>> configured in output which forbid fastboot and stm32prog button usage.
>>
>> To avoid this, remove the "default-state" property from red led node.
>>
>> This will avoid to trigger the led driver probe() to configure the led
>> default state during startup.
>>
>> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
>> ---
>>
>> arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi | 1 -
>> 1 file changed, 1 deletion(-)
>>
>> diff --git a/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi b/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
>> index 5d49b09c35d..8760d6c7d93 100644
>> --- a/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
>> +++ b/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
>> @@ -39,7 +39,6 @@
>> red {
>> label = "error";
>> gpios = <&gpioa 13 GPIO_ACTIVE_LOW>;
>> - default-state = "off";
>> status = "okay";
>> };
>> };
>
>
>
> Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
>
> Thanks
> Patrick
>
Applied on u-boot-stm32/master
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 12/25] ARM: dts: stm32: Update red led node for stm32mp157a-dk1-scmi-u-boot
2024-04-17 9:08 ` Patrick DELAUNAY
@ 2024-04-19 12:21 ` Patrice CHOTARD
0 siblings, 0 replies; 79+ messages in thread
From: Patrice CHOTARD @ 2024-04-19 12:21 UTC (permalink / raw)
To: Patrick DELAUNAY, u-boot; +Cc: U-Boot STM32, Tom Rini
On 4/17/24 11:08, Patrick DELAUNAY wrote:
> Hi,
>
> On 4/9/24 17:02, Patrice Chotard wrote:
>> As indicated in kernel led dt-bindings, label is a deprecated
>> property, so remove it and use red led node's name instead
>> for u-boot,error-led property.
>> Rename "red" led node's name to "led-red".
>> Remove status property which is useless.
>>
>> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
>> ---
>>
>> arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi | 6 ++----
>> 1 file changed, 2 insertions(+), 4 deletions(-)
>>
>> diff --git a/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi b/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
>> index 8760d6c7d93..e61814fd66e 100644
>> --- a/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
>> +++ b/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
>> @@ -14,7 +14,7 @@
>> config {
>> u-boot,boot-led = "heartbeat";
>> - u-boot,error-led = "error";
>> + u-boot,error-led = "led-red";
>> u-boot,mmc-env-partition = "u-boot-env";
>> st,adc_usb_pd = <&adc1 18>, <&adc1 19>;
>> };
>> @@ -36,10 +36,8 @@
>> };
>> led {
>> - red {
>> - label = "error";
>> + led-red {
>> gpios = <&gpioa 13 GPIO_ACTIVE_LOW>;
>> - status = "okay";
>> };
>> };
>> };
>
>
>
> Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
>
> Thanks
> Patrick
>
Applied on u-boot-stm32/master
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 13/25] ARM: dts: stm32: Add led-blue for stm32mp157a-dk1-scmi-u-boot
2024-04-17 9:09 ` Patrick DELAUNAY
@ 2024-04-19 12:21 ` Patrice CHOTARD
0 siblings, 0 replies; 79+ messages in thread
From: Patrice CHOTARD @ 2024-04-19 12:21 UTC (permalink / raw)
To: Patrick DELAUNAY, u-boot; +Cc: U-Boot STM32, Tom Rini
On 4/17/24 11:09, Patrick DELAUNAY wrote:
> Hi,
>
> On 4/9/24 17:02, Patrice Chotard wrote:
>> As indicated in kernel led dt-bindings, label is a deprecated
>> property, so remove it and use blue led node's name instead
>> for u-boot,boot-led property.
>>
>> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
>> ---
>>
>> arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi | 6 +++++-
>> 1 file changed, 5 insertions(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi b/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
>> index e61814fd66e..a5158fec7ef 100644
>> --- a/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
>> +++ b/arch/arm/dts/stm32mp157a-dk1-scmi-u-boot.dtsi
>> @@ -13,7 +13,7 @@
>> };
>> config {
>> - u-boot,boot-led = "heartbeat";
>> + u-boot,boot-led = "led-blue";
>> u-boot,error-led = "led-red";
>> u-boot,mmc-env-partition = "u-boot-env";
>> st,adc_usb_pd = <&adc1 18>, <&adc1 19>;
>> @@ -36,6 +36,10 @@
>> };
>> led {
>> + led-blue {
>> + /delete-property/label;
>> + };
>> +
>> led-red {
>> gpios = <&gpioa 13 GPIO_ACTIVE_LOW>;
>> };
>
>
>
> Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
>
> Thanks
> Patrick
>
Applied on u-boot-stm32/master
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 14/25] ARM: dts: stm32: Add gpio-keys for stm32mp157a-dk1-u-boot
2024-04-17 9:09 ` Patrick DELAUNAY
@ 2024-04-19 12:26 ` Patrice CHOTARD
0 siblings, 0 replies; 79+ messages in thread
From: Patrice CHOTARD @ 2024-04-19 12:26 UTC (permalink / raw)
To: Patrick DELAUNAY, u-boot; +Cc: U-Boot STM32, Simon Glass, Tom Rini
On 4/17/24 11:09, Patrick DELAUNAY wrote:
> Hi,
>
> On 4/9/24 17:02, Patrice Chotard wrote:
>> Instead of using "st,fastboot-gpios" and "st,stm32prog-gpios", declare
>> 2 gpio-keys.
>>
>> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
>> ---
>>
>> arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi | 19 +++++++++++++++++--
>> 1 file changed, 17 insertions(+), 2 deletions(-)
>>
>
>
> Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
>
> Thanks
> Patrick
>
Applied on u-boot-stm32/master
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 15/25] ARM: dts: stm32: Don't probe red led at boot for stm32mp157a-dk1-u-boot
2024-04-17 9:09 ` Patrick DELAUNAY
@ 2024-04-19 12:27 ` Patrice CHOTARD
0 siblings, 0 replies; 79+ messages in thread
From: Patrice CHOTARD @ 2024-04-19 12:27 UTC (permalink / raw)
To: Patrick DELAUNAY, u-boot; +Cc: U-Boot STM32, Simon Glass, Tom Rini
On 4/17/24 11:09, Patrick DELAUNAY wrote:
> Hi,
>
> On 4/9/24 17:02, Patrice Chotard wrote:
>> red led and button dedicated to fastboot share the same gpio GPIOA13.
>> Led driver is probed early so the corresponding gpio is taken and
>> configured in output which forbid fastboot and stm32prog button usage.
>>
>> To avoid this, remove the "default-state" property from red led node.
>>
>> This will avoid to trigger the led driver probe() to configure the led
>> default state during startup.
>>
>> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
>> ---
>>
>> arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi | 1 -
>> 1 file changed, 1 deletion(-)
>>
>> diff --git a/arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi b/arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi
>> index 6bf6136c5fd..ee9b51d42b7 100644
>> --- a/arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi
>> +++ b/arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi
>> @@ -67,7 +67,6 @@
>> red {
>> label = "error";
>> gpios = <&gpioa 13 GPIO_ACTIVE_LOW>;
>> - default-state = "off";
>> status = "okay";
>> };
>> };
>
>
>
> Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
>
> Thanks
> Patrick
>
Applied on u-boot-stm32/master
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 16/25] ARM: dts: stm32: Update red led node for stm32mp157a-dk1-u-boot
2024-04-17 9:10 ` Patrick DELAUNAY
@ 2024-04-19 12:27 ` Patrice CHOTARD
0 siblings, 0 replies; 79+ messages in thread
From: Patrice CHOTARD @ 2024-04-19 12:27 UTC (permalink / raw)
To: Patrick DELAUNAY, u-boot; +Cc: U-Boot STM32, Simon Glass, Tom Rini
On 4/17/24 11:10, Patrick DELAUNAY wrote:
> Hi,
>
> On 4/9/24 17:02, Patrice Chotard wrote:
>> As indicated in kernel led dt-bindings, label is a deprecated
>> property, so remove it and use red led node's name instead
>> for u-boot,error-led property.
>> Rename red led node's name to led-red.
>> Remove status property which is useless.
>>
>> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
>> ---
>>
>> arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi | 6 ++----
>> 1 file changed, 2 insertions(+), 4 deletions(-)
>
>
>
> Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
>
> Thanks
> Patrick
>
Applied on u-boot-stm32/master
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 17/25] ARM: dts: stm32: Update u-boot,boot-led for stm32mp157a-dk1-u-boot
2024-04-17 9:10 ` [PATCH v1 17/25] ARM: dts: stm32: Update u-boot,boot-led " Patrick DELAUNAY
@ 2024-04-19 12:27 ` Patrice CHOTARD
0 siblings, 0 replies; 79+ messages in thread
From: Patrice CHOTARD @ 2024-04-19 12:27 UTC (permalink / raw)
To: Patrick DELAUNAY, u-boot; +Cc: U-Boot STM32, Simon Glass, Tom Rini
On 4/17/24 11:10, Patrick DELAUNAY wrote:
> Hi,
>
> On 4/9/24 17:02, Patrice Chotard wrote:
>> As indicated in kernel led dt-bindings, label is a deprecated
>> property, so remove it and use blue led node's name instead
>> for u-boot,boot-led property.
>>
>> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
>> ---
>>
>> arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi | 6 +++++-
>> 1 file changed, 5 insertions(+), 1 deletion(-)
>>
>
>
> Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
>
> Thanks
> Patrick
>
Applied on u-boot-stm32/master
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 18/25] ARM: dts: stm32: Add gpio-keys for stm32mp157c-ed1-u-boot
2024-04-17 9:11 ` Patrick DELAUNAY
@ 2024-04-19 12:27 ` Patrice CHOTARD
0 siblings, 0 replies; 79+ messages in thread
From: Patrice CHOTARD @ 2024-04-19 12:27 UTC (permalink / raw)
To: Patrick DELAUNAY, u-boot; +Cc: U-Boot STM32, Simon Glass, Tom Rini
On 4/17/24 11:11, Patrick DELAUNAY wrote:
> Hi,
>
> On 4/9/24 17:02, Patrice Chotard wrote:
>> Add 2 gpio-keys :
>> _ button-user-1 for stm32prog mode activation.
>> _ button-user-2 for fastboot mode activation.
>>
>> Remove proprietary st,fastboot-gpios and st,stm32prog-gpios.
>>
>> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
>> ---
>>
>> arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi | 19 +++++++++++++++++--
>> 1 file changed, 17 insertions(+), 2 deletions(-)
>>
>
> Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
>
> Thanks
> Patrick
>
Applied on u-boot-stm32/master
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 19/25] ARM: dts: stm32: Don't probe red led at boot for stm32mp157c-ed1-u-boot
2024-04-17 9:11 ` Patrick DELAUNAY
@ 2024-04-19 12:27 ` Patrice CHOTARD
0 siblings, 0 replies; 79+ messages in thread
From: Patrice CHOTARD @ 2024-04-19 12:27 UTC (permalink / raw)
To: Patrick DELAUNAY, u-boot; +Cc: U-Boot STM32, Simon Glass, Tom Rini
On 4/17/24 11:11, Patrick DELAUNAY wrote:
> Hi,
>
> On 4/9/24 17:02, Patrice Chotard wrote:
>> red led and button dedicated to fastboot share the same gpio GPIOA13.
>> Led driver is probed early so the corresponding gpio is taken and
>> configured in output which forbid fastboot and stm32prog button usage.
>>
>> To avoid this, remove the "default-state" property from red led node.
>>
>> This will avoid to trigger the led driver probe() to configure the led
>> default state during startup.
>>
>> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
>> ---
>>
>> arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi | 1 -
>> 1 file changed, 1 deletion(-)
>>
>
>
> Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
>
> Thanks
> Patrick
>
Applied on u-boot-stm32/master
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 20/25] ARM: dts: stm32: Update red led node for stm32mp157c-ed1-u-boot
2024-04-17 9:11 ` Patrick DELAUNAY
@ 2024-04-19 12:27 ` Patrice CHOTARD
0 siblings, 0 replies; 79+ messages in thread
From: Patrice CHOTARD @ 2024-04-19 12:27 UTC (permalink / raw)
To: Patrick DELAUNAY, u-boot; +Cc: U-Boot STM32, Simon Glass, Tom Rini
On 4/17/24 11:11, Patrick DELAUNAY wrote:
> Hi,
>
> On 4/9/24 17:02, Patrice Chotard wrote:
>> As indicated in kernel led dt-bindings, label is a deprecated
>> property, so remove it and use led node's name instead for
>> u-boot,error-led property.
>> Rename red led node's name to led-red.
>> Remove status property which is useless.
>> Add compatible = "gpio-leds" which is not present in kernel DT.
>>
>> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
>> ---
>>
>> arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi | 8 ++++----
>> 1 file changed, 4 insertions(+), 4 deletions(-)
>>
>
>
> Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
>
> Thanks
> Patrick
>
Applied on u-boot-stm32/master
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 21/25] ARM: dts: stm32: Add led-blue for stm32mp157c-ed1-u-boot
2024-04-17 9:11 ` Patrick DELAUNAY
@ 2024-04-19 12:30 ` Patrice CHOTARD
0 siblings, 0 replies; 79+ messages in thread
From: Patrice CHOTARD @ 2024-04-19 12:30 UTC (permalink / raw)
To: Patrick DELAUNAY, u-boot; +Cc: U-Boot STM32, Simon Glass, Tom Rini
On 4/17/24 11:11, Patrick DELAUNAY wrote:
> Hi,
>
> On 4/9/24 17:02, Patrice Chotard wrote:
>> The blue led is used to indicate U-Boot entering / exit indication
>> then Linux heartbeat.
>>
>> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
>> ---
>>
>> arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi | 6 +++++-
>> 1 file changed, 5 insertions(+), 1 deletion(-)
>>
>
>
> Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
>
> Thanks
> Patrick
>
Applied on u-boot-stm32/master
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 22/25] ARM: dts: stm32: Add gpio-keys for stm32mp157c-ed1-scmi-u-boot
2024-04-17 9:12 ` Patrick DELAUNAY
@ 2024-04-19 12:30 ` Patrice CHOTARD
0 siblings, 0 replies; 79+ messages in thread
From: Patrice CHOTARD @ 2024-04-19 12:30 UTC (permalink / raw)
To: Patrick DELAUNAY, u-boot; +Cc: U-Boot STM32, Tom Rini
On 4/17/24 11:12, Patrick DELAUNAY wrote:
> Hi,
>
> On 4/9/24 17:02, Patrice Chotard wrote:
>> Add 2 gpio-keys :
>> _ button-user-1 for stm32prog mode activation.
>> _ button-user-2 for fastboot mode activation.
>>
>> Remove proprietary st,fastboot-gpios and st,stm32prog-gpios.
>>
>> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
>> ---
>>
>> arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi | 19 +++++++++++++++++--
>> 1 file changed, 17 insertions(+), 2 deletions(-)
>>
>
> Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
>
> Thanks
> Patrick
>
Applied on u-boot-stm32/master
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 23/25] ARM: dts: stm32: Don't probe red led at boot for stm32mp157c-ed1-scmi-u-boot
2024-04-09 15:02 ` [PATCH v1 23/25] ARM: dts: stm32: Don't probe red led at boot " Patrice Chotard
2024-04-17 9:13 ` Patrick DELAUNAY
@ 2024-04-19 12:30 ` Patrice CHOTARD
1 sibling, 0 replies; 79+ messages in thread
From: Patrice CHOTARD @ 2024-04-19 12:30 UTC (permalink / raw)
To: u-boot; +Cc: Patrick DELAUNAY, U-Boot STM32, Tom Rini
On 4/9/24 17:02, Patrice Chotard wrote:
> red led and button dedicated to fastboot share the same gpio GPIOA13.
> Led driver is probed early so the corresponding gpio is taken and
> configured in output which forbid fastboot and stm32prog button usage.
>
> To avoid this, remove the "default-state" property from red led node.
>
> This will avoid to trigger the led driver probe() to configure the led
> default state during startup.
>
> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
> ---
>
> arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi b/arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi
> index 84920f53496..2abd512e297 100644
> --- a/arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi
> +++ b/arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi
> @@ -37,7 +37,6 @@
> red {
> label = "error";
> gpios = <&gpioa 13 GPIO_ACTIVE_LOW>;
> - default-state = "off";
> status = "okay";
> };
> };
Applied on u-boot-stm32/master
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 23/25] ARM: dts: stm32: Don't probe red led at boot for stm32mp157c-ed1-scmi-u-boot
2024-04-17 9:13 ` Patrick DELAUNAY
@ 2024-04-19 12:30 ` Patrice CHOTARD
0 siblings, 0 replies; 79+ messages in thread
From: Patrice CHOTARD @ 2024-04-19 12:30 UTC (permalink / raw)
To: Patrick DELAUNAY, u-boot; +Cc: U-Boot STM32, Tom Rini
On 4/17/24 11:13, Patrick DELAUNAY wrote:
> Hi,
>
> On 4/9/24 17:02, Patrice Chotard wrote:
>> red led and button dedicated to fastboot share the same gpio GPIOA13.
>> Led driver is probed early so the corresponding gpio is taken and
>> configured in output which forbid fastboot and stm32prog button usage.
>>
>> To avoid this, remove the "default-state" property from red led node.
>>
>> This will avoid to trigger the led driver probe() to configure the led
>> default state during startup.
>>
>> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
>> ---
>>
>> arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi | 1 -
>> 1 file changed, 1 deletion(-)
>>
>
>
> Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
>
> Thanks
> Patrick
>
Applied on u-boot-stm32/master
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 24/25] ARM: dts: stm32: Update red led node for stm32mp157c-ed1-scmi-u-boot
2024-04-17 9:13 ` Patrick DELAUNAY
@ 2024-04-19 12:30 ` Patrice CHOTARD
0 siblings, 0 replies; 79+ messages in thread
From: Patrice CHOTARD @ 2024-04-19 12:30 UTC (permalink / raw)
To: Patrick DELAUNAY, u-boot; +Cc: U-Boot STM32, Tom Rini
On 4/17/24 11:13, Patrick DELAUNAY wrote:
> Hi,
>
> On 4/9/24 17:02, Patrice Chotard wrote:
>> As indicated in kernel led dt-bindings, label is a deprecated
>> property, so remove it and use led node's name instead for
>> u-boot,error-led property.
>> Rename red led node's name to led-red.
>> Remove status property which is useless.
>> Add compatible = "gpio-leds"; which is not present in kernel DT.
>>
>> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
>> ---
>>
>> arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi | 8 ++++----
>> 1 file changed, 4 insertions(+), 4 deletions(-)
>>
>
>
> Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
>
> Thanks
> Patrick
>
Applied on u-boot-stm32/master
^ permalink raw reply [flat|nested] 79+ messages in thread
* Re: [PATCH v1 25/25] ARM: dts: stm32: Add led-blue for stm32mp157c-ed1-scmi-u-boot
2024-04-17 9:28 ` Patrick DELAUNAY
@ 2024-04-19 12:32 ` Patrice CHOTARD
0 siblings, 0 replies; 79+ messages in thread
From: Patrice CHOTARD @ 2024-04-19 12:32 UTC (permalink / raw)
To: Patrick DELAUNAY, u-boot; +Cc: U-Boot STM32, Tom Rini
On 4/17/24 11:28, Patrick DELAUNAY wrote:
> Hi,
>
> On 4/9/24 17:02, Patrice Chotard wrote:
>> The blue led is used to indicate U-Boot entering / exit indication
>> then Linux heartbeat.
>>
>> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
>> ---
>>
>> arch/arm/dts/stm32mp157c-ed1-scmi-u-boot.dtsi | 6 +++++-
>> 1 file changed, 5 insertions(+), 1 deletion(-)
>>
>
> Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
>
> Thanks
> Patrick
>
Applied on u-boot-stm32/master
^ permalink raw reply [flat|nested] 79+ messages in thread
end of thread, other threads:[~2024-04-19 12:32 UTC | newest]
Thread overview: 79+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-04-09 15:01 [PATCH v1 00/25] stm32mp: Fix keys & leds management Patrice Chotard
2024-04-09 15:01 ` [PATCH v1 01/25] configs: stm32mp13: Enable FASTBOOT Patrice Chotard
2024-04-17 9:03 ` Patrick DELAUNAY
2024-04-18 11:48 ` Igor Opaniuk
2024-04-19 12:16 ` Patrice CHOTARD
2024-04-09 15:01 ` [PATCH v1 02/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp15_defconfig Patrice Chotard
2024-04-17 9:03 ` Patrick DELAUNAY
2024-04-19 12:16 ` Patrice CHOTARD
2024-04-09 15:01 ` [PATCH v1 03/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp15_basic_defconfig Patrice Chotard
2024-04-17 9:04 ` Patrick DELAUNAY
2024-04-19 12:16 ` Patrice CHOTARD
2024-04-09 15:01 ` [PATCH v1 04/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp15_trusted_defconfig Patrice Chotard
2024-04-17 9:04 ` Patrick DELAUNAY
2024-04-19 12:16 ` Patrice CHOTARD
2024-04-09 15:01 ` [PATCH v1 05/25] configs: stm32mp1: Enable BUTTON_GPIO flag for stm32mp13_defconfig Patrice Chotard
2024-04-17 9:04 ` Patrick DELAUNAY
2024-04-18 11:48 ` Igor Opaniuk
2024-04-19 12:16 ` Patrice CHOTARD
2024-04-09 15:01 ` [PATCH v1 06/25] board: st: stmp32mp1: Use BUTTON UCLASS in board_key_check() Patrice Chotard
2024-04-17 9:05 ` Patrick DELAUNAY
2024-04-19 12:16 ` Patrice CHOTARD
2024-04-09 15:01 ` [PATCH v1 07/25] ARM: dts: stm32: Add gpio-keys for stm32mp135f-dk-u-boot Patrice Chotard
2024-04-17 9:06 ` Patrick DELAUNAY
2024-04-19 12:17 ` Patrice CHOTARD
2024-04-09 15:01 ` [PATCH v1 08/25] ARM: dts: stm32: Don't probe led-red/led-blue at boot " Patrice Chotard
2024-04-17 9:06 ` Patrick DELAUNAY
2024-04-19 12:17 ` Patrice CHOTARD
2024-04-09 15:01 ` [PATCH v1 09/25] ARM: dts: stm32: Clean led-red node " Patrice Chotard
2024-04-17 9:07 ` Patrick DELAUNAY
2024-04-19 12:21 ` Patrice CHOTARD
2024-04-09 15:02 ` [PATCH v1 10/25] ARM: dts: stm32: Add gpio-keys for stm32mp157a-dk1-scmi-u-boot Patrice Chotard
2024-04-17 9:07 ` Patrick DELAUNAY
2024-04-19 12:21 ` Patrice CHOTARD
2024-04-09 15:02 ` [PATCH v1 11/25] ARM: dts: stm32: Don't probe red led at boot " Patrice Chotard
2024-04-17 9:07 ` Patrick DELAUNAY
2024-04-19 12:21 ` Patrice CHOTARD
2024-04-09 15:02 ` [PATCH v1 12/25] ARM: dts: stm32: Update red led node " Patrice Chotard
2024-04-17 9:08 ` Patrick DELAUNAY
2024-04-19 12:21 ` Patrice CHOTARD
2024-04-09 15:02 ` [PATCH v1 13/25] ARM: dts: stm32: Add led-blue " Patrice Chotard
2024-04-17 9:09 ` Patrick DELAUNAY
2024-04-19 12:21 ` Patrice CHOTARD
2024-04-09 15:02 ` [PATCH v1 14/25] ARM: dts: stm32: Add gpio-keys for stm32mp157a-dk1-u-boot Patrice Chotard
2024-04-17 9:09 ` Patrick DELAUNAY
2024-04-19 12:26 ` Patrice CHOTARD
2024-04-09 15:02 ` [PATCH v1 15/25] ARM: dts: stm32: Don't probe red led at boot " Patrice Chotard
2024-04-17 9:09 ` Patrick DELAUNAY
2024-04-19 12:27 ` Patrice CHOTARD
2024-04-09 15:02 ` [PATCH v1 16/25] ARM: dts: stm32: Update red led node " Patrice Chotard
2024-04-17 9:10 ` Patrick DELAUNAY
2024-04-19 12:27 ` Patrice CHOTARD
2024-04-09 15:02 ` [PATCH v1 17/25] ARM: dts: stm32: Update u-boot, boot-led " Patrice Chotard
2024-04-17 9:10 ` [PATCH v1 17/25] ARM: dts: stm32: Update u-boot,boot-led " Patrick DELAUNAY
2024-04-19 12:27 ` Patrice CHOTARD
2024-04-09 15:02 ` [PATCH v1 18/25] ARM: dts: stm32: Add gpio-keys for stm32mp157c-ed1-u-boot Patrice Chotard
2024-04-17 9:11 ` Patrick DELAUNAY
2024-04-19 12:27 ` Patrice CHOTARD
2024-04-09 15:02 ` [PATCH v1 19/25] ARM: dts: stm32: Don't probe red led at boot " Patrice Chotard
2024-04-17 9:11 ` Patrick DELAUNAY
2024-04-19 12:27 ` Patrice CHOTARD
2024-04-09 15:02 ` [PATCH v1 20/25] ARM: dts: stm32: Update red led node " Patrice Chotard
2024-04-17 9:11 ` Patrick DELAUNAY
2024-04-19 12:27 ` Patrice CHOTARD
2024-04-09 15:02 ` [PATCH v1 21/25] ARM: dts: stm32: Add led-blue " Patrice Chotard
2024-04-17 9:11 ` Patrick DELAUNAY
2024-04-19 12:30 ` Patrice CHOTARD
2024-04-09 15:02 ` [PATCH v1 22/25] ARM: dts: stm32: Add gpio-keys for stm32mp157c-ed1-scmi-u-boot Patrice Chotard
2024-04-17 9:12 ` Patrick DELAUNAY
2024-04-19 12:30 ` Patrice CHOTARD
2024-04-09 15:02 ` [PATCH v1 23/25] ARM: dts: stm32: Don't probe red led at boot " Patrice Chotard
2024-04-17 9:13 ` Patrick DELAUNAY
2024-04-19 12:30 ` Patrice CHOTARD
2024-04-19 12:30 ` Patrice CHOTARD
2024-04-09 15:02 ` [PATCH v1 24/25] ARM: dts: stm32: Update red led node " Patrice Chotard
2024-04-17 9:13 ` Patrick DELAUNAY
2024-04-19 12:30 ` Patrice CHOTARD
2024-04-09 15:02 ` [PATCH v1 25/25] ARM: dts: stm32: Add led-blue " Patrice Chotard
2024-04-17 9:28 ` Patrick DELAUNAY
2024-04-19 12:32 ` Patrice CHOTARD
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox