Re: [PATCH 3/3] ARM64: zynqmp: Use 64bit size cell format

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On Thu, Feb 11, 2016 at 6:26 AM, Michal Simek <michal.simek@xxxxxxxxxx> wrote:
> Use 64bit size cell format instead of 32bit for memory
> description. Change 64bit sizes also for all others IPs.

Why? As is, this change is completely pointless because nothing needs
a >4GB size. Do you have peripherals with >4GB size?

> Signed-off-by: Michal Simek <michal.simek@xxxxxxxxxx>
> ---
>
>  arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts |  2 +-
>  arch/arm64/boot/dts/xilinx/zynqmp.dtsi      | 63 ++++++++++++++---------------
>  2 files changed, 32 insertions(+), 33 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts b/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
> index acb0527fdc4a..358089687a69 100644
> --- a/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
> +++ b/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts
> @@ -29,7 +29,7 @@
>
>         memory {
>                 device_type = "memory";
> -               reg = <0x0 0x0 0x40000000>;
> +               reg = <0x0 0x0 0x0 0x40000000>;
>         };
>  };
>
> diff --git a/arch/arm64/boot/dts/xilinx/zynqmp.dtsi b/arch/arm64/boot/dts/xilinx/zynqmp.dtsi
> index e595f22e7e4b..ef8fe8e6c868 100644
> --- a/arch/arm64/boot/dts/xilinx/zynqmp.dtsi
> +++ b/arch/arm64/boot/dts/xilinx/zynqmp.dtsi
> @@ -14,7 +14,7 @@
>  / {
>         compatible = "xlnx,zynqmp";
>         #address-cells = <2>;
> -       #size-cells = <1>;
> +       #size-cells = <2>;
>
>         cpus {
>                 #address-cells = <1>;
> @@ -74,16 +74,15 @@
>         amba_apu {
>                 compatible = "simple-bus";
>                 #address-cells = <2>;
> -               #size-cells = <1>;
> +               #size-cells = <2>;
>                 ranges;
> -
>                 gic: interrupt-controller@f9010000 {
>                         compatible = "arm,gic-400", "arm,cortex-a15-gic";
>                         #interrupt-cells = <3>;
> -                       reg = <0x0 0xf9010000 0x10000>,
> -                             <0x0 0xf902f000 0x2000>,
> -                             <0x0 0xf9040000 0x20000>,
> -                             <0x0 0xf906f000 0x2000>;
> +                       reg = <0x0 0xf9010000 0x0 0x10000>,
> +                             <0x0 0xf902f000 0x0 0x2000>,
> +                             <0x0 0xf9040000 0x0 0x20000>,
> +                             <0x0 0xf906f000 0x0 0x2000>;
>                         interrupt-controller;
>                         interrupt-parent = <&gic>;
>                         interrupts = <1 9 0xf04>;
> @@ -93,14 +92,14 @@
>         amba: amba {
>                 compatible = "simple-bus";
>                 #address-cells = <2>;
> -               #size-cells = <1>;
> +               #size-cells = <2>;
>                 ranges;
>
>                 can0: can@ff060000 {
>                         compatible = "xlnx,zynq-can-1.0";
>                         status = "disabled";
>                         clock-names = "can_clk", "pclk";
> -                       reg = <0x0 0xff060000 0x1000>;
> +                       reg = <0x0 0xff060000 0x0 0x1000>;
>                         interrupts = <0 23 4>;
>                         interrupt-parent = <&gic>;
>                         tx-fifo-depth = <0x40>;
> @@ -111,7 +110,7 @@
>                         compatible = "xlnx,zynq-can-1.0";
>                         status = "disabled";
>                         clock-names = "can_clk", "pclk";
> -                       reg = <0x0 0xff070000 0x1000>;
> +                       reg = <0x0 0xff070000 0x0 0x1000>;
>                         interrupts = <0 24 4>;
>                         interrupt-parent = <&gic>;
>                         tx-fifo-depth = <0x40>;
> @@ -123,7 +122,7 @@
>                         status = "disabled";
>                         interrupt-parent = <&gic>;
>                         interrupts = <0 57 4>, <0 57 4>;
> -                       reg = <0x0 0xff0b0000 0x1000>;
> +                       reg = <0x0 0xff0b0000 0x0 0x1000>;
>                         clock-names = "pclk", "hclk", "tx_clk";
>                         #address-cells = <1>;
>                         #size-cells = <0>;
> @@ -134,7 +133,7 @@
>                         status = "disabled";
>                         interrupt-parent = <&gic>;
>                         interrupts = <0 59 4>, <0 59 4>;
> -                       reg = <0x0 0xff0c0000 0x1000>;
> +                       reg = <0x0 0xff0c0000 0x0 0x1000>;
>                         clock-names = "pclk", "hclk", "tx_clk";
>                         #address-cells = <1>;
>                         #size-cells = <0>;
> @@ -145,7 +144,7 @@
>                         status = "disabled";
>                         interrupt-parent = <&gic>;
>                         interrupts = <0 61 4>, <0 61 4>;
> -                       reg = <0x0 0xff0d0000 0x1000>;
> +                       reg = <0x0 0xff0d0000 0x0 0x1000>;
>                         clock-names = "pclk", "hclk", "tx_clk";
>                         #address-cells = <1>;
>                         #size-cells = <0>;
> @@ -156,7 +155,7 @@
>                         status = "disabled";
>                         interrupt-parent = <&gic>;
>                         interrupts = <0 63 4>, <0 63 4>;
> -                       reg = <0x0 0xff0e0000 0x1000>;
> +                       reg = <0x0 0xff0e0000 0x0 0x1000>;
>                         clock-names = "pclk", "hclk", "tx_clk";
>                         #address-cells = <1>;
>                         #size-cells = <0>;
> @@ -170,7 +169,7 @@
>                         interrupts = <0 16 4>;
>                         interrupt-controller;
>                         #interrupt-cells = <2>;
> -                       reg = <0x0 0xff0a0000 0x1000>;
> +                       reg = <0x0 0xff0a0000 0x0 0x1000>;
>                 };
>
>                 i2c0: i2c@ff020000 {
> @@ -178,7 +177,7 @@
>                         status = "disabled";
>                         interrupt-parent = <&gic>;
>                         interrupts = <0 17 4>;
> -                       reg = <0x0 0xff020000 0x1000>;
> +                       reg = <0x0 0xff020000 0x0 0x1000>;
>                         #address-cells = <1>;
>                         #size-cells = <0>;
>                 };
> @@ -188,7 +187,7 @@
>                         status = "disabled";
>                         interrupt-parent = <&gic>;
>                         interrupts = <0 18 4>;
> -                       reg = <0x0 0xff030000 0x1000>;
> +                       reg = <0x0 0xff030000 0x0 0x1000>;
>                         #address-cells = <1>;
>                         #size-cells = <0>;
>                 };
> @@ -196,7 +195,7 @@
>                 sata: ahci@fd0c0000 {
>                         compatible = "ceva,ahci-1v84";
>                         status = "disabled";
> -                       reg = <0x0 0xfd0c0000 0x2000>;
> +                       reg = <0x0 0xfd0c0000 0x0 0x2000>;
>                         interrupt-parent = <&gic>;
>                         interrupts = <0 133 4>;
>                 };
> @@ -206,7 +205,7 @@
>                         status = "disabled";
>                         interrupt-parent = <&gic>;
>                         interrupts = <0 48 4>;
> -                       reg = <0x0 0xff160000 0x1000>;
> +                       reg = <0x0 0xff160000 0x0 0x1000>;
>                         clock-names = "clk_xin", "clk_ahb";
>                 };
>
> @@ -215,13 +214,13 @@
>                         status = "disabled";
>                         interrupt-parent = <&gic>;
>                         interrupts = <0 49 4>;
> -                       reg = <0x0 0xff170000 0x1000>;
> +                       reg = <0x0 0xff170000 0x0 0x1000>;
>                         clock-names = "clk_xin", "clk_ahb";
>                 };
>
>                 smmu: smmu@fd800000 {
>                         compatible = "arm,mmu-500";
> -                       reg = <0x0 0xfd800000 0x20000>;
> +                       reg = <0x0 0xfd800000 0x0 0x20000>;
>                         #global-interrupts = <1>;
>                         interrupt-parent = <&gic>;
>                         interrupts = <0 157 4>,
> @@ -236,7 +235,7 @@
>                         status = "disabled";
>                         interrupt-parent = <&gic>;
>                         interrupts = <0 19 4>;
> -                       reg = <0x0 0xff040000 0x1000>;
> +                       reg = <0x0 0xff040000 0x0 0x1000>;
>                         clock-names = "ref_clk", "pclk";
>                         #address-cells = <1>;
>                         #size-cells = <0>;
> @@ -247,7 +246,7 @@
>                         status = "disabled";
>                         interrupt-parent = <&gic>;
>                         interrupts = <0 20 4>;
> -                       reg = <0x0 0xff050000 0x1000>;
> +                       reg = <0x0 0xff050000 0x0 0x1000>;
>                         clock-names = "ref_clk", "pclk";
>                         #address-cells = <1>;
>                         #size-cells = <0>;
> @@ -258,7 +257,7 @@
>                         status = "disabled";
>                         interrupt-parent = <&gic>;
>                         interrupts = <0 36 4>, <0 37 4>, <0 38 4>;
> -                       reg = <0x0 0xff110000 0x1000>;
> +                       reg = <0x0 0xff110000 0x0 0x1000>;
>                         timer-width = <32>;
>                 };
>
> @@ -267,7 +266,7 @@
>                         status = "disabled";
>                         interrupt-parent = <&gic>;
>                         interrupts = <0 39 4>, <0 40 4>, <0 41 4>;
> -                       reg = <0x0 0xff120000 0x1000>;
> +                       reg = <0x0 0xff120000 0x0 0x1000>;
>                         timer-width = <32>;
>                 };
>
> @@ -276,7 +275,7 @@
>                         status = "disabled";
>                         interrupt-parent = <&gic>;
>                         interrupts = <0 42 4>, <0 43 4>, <0 44 4>;
> -                       reg = <0x0 0xff130000 0x1000>;
> +                       reg = <0x0 0xff130000 0x0 0x1000>;
>                         timer-width = <32>;
>                 };
>
> @@ -285,7 +284,7 @@
>                         status = "disabled";
>                         interrupt-parent = <&gic>;
>                         interrupts = <0 45 4>, <0 46 4>, <0 47 4>;
> -                       reg = <0x0 0xff140000 0x1000>;
> +                       reg = <0x0 0xff140000 0x0 0x1000>;
>                         timer-width = <32>;
>                 };
>
> @@ -294,7 +293,7 @@
>                         status = "disabled";
>                         interrupt-parent = <&gic>;
>                         interrupts = <0 21 4>;
> -                       reg = <0x0 0xff000000 0x1000>;
> +                       reg = <0x0 0xff000000 0x0 0x1000>;
>                         clock-names = "uart_clk", "pclk";
>                 };
>
> @@ -303,7 +302,7 @@
>                         status = "disabled";
>                         interrupt-parent = <&gic>;
>                         interrupts = <0 22 4>;
> -                       reg = <0x0 0xff010000 0x1000>;
> +                       reg = <0x0 0xff010000 0x0 0x1000>;
>                         clock-names = "uart_clk", "pclk";
>                 };
>
> @@ -312,7 +311,7 @@
>                         status = "disabled";
>                         interrupt-parent = <&gic>;
>                         interrupts = <0 65 4>;
> -                       reg = <0x0 0xfe200000 0x40000>;
> +                       reg = <0x0 0xfe200000 0x0 0x40000>;
>                         clock-names = "clk_xin", "clk_ahb";
>                 };
>
> @@ -321,7 +320,7 @@
>                         status = "disabled";
>                         interrupt-parent = <&gic>;
>                         interrupts = <0 70 4>;
> -                       reg = <0x0 0xfe300000 0x40000>;
> +                       reg = <0x0 0xfe300000 0x0 0x40000>;
>                         clock-names = "clk_xin", "clk_ahb";
>                 };
>
> @@ -330,7 +329,7 @@
>                         status = "disabled";
>                         interrupt-parent = <&gic>;
>                         interrupts = <0 52 1>;
> -                       reg = <0x0 0xfd4d0000 0x1000>;
> +                       reg = <0x0 0xfd4d0000 0x0 0x1000>;
>                         timeout-sec = <10>;
>                 };
>         };
> --
> 1.9.1
>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux