From: Kevin Hilman <khilman-rdvid1DuHRBWk0Htik3J/w@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>,
"Viresh Kumar" <vireshk-DgEjT+Ai2ygdnm+yROfE0A@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
Subject: Re: [PATCH] ARM: dts: Kill off skeleton{64}.dtsi
Date: Tue, 04 Dec 2018 14:36:10 -0800 [thread overview]
Message-ID: <7h7egoor6t.fsf@baylibre.com> (raw)
In-Reply-To: <20181130170634.9467-1-robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> 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 <khilman-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
> 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 <khilman-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
Acked-by: Kevin Hilman <khilman-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
Kevin
next prev parent reply other threads:[~2018-12-04 22:36 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
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 [this message]
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=7h7egoor6t.fsf@baylibre.com \
--to=khilman-rdvid1duhrbwk0htik3j/w@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=f.fainelli-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
--cc=gregory.clement-LDxbnhwyfcJBDgjK7y7TUQ@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=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=vireshk-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;
as well as URLs for NNTP newsgroup(s).