'/memory' nodes always have a 'reg' property, and therefore should have
a unit-address with just plain hex (i.e. no commas). Fix all the arm64
'/memory' nodes.
It's possible that some bootloader depends on /memory (arm32 ATAG to DT
code does for example). If so, the memory node should be commented with
that requirement.
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
Reviewed-by: Florian Fainelli <florian.fainelli@broadcom.com>
Reviewed-by: Heiko Stuebner <heiko@sntech.de>
Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Reviewed-by: Chanho Min <chanho.min@lge.com>
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Link: https://lore.kernel.org/r/20240430191856.874600-2-robh@kernel.org
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
                reg = <0x0 0x0 0x0 0x80000000>;
        };
 
-       memory@1,e0000000 {
+       memory@1e0000000 {
                device_type = "memory";
                reg = <0x1 0xe0000000 0x0 0x0>;
        };
 
 
        chosen { };
 
-       memory {
+       memory@100000000 {
                device_type = "memory";
                reg = < 0x1 0x00000000 0x0 0x80000000 >;
        };
 
 
        chosen { };
 
-       memory {
+       memory@100000000 {
                device_type = "memory";
                reg = < 0x1 0x00000000 0x0 0x80000000 >; /* Updated by bootloader */
        };
 
                bootargs = "earlycon=uart8250,mmio32,0x66130000";
        };
 
-       memory {
+       memory@80000000 {
                device_type = "memory";
                reg = <0x00000000 0x80000000 0x00000000 0x40000000>;
        };
 
                bootargs = "earlycon=uart8250,mmio32,0x66130000";
        };
 
-       memory {
+       memory@80000000 {
                device_type = "memory";
                reg = <0x00000000 0x80000000 0x00000001 0x00000000>;
        };
 
        model = "Cavium ThunderX2 CN99XX";
        compatible = "cavium,thunderx2-cn9900", "brcm,vulcan-soc";
 
-       memory {
+       memory@80000000 {
                device_type = "memory";
                reg = <0x00000000 0x80000000 0x0 0x80000000>,  /* 2G @ 2G  */
                      <0x00000008 0x80000000 0x0 0x80000000>;  /* 2G @ 34G */
 
                serial2 = &uart2;
        };
 
-       memory {
+       memory@0 {
                device_type = "memory";
                reg = <0x0 0x00000000 0x20000000>;
        };
 
                serial2 = &uart2;
        };
 
-       memory {
+       memory@0 {
                device_type = "memory";
                reg = <0x0 0x00000000 0x20000000>;
        };
 
                stdout-path = &serial0;
        };
 
-       memory {
+       memory@0 {
                reg = <0x0 0x0 0x0 0x40000000>;
        };
 };
 
                stdout-path = "serial0:115200n8";
        };
 
-       memory {
+       memory@80000000 {
                device_type = "memory";
                reg = <0x0 0x80000000 0x0 0xc0000000>;
        };
 
                stdout-path = "serial0";
        };
 
-       memory {
+       memory@40000000 {
                device_type = "memory";
                reg = <0x0 0x40000000 0x0 0x20000000>;
        };
 
                stdout-path = "serial2:115200n8";
        };
 
-       memory {
+       memory@0 {
                device_type = "memory";
                reg = <0x0 0x0 0x0 0x40000000>;
        };
 
                stdout-path = "serial2:115200n8";
        };
 
-       memory {
+       memory@0 {
                device_type = "memory";
                reg = <0x0 0x0 0x0 0x80000000>;
        };
 
                stdout-path = "serial2:115200n8";
        };
 
-       memory {
+       memory@0 {
                device_type = "memory";
                reg = <0x0 0x0 0x0 0x40000000>;
        };
 
                spi0 = &adi_bus;
        };
 
-       memory {
+       memory@80000000 {
                device_type = "memory";
                reg = <0x0 0x80000000 0 0x60000000>,
                      <0x1 0x80000000 0 0x60000000>;