[PATCH linux 1/3] ARM: dts: aspeed: Update palmetto device tree
Joel Stanley
joel at jms.id.au
Tue Dec 13 11:44:35 AEDT 2016
Hi Cyril,
On Fri, Dec 9, 2016 at 4:43 PM, Cyril Bur <cyrilbur at gmail.com> wrote:
> Palmettos have 512mb of ram. There is always framebuffer memory at the
> top of ram.
>
> This patch also reserves BMC ram for host to BMC communication.
>
> Signed-off-by: Cyril Bur <cyrilbur at gmail.com>
> ---
> arch/arm/boot/dts/aspeed-bmc-opp-palmetto.dts | 25 ++++++++++++++++++++++++-
> arch/arm/boot/dts/aspeed-g4.dtsi | 6 ++++++
> 2 files changed, 30 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-palmetto.dts b/arch/arm/boot/dts/aspeed-bmc-opp-palmetto.dts
> index acaceda2..e810ae7 100644
> --- a/arch/arm/boot/dts/aspeed-bmc-opp-palmetto.dts
> +++ b/arch/arm/boot/dts/aspeed-bmc-opp-palmetto.dts
> @@ -17,7 +17,30 @@
> };
>
> memory {
> - reg = <0x40000000 0x10000000>;
> + reg = <0x40000000 0x20000000>;
> + };
Can you send this in a separate patch? It should go into the tree now.
> +
> + reserved-memory {
> + #address-cells = <1>;
> + #size-cells= <1>;
> + ranges;
> +
> + flash_mem: region at 52000000 {
> + compatible = "aspeed,lpc-ctrl";
> + no-map;
> + reg = <0x54000000 0x04000000>; /* 64m */
> + };
> +
> + fb_mem: framebuffer at 5f000000 {
> + no-map;
> + reg = <0x5f000000 0x01000000>; /* 16m */
> + };
This should be a separate patch too, which we can take straight away.
> + };
> +
> + flash_buffer {
It's convention to put the reg value after @ in the node name:
flash_buffer at 1e789000
> + compatible = "aspeed,lpc-ctrl";
Upstream demands we put the SoC in the compatible string
aspeed,ast2400-lpc-ctrl.
> + memory-region = <&flash_mem>;
> + reg = <0x1e789000 8>;
Make the second cell be 0x8 for consistency.
> };
> --- a/arch/arm/boot/dts/aspeed-g4.dtsi
> +++ b/arch/arm/boot/dts/aspeed-g4.dtsi
> @@ -870,6 +870,12 @@
> interrupts = <8>;
> };
>
> + mbox: mbox at 1e789200 {
> + compatible = "aspeed,mbox-host";
aspeed,ast2400-mbox-host
Not sure on the name here.
> + reg = <0x1e789200 0x5c>;
> + interrupts = <46>;
> + };
> +
> wdt1: wdt at 1e785000 {
> compatible = "aspeed,ast2400-wdt";
> reg = <0x1e785000 0x1c>;
> --
> 2.10.2
>
> _______________________________________________
> openbmc mailing list
> openbmc at lists.ozlabs.org
> https://lists.ozlabs.org/listinfo/openbmc
More information about the openbmc
mailing list