uboot-mediatek: u7623: remove keys pin function
authorTomasz Maciej Nowak <[email protected]>
Tue, 4 Mar 2025 16:43:10 +0000 (17:43 +0100)
committerHauke Mehrtens <[email protected]>
Mon, 17 Mar 2025 15:21:23 +0000 (16:21 +0100)
Apparently U-Boot will discard whole node if requested pin function is
unknown to the driver. This resulted in inability to interact with
U-Boot on the said board, as U-Boot always assumed the recovery key
pressed and issued recovery procedure. Log snippet:

button_gpio gpio-keys: pinctrl_select_state_full: pinctrl_config_one: err=-38
reset button found
button pushed, resetting environment

Recovery procedure also booted recovery image, which didn't affect much
the 23.05.x release, since the root fs argument was valid, so changes
persisted. But as 24.10.x hit with fitblk, the board will boot only
recovery image (initramfs) because of default bootargs will reset on each
boot and U-Boot provided bootargs took precedence.

Fixes: 42eeb22450f2 ("uboot-mediatek: fix factory/reset button")
Signed-off-by: Tomasz Maciej Nowak <[email protected]>
Link: https://patchwork.ozlabs.org/project/openwrt/patch/[email protected]/
Signed-off-by: Hauke Mehrtens <[email protected]>
(cherry picked from commit f8a2e1c68b48d72234172543f4232e77f065ca4d)

package/boot/uboot-mediatek/patches/401-update-u7623-defconfig.patch

index 1fb0ae9c9eb0b78e99e5dc9861c759074f30dd2e..7d1936967202148a2c8354e9bf6625ca880ea2ec 100644 (file)
        leds {
                compatible = "gpio-leds";
  
-@@ -109,6 +122,19 @@
+@@ -109,6 +122,18 @@
                };
        };
  
 +      key_pins_a: keys-alt {
 +              mux {
-+                      function = "gpio";
 +                      groups = "msdc3";
 +              };
 +