[PATCH v2] ARM: dts: aspeed: Update reserved memory nodes for zaius and witherspoon

Joel Stanley joel at jms.id.au
Fri Feb 17 16:38:40 AEDT 2017


On Fri, Feb 17, 2017 at 1:42 PM, Cyril Bur <cyrilbur at gmail.com> wrote:
> Zaius platform doesn't use VGA so don't reserve that memory
>
> Signed-off-by: Cyril Bur <cyrilbur at gmail.com>
> ---
> v2: Drop reserving the framebuffer on Zaius
>     Both platforms have 1GB not 512MB - adjust numbers

Witherspoon has 512MB of DDR. I think Lei was trying to point out that
the memory node needed fixing.

(The pass1 BMC boards for Witherspoon did have 1GB, but it was changed
to 512MB. As far as I know we got rid of the pass1 boards so we only
need to support 512MB systems).

Cheers,

Joel


>     Add comment requested by Joel.
>
>  arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts | 15 +++++++++++++--
>  arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts       | 12 ++++++++++--
>  2 files changed, 23 insertions(+), 4 deletions(-)
>
> diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts b/arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts
> index 4b2d8aa8abb6..b79cadb2f6d7 100644
> --- a/arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts
> +++ b/arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts
> @@ -25,9 +25,20 @@
>                 #size-cells = <1>;
>                 ranges;
>
> -               flash_memory: region at 94000000 {
> +               /*
> +                * The addreses for the flash_memory reserved range are the
> +                * highest allowable for the size of the mapping (ie, the
> +                * address has no set bits greater than the size)
> +                */
> +
> +               flash_memory: region at b8000000 {
> +                       no-map;
> +                       reg = <0xb8000000 0x04000000>; /* 64M */
> +               };
> +
> +               vga_memory: framebuffer at bc000000 {
>                         no-map;
> -                       reg = <0x94000000 0x04000000>; /* 64M */
> +                       reg = <0xbc000000 0x04000000>; /* 64M */
>                 };
>         };
>
> diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts b/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts
> index e1c9b3f4fe44..5a06da43801b 100644
> --- a/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts
> +++ b/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts
> @@ -25,10 +25,18 @@
>                 #size-cells = <1>;
>                 ranges;
>
> -               flash_memory: region at 94000000 {
> +               /*
> +                * The addreses for the flash_memory reserved range are the
> +                * highest allowable for the size of the mapping (ie, the
> +                * address has no set bits greater than the size)
> +                */
> +
> +               flash_memory: region at bc000000 {
>                         no-map;
> -                       reg = <0x94000000 0x04000000>; /* 64M */
> +                       reg = <0xbc000000 0x04000000>; /* 64M */
>                 };
> +
> +               /* Zaius doesn't use VGA so don't reserve VGA buffer */
>         };
>
>         onewire0 {
> --
> 2.11.1
>


More information about the openbmc mailing list