From: Geert Uytterhoeven <geert+renesas@glider.be>
To: Simon Horman <horms@verge.net.au>,
Magnus Damm <magnus.damm@gmail.com>,
Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>,
Rob Herring <robh+dt@kernel.org>
Cc: Mark Rutland <mark.rutland@arm.com>,
Ian Campbell <ijc+devicetree@hellion.org.uk>,
Kumar Gala <galak@codeaurora.org>,
Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>,
devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
linux-renesas-soc@vger.kernel.org,
Geert Uytterhoeven <geert+renesas@glider.be>
Subject: [PATCH v2 00/23] ARM/arm64: dts: renesas: Fix W=1 dtc warnings
Date: Fri, 20 May 2016 09:09:51 +0200 [thread overview]
Message-ID: <1463728214-13838-1-git-send-email-geert+renesas@glider.be> (raw)
Hi Simon, Magnus, Morimoto-san, Rob,
This patch series fixes the new warnings related to Renesas arm32/arm64
DTSes that can be seen in -next using "make W=1 dtbs", courtesy of
commit b993734718c01064 ("scripts/dtc: Update to upstream version
53bf130b1cdd") in Rob herring's for-next branch. For testing, you can just
cherry-pick that commit (showing the warning for W=1 only was a separate
commit).
All warnings were of the form:
Warning (unit_address_vs_reg): Node /cache-controller@0 has a unit name, but no reg property
Warning (unit_address_vs_reg): Node /cache-controller has a reg or ranges property, but no unit name
Warning (unit_address_vs_reg): Node /clocks@e0110000/iic0_sclkdiv has a reg or ranges property, but no unit name
Warning (unit_address_vs_reg): Node /i2c@2 has a unit name, but no reg property
Warning (unit_address_vs_reg): Node /keyboard/button@1 has a unit name, but no reg property
Warning (unit_address_vs_reg): Node /memory has a reg or ranges property, but no unit name
Warning (unit_address_vs_reg): Node /pfc@e0140200/serial@e1030000 has a unit name, but no reg property
Warning (unit_address_vs_reg): Node /regulator@0 has a unit name, but no reg property
Warning (unit_address_vs_reg): Node /sound@ec500000/rcar_sound,ctu/ctu@0 has a unit name, but no reg property
Warning (unit_address_vs_reg): Node /sound@ec500000/rcar_sound,dvc/dvc@0 has a unit name, but no reg property
Warning (unit_address_vs_reg): Node /sound@ec500000/rcar_sound,mix/mix@0 has a unit name, but no reg property
Warning (unit_address_vs_reg): Node /sound@ec500000/rcar_sound,src/src@0 has a unit name, but no reg property
Warning (unit_address_vs_reg): Node /sound@ec500000/rcar_sound,ssi/ssi@0 has a unit name, but no reg property
Fixes:
- For cache-controllers of Cortex A7, A15, A53, and A57, I moved them
inside the /cpus node, and made their unit names and reg properties
match the MPIDR values,
- For clocks, I added the missing unit names,
- For regulators, i2c-gpios buses, and sound nodes, I (more-or-less)
replaced the '@' by a dash,
- For buttons, I changed the node name to match the key events,
- For pinctrl, I changed the node name to the device name.
Note that to get rid of all warnings for the /memory node,
arch/arm/boot/dts/skeleton.dtsi should be fixed, too.
Changes compared to v1:
- Add missing 0x-prefix to non-zero cache-controller reg properties,
- Change regulator names from fixedregulator3v3 to regulator-3v3.
Morimoto-san: If this is accepted, the examples in
Documentation/devicetree/bindings/sound/renesas,rsnd.txt should be
updated, too.
Boot-tested on r8a73a4/ape6evm, r8a7740/armadillo, r8a7791/koelsch,
sh73a0/kzm9g, r8a7795/salvator-x. I didn't test sound, but the kernel
log didn't show any failures.
The patches are against renesas-devel-20160509-v4.6-rc7.
Thanks for your comments!
Geert Uytterhoeven (23):
ARM: dts: emev2: Fix W=1 dtc warnings
ARM: dts: r8a73a4: Fix W=1 dtc warnings
ARM: dts: r8a7740: Fix W=1 dtc warnings
ARM: dts: r8a7778: Fix W=1 dtc warnings
ARM: dts: r8a7790: Fix W=1 dtc warnings
ARM: dts: r8a7791: Fix W=1 dtc warnings
ARM: dts: r8a7793: Fix W=1 dtc warnings
ARM: dts: r8a7794: Fix W=1 dtc warnings
ARM: dts: sh73a0: Fix W=1 dtc warnings
ARM: dts: ape6evm: Fix W=1 dtc warnings
ARM: dts: armadillo800eva: Fix W=1 dtc warnings
ARM: dts: bockw: Fix W=1 dtc warnings
ARM: dts: genmai: Fix W=1 dtc warnings
ARM: dts: gose: Fix W=1 dtc warnings
ARM: dts: koelsch: Fix W=1 dtc warnings
ARM: dts: kzm9d: Fix W=1 dtc warnings
ARM: dts: kzm9g: Fix W=1 dtc warnings
ARM: dts: lager: Fix W=1 dtc warnings
ARM: dts: marzen: Fix W=1 dtc warnings
ARM: dts: porter: Fix W=1 dtc warnings
ARM: dts: silk: Fix W=1 dtc warnings
arm64: dts: r8a7795: Fix W=1 dtc warnings
arm64: dts: salvator-x: Fix W=1 dtc warnings
arch/arm/boot/dts/emev2-kzm9d.dts | 16 ++--
arch/arm/boot/dts/emev2.dtsi | 26 +++---
arch/arm/boot/dts/r7s72100-genmai.dts | 2 +-
arch/arm/boot/dts/r8a73a4-ape6evm.dts | 8 +-
arch/arm/boot/dts/r8a73a4.dtsi | 34 ++++----
arch/arm/boot/dts/r8a7740-armadillo800eva.dts | 12 +--
arch/arm/boot/dts/r8a7740.dtsi | 2 +-
arch/arm/boot/dts/r8a7778-bockw.dts | 4 +-
arch/arm/boot/dts/r8a7778.dtsi | 28 +++----
arch/arm/boot/dts/r8a7779-marzen.dts | 4 +-
arch/arm/boot/dts/r8a7790-lager.dts | 20 ++---
arch/arm/boot/dts/r8a7790.dtsi | 94 +++++++++++-----------
arch/arm/boot/dts/r8a7791-koelsch.dts | 12 +--
arch/arm/boot/dts/r8a7791-porter.dts | 8 +-
arch/arm/boot/dts/r8a7791.dtsi | 79 +++++++++---------
arch/arm/boot/dts/r8a7793-gose.dts | 12 +--
arch/arm/boot/dts/r8a7793.dtsi | 59 +++++++-------
arch/arm/boot/dts/r8a7794-silk.dts | 6 +-
arch/arm/boot/dts/r8a7794.dtsi | 13 +--
arch/arm/boot/dts/sh73a0-kzm9g.dts | 10 +--
arch/arm/boot/dts/sh73a0.dtsi | 2 +-
arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts | 8 +-
arch/arm64/boot/dts/renesas/r8a7795.dtsi | 70 ++++++++--------
23 files changed, 269 insertions(+), 260 deletions(-)
--
1.9.1
next reply other threads:[~2016-05-20 7:09 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-05-20 7:09 Geert Uytterhoeven [this message]
2016-05-20 7:09 ` [PATCH v2 02/23] ARM: dts: r8a73a4: Fix W=1 dtc warnings Geert Uytterhoeven
2016-05-20 7:09 ` [PATCH v2 04/23] ARM: dts: r8a7778: " Geert Uytterhoeven
2016-05-20 7:09 ` [PATCH v2 07/23] ARM: dts: r8a7793: " Geert Uytterhoeven
2016-05-20 7:10 ` [PATCH v2 09/23] ARM: dts: sh73a0: " Geert Uytterhoeven
[not found] ` <1463728214-13838-1-git-send-email-geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
2016-05-20 7:09 ` [PATCH v2 01/23] ARM: dts: emev2: " Geert Uytterhoeven
2016-05-20 7:09 ` [PATCH v2 03/23] ARM: dts: r8a7740: " Geert Uytterhoeven
2016-05-20 7:09 ` [PATCH v2 05/23] ARM: dts: r8a7790: " Geert Uytterhoeven
2016-05-20 7:09 ` [PATCH v2 06/23] ARM: dts: r8a7791: " Geert Uytterhoeven
2016-05-20 7:09 ` [PATCH v2 08/23] ARM: dts: r8a7794: " Geert Uytterhoeven
2016-05-20 7:10 ` [PATCH v2 10/23] ARM: dts: ape6evm: " Geert Uytterhoeven
2016-05-20 7:10 ` [PATCH v2 11/23] ARM: dts: armadillo800eva: " Geert Uytterhoeven
[not found] ` <1463728214-13838-12-git-send-email-geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
2016-05-20 15:38 ` Sergei Shtylyov
[not found] ` <e0a1fbc1-ac95-2c79-65ed-d0328679bb8a-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org>
2016-05-23 12:52 ` Geert Uytterhoeven
2016-05-24 0:13 ` Simon Horman
2016-05-24 0:33 ` Simon Horman
2016-05-24 7:37 ` Geert Uytterhoeven
2016-05-20 7:10 ` [PATCH v2 16/23] ARM: dts: kzm9d: " Geert Uytterhoeven
2016-05-20 7:10 ` [PATCH v2 19/23] ARM: dts: marzen: " Geert Uytterhoeven
2016-05-20 7:10 ` [PATCH v2 20/23] ARM: dts: porter: " Geert Uytterhoeven
2016-05-20 7:10 ` [PATCH v2 22/23] arm64: dts: r8a7795: " Geert Uytterhoeven
2016-05-20 7:10 ` [PATCH v2 12/23] ARM: dts: bockw: " Geert Uytterhoeven
2016-05-20 7:10 ` [PATCH v2 13/23] ARM: dts: genmai: " Geert Uytterhoeven
2016-05-20 7:10 ` [PATCH v2 14/23] ARM: dts: gose: " Geert Uytterhoeven
2016-05-20 7:10 ` [PATCH v2 15/23] ARM: dts: koelsch: " Geert Uytterhoeven
2016-05-20 7:10 ` [PATCH v2 17/23] ARM: dts: kzm9g: " Geert Uytterhoeven
2016-05-20 7:10 ` [PATCH v2 18/23] ARM: dts: lager: " Geert Uytterhoeven
2016-05-20 7:10 ` [PATCH v2 21/23] ARM: dts: silk: " Geert Uytterhoeven
2016-05-20 7:10 ` [PATCH v2 23/23] arm64: dts: salvator-x: " Geert Uytterhoeven
2016-05-24 0:34 ` [PATCH v2 00/23] ARM/arm64: dts: renesas: " Simon Horman
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=1463728214-13838-1-git-send-email-geert+renesas@glider.be \
--to=geert+renesas@glider.be \
--cc=devicetree@vger.kernel.org \
--cc=galak@codeaurora.org \
--cc=horms@verge.net.au \
--cc=ijc+devicetree@hellion.org.uk \
--cc=kuninori.morimoto.gx@renesas.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=magnus.damm@gmail.com \
--cc=mark.rutland@arm.com \
--cc=robh+dt@kernel.org \
--cc=sergei.shtylyov@cogentembedded.com \
/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).