* [PATCH] ARC: [hsdk] Make it easier to add PAE40 region to DTB
@ 2019-04-02 19:18 ` Vineet Gupta
0 siblings, 0 replies; 6+ messages in thread
From: Vineet Gupta @ 2019-04-02 19:18 UTC (permalink / raw)
To: linux-snps-arc; +Cc: linux-kernel, eugeniy.paltsev, robh, Vineet Gupta
1. Bump top level address-cells/size-cells nodes to 2 (to ensure all
down stream addresses are 64-bits, unless explicitly specified
otherwise (in "soc" bus with all peripherals)
2. "memory" also specified with address/size 2
3. Add a commented reference for PAE40 region beyond 4GB physical
address space
Signed-off-by: Vineet Gupta <vgupta@synopsys.com>
---
arch/arc/boot/dts/hsdk.dts | 13 +++++++------
1 file changed, 7 insertions(+), 6 deletions(-)
diff --git a/arch/arc/boot/dts/hsdk.dts b/arch/arc/boot/dts/hsdk.dts
index 69bc1c9e8e50..7425bb0f2d1b 100644
--- a/arch/arc/boot/dts/hsdk.dts
+++ b/arch/arc/boot/dts/hsdk.dts
@@ -18,8 +18,8 @@
model = "snps,hsdk";
compatible = "snps,hsdk";
- #address-cells = <1>;
- #size-cells = <1>;
+ #address-cells = <2>;
+ #size-cells = <2>;
chosen {
bootargs = "earlycon=uart8250,mmio32,0xf0005000,115200n8 console=ttyS0,115200n8 debug print-fatal-signals=1";
@@ -105,7 +105,7 @@
#size-cells = <1>;
interrupt-parent = <&idu_intc>;
- ranges = <0x00000000 0xf0000000 0x10000000>;
+ ranges = <0x00000000 0x0 0xf0000000 0x10000000>;
cgu_rst: reset-controller@8a0 {
compatible = "snps,hsdk-reset";
@@ -269,9 +269,10 @@
};
memory@80000000 {
- #address-cells = <1>;
- #size-cells = <1>;
+ #address-cells = <2>;
+ #size-cells = <2>;
device_type = "memory";
- reg = <0x80000000 0x40000000>; /* 1 GiB */
+ reg = <0x0 0x80000000 0x0 0x40000000>; /* 1 GB lowmem */
+ /* 0x1 0x00000000 0x0 0x40000000>; 1 GB highmem */
};
};
--
2.7.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH] ARC: [hsdk] Make it easier to add PAE40 region to DTB
@ 2019-04-02 19:18 ` Vineet Gupta
0 siblings, 0 replies; 6+ messages in thread
From: Vineet Gupta @ 2019-04-02 19:18 UTC (permalink / raw)
To: linux-snps-arc
1. Bump top level address-cells/size-cells nodes to 2 (to ensure all
down stream addresses are 64-bits, unless explicitly specified
otherwise (in "soc" bus with all peripherals)
2. "memory" also specified with address/size 2
3. Add a commented reference for PAE40 region beyond 4GB physical
address space
Signed-off-by: Vineet Gupta <vgupta at synopsys.com>
---
arch/arc/boot/dts/hsdk.dts | 13 +++++++------
1 file changed, 7 insertions(+), 6 deletions(-)
diff --git a/arch/arc/boot/dts/hsdk.dts b/arch/arc/boot/dts/hsdk.dts
index 69bc1c9e8e50..7425bb0f2d1b 100644
--- a/arch/arc/boot/dts/hsdk.dts
+++ b/arch/arc/boot/dts/hsdk.dts
@@ -18,8 +18,8 @@
model = "snps,hsdk";
compatible = "snps,hsdk";
- #address-cells = <1>;
- #size-cells = <1>;
+ #address-cells = <2>;
+ #size-cells = <2>;
chosen {
bootargs = "earlycon=uart8250,mmio32,0xf0005000,115200n8 console=ttyS0,115200n8 debug print-fatal-signals=1";
@@ -105,7 +105,7 @@
#size-cells = <1>;
interrupt-parent = <&idu_intc>;
- ranges = <0x00000000 0xf0000000 0x10000000>;
+ ranges = <0x00000000 0x0 0xf0000000 0x10000000>;
cgu_rst: reset-controller at 8a0 {
compatible = "snps,hsdk-reset";
@@ -269,9 +269,10 @@
};
memory at 80000000 {
- #address-cells = <1>;
- #size-cells = <1>;
+ #address-cells = <2>;
+ #size-cells = <2>;
device_type = "memory";
- reg = <0x80000000 0x40000000>; /* 1 GiB */
+ reg = <0x0 0x80000000 0x0 0x40000000>; /* 1 GB lowmem */
+ /* 0x1 0x00000000 0x0 0x40000000>; 1 GB highmem */
};
};
--
2.7.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] ARC: [hsdk] Make it easier to add PAE40 region to DTB
2019-04-02 19:18 ` Vineet Gupta
@ 2019-04-03 16:34 ` Eugeniy Paltsev
-1 siblings, 0 replies; 6+ messages in thread
From: Eugeniy Paltsev @ 2019-04-03 16:34 UTC (permalink / raw)
To: Vineet Gupta, linux-snps-arc; +Cc: robh, linux-kernel
Hi Vineet,
On Tue, 2019-04-02 at 12:18 -0700, Vineet Gupta wrote:
> 1. Bump top level address-cells/size-cells nodes to 2 (to ensure all
> down stream addresses are 64-bits, unless explicitly specified
> otherwise (in "soc" bus with all peripherals)
>
> 2. "memory" also specified with address/size 2
>
> 3. Add a commented reference for PAE40 region beyond 4GB physical
> address space
[see below]
>
> Signed-off-by: Vineet Gupta <vgupta@synopsys.com>
> ---
> arch/arc/boot/dts/hsdk.dts | 13 +++++++------
> 1 file changed, 7 insertions(+), 6 deletions(-)
>
> diff --git a/arch/arc/boot/dts/hsdk.dts b/arch/arc/boot/dts/hsdk.dts
> index 69bc1c9e8e50..7425bb0f2d1b 100644
> --- a/arch/arc/boot/dts/hsdk.dts
> +++ b/arch/arc/boot/dts/hsdk.dts
> @@ -18,8 +18,8 @@
> model = "snps,hsdk";
> compatible = "snps,hsdk";
>
> - #address-cells = <1>;
> - #size-cells = <1>;
> + #address-cells = <2>;
> + #size-cells = <2>;
>
> chosen {
> bootargs = "earlycon=uart8250,mmio32,0xf0005000,115200n8 console=ttyS0,115200n8 debug print-fatal-signals=1";
> @@ -105,7 +105,7 @@
> #size-cells = <1>;
> interrupt-parent = <&idu_intc>;
>
> - ranges = <0x00000000 0xf0000000 0x10000000>;
> + ranges = <0x00000000 0x0 0xf0000000 0x10000000>;
>
> cgu_rst: reset-controller@8a0 {
> compatible = "snps,hsdk-reset";
> @@ -269,9 +269,10 @@
> };
>
> memory@80000000 {
> - #address-cells = <1>;
> - #size-cells = <1>;
> + #address-cells = <2>;
> + #size-cells = <2>;
> device_type = "memory";
> - reg = <0x80000000 0x40000000>; /* 1 GiB */
> + reg = <0x0 0x80000000 0x0 0x40000000>; /* 1 GB lowmem */
> + /* 0x1 0x00000000 0x0 0x40000000>; 1 GB highmem */
Could you please get rid of comment with reference for PAE40/HIGHMEM region in this patch?
It gives wrong expectations that if we uncomment it and enable PAE40 in config we'll get working system.
But it isn't true - we also need to remap AXI apertures appropriately.
Otherwise kernel will crush somewhere. Moreover kernel will crush for different reasons in case of launch
via U-boot and via MDB (because we remap AXI apertures in U-boot for internal purposes).
If you really want to keep this - I can send you patch with AXI apertures remapping we need to apply firstly.
Thanks.
> };
> };
--
Eugeniy Paltsev
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] ARC: [hsdk] Make it easier to add PAE40 region to DTB
@ 2019-04-03 16:34 ` Eugeniy Paltsev
0 siblings, 0 replies; 6+ messages in thread
From: Eugeniy Paltsev @ 2019-04-03 16:34 UTC (permalink / raw)
To: linux-snps-arc
Hi Vineet,
On Tue, 2019-04-02@12:18 -0700, Vineet Gupta wrote:
> 1. Bump top level address-cells/size-cells nodes to 2 (to ensure all
> down stream addresses are 64-bits, unless explicitly specified
> otherwise (in "soc" bus with all peripherals)
>
> 2. "memory" also specified with address/size 2
>
> 3. Add a commented reference for PAE40 region beyond 4GB physical
> address space
[see below]
>
> Signed-off-by: Vineet Gupta <vgupta at synopsys.com>
> ---
> arch/arc/boot/dts/hsdk.dts | 13 +++++++------
> 1 file changed, 7 insertions(+), 6 deletions(-)
>
> diff --git a/arch/arc/boot/dts/hsdk.dts b/arch/arc/boot/dts/hsdk.dts
> index 69bc1c9e8e50..7425bb0f2d1b 100644
> --- a/arch/arc/boot/dts/hsdk.dts
> +++ b/arch/arc/boot/dts/hsdk.dts
> @@ -18,8 +18,8 @@
> model = "snps,hsdk";
> compatible = "snps,hsdk";
>
> - #address-cells = <1>;
> - #size-cells = <1>;
> + #address-cells = <2>;
> + #size-cells = <2>;
>
> chosen {
> bootargs = "earlycon=uart8250,mmio32,0xf0005000,115200n8 console=ttyS0,115200n8 debug print-fatal-signals=1";
> @@ -105,7 +105,7 @@
> #size-cells = <1>;
> interrupt-parent = <&idu_intc>;
>
> - ranges = <0x00000000 0xf0000000 0x10000000>;
> + ranges = <0x00000000 0x0 0xf0000000 0x10000000>;
>
> cgu_rst: reset-controller at 8a0 {
> compatible = "snps,hsdk-reset";
> @@ -269,9 +269,10 @@
> };
>
> memory at 80000000 {
> - #address-cells = <1>;
> - #size-cells = <1>;
> + #address-cells = <2>;
> + #size-cells = <2>;
> device_type = "memory";
> - reg = <0x80000000 0x40000000>; /* 1 GiB */
> + reg = <0x0 0x80000000 0x0 0x40000000>; /* 1 GB lowmem */
> + /* 0x1 0x00000000 0x0 0x40000000>; 1 GB highmem */
Could you please get rid of comment with reference for PAE40/HIGHMEM region in this patch?
It gives wrong expectations that if we uncomment it and enable PAE40 in config we'll get working system.
But it isn't true - we also need to remap AXI apertures appropriately.
Otherwise kernel will crush somewhere. Moreover kernel will crush for different reasons in case of launch
via U-boot and via MDB (because we remap AXI apertures in U-boot for internal purposes).
If you really want to keep this - I can send you patch with AXI apertures remapping we need to apply firstly.
Thanks.
> };
> };
--
Eugeniy Paltsev
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ARC: [hsdk] Make it easier to add PAE40 region to DTB
2019-04-03 16:34 ` Eugeniy Paltsev
@ 2019-04-03 16:44 ` Vineet Gupta
-1 siblings, 0 replies; 6+ messages in thread
From: Vineet Gupta @ 2019-04-03 16:44 UTC (permalink / raw)
To: Eugeniy Paltsev, linux-snps-arc; +Cc: robh, linux-kernel
On 4/3/19 9:34 AM, Eugeniy Paltsev wrote:
>> - reg = <0x80000000 0x40000000>; /* 1 GiB */
>> + reg = <0x0 0x80000000 0x0 0x40000000>; /* 1 GB lowmem */
>> + /* 0x1 0x00000000 0x0 0x40000000>; 1 GB highmem */
> Could you please get rid of comment with reference for PAE40/HIGHMEM region in this patch?
> It gives wrong expectations that if we uncomment it and enable PAE40 in config we'll get working system.
> But it isn't true - we also need to remap AXI apertures appropriately.
Well I didn't have to do any remapping for successfully running a cross glibc
testsuite over ssh.
This was v5.1-rc1 + HIGHMEM+ PAE + patch to disable ioc + this patch for DT update.
Perhaps the remapping is needed when you are remapping DDR to 0 (for HIGHMEM + !PAE)
> Otherwise kernel will crush somewhere. Moreover kernel will crush for different reasons in case of launch
> via U-boot and via MDB (because we remap AXI apertures in U-boot for internal purposes).
This was all via mdb !
> If you really want to keep this - I can send you patch with AXI apertures remapping we need to apply firstly.
Well I can drop the last commented line but I don't want to do wholesale changes
to DT for enabling DT, so would liek to keep the rest of the patch anyways.
But sure send over the AXI remapping stuff - is that described in hsdk specs ?
-Vineet
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] ARC: [hsdk] Make it easier to add PAE40 region to DTB
@ 2019-04-03 16:44 ` Vineet Gupta
0 siblings, 0 replies; 6+ messages in thread
From: Vineet Gupta @ 2019-04-03 16:44 UTC (permalink / raw)
To: linux-snps-arc
On 4/3/19 9:34 AM, Eugeniy Paltsev wrote:
>> - reg = <0x80000000 0x40000000>; /* 1 GiB */
>> + reg = <0x0 0x80000000 0x0 0x40000000>; /* 1 GB lowmem */
>> + /* 0x1 0x00000000 0x0 0x40000000>; 1 GB highmem */
> Could you please get rid of comment with reference for PAE40/HIGHMEM region in this patch?
> It gives wrong expectations that if we uncomment it and enable PAE40 in config we'll get working system.
> But it isn't true - we also need to remap AXI apertures appropriately.
Well I didn't have to do any remapping for successfully running a cross glibc
testsuite over ssh.
This was v5.1-rc1 + HIGHMEM+ PAE + patch to disable ioc + this patch for DT update.
Perhaps the remapping is needed when you are remapping DDR to 0 (for HIGHMEM + !PAE)
> Otherwise kernel will crush somewhere. Moreover kernel will crush for different reasons in case of launch
> via U-boot and via MDB (because we remap AXI apertures in U-boot for internal purposes).
This was all via mdb !
> If you really want to keep this - I can send you patch with AXI apertures remapping we need to apply firstly.
Well I can drop the last commented line but I don't want to do wholesale changes
to DT for enabling DT, so would liek to keep the rest of the patch anyways.
But sure send over the AXI remapping stuff - is that described in hsdk specs ?
-Vineet
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-04-03 16:44 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-04-02 19:18 [PATCH] ARC: [hsdk] Make it easier to add PAE40 region to DTB Vineet Gupta
2019-04-02 19:18 ` Vineet Gupta
2019-04-03 16:34 ` Eugeniy Paltsev
2019-04-03 16:34 ` Eugeniy Paltsev
2019-04-03 16:44 ` Vineet Gupta
2019-04-03 16:44 ` Vineet Gupta
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.