From mboxrd@z Thu Jan 1 00:00:00 1970 From: Florian Fainelli Subject: Re: [PATCH] ARM: dts: Kill off skeleton{64}.dtsi Date: Fri, 30 Nov 2018 09:52:15 -0800 Message-ID: <3d46df10-9b85-7ecf-9302-0cde08b9283a@gmail.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> Content-Language: en-US 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 , Kevin Hilman , Gregory Clement , =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= , 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 On 11/30/2018 9:06 AM, 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 > --- > arch/arm/boot/dts/bcm-cygnus.dtsi | 9 +++++++-- > arch/arm/boot/dts/bcm-nsp.dtsi | 4 ++-- > arch/arm/boot/dts/bcm11351.dtsi | 4 ++-- > arch/arm/boot/dts/bcm21664-garnet.dts | 1 + > arch/arm/boot/dts/bcm21664.dtsi | 4 ++-- > arch/arm/boot/dts/bcm23550-sparrow.dts | 1 + > arch/arm/boot/dts/bcm23550.dtsi | 4 ++-- > arch/arm/boot/dts/bcm28155-ap.dts | 1 + > arch/arm/boot/dts/bcm47081-asus-rt-n18u.dts | 1 + > .../boot/dts/bcm47081-buffalo-wzr-600dhp2.dts | 1 + > .../boot/dts/bcm47081-buffalo-wzr-900dhp.dts | 1 + > arch/arm/boot/dts/bcm47081-luxul-xap-1410.dts | 1 + > arch/arm/boot/dts/bcm47081-luxul-xwr-1200.dts | 1 + > .../boot/dts/bcm47081-tplink-archer-c5-v2.dts | 1 + > arch/arm/boot/dts/bcm47094-dlink-dir-885l.dts | 1 + > .../arm/boot/dts/bcm47094-linksys-panamera.dts | 1 + > arch/arm/boot/dts/bcm47094-luxul-abr-4500.dts | 1 + > arch/arm/boot/dts/bcm47094-luxul-xap-1610.dts | 1 + > arch/arm/boot/dts/bcm47094-luxul-xbr-4500.dts | 1 + > arch/arm/boot/dts/bcm47094-luxul-xwr-3100.dts | 1 + > .../boot/dts/bcm47094-luxul-xwr-3150-v1.dts | 1 + > arch/arm/boot/dts/bcm47094-netgear-r8500.dts | 1 + > arch/arm/boot/dts/bcm47189-luxul-xap-1440.dts | 1 + > arch/arm/boot/dts/bcm47189-luxul-xap-810.dts | 1 + > arch/arm/boot/dts/bcm47189-tenda-ac9.dts | 1 + > arch/arm/boot/dts/bcm5301x.dtsi | 3 ++- > .../boot/dts/bcm53340-ubnt-unifi-switch8.dts | 1 + > arch/arm/boot/dts/bcm53573.dtsi | 3 ++- > arch/arm/boot/dts/bcm63138.dtsi | 4 ++-- > arch/arm/boot/dts/bcm7445.dtsi | 2 -- > arch/arm/boot/dts/bcm947189acdbmr.dts | 1 + > arch/arm/boot/dts/bcm953012er.dts | 1 + > arch/arm/boot/dts/bcm953012hr.dts | 1 + > arch/arm/boot/dts/bcm953012k.dts | 1 + For these files above (Broadcom): Reviewed-by: Florian Fainelli Thanks! -- Florian