diff mbox series

[v2,11/11] ARM: dts: amlogic: meson8b-ec100: add missing gpio-line-names entry

Message ID 20240916-topic-amlogic-arm32-upstream-bindings-fixes-v2-11-0b20ed4f0571@linaro.org (mailing list archive)
State Accepted
Headers show
Series ARM: dts: amlogic: first easy dtbs fixes | expand

Commit Message

Neil Armstrong Sept. 16, 2024, 8:08 a.m. UTC 
The EC100 gpio-line-names table is not the right size, add the missing
cell and fix:
meson8b-ec100.dtb: pinctrl@9880: bank@80b0:gpio-line-names: [...] is too short

Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
---
 arch/arm/boot/dts/amlogic/meson8b-ec100.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Martin Blumenstingl Sept. 25, 2024, 5:34 p.m. UTC | #1 
On Mon, Sep 16, 2024 at 10:08 AM Neil Armstrong
<neil.armstrong@linaro.org> wrote:
>
> The EC100 gpio-line-names table is not the right size, add the missing
> cell and fix:
> meson8b-ec100.dtb: pinctrl@9880: bank@80b0:gpio-line-names: [...] is too short
>
> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
Reviewed-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com>

I've been staring at that table multiple times but never noticed any issue.
Great to have code in place to validate it :-)
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/amlogic/meson8b-ec100.dts b/arch/arm/boot/dts/amlogic/meson8b-ec100.dts
index bf1ef07de6c1..18ea6592b7d7 100644
--- a/arch/arm/boot/dts/amlogic/meson8b-ec100.dts
+++ b/arch/arm/boot/dts/amlogic/meson8b-ec100.dts
@@ -431,7 +431,7 @@  &gpio {
 			  "NAND_CS1 (EMMC)", "NAND_CS2 iNAND_RS1 (EMMC)",
 			  "NAND_nR/B iNAND_CMD (EMMC)", "NAND_ALE (EMMC)",
 			  "NAND_CLE (EMMC)", "nRE_S1 NAND_nRE (EMMC)",
-			  "nWE_S1 NAND_nWE (EMMC)",  "", "", "SPI_CS",
+			  "nWE_S1 NAND_nWE (EMMC)",  "", "", "", "SPI_CS",
 			  /* Bank DIF */
 			  "RMII_RXD1", "RMII_RXD0", "RMII_CRS_DV",
 			  "RMII_50M_IN", "GPIODIF_4", "GPIODIF_5",