From: Florian Fainelli <f.fainelli-Re5JQEeQqe8AvxtiuMwx3w@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,
"Alexandre Torgue"
<alexandre.torgue-qxv4g6HH51o@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>
Subject: Re: [PATCH] ARM: dts: Kill off skeleton{64}.dtsi
Date: Fri, 30 Nov 2018 09:52:15 -0800 [thread overview]
Message-ID: <3d46df10-9b85-7ecf-9302-0cde08b9283a@gmail.com> (raw)
In-Reply-To: <20181130170634.9467-1-robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.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 <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
> ---
> 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 <f.fainelli-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Thanks!
--
Florian
next prev parent reply other threads:[~2018-11-30 17:52 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-11-30 17:06 [PATCH] ARM: dts: Kill off skeleton{64}.dtsi Rob Herring
[not found] ` <20181130170634.9467-1-robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2018-11-30 17:21 ` Linus Walleij
2018-11-30 17:52 ` Florian Fainelli [this message]
2018-12-02 19:44 ` Matthias Brugger
2018-12-03 4:02 ` Chen-Yu Tsai
2018-12-03 5:54 ` Viresh Kumar
2018-12-03 7:50 ` Maxime Ripard
2018-12-03 8:19 ` Alexandre Belloni
2018-12-03 9:01 ` Neil Armstrong
2018-12-03 9:23 ` Antoine Tenart
2018-12-03 10:29 ` Alexandre Torgue
2018-12-03 21:56 ` Robert Jarzmik
2018-12-03 22:27 ` Vladimir Zapolskiy
2018-12-04 22:36 ` Kevin Hilman
2018-12-05 22:21 ` Martin Blumenstingl
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=3d46df10-9b85-7ecf-9302-0cde08b9283a@gmail.com \
--to=f.fainelli-re5jqeeqqe8avxtiumwx3w@public.gmane.org \
--cc=alexandre.belloni-LDxbnhwyfcJBDgjK7y7TUQ@public.gmane.org \
--cc=alexandre.torgue-qxv4g6HH51o@public.gmane.org \
--cc=andrew-g2DYL2Zd6BY@public.gmane.org \
--cc=arm-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
--cc=baruch-NswTu9S1W3P6gbPvEgmw2w@public.gmane.org \
--cc=bcm-kernel-feedback-list-dY08KVG/lbpWk0Htik3J/w@public.gmane.org \
--cc=david.brown-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
--cc=gregory.clement-LDxbnhwyfcJBDgjK7y7TUQ@public.gmane.org \
--cc=hauke-5/S+JYg5SzeELgA04lAiVw@public.gmane.org \
--cc=jamie-wmLquQDDieKakBO8gow8eQ@public.gmane.org \
--cc=jesper.nilsson-VrBV9hrLPhE@public.gmane.org \
--cc=jonmason-dY08KVG/lbpWk0Htik3J/w@public.gmane.org \
--cc=khilman-rdvid1DuHRBWk0Htik3J/w@public.gmane.org \
--cc=linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
--cc=linux-arm-kernel-VrBV9hrLPhE@public.gmane.org \
--cc=linux-stm32-XDFAJ8BFU24N7RejjzZ/Li2xQDfSxrLKVpNB7YpNyf8@public.gmane.org \
--cc=ludovic.desroches-UWL1GkI3JZL3oGB3hsPCZA@public.gmane.org \
--cc=maxime.ripard-LDxbnhwyfcJBDgjK7y7TUQ@public.gmane.org \
--cc=narmstrong-rdvid1DuHRBWk0Htik3J/w@public.gmane.org \
--cc=robert.jarzmik-GANU6spQydw@public.gmane.org \
--cc=robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
--cc=wens-jdAy2FN1RRM@public.gmane.org \
--cc=zajec5-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
/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