From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kevin Hilman Subject: Re: [PATCH] ARM: dts: Kill off skeleton{64}.dtsi Date: Tue, 04 Dec 2018 14:36:10 -0800 Message-ID: <7h7egoor6t.fsf@baylibre.com> References: <20181130170634.9467-1-robh@kernel.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20181130170634.9467-1-robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+glpam-linux-mediatek=m.gmane.org-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org To: Rob Herring , arm-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org Cc: Andrew Lunn , Alexandre Belloni , Neil Armstrong , Maxime Ripard , Linus Walleij , Baruch Siach , David Brown , Jamie Iles , Robert Jarzmik , linux-stm32-XDFAJ8BFU24N7RejjzZ/Li2xQDfSxrLKVpNB7YpNyf8@public.gmane.org, Alexandre Torgue , Jesper Nilsson , Florian Fainelli , Viresh Kumar , Gregory Clement , =?utf-8?Q?Rafa?= =?utf-8?Q?=C5=82_Mi=C5=82ecki?= , Chen-Yu Tsai , linux-arm-kernel-VrBV9hrLPhE@public.gmane.org, Jon Mason , Ludovic Desroches , bcm-kernel-feedback-list-dY08KVG/lbpWk0Htik3J/w@public.gmane.org, Hauke Mehrtens List-Id: devicetree@vger.kernel.org Rob Herring writes: > 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. [...] I gave this a spin through the kernelCI on multiple SoCs, and builds and no new regressions for build or boot that I spotted. Tested-by: Kevin Hilman > arch/arm/boot/dts/meson.dtsi | 3 ++- > arch/arm/boot/dts/meson6-atv1200.dts | 1 + > arch/arm/boot/dts/meson8-minix-neo-x8.dts | 1 + > arch/arm/boot/dts/meson8b-ec100.dts | 1 + > arch/arm/boot/dts/meson8b-mxq.dts | 1 + > arch/arm/boot/dts/meson8b-odroidc1.dts | 1 + > arch/arm/boot/dts/meson8m2-mxiii-plus.dts | 1 + also for the amlogic/meson platforms, Reviewed-by: Kevin Hilman Acked-by: Kevin Hilman Kevin