Re: [RFT PATCH] ARM64: dts: meson-gxbb: Add reserved memory zone and usable memory range

From	Andreas Färber <afaerber@suse.de>
Newsgroups	linux.kernel
Subject	Re: [RFT PATCH] ARM64: dts: meson-gxbb: Add reserved memory zone and usable memory range
Date	2017-01-15 16:50 +0100
Message-ID	<sZVeb-4eg-21@gated-at.bofh.it> (permalink)
References	<sNvSb-341-59@gated-at.bofh.it> <sNGkx-XE-17@gated-at.bofh.it> <sOevU-4RP-19@gated-at.bofh.it> <sR8tX-87O-7@gated-at.bofh.it> <sRuE9-5au-1@gated-at.bofh.it>
Organization	SUSE Linux GmbH

Show all headers | View raw

Am 23.12.2016 um 10:42 schrieb Heinrich Schuchardt:
> it really makes a difference if we write
> 
>  	memory@0 {
>  		device_type = "memory";
>  		linux,usable-memory = <0x0 0x1000000 0x0 0x7f000000>;
>  	};
> 
> or
> 
>  	memory@0 {
>  		device_type = "memory";
>  		reg = <0x0 0x1000000 0x0 0x7f000000>;
>  	};
> 
> The second version leads to failure of the Odroid C2.
> 
> When I looked at /sys/firmware/fdt I saw this difference:
> 
> --- fails
> +++ works
> 
>         memory@0 {
> -               device_type = "memory";
>                 reg = <0x0 0x0 0x0 0x78000000>;
> +               device_type = "memory";
> +               linux,usable-memory = <0x0 0x1000000 0x0 0x7f000000>;
>         };
> 
> I found the following sentence in the NXP forum:
> In case you want to overwrite the memory usage passed from u-boot, you
> can use "linux,usable-memory".
> https://community.nxp.com/thread/382284

The Odroid-C2 is in mainline U-Boot. Please submit a patch to U-Boot
instead of forcing the creation of unnecessary new .dts files onto
everyone due to hardcoded linux,usable-memory properties. In fact, it
already reserves 0x1000000, so it seems you are merely using an older
U-Boot.

http://git.denx.de/?p=u-boot.git;a=blob;f=arch/arm/mach-meson/board.c;h=f159cbf849f75ab046e6f3a025bbc97c0bcfd59d;hb=HEAD#l39

I would bet that the upper limit is unrelated here.

Regards,
Andreas

-- 
SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer, Jane Smithard, Graham Norton
HRB 21284 (AG Nürnberg)

Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread

Thread

Re: [RFT PATCH] ARM64: dts: meson-gxbb: Add reserved memory zone and  usable memory range Andreas Färber <afaerber@suse.de> - 2017-01-15 16:50 +0100
  Re: [RFT PATCH] ARM64: dts: meson-gxbb: Add reserved memory zone and  usable memory range Neil Armstrong <narmstrong@baylibre.com> - 2017-01-16 11:30 +0100
    Re: [RFT PATCH] ARM64: dts: meson-gxbb: Add reserved memory zone and  usable memory range Andreas Färber <afaerber@suse.de> - 2017-01-16 15:10 +0100

csiph-web