On Sun, Oct 24, 2021 at 5:40 PM Jesse Taube [off-list ref] wrote:

+               pinctrl_semc: semcgrp {
+                       fsl,pins = <

This node doesn't appear to be referenced from anywhere. Is it actually needed?

+
+/ {
+       #address-cells = <1>;
+       #size-cells = <1>;
+
+       aliases {
+               gpio0 = &gpio1;
+               gpio1 = &gpio2;
+               gpio2 = &gpio3;
+               gpio3 = &gpio4;
+               gpio4 = &gpio5;
+               mmc0 = &usdhc1;
+               serial0 = &lpuart1;
+       };

The aliases should go into the .dts file.

+               edma1: dma-controller@400E8000 {
+                       #dma-cells = <2>;
+                       compatible = "fsl,imx7ulp-edma";
+                       reg = <0x400E8000 0x4000>,
+                               <0x400EC000 0x4000>;

Use lowercase letters in hex numbers here.

        Arnd

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel