ARM: dts: BCM5301X: Extend RAM to full 256MB for Linksys EA6500 V2

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

Posted by Christian Marangi 1 year, 9 months ago

From: Aleksey Nasibulin <alealexpro100@ya.ru>

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 <alealexpro100@ya.ru>
Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
Cc: stable@vger.kernel.org
---
 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

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

Posted by Andrew Lunn 1 year, 9 months ago

> --- 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

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

Posted by Rafał Miłecki 1 year, 9 months ago

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.

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

Posted by Rafał Miłecki 1 year, 9 months ago

On 12.07.2023 03:40, Christian Marangi wrote:
> From: Aleksey Nasibulin <alealexpro100@ya.ru>
> 
> 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 <alealexpro100@ya.ru>
> Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
> Cc: stable@vger.kernel.org

Acked-by: Rafał Miłecki <rafal@milecki.pl>

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

Posted by Florian Fainelli 1 year, 9 months ago

From: Florian Fainelli <f.fainelli@gmail.com>

On Wed, 12 Jul 2023 03:40:17 +0200, Christian Marangi <ansuelsmth@gmail.com> wrote:
> From: Aleksey Nasibulin <alealexpro100@ya.ru>
> 
> 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 <alealexpro100@ya.ru>
> Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
> Cc: stable@vger.kernel.org
> ---

Applied to https://github.com/Broadcom/stblinux/commits/devicetree/next, thanks!
--
Florian