Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752641AbdGGT5x (ORCPT ); Fri, 7 Jul 2017 15:57:53 -0400 Received: from us01smtprelay-2.synopsys.com ([198.182.47.9]:59861 "EHLO smtprelay.synopsys.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752596AbdGGT5t (ORCPT ); Fri, 7 Jul 2017 15:57:49 -0400 Subject: Re: [PATCH] ARC: AXS: prepare dts files for enabling PAE40 on axs103 To: Eugeniy Paltsev , "linux-snps-arc@lists.infradead.org" CC: "linux-kernel@vger.kernel.org" , "devicetree@vger.kernel.org" , Rob Herring , Alexey Brodkin References: <20170626114725.9678-1-Eugeniy.Paltsev@synopsys.com> From: Vineet Gupta Message-ID: Date: Fri, 7 Jul 2017 12:57:29 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.1 MIME-Version: 1.0 In-Reply-To: <20170626114725.9678-1-Eugeniy.Paltsev@synopsys.com> Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 7bit Content-Language: en-US X-Originating-IP: [10.10.161.108] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 6398 Lines: 214 On 06/26/2017 04:47 AM, Eugeniy Paltsev wrote: > Enable 64bit adressing, where it needed, to make possible > enabling PAE40 on axs103. > > This patch doesn't affect on any functionality. > > Signed-off-by: Eugeniy Paltsev > --- > arch/arc/boot/dts/axc001.dtsi | 20 +++++++++----------- > arch/arc/boot/dts/axc003.dtsi | 21 ++++++++++----------- > arch/arc/boot/dts/axc003_idu.dtsi | 21 ++++++++++----------- > arch/arc/boot/dts/axs10x_mb.dtsi | 2 +- > 4 files changed, 30 insertions(+), 34 deletions(-) > > diff --git a/arch/arc/boot/dts/axc001.dtsi b/arch/arc/boot/dts/axc001.dtsi > index 53ce226..a380ffa 100644 > --- a/arch/arc/boot/dts/axc001.dtsi > +++ b/arch/arc/boot/dts/axc001.dtsi > @@ -15,15 +15,15 @@ > > / { > compatible = "snps,arc"; > - #address-cells = <1>; > - #size-cells = <1>; > + #address-cells = <2>; > + #size-cells = <2>; > > cpu_card { > compatible = "simple-bus"; > #address-cells = <1>; > #size-cells = <1>; > > - ranges = <0x00000000 0xf0000000 0x10000000>; > + ranges = <0x00000000 0x0 0xf0000000 0x10000000>; AXS101 can't support PAE etc - but I guess you still need to make the change here because the DT bits for MB are shared ? > > core_clk: core_clk { > #clock-cells = <0>; > @@ -91,23 +91,21 @@ > mb_intc: dw-apb-ictl@0xe0012000 { > #interrupt-cells = <1>; > compatible = "snps,dw-apb-ictl"; > - reg = < 0xe0012000 0x200 >; > + reg = < 0x0 0xe0012000 0x0 0x200 >; > interrupt-controller; > interrupt-parent = <&core_intc>; > interrupts = < 7 >; > }; > > memory { > - #address-cells = <1>; > - #size-cells = <1>; > - ranges = <0x00000000 0x80000000 0x20000000>; > device_type = "memory"; > - reg = <0x80000000 0x1b000000>; /* (512 - 32) MiB */ > + /* CONFIG_KERNEL_RAM_BASE_ADDRESS needs to match low mem start */ > + reg = <0x0 0x80000000 0x0 0x1b000000>; /* (512 - 32) MiB */ > }; > > reserved-memory { > - #address-cells = <1>; > - #size-cells = <1>; > + #address-cells = <2>; > + #size-cells = <2>; > ranges; > /* > * We just move frame buffer area to the very end of > @@ -118,7 +116,7 @@ > */ > frame_buffer: frame_buffer@9e000000 { > compatible = "shared-dma-pool"; > - reg = <0x9e000000 0x2000000>; > + reg = <0x0 0x9e000000 0x0 0x2000000>; > no-map; > }; > }; > diff --git a/arch/arc/boot/dts/axc003.dtsi b/arch/arc/boot/dts/axc003.dtsi > index 14df46f..cc9239e 100644 > --- a/arch/arc/boot/dts/axc003.dtsi > +++ b/arch/arc/boot/dts/axc003.dtsi > @@ -14,15 +14,15 @@ > > / { > compatible = "snps,arc"; > - #address-cells = <1>; > - #size-cells = <1>; > + #address-cells = <2>; > + #size-cells = <2>; > > cpu_card { > compatible = "simple-bus"; > #address-cells = <1>; > #size-cells = <1>; > > - ranges = <0x00000000 0xf0000000 0x10000000>; > + ranges = <0x00000000 0x0 0xf0000000 0x10000000>; > > core_clk: core_clk { > #clock-cells = <0>; > @@ -94,30 +94,29 @@ > mb_intc: dw-apb-ictl@0xe0012000 { > #interrupt-cells = <1>; > compatible = "snps,dw-apb-ictl"; > - reg = < 0xe0012000 0x200 >; > + reg = < 0x0 0xe0012000 0x0 0x200 >; > interrupt-controller; > interrupt-parent = <&core_intc>; > interrupts = < 24 >; > }; > > memory { > - #address-cells = <1>; > - #size-cells = <1>; > - ranges = <0x00000000 0x80000000 0x40000000>; > device_type = "memory"; > - reg = <0x80000000 0x20000000>; /* 512MiB */ > + /* CONFIG_KERNEL_RAM_BASE_ADDRESS needs to match low mem start */ > + reg = <0x0 0x80000000 0x0 0x20000000 /* 512 MiB low mem */ > + 0x1 0xc0000000 0x0 0x40000000>; /* 1 GiB highmem */ > }; > > reserved-memory { > - #address-cells = <1>; > - #size-cells = <1>; > + #address-cells = <2>; > + #size-cells = <2>; > ranges; > /* > * Move frame buffer out of IOC aperture (0x8z-0xAz). > */ > frame_buffer: frame_buffer@be000000 { > compatible = "shared-dma-pool"; > - reg = <0xbe000000 0x2000000>; > + reg = <0x0 0xbe000000 0x0 0x2000000>; > no-map; > }; > }; > diff --git a/arch/arc/boot/dts/axc003_idu.dtsi b/arch/arc/boot/dts/axc003_idu.dtsi > index 695f9fa..4ebb2170 100644 > --- a/arch/arc/boot/dts/axc003_idu.dtsi > +++ b/arch/arc/boot/dts/axc003_idu.dtsi > @@ -14,15 +14,15 @@ > > / { > compatible = "snps,arc"; > - #address-cells = <1>; > - #size-cells = <1>; > + #address-cells = <2>; > + #size-cells = <2>; > > cpu_card { > compatible = "simple-bus"; > #address-cells = <1>; > #size-cells = <1>; > > - ranges = <0x00000000 0xf0000000 0x10000000>; > + ranges = <0x00000000 0x0 0xf0000000 0x10000000>; > > core_clk: core_clk { > #clock-cells = <0>; > @@ -100,30 +100,29 @@ > mb_intc: dw-apb-ictl@0xe0012000 { > #interrupt-cells = <1>; > compatible = "snps,dw-apb-ictl"; > - reg = < 0xe0012000 0x200 >; > + reg = < 0x0 0xe0012000 0x0 0x200 >; > interrupt-controller; > interrupt-parent = <&idu_intc>; > interrupts = <0>; > }; > > memory { > - #address-cells = <1>; > - #size-cells = <1>; > - ranges = <0x00000000 0x80000000 0x40000000>; > device_type = "memory"; > - reg = <0x80000000 0x20000000>; /* 512MiB */ > + /* CONFIG_KERNEL_RAM_BASE_ADDRESS needs to match low mem start */ > + reg = <0x0 0x80000000 0x0 0x20000000 /* 512 MiB low mem */ > + 0x1 0xc0000000 0x0 0x40000000>; /* 1 GiB highmem */ > }; > > reserved-memory { > - #address-cells = <1>; > - #size-cells = <1>; > + #address-cells = <2>; > + #size-cells = <2>; > ranges; > /* > * Move frame buffer out of IOC aperture (0x8z-0xAz). > */ > frame_buffer: frame_buffer@be000000 { > compatible = "shared-dma-pool"; > - reg = <0xbe000000 0x2000000>; > + reg = <0x0 0xbe000000 0x0 0x2000000>; > no-map; > }; > }; > diff --git a/arch/arc/boot/dts/axs10x_mb.dtsi b/arch/arc/boot/dts/axs10x_mb.dtsi > index 41cfb29..0ff7e07 100644 > --- a/arch/arc/boot/dts/axs10x_mb.dtsi > +++ b/arch/arc/boot/dts/axs10x_mb.dtsi > @@ -13,7 +13,7 @@ > compatible = "simple-bus"; > #address-cells = <1>; > #size-cells = <1>; > - ranges = <0x00000000 0xe0000000 0x10000000>; > + ranges = <0x00000000 0x0 0xe0000000 0x10000000>; > interrupt-parent = <&mb_intc>; > > i2sclk: i2sclk@100a0 {