doc: FIT image: fix incorrect examples of DT node unit address
authorAndre Przywara <andre.przywara@arm.com>
Mon, 4 Dec 2017 02:05:08 +0000 (02:05 +0000)
committerSimon Glass <sjg@chromium.org>
Tue, 16 Jan 2018 01:29:21 +0000 (18:29 -0700)
The DT spec demands a unit-address of a node name to match the "reg"
property in that node. Newer dtc versions will throw warnings if this is
not the case.
Fix all occurences in the FIT image example files where this was not
observed, to not give bad examples to the reader.

Signed-off-by: Andre Przywara <andre.przywara@arm.com>
doc/uImage.FIT/kernel.its
doc/uImage.FIT/kernel_fdt.its
doc/uImage.FIT/multi-with-fpga.its
doc/uImage.FIT/multi-with-loadables.its
doc/uImage.FIT/multi.its
doc/uImage.FIT/multi_spl.its
doc/uImage.FIT/sign-configs.its
doc/uImage.FIT/sign-images.its
doc/uImage.FIT/update3.its
doc/uImage.FIT/update_uboot.its

index 0aaf47e6c6bb2a11c067a824830775816b27266e..77ddf622deb84da04e3be6859da884496b39d6a5 100644 (file)
@@ -9,7 +9,7 @@
        #address-cells = <1>;
 
        images {
-               kernel@1 {
+               kernel {
                        description = "Vanilla Linux kernel";
                        data = /incbin/("./vmlinux.bin.gz");
                        type = "kernel";
                        compression = "gzip";
                        load = <00000000>;
                        entry = <00000000>;
-                       hash@1 {
+                       hash-1 {
                                algo = "crc32";
                        };
-                       hash@2 {
+                       hash-2 {
                                algo = "sha1";
                        };
                };
        };
 
        configurations {
-               default = "config@1";
-               config@1 {
+               default = "config-1";
+               config-1 {
                        description = "Boot Linux kernel";
-                       kernel = "kernel@1";
+                       kernel = "kernel";
                };
        };
 };
@@ -47,7 +47,7 @@ For x86 a setup node is also required: see x86-fit-boot.txt.
        #address-cells = <1>;
 
        images {
-               kernel@1 {
+               kernel {
                        description = "Vanilla Linux kernel";
                        data = /incbin/("./image.bin.lzo");
                        type = "kernel";
@@ -56,12 +56,12 @@ For x86 a setup node is also required: see x86-fit-boot.txt.
                        compression = "lzo";
                        load = <0x01000000>;
                        entry = <0x00000000>;
-                       hash@2 {
+                       hash-2 {
                                algo = "sha1";
                        };
                };
 
-               setup@1 {
+               setup {
                        description = "Linux setup.bin";
                        data = /incbin/("./setup.bin");
                        type = "x86_setup";
@@ -70,18 +70,18 @@ For x86 a setup node is also required: see x86-fit-boot.txt.
                        compression = "none";
                        load = <0x00090000>;
                        entry = <0x00090000>;
-                       hash@2 {
+                       hash-2 {
                                algo = "sha1";
                        };
                };
        };
 
        configurations {
-               default = "config@1";
-               config@1 {
+               default = "config-1";
+               config-1 {
                        description = "Boot Linux kernel";
-                       kernel = "kernel@1";
-                       setup = "setup@1";
+                       kernel = "kernel";
+                       setup = "setup";
                };
        };
 };
index 7c521486ef79540891cfda98ed46cec3cf183abf..000d85b8e09cf4fb5241e41db610317871582aa2 100644 (file)
@@ -9,7 +9,7 @@
        #address-cells = <1>;
 
        images {
-               kernel@1 {
+               kernel {
                        description = "Vanilla Linux kernel";
                        data = /incbin/("./vmlinux.bin.gz");
                        type = "kernel";
                        compression = "gzip";
                        load = <00000000>;
                        entry = <00000000>;
-                       hash@1 {
+                       hash-1 {
                                algo = "crc32";
                        };
-                       hash@2 {
+                       hash-2 {
                                algo = "sha1";
                        };
                };
-               fdt@1 {
+               fdt-1 {
                        description = "Flattened Device Tree blob";
                        data = /incbin/("./target.dtb");
                        type = "flat_dt";
                        arch = "ppc";
                        compression = "none";
-                       hash@1 {
+                       hash-1 {
                                algo = "crc32";
                        };
-                       hash@2 {
+                       hash-2 {
                                algo = "sha1";
                        };
                };
        };
 
        configurations {
-               default = "conf@1";
-               conf@1 {
+               default = "conf-1";
+               conf-1 {
                        description = "Boot Linux kernel with FDT blob";
-                       kernel = "kernel@1";
-                       fdt = "fdt@1";
+                       kernel = "kernel";
+                       fdt = "fdt-1";
                };
        };
 };
index 0cdb31fe91c4f6c580058a9d7d1f70f930219e50..47ee5760c494663e508a05fb87bed28511dd3fc0 100644 (file)
        #address-cells = <1>;
 
        images {
-               fdt@1 {
+               fdt-1 {
                        description = "zc706";
                        data = /incbin/("/tftpboot/devicetree.dtb");
                        type = "flat_dt";
                        arch = "arm";
                        compression = "none";
                        load = <0x10000000>;
-                       hash@1 {
+                       hash-1 {
                                algo = "md5";
                        };
                };
 
-               fpga@1 {
+               fpga {
                        description = "FPGA";
                        data = /incbin/("/tftpboot/download.bit");
                        type = "fpga";
                        arch = "arm";
                        compression = "none";
                        load = <0x30000000>;
-                       hash@1 {
+                       hash-1 {
                                algo = "md5";
                        };
                };
 
-               linux_kernel@1 {
+               linux_kernel {
                        description = "Linux";
                        data = /incbin/("/tftpboot/zImage");
                        type = "kernel";
                        compression = "none";
                        load = <0x8000>;
                        entry = <0x8000>;
-                       hash@1 {
+                       hash-1 {
                                algo = "md5";
                        };
                };
        };
 
        configurations {
-               default = "config@2";
-               config@1 {
+               default = "config-2";
+               config-1 {
                        description = "Linux";
-                       kernel = "linux_kernel@1";
-                       fdt = "fdt@1";
+                       kernel = "linux_kernel";
+                       fdt = "fdt-1";
                };
 
-               config@2 {
+               config-2 {
                        description = "Linux with fpga";
-                       kernel = "linux_kernel@1";
-                       fdt = "fdt@1";
-                       fpga = "fpga@1";
+                       kernel = "linux_kernel";
+                       fdt = "fdt-1";
+                       fpga = "fpga";
                };
        };
 };
index a8545d245c992e2f101b2df677c8692ac4ccdae3..4d4909f83207ab8a6780b93ecefd61130773be59 100644 (file)
@@ -10,7 +10,7 @@
        #address-cells = <1>;
 
        images {
-               xen_kernel@1 {
+               xen_kernel {
                        description = "xen binary";
                        data = /incbin/("./xen");
                        type = "kernel";
                        compression = "none";
                        load = <0xa0000000>;
                        entry = <0xa0000000>;
-                       hash@1 {
+                       hash-1 {
                                algo = "md5";
                        };
                };
 
-               fdt@1 {
+               fdt-1 {
                        description = "xexpress-ca15 tree blob";
                        data = /incbin/("./vexpress-v2p-ca15-tc1.dtb");
                        type = "flat_dt";
                        arch = "arm";
                        compression = "none";
                        load = <0xb0000000>;
-                       hash@1 {
+                       hash-1 {
                                algo = "md5";
                        };
                };
 
-               fdt@2 {
+               fdt-2 {
                        description = "xexpress-ca15 tree blob";
                        data = /incbin/("./vexpress-v2p-ca15-tc1.dtb");
                        type = "flat_dt";
                        arch = "arm";
                        compression = "none";
                        load = <0xb0400000>;
-                       hash@1 {
+                       hash-1 {
                                algo = "md5";
                        };
                };
 
-               linux_kernel@1 {
+               linux_kernel {
                        description = "Linux Image";
                        data = /incbin/("./Image");
                        type = "kernel";
                        compression = "none";
                        load = <0xa0000000>;
                        entry = <0xa0000000>;
-                       hash@1 {
+                       hash-1 {
                                algo = "md5";
                        };
                };
        };
 
        configurations {
-               default = "config@2";
+               default = "config-2";
 
-               config@1 {
+               config-1 {
                        description = "Just plain Linux";
-                       kernel = "linux_kernel@1";
-                       fdt = "fdt@1";
+                       kernel = "linux_kernel";
+                       fdt = "fdt-1";
                };
 
-               config@2 {
+               config-2 {
                        description = "Xen one loadable";
-                       kernel = "xen_kernel@1";
-                       fdt = "fdt@1";
-                       loadables = "linux_kernel@1";
+                       kernel = "xen_kernel";
+                       fdt = "fdt-1";
+                       loadables = "linux_kernel";
                };
 
-               config@3 {
+               config-3 {
                        description = "Xen two loadables";
-                       kernel = "xen_kernel@1";
-                       fdt = "fdt@1";
-                       loadables = "linux_kernel@1", "fdt@2";
+                       kernel = "xen_kernel";
+                       fdt = "fdt-1";
+                       loadables = "linux_kernel", "fdt-2";
                };
        };
 };
index 37369ecc82d0c410f4ad9113c725560d5e9416a9..26c8dad6a2b204177c1bb4bd527c71dbcc5714df 100644 (file)
@@ -9,7 +9,7 @@
        #address-cells = <1>;
 
        images {
-               kernel@1 {
+               kernel-1 {
                        description = "vanilla-2.6.23";
                        data = /incbin/("./vmlinux.bin.gz");
                        type = "kernel";
                        compression = "gzip";
                        load = <00000000>;
                        entry = <00000000>;
-                       hash@1 {
+                       hash-1 {
                                algo = "md5";
                        };
-                       hash@2 {
+                       hash-2 {
                                algo = "sha1";
                        };
                };
 
-               kernel@2 {
+               kernel-2 {
                        description = "2.6.23-denx";
                        data = /incbin/("./2.6.23-denx.bin.gz");
                        type = "kernel";
                        compression = "gzip";
                        load = <00000000>;
                        entry = <00000000>;
-                       hash@1 {
+                       hash-1 {
                                algo = "sha1";
                        };
                };
 
-               kernel@3 {
+               kernel-3 {
                        description = "2.4.25-denx";
                        data = /incbin/("./2.4.25-denx.bin.gz");
                        type = "kernel";
                        compression = "gzip";
                        load = <00000000>;
                        entry = <00000000>;
-                       hash@1 {
+                       hash-1 {
                                algo = "md5";
                        };
                };
 
-               ramdisk@1 {
+               ramdisk-1 {
                        description = "eldk-4.2-ramdisk";
                        data = /incbin/("./eldk-4.2-ramdisk");
                        type = "ramdisk";
                        compression = "gzip";
                        load = <00000000>;
                        entry = <00000000>;
-                       hash@1 {
+                       hash-1 {
                                algo = "sha1";
                        };
                };
 
-               ramdisk@2 {
+               ramdisk-2 {
                        description = "eldk-3.1-ramdisk";
                        data = /incbin/("./eldk-3.1-ramdisk");
                        type = "ramdisk";
                        compression = "gzip";
                        load = <00000000>;
                        entry = <00000000>;
-                       hash@1 {
+                       hash-1 {
                                algo = "crc32";
                        };
                };
 
-               fdt@1 {
+               fdt-1 {
                        description = "tqm5200-fdt";
                        data = /incbin/("./tqm5200.dtb");
                        type = "flat_dt";
                        arch = "ppc";
                        compression = "none";
-                       hash@1 {
+                       hash-1 {
                                algo = "crc32";
                        };
                };
 
-               fdt@2 {
+               fdt-2 {
                        description = "tqm5200s-fdt";
                        data = /incbin/("./tqm5200s.dtb");
                        type = "flat_dt";
                        arch = "ppc";
                        compression = "none";
                        load = <00700000>;
-                       hash@1 {
+                       hash-1 {
                                algo = "sha1";
                        };
                };
        };
 
        configurations {
-               default = "config@1";
+               default = "config-1";
 
-               config@1 {
+               config-1 {
                        description = "tqm5200 vanilla-2.6.23 configuration";
-                       kernel = "kernel@1";
-                       ramdisk = "ramdisk@1";
-                       fdt = "fdt@1";
+                       kernel = "kernel-1";
+                       ramdisk = "ramdisk-1";
+                       fdt = "fdt-1";
                };
 
-               config@2 {
+               config-2 {
                        description = "tqm5200s denx-2.6.23 configuration";
-                       kernel = "kernel@2";
-                       ramdisk = "ramdisk@1";
-                       fdt = "fdt@2";
+                       kernel = "kernel-2";
+                       ramdisk = "ramdisk-1";
+                       fdt = "fdt-2";
                };
 
-               config@3 {
+               config-3 {
                        description = "tqm5200s denx-2.4.25 configuration";
-                       kernel = "kernel@3";
-                       ramdisk = "ramdisk@2";
+                       kernel = "kernel-3";
+                       ramdisk = "ramdisk-2";
                };
        };
 };
index d43563d87a2a6a4f14c4bd494ae41d3226afbe57..59421997441228e09c952879e62bdcf96724976b 100644 (file)
@@ -45,7 +45,7 @@
                        load = <0x40000>;
                };
 
-               fdt@1 {
+               fdt-1 {
                        description = "Pine64+ DT";
                        type = "flat_dt";
                        compression = "none";
@@ -53,7 +53,7 @@
                        arch = "arm64";
                };
 
-               fdt@2 {
+               fdt-2 {
                        description = "Pine64 DT";
                        type = "flat_dt";
                        compression = "none";
        };
 
        configurations {
-               default = "config@1";
+               default = "config-1";
 
-               config@1 {
+               config-1 {
                        description = "sun50i-a64-pine64-plus";
                        loadables = "uboot", "atf", "kernel", "initrd";
-                       fdt = "fdt@1";
+                       fdt = "fdt-1";
                };
 
-               config@2 {
+               config-2 {
                        description = "sun50i-a64-pine64";
                        loadables = "uboot", "atf", "mgmt-firmware";
-                       fdt = "fdt@2";
+                       fdt = "fdt-2";
                };
        };
 };
index 3c17f040de0f8760d97a5bf457158b3803e8dd6d..9e992c198826a65eba7e9004f7f8b57786c502f2 100644 (file)
@@ -5,7 +5,7 @@
        #address-cells = <1>;
 
        images {
-               kernel@1 {
+               kernel {
                        data = /incbin/("test-kernel.bin");
                        type = "kernel_noload";
                        arch = "sandbox";
                        load = <0x4>;
                        entry = <0x8>;
                        kernel-version = <1>;
-                       hash@1 {
+                       hash-1 {
                                algo = "sha1";
                        };
                };
-               fdt@1 {
+               fdt-1 {
                        description = "snow";
                        data = /incbin/("sandbox-kernel.dtb");
                        type = "flat_dt";
                        arch = "sandbox";
                        compression = "none";
                        fdt-version = <1>;
-                       hash@1 {
+                       hash-1 {
                                algo = "sha1";
                        };
                };
        };
        configurations {
-               default = "conf@1";
-               conf@1 {
-                       kernel = "kernel@1";
-                       fdt = "fdt@1";
-                       signature@1 {
+               default = "conf-1";
+               conf-1 {
+                       kernel = "kernel";
+                       fdt = "fdt-1";
+                       signature {
                                algo = "sha1,rsa2048";
                                key-name-hint = "dev";
                                sign-images = "fdt", "kernel";
index f69326a39bce1bc78b9fe43b137f84ddaa666a75..18c759e9e65cbb67185a1d76d7568972f040be19 100644 (file)
@@ -5,7 +5,7 @@
        #address-cells = <1>;
 
        images {
-               kernel@1 {
+               kernel {
                        data = /incbin/("test-kernel.bin");
                        type = "kernel_noload";
                        arch = "sandbox";
                        load = <0x4>;
                        entry = <0x8>;
                        kernel-version = <1>;
-                       signature@1 {
+                       signature {
                                algo = "sha1,rsa2048";
                                key-name-hint = "dev";
                        };
                };
-               fdt@1 {
+               fdt-1 {
                        description = "snow";
                        data = /incbin/("sandbox-kernel.dtb");
                        type = "flat_dt";
                        arch = "sandbox";
                        compression = "none";
                        fdt-version = <1>;
-                       signature@1 {
+                       signature {
                                algo = "sha1,rsa2048";
                                key-name-hint = "dev";
                        };
                };
        };
        configurations {
-               default = "conf@1";
-               conf@1 {
-                       kernel = "kernel@1";
-                       fdt = "fdt@1";
+               default = "conf-1";
+               conf-1 {
+                       kernel = "kernel";
+                       fdt = "fdt-1";
                };
        };
 };
index a6eaef691e266b35f5f84125d63b2cce17b5584c..0659f20002d57dc0382bdef41d1558a62dafcf56 100644 (file)
@@ -9,34 +9,34 @@
        #address-cells = <1>;
 
        images {
-               update@1 {
+               update-1 {
                        description = "Linux kernel binary";
                        data = /incbin/("./vmlinux.bin.gz");
                        compression = "none";
                        type = "firmware";
                        load = <FF700000>;
-                       hash@1 {
+                       hash-1 {
                                algo = "sha1";
                        };
                };
-               update@2 {
+               update-2 {
                        description = "Ramdisk image";
                        data = /incbin/("./ramdisk_image.gz");
                        compression = "none";
                        type = "firmware";
                        load = <FF8E0000>;
-                       hash@1 {
+                       hash-1 {
                                algo = "sha1";
                        };
                };
 
-               update@3 {
+               update-3 {
                        description = "FDT blob";
                        data = /incbin/("./blob.fdt");
                        compression = "none";
                        type = "firmware";
                        load = <FFAC0000>;
-                       hash@1 {
+                       hash-1 {
                                algo = "sha1";
                        };
                };
index 846723e2d6272c365d6bcf7541be7f16e30a9b59..c2ac2663e625251b4bfc4e1688aecbf159cd83b4 100644 (file)
        #address-cells = <1>;
 
        images {
-               update@1 {
+               update-1 {
                        description = "U-Boot binary";
                        data = /incbin/("./u-boot.bin");
                        compression = "none";
                        type = "firmware";
                        load = <FFFC0000>;
-                       hash@1 {
+                       hash-1 {
                                algo = "sha1";
                        };
                };