From 79c2af05fd48fecd917785ab4abbcbebd36eb35e Mon Sep 17 00:00:00 2001 From: Bogdan Ivanus Date: Tue, 2 Dec 2025 15:34:04 +0200 Subject: [PATCH 1/3] Variants: Changed 'built-in' LED from RED to GREEN on R1, C33 and H7. --- .../arduino_giga_r1_stm32h747xx_m7.overlay | 24 +++++++++++++-- ...arduino_portenta_c33_r7fa6m5bh3cfc.overlay | 30 +++++++++++++++++-- ...arduino_portenta_h7_stm32h747xx_m7.overlay | 27 +++++++++++++++-- 3 files changed, 75 insertions(+), 6 deletions(-) diff --git a/variants/arduino_giga_r1_stm32h747xx_m7/arduino_giga_r1_stm32h747xx_m7.overlay b/variants/arduino_giga_r1_stm32h747xx_m7/arduino_giga_r1_stm32h747xx_m7.overlay index fdf6eb3b..a86a5961 100644 --- a/variants/arduino_giga_r1_stm32h747xx_m7/arduino_giga_r1_stm32h747xx_m7.overlay +++ b/variants/arduino_giga_r1_stm32h747xx_m7/arduino_giga_r1_stm32h747xx_m7.overlay @@ -463,8 +463,8 @@ <&gpioz 2 0>, /* analog only A10 */ <&gpioz 3 0>; /* analog only A11 */ - builtin-led-gpios = <&gpioi 12 GPIO_ACTIVE_LOW>, - <&gpioj 13 GPIO_ACTIVE_LOW>, + builtin-led-gpios = <&gpioj 13 GPIO_ACTIVE_LOW>, + <&gpioi 12 GPIO_ACTIVE_LOW>, <&gpioe 3 GPIO_ACTIVE_LOW>; pwm-pin-gpios = <&gpioj 9 0>, @@ -540,3 +540,23 @@ /* Include common flash filesystem configuration */ qspi_flash: &n25q128a1 {}; #include "../common/arduino_flash_fs.dtsi" + +/ { + zephyr,user { + /* Override builtin LED GPIO to point to the green LED */ + builtin_led_gpios = <&gpioj 13 0>; /* Pin 13 on GPIOJ for Green LED, active-low */ + }; + + leds { + compatible = "gpio-leds"; + red_led: led_0 { + gpios = <&gpioi 12 GPIO_ACTIVE_LOW>; /* Red LED on GPIO pin 12 of GPIOI, active-low */ + }; + green_led: led_1 { + gpios = <&gpioj 13 GPIO_ACTIVE_LOW>; /* Green LED on GPIO pin 13 of GPIOJ, active-low */ + }; + blue_led: led_2 { + gpios = <&gpioe 3 GPIO_ACTIVE_LOW>; /* Blue LED on GPIO pin 3 of GPIOE, active-low */ + }; + }; +}; \ No newline at end of file diff --git a/variants/arduino_portenta_c33_r7fa6m5bh3cfc/arduino_portenta_c33_r7fa6m5bh3cfc.overlay b/variants/arduino_portenta_c33_r7fa6m5bh3cfc/arduino_portenta_c33_r7fa6m5bh3cfc.overlay index 806ef2da..43397bff 100644 --- a/variants/arduino_portenta_c33_r7fa6m5bh3cfc/arduino_portenta_c33_r7fa6m5bh3cfc.overlay +++ b/variants/arduino_portenta_c33_r7fa6m5bh3cfc/arduino_portenta_c33_r7fa6m5bh3cfc.overlay @@ -267,8 +267,8 @@ - builtin-led-gpios = <&ioport1 7 GPIO_ACTIVE_LOW>, - <&ioport4 0 GPIO_ACTIVE_LOW>, + builtin-led-gpios = <&ioport4 0 GPIO_ACTIVE_LOW>, + <&ioport1 7 GPIO_ACTIVE_LOW>, <&ioport8 0 GPIO_ACTIVE_LOW>; pwm-pin-gpios = <&ioport6 0 0>; @@ -302,3 +302,29 @@ /* Include common flash filesystem configuration */ qspi_flash: &at25sf128a {}; #include "../common/arduino_flash_fs.dtsi" + +/ { + zephyr,user { + /* Override builtin LED GPIO to point to the green LED (led2) */ + builtin_led_gpios = <&ioport4 0 0>; /* GPIO pin 0 on ioport4 (green LED) */ + }; + + leds { + compatible = "gpio-leds"; + + led1: led1 { + gpios = <&ioport1 7 GPIO_ACTIVE_LOW>; /* Red LED on GPIO pin 7 of ioport1, active-low */ + label = "LEDR"; /* Label changed to Red LED */ + }; + + led2: led2 { + gpios = <&ioport4 0 GPIO_ACTIVE_LOW>; /* Green LED on GPIO pin 0 of ioport4, active-low */ + label = "LEDG"; /* Label changed to Green LED */ + }; + + led3: led3 { + gpios = <&ioport8 0 GPIO_ACTIVE_LOW>; /* Blue LED on GPIO pin 0 of ioport8, active-low */ + label = "LEDB"; /* Label changed to Blue LED */ + }; + }; +}; \ No newline at end of file diff --git a/variants/arduino_portenta_h7_stm32h747xx_m7/arduino_portenta_h7_stm32h747xx_m7.overlay b/variants/arduino_portenta_h7_stm32h747xx_m7/arduino_portenta_h7_stm32h747xx_m7.overlay index 72ae8ffe..71a01861 100644 --- a/variants/arduino_portenta_h7_stm32h747xx_m7/arduino_portenta_h7_stm32h747xx_m7.overlay +++ b/variants/arduino_portenta_h7_stm32h747xx_m7/arduino_portenta_h7_stm32h747xx_m7.overlay @@ -337,8 +337,8 @@ qspi_flash: &mx25l12833f {}; <&gpiok 6 0>, /* LEDG */ <&gpiok 7 0>; /* LEDB */ - builtin-led-gpios = <&gpiok 5 0>, - <&gpiok 6 0>, + builtin-led-gpios = <&gpiok 6 0>, + <&gpiok 5 0>, <&gpiok 7 0>; pwm-pin-gpios = <&gpioa 8 0>, @@ -388,3 +388,26 @@ qspi_flash: &mx25l12833f {}; <&adc1 13>; /* Hack for D20 */ }; }; + +/ { + zephyr,user { + /* Override builtin LED GPIO to point to the green LED */ + builtin_led_gpios = <&gpiok 6 0>; /* Pin 6 on GPIOK, active low */ + }; + + leds { + compatible = "gpio-leds"; + + red_led: led_0 { + gpios = <&gpiok 5 GPIO_ACTIVE_LOW>; /* Red LED on GPIO pin 5, active-low */ + }; + + green_led: led_1 { + gpios = <&gpiok 6 GPIO_ACTIVE_LOW>; /* Green LED on GPIO pin 6, active-low */ + }; + + blue_led: led_2 { + gpios = <&gpiok 7 GPIO_ACTIVE_LOW>; /* Blue LED on GPIO pin 7, active-low */ + }; + }; +}; \ No newline at end of file From cad1828d6f7182dffee497de2b568c9c7f6372e9 Mon Sep 17 00:00:00 2001 From: Bogdan Ivanus Date: Wed, 3 Dec 2025 13:00:50 +0200 Subject: [PATCH 2/3] Variants: Changed OPTA built-in LED change to GREEN as well. --- .../arduino_giga_r1_stm32h747xx_m7.overlay | 20 -------------- .../arduino_opta_stm32h747xx_m7.overlay | 4 +-- ...arduino_portenta_c33_r7fa6m5bh3cfc.overlay | 26 ------------------- ...arduino_portenta_h7_stm32h747xx_m7.overlay | 23 ---------------- 4 files changed, 2 insertions(+), 71 deletions(-) diff --git a/variants/arduino_giga_r1_stm32h747xx_m7/arduino_giga_r1_stm32h747xx_m7.overlay b/variants/arduino_giga_r1_stm32h747xx_m7/arduino_giga_r1_stm32h747xx_m7.overlay index a86a5961..ed5ae090 100644 --- a/variants/arduino_giga_r1_stm32h747xx_m7/arduino_giga_r1_stm32h747xx_m7.overlay +++ b/variants/arduino_giga_r1_stm32h747xx_m7/arduino_giga_r1_stm32h747xx_m7.overlay @@ -540,23 +540,3 @@ /* Include common flash filesystem configuration */ qspi_flash: &n25q128a1 {}; #include "../common/arduino_flash_fs.dtsi" - -/ { - zephyr,user { - /* Override builtin LED GPIO to point to the green LED */ - builtin_led_gpios = <&gpioj 13 0>; /* Pin 13 on GPIOJ for Green LED, active-low */ - }; - - leds { - compatible = "gpio-leds"; - red_led: led_0 { - gpios = <&gpioi 12 GPIO_ACTIVE_LOW>; /* Red LED on GPIO pin 12 of GPIOI, active-low */ - }; - green_led: led_1 { - gpios = <&gpioj 13 GPIO_ACTIVE_LOW>; /* Green LED on GPIO pin 13 of GPIOJ, active-low */ - }; - blue_led: led_2 { - gpios = <&gpioe 3 GPIO_ACTIVE_LOW>; /* Blue LED on GPIO pin 3 of GPIOE, active-low */ - }; - }; -}; \ No newline at end of file diff --git a/variants/arduino_opta_stm32h747xx_m7/arduino_opta_stm32h747xx_m7.overlay b/variants/arduino_opta_stm32h747xx_m7/arduino_opta_stm32h747xx_m7.overlay index 53c81950..23c47f1a 100644 --- a/variants/arduino_opta_stm32h747xx_m7/arduino_opta_stm32h747xx_m7.overlay +++ b/variants/arduino_opta_stm32h747xx_m7/arduino_opta_stm32h747xx_m7.overlay @@ -86,8 +86,8 @@ <&gpiob 14 0>, /* Modbus DE */ <&gpiob 13 0>; /* Modbus RE */ - builtin-led-gpios = <&gpioe 5 0>, - <&gpioh 12 0>, + builtin-led-gpios = <&gpioh 12 0>, + <&gpioe 5 0>, <&gpioh 11 0>; adc-pin-gpios = <&gpioz 0 0>, diff --git a/variants/arduino_portenta_c33_r7fa6m5bh3cfc/arduino_portenta_c33_r7fa6m5bh3cfc.overlay b/variants/arduino_portenta_c33_r7fa6m5bh3cfc/arduino_portenta_c33_r7fa6m5bh3cfc.overlay index 43397bff..15b6d614 100644 --- a/variants/arduino_portenta_c33_r7fa6m5bh3cfc/arduino_portenta_c33_r7fa6m5bh3cfc.overlay +++ b/variants/arduino_portenta_c33_r7fa6m5bh3cfc/arduino_portenta_c33_r7fa6m5bh3cfc.overlay @@ -302,29 +302,3 @@ /* Include common flash filesystem configuration */ qspi_flash: &at25sf128a {}; #include "../common/arduino_flash_fs.dtsi" - -/ { - zephyr,user { - /* Override builtin LED GPIO to point to the green LED (led2) */ - builtin_led_gpios = <&ioport4 0 0>; /* GPIO pin 0 on ioport4 (green LED) */ - }; - - leds { - compatible = "gpio-leds"; - - led1: led1 { - gpios = <&ioport1 7 GPIO_ACTIVE_LOW>; /* Red LED on GPIO pin 7 of ioport1, active-low */ - label = "LEDR"; /* Label changed to Red LED */ - }; - - led2: led2 { - gpios = <&ioport4 0 GPIO_ACTIVE_LOW>; /* Green LED on GPIO pin 0 of ioport4, active-low */ - label = "LEDG"; /* Label changed to Green LED */ - }; - - led3: led3 { - gpios = <&ioport8 0 GPIO_ACTIVE_LOW>; /* Blue LED on GPIO pin 0 of ioport8, active-low */ - label = "LEDB"; /* Label changed to Blue LED */ - }; - }; -}; \ No newline at end of file diff --git a/variants/arduino_portenta_h7_stm32h747xx_m7/arduino_portenta_h7_stm32h747xx_m7.overlay b/variants/arduino_portenta_h7_stm32h747xx_m7/arduino_portenta_h7_stm32h747xx_m7.overlay index 71a01861..0bdfc7c2 100644 --- a/variants/arduino_portenta_h7_stm32h747xx_m7/arduino_portenta_h7_stm32h747xx_m7.overlay +++ b/variants/arduino_portenta_h7_stm32h747xx_m7/arduino_portenta_h7_stm32h747xx_m7.overlay @@ -388,26 +388,3 @@ qspi_flash: &mx25l12833f {}; <&adc1 13>; /* Hack for D20 */ }; }; - -/ { - zephyr,user { - /* Override builtin LED GPIO to point to the green LED */ - builtin_led_gpios = <&gpiok 6 0>; /* Pin 6 on GPIOK, active low */ - }; - - leds { - compatible = "gpio-leds"; - - red_led: led_0 { - gpios = <&gpiok 5 GPIO_ACTIVE_LOW>; /* Red LED on GPIO pin 5, active-low */ - }; - - green_led: led_1 { - gpios = <&gpiok 6 GPIO_ACTIVE_LOW>; /* Green LED on GPIO pin 6, active-low */ - }; - - blue_led: led_2 { - gpios = <&gpiok 7 GPIO_ACTIVE_LOW>; /* Blue LED on GPIO pin 7, active-low */ - }; - }; -}; \ No newline at end of file From 7991d7277c4f7e647cebb45573c8750f70af5641 Mon Sep 17 00:00:00 2001 From: Bogdan Ivanus Date: Thu, 11 Dec 2025 14:43:18 +0200 Subject: [PATCH 3/3] Variants: Changed 'built-in' LED to resemble G-R-G-B pattern. --- .../arduino_giga_r1_stm32h747xx_m7.overlay | 1 + .../arduino_opta_stm32h747xx_m7.overlay | 5 +++-- .../arduino_portenta_c33_r7fa6m5bh3cfc.overlay | 1 + .../arduino_portenta_h7_stm32h747xx_m7.overlay | 1 + 4 files changed, 6 insertions(+), 2 deletions(-) diff --git a/variants/arduino_giga_r1_stm32h747xx_m7/arduino_giga_r1_stm32h747xx_m7.overlay b/variants/arduino_giga_r1_stm32h747xx_m7/arduino_giga_r1_stm32h747xx_m7.overlay index ed5ae090..4d9269ca 100644 --- a/variants/arduino_giga_r1_stm32h747xx_m7/arduino_giga_r1_stm32h747xx_m7.overlay +++ b/variants/arduino_giga_r1_stm32h747xx_m7/arduino_giga_r1_stm32h747xx_m7.overlay @@ -465,6 +465,7 @@ builtin-led-gpios = <&gpioj 13 GPIO_ACTIVE_LOW>, <&gpioi 12 GPIO_ACTIVE_LOW>, + <&gpioj 13 GPIO_ACTIVE_LOW>, <&gpioe 3 GPIO_ACTIVE_LOW>; pwm-pin-gpios = <&gpioj 9 0>, diff --git a/variants/arduino_opta_stm32h747xx_m7/arduino_opta_stm32h747xx_m7.overlay b/variants/arduino_opta_stm32h747xx_m7/arduino_opta_stm32h747xx_m7.overlay index 23c47f1a..d018328e 100644 --- a/variants/arduino_opta_stm32h747xx_m7/arduino_opta_stm32h747xx_m7.overlay +++ b/variants/arduino_opta_stm32h747xx_m7/arduino_opta_stm32h747xx_m7.overlay @@ -87,8 +87,9 @@ <&gpiob 13 0>; /* Modbus RE */ builtin-led-gpios = <&gpioh 12 0>, - <&gpioe 5 0>, - <&gpioh 11 0>; + <&gpioh 11 0>, + <&gpioh 12 0>, + <&gpioe 5 0>; adc-pin-gpios = <&gpioz 0 0>, <&gpioz 1 0>, diff --git a/variants/arduino_portenta_c33_r7fa6m5bh3cfc/arduino_portenta_c33_r7fa6m5bh3cfc.overlay b/variants/arduino_portenta_c33_r7fa6m5bh3cfc/arduino_portenta_c33_r7fa6m5bh3cfc.overlay index 15b6d614..ecaaa65e 100644 --- a/variants/arduino_portenta_c33_r7fa6m5bh3cfc/arduino_portenta_c33_r7fa6m5bh3cfc.overlay +++ b/variants/arduino_portenta_c33_r7fa6m5bh3cfc/arduino_portenta_c33_r7fa6m5bh3cfc.overlay @@ -269,6 +269,7 @@ builtin-led-gpios = <&ioport4 0 GPIO_ACTIVE_LOW>, <&ioport1 7 GPIO_ACTIVE_LOW>, + <&ioport4 0 GPIO_ACTIVE_LOW>, <&ioport8 0 GPIO_ACTIVE_LOW>; pwm-pin-gpios = <&ioport6 0 0>; diff --git a/variants/arduino_portenta_h7_stm32h747xx_m7/arduino_portenta_h7_stm32h747xx_m7.overlay b/variants/arduino_portenta_h7_stm32h747xx_m7/arduino_portenta_h7_stm32h747xx_m7.overlay index 0bdfc7c2..f3f377b3 100644 --- a/variants/arduino_portenta_h7_stm32h747xx_m7/arduino_portenta_h7_stm32h747xx_m7.overlay +++ b/variants/arduino_portenta_h7_stm32h747xx_m7/arduino_portenta_h7_stm32h747xx_m7.overlay @@ -339,6 +339,7 @@ qspi_flash: &mx25l12833f {}; builtin-led-gpios = <&gpiok 6 0>, <&gpiok 5 0>, + <&gpiok 6 0>, <&gpiok 7 0>; pwm-pin-gpios = <&gpioa 8 0>,