diff mbox series

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

Message ID 20230712014017.28123-1-ansuelsmth@gmail.com
State New
Headers
Series ARM: dts: BCM5301X: Extend RAM to full 256MB for Linksys EA6500 V2 |

Commit Message

Christian Marangi July 12, 2023, 1:40 a.m. UTC 
  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(-)

Comments

Andrew Lunn July 12, 2023, 3:23 p.m. UTC | #1 
> --- 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
Rafał Miłecki July 12, 2023, 5:07 p.m. UTC | #2 
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.
Rafał Miłecki July 12, 2023, 5:07 p.m. UTC | #3 
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>
Florian Fainelli July 12, 2023, 5:23 p.m. UTC | #4 
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
diff mbox series

Patch

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 {