linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Enric Balletbo Serra <eballetbo@gmail.com>
To: Javier Martinez Canillas <javier@osg.samsung.com>
Cc: linux-kernel <linux-kernel@vger.kernel.org>,
	Mark Rutland <mark.rutland@arm.com>,
	Rob Herring <robh+dt@kernel.org>,
	Tony Lindgren <tony@atomide.com>,
	"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>,
	Enric Balletbo i Serra <enric.balletbo@collabora.com>
Subject: Re: [PATCH 01/23] ARM: am335x: Add missing memory node in sl50 board
Date: Wed, 31 Aug 2016 15:09:17 +0200	[thread overview]
Message-ID: <CAFqH_51F-1n3mp8_Ags4LSCR5DX10vDVEvLCXpnWvHY=Mi0cTA@mail.gmail.com> (raw)
In-Reply-To: <1472639737-17586-2-git-send-email-javier@osg.samsung.com>

2016-08-31 12:35 GMT+02:00 Javier Martinez Canillas <javier@osg.samsung.com>:
> The skeleton.dtsi file was removed in ARM64 for different reasons as
> explained in commit ("3ebee5a2e141 arm64: dts: kill skeleton.dtsi").
>
> These also applies to ARM and it will also allow to get rid of the
> following DTC warnings in the future:
>
> "Node /memory has a reg or ranges property, but no unit name"
>
> But the sl50 board doesn't have a, so removing the  skeleton.dtsi
> inclusion from am33xx.dtsi will cause a change in the compiled DTB.
>
> The board has 512 MiB of RAM and its starting address is 0x80000000,
> so add a proper memory device node in the DTS.
>
> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
>
> ---
>
>  arch/arm/boot/dts/am335x-sl50.dts | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/arch/arm/boot/dts/am335x-sl50.dts b/arch/arm/boot/dts/am335x-sl50.dts
> index f802adb23ff9..e3be7a6bf8f1 100644
> --- a/arch/arm/boot/dts/am335x-sl50.dts
> +++ b/arch/arm/boot/dts/am335x-sl50.dts
> @@ -19,6 +19,11 @@
>                 };
>         };
>
> +       memory {
> +               device_type = "memory";
> +               reg = <0x80000000 0x20000000>; /* 512 MB */
> +       };
> +
>         chosen {
>                 stdout-path = &uart0;
>         };
> --
> 2.7.4
>

Acked-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>

Thanks,
 Enric

  reply	other threads:[~2016-08-31 13:09 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-31 10:35 [PATCH 00/23] ARM: dts: omap: Remove skeleton.dtsi usage and fix memory node DTC warning" Javier Martinez Canillas
2016-08-31 10:35 ` [PATCH 01/23] ARM: am335x: Add missing memory node in sl50 board Javier Martinez Canillas
2016-08-31 13:09   ` Enric Balletbo Serra [this message]
2016-08-31 10:35 ` [PATCH 02/23] ARM: omap3: Add missing memory node in overo and torpedo boards Javier Martinez Canillas
2016-08-31 10:35 ` [PATCH 03/23] ARM: dts: omap2: Remove skeleton.dtsi usage Javier Martinez Canillas
2016-08-31 10:35 ` [PATCH 04/23] ARM: dts: omap3: " Javier Martinez Canillas
2016-08-31 10:35 ` [PATCH 05/23] ARM: dts: omap4: " Javier Martinez Canillas
2016-08-31 10:35 ` [PATCH 06/23] ARM: dts: omap5: " Javier Martinez Canillas
2016-08-31 10:35 ` [PATCH 07/23] ARM: dts: am33xx: " Javier Martinez Canillas
2016-08-31 10:35 ` [PATCH 08/23] ARM: dts: dm814x: " Javier Martinez Canillas
2016-08-31 10:35 ` [PATCH 09/23] ARM: dts: dm816x: " Javier Martinez Canillas
2016-08-31 10:35 ` [PATCH 10/23] ARM: dts: dra7: " Javier Martinez Canillas
2016-08-31 10:35 ` [PATCH 11/23] ARM: dts: am4372: " Javier Martinez Canillas
2016-08-31 10:35 ` [PATCH 12/23] ARM: dts: omap2: Add missing unit name to memory nodes Javier Martinez Canillas
2016-08-31 10:35 ` [PATCH 13/23] ARM: dts: omap3: " Javier Martinez Canillas
2016-08-31 10:35 ` [PATCH 14/23] ARM: dts: omap4: " Javier Martinez Canillas
2016-08-31 10:35 ` [PATCH 15/23] ARM: dts: omap5: " Javier Martinez Canillas
2016-08-31 10:35 ` [PATCH 16/23] ARM: dts: am335x: " Javier Martinez Canillas
2016-08-31 10:35 ` [PATCH 17/23] ARM: dts: am3517: " Javier Martinez Canillas
2016-08-31 10:35 ` [PATCH 18/23] ARM: dts: am4372: " Javier Martinez Canillas
2016-08-31 10:35 ` [PATCH 19/23] ARM: dts: am572x: " Javier Martinez Canillas
2016-08-31 10:35 ` [PATCH 20/23] ARM: dts: dm8148: " Javier Martinez Canillas
2016-08-31 10:35 ` [PATCH 21/23] ARM: dts: dm8168: " Javier Martinez Canillas
2016-08-31 10:35 ` [PATCH 22/23] ARM: dts: dra62x: " Javier Martinez Canillas
2016-08-31 10:35 ` [PATCH 23/23] ARM: dts: dra7: " Javier Martinez Canillas
2016-08-31 14:24 ` [PATCH 00/23] ARM: dts: omap: Remove skeleton.dtsi usage and fix memory node DTC warning" Tony Lindgren
2016-08-31 17:54   ` Tony Lindgren
2016-09-01  8:46     ` Javier Martinez Canillas

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAFqH_51F-1n3mp8_Ags4LSCR5DX10vDVEvLCXpnWvHY=Mi0cTA@mail.gmail.com' \
    --to=eballetbo@gmail.com \
    --cc=enric.balletbo@collabora.com \
    --cc=javier@osg.samsung.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=robh+dt@kernel.org \
    --cc=tony@atomide.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).