[v2] ARM: dts: amlogic: first easy dtbs fixes

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

Posted by Neil Armstrong 2 months, 2 weeks ago

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(-)

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",

-- 
2.34.1

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

Posted by Martin Blumenstingl 2 months ago

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 :-)