Re: [PATCH] ARM: dts: Kill off skeleton{64}.dtsi

From: Alexandre Torgue <alexandre.torgue-qxv4g6HH51o@public.gmane.org>
To: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
	arm-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org
Cc: "Andrew Lunn" <andrew-g2DYL2Zd6BY@public.gmane.org>,
	"Alexandre Belloni"
	<alexandre.belloni-LDxbnhwyfcJBDgjK7y7TUQ@public.gmane.org>,
	"Neil Armstrong"
	<narmstrong-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>,
	"Maxime Ripard"
	<maxime.ripard-LDxbnhwyfcJBDgjK7y7TUQ@public.gmane.org>,
	"Linus Walleij"
	<linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
	"Baruch Siach" <baruch-NswTu9S1W3P6gbPvEgmw2w@public.gmane.org>,
	"David Brown"
	<david.brown-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
	"Jamie Iles" <jamie-wmLquQDDieKakBO8gow8eQ@public.gmane.org>,
	"Robert Jarzmik" <robert.jarzmik-GANU6spQydw@public.gmane.org>,
	linux-stm32-XDFAJ8BFU24N7RejjzZ/Li2xQDfSxrLKVpNB7YpNyf8@public.gmane.org,
	"Jesper Nilsson" <jesper.nilsson-VrBV9hrLPhE@public.gmane.org>,
	"Florian Fainelli"
	<f.fainelli-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	"Kevin Hilman" <khilman-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>,
	"Gregory Clement"
	<gregory.clement-LDxbnhwyfcJBDgjK7y7TUQ@public.gmane.org>,
	"Rafał Miłecki" <zajec5-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	"Chen-Yu Tsai" <wens-jdAy2FN1RRM@public.gmane.org>,
	linux-arm-kernel-VrBV9hrLPhE@public.gmane.org,
	"Jon Mason" <jonmason-dY08KVG/lbpWk0Htik3J/w@public.gmane.org>,
	"Ludovic Desroches"
	<ludovic.desroches-UWL1GkI3JZL3oGB3hsPCZA@public.gmane.org>,
	bcm-kernel-feedback-list-dY08KVG/lbpWk0Htik3J/w@public.gmane.org,
	"Hauke Mehrtens" <hauke-5/S+JYg5SzeELgA04lAiVw@public.gmane.org>,
	"Andy Gross" <andy.gross-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
Subject: Re: [PATCH] ARM: dts: Kill off skeleton{64}.dtsi
Date: Mon, 3 Dec 2018 11:29:09 +0100	[thread overview]
Message-ID: <b34072f1-d0f3-8115-741b-fa70d5723f0f@st.com> (raw)
In-Reply-To: <20181130170634.9467-1-robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>

Hi Rob

On 11/30/18 6:06 PM, Rob Herring wrote:
> Remove the usage of skeleton.dtsi in the remaining dts files. It was
> deprecated since commit 9c0da3cc61f1 ("ARM: dts: explicitly mark
> skeleton.dtsi as deprecated"). This will make adding a unit-address to
> memory nodes easier.
> 
> The main tricky part to removing skeleton.dtsi is we could end up with
> no /memory node at all when a bootloader depends on one being present. I
> hacked up dtc to check for this condition.
> 
> Signed-off-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
> ---
> Yes, I'm being lazy and not splitting this up into dozens of sub-arch
> patches. I don't think this should conflict nor should we be getting new
> dependencies on skeleton.dtsi. So please apply directly to arm-soc.
> 
> Rob

...

>   arch/arm/boot/dts/stm32429i-eval.dts           |  1 +
>   arch/arm/boot/dts/stm32746g-eval.dts           |  1 +
>   arch/arm/boot/dts/stm32f429-disco.dts          |  1 +
>   arch/arm/boot/dts/stm32f429.dtsi               |  4 +++-
>   arch/arm/boot/dts/stm32f469-disco.dts          |  1 +
>   arch/arm/boot/dts/stm32f746-disco.dts          |  1 +
>   arch/arm/boot/dts/stm32f746.dtsi               |  4 +++-
>   arch/arm/boot/dts/stm32f769-disco.dts          |  1 +
>   arch/arm/boot/dts/stm32h743.dtsi               |  4 +++-
>   arch/arm/boot/dts/stm32h743i-disco.dts         |  1 +
>   arch/arm/boot/dts/stm32h743i-eval.dts          |  1 +
>   arch/arm/boot/dts/stm32mp157c-ed1.dts          |  1 +

For STM32 files:

Acked-by: Alexandre TORGUE <alexandre.torgue-qxv4g6HH51o@public.gmane.org>

Regards
Alex