2023-07-12 09:34:15

by Christian Marangi

[permalink] [raw]
Subject: [PATCH] ARM: dts: BCM5301X: Extend RAM to full 256MB for Linksys EA6500 V2

From: Aleksey Nasibulin <[email protected]>

Linksys ea6500-v2 have 256MB of ram. Currently we only use 128MB.
Expand the definition to use all the available RAM.

Fixes: 03e96644d7a8 ("ARM: dts: BCM5301X: Add basic DT for Linksys EA6500 V2")
Signed-off-by: Aleksey Nasibulin <[email protected]>
Signed-off-by: Christian Marangi <[email protected]>
Cc: [email protected]
---
arch/arm/boot/dts/broadcom/bcm4708-linksys-ea6500-v2.dts | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/broadcom/bcm4708-linksys-ea6500-v2.dts b/arch/arm/boot/dts/broadcom/bcm4708-linksys-ea6500-v2.dts
index f1412ba83def..0454423fe166 100644
--- a/arch/arm/boot/dts/broadcom/bcm4708-linksys-ea6500-v2.dts
+++ b/arch/arm/boot/dts/broadcom/bcm4708-linksys-ea6500-v2.dts
@@ -19,7 +19,8 @@ chosen {

memory@0 {
device_type = "memory";
- reg = <0x00000000 0x08000000>;
+ reg = <0x00000000 0x08000000>,
+ <0x88000000 0x08000000>;
};

gpio-keys {
--
2.40.1



2023-07-12 15:44:10

by Andrew Lunn

[permalink] [raw]
Subject: Re: [PATCH] ARM: dts: BCM5301X: Extend RAM to full 256MB for Linksys EA6500 V2

> --- a/arch/arm/boot/dts/broadcom/bcm4708-linksys-ea6500-v2.dts
> +++ b/arch/arm/boot/dts/broadcom/bcm4708-linksys-ea6500-v2.dts
> @@ -19,7 +19,8 @@ chosen {
>
> memory@0 {
> device_type = "memory";
> - reg = <0x00000000 0x08000000>;
> + reg = <0x00000000 0x08000000>,
> + <0x88000000 0x08000000>;

So there is a hole in the middle? Does this require any special
CONFIG_ option to support sparse memory?

Andrew

2023-07-12 17:35:27

by Rafał Miłecki

[permalink] [raw]
Subject: Re: [PATCH] ARM: dts: BCM5301X: Extend RAM to full 256MB for Linksys EA6500 V2

On 12.07.2023 17:23, Andrew Lunn wrote:
>> --- a/arch/arm/boot/dts/broadcom/bcm4708-linksys-ea6500-v2.dts
>> +++ b/arch/arm/boot/dts/broadcom/bcm4708-linksys-ea6500-v2.dts
>> @@ -19,7 +19,8 @@ chosen {
>>
>> memory@0 {
>> device_type = "memory";
>> - reg = <0x00000000 0x08000000>;
>> + reg = <0x00000000 0x08000000>,
>> + <0x88000000 0x08000000>;
>
> So there is a hole in the middle? Does this require any special
> CONFIG_ option to support sparse memory?

This is how RAM is mapped on all known Northstar devices. Using two
blocks.

I've heard there is a way to design Northstar SoC board to have all RAM
in one region but thay requires disabling something (I don't remember
what) and in practice to vendor decided to do that.

AFAIK no extra CONFIG_* option is needed to make Linux map multiple RAM
regions.

2023-07-12 17:38:27

by Rafał Miłecki

[permalink] [raw]
Subject: Re: [PATCH] ARM: dts: BCM5301X: Extend RAM to full 256MB for Linksys EA6500 V2

On 12.07.2023 03:40, Christian Marangi wrote:
> From: Aleksey Nasibulin <[email protected]>
>
> Linksys ea6500-v2 have 256MB of ram. Currently we only use 128MB.
> Expand the definition to use all the available RAM.
>
> Fixes: 03e96644d7a8 ("ARM: dts: BCM5301X: Add basic DT for Linksys EA6500 V2")
> Signed-off-by: Aleksey Nasibulin <[email protected]>
> Signed-off-by: Christian Marangi <[email protected]>
> Cc: [email protected]

Acked-by: Rafał Miłecki <[email protected]>