devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/7] arm: dts: renesas: Group tuples in reg/ranges/dma-ranges/states properties
@ 2019-12-13 16:41 Geert Uytterhoeven
  2019-12-13 16:41 ` [PATCH 1/7] ARM: dts: renesas: Group tuples in regulator-gpio states properties Geert Uytterhoeven
                   ` (8 more replies)
  0 siblings, 9 replies; 12+ messages in thread
From: Geert Uytterhoeven @ 2019-12-13 16:41 UTC (permalink / raw)
  To: Magnus Damm
  Cc: Rob Herring, Mark Rutland, linux-renesas-soc, devicetree,
	Geert Uytterhoeven

	Hi all,

To improve human readability and enable automatic validation, tuples
in various properties should be grouped.  While "make dtbs_check" does
not impose this yet for all properties, it does for some, hence
triggering me to fix (I hope) all of them.

Unfortunately even after this, a few "... is too long" warnings are
still printed (e.g. for PCI "ranges"), which I believe are false
positives.

This series is against renesas-devel-2019-12-13-v5.5-rc1[*] with
"[PATCH] ARM: dts: rcar-gen2: Fix PCI high address in
interrupt-map-mask" applied on top.

Thanks for your comments!

[*] Will be pushed out when gitolite.kernel.org is back.

Geert Uytterhoeven (7):
  ARM: dts: renesas: Group tuples in regulator-gpio states properties
  ARM: dts: renesas: Group tuples in interrupt properties
  ARM: dts: renesas: Group tuples in pci ranges and dma-ranges
    properties
  arm64: dts: renesas: Group tuples in regulator-gpio states properties
  arm64: dts: renesas: Group tuples in interrupt properties
  arm64: dts: renesas: Group tuples in pci ranges and dma-ranges
    properties
  arm64: dts: renesas: r8a77970: Group tuples in thermal reg property

 arch/arm/boot/dts/iwg20d-q7-common.dtsi       |   3 +-
 arch/arm/boot/dts/r7s72100.dtsi               |  18 +-
 arch/arm/boot/dts/r8a73a4.dtsi                |  42 ++--
 arch/arm/boot/dts/r8a7740-armadillo800eva.dts |   3 +-
 arch/arm/boot/dts/r8a7740.dtsi                | 102 ++++-----
 arch/arm/boot/dts/r8a7743.dtsi                | 152 ++++++-------
 arch/arm/boot/dts/r8a7744.dtsi                | 152 ++++++-------
 arch/arm/boot/dts/r8a7745-iwg22d-sodimm.dts   |   3 +-
 arch/arm/boot/dts/r8a7745.dtsi                | 112 ++++-----
 arch/arm/boot/dts/r8a77470-iwg23s-sbc.dts     |   3 +-
 arch/arm/boot/dts/r8a77470.dtsi               |  80 +++----
 arch/arm/boot/dts/r8a7778.dtsi                |   8 +-
 arch/arm/boot/dts/r8a7779-marzen.dts          |   3 +-
 arch/arm/boot/dts/r8a7779.dtsi                |   8 +-
 arch/arm/boot/dts/r8a7790-lager.dts           |   6 +-
 arch/arm/boot/dts/r8a7790.dtsi                | 158 ++++++-------
 arch/arm/boot/dts/r8a7791-koelsch.dts         |   9 +-
 arch/arm/boot/dts/r8a7791-porter.dts          |   6 +-
 arch/arm/boot/dts/r8a7791.dtsi                | 152 ++++++-------
 arch/arm/boot/dts/r8a7792.dtsi                |  64 +++---
 arch/arm/boot/dts/r8a7793-gose.dts            |   9 +-
 arch/arm/boot/dts/r8a7793.dtsi                | 120 +++++-----
 arch/arm/boot/dts/r8a7794-alt.dts             |   6 +-
 arch/arm/boot/dts/r8a7794-silk.dts            |   3 +-
 arch/arm/boot/dts/r8a7794.dtsi                | 104 ++++-----
 arch/arm/boot/dts/sh73a0.dtsi                 | 122 +++++-----
 .../arm64/boot/dts/renesas/hihope-common.dtsi |   3 +-
 arch/arm64/boot/dts/renesas/r8a774a1.dtsi     | 206 ++++++++---------
 arch/arm64/boot/dts/renesas/r8a774b1.dtsi     | 206 ++++++++---------
 .../boot/dts/renesas/r8a774c0-cat874.dts      |   3 +-
 arch/arm64/boot/dts/renesas/r8a774c0.dtsi     | 164 +++++++-------
 arch/arm64/boot/dts/renesas/r8a7795.dtsi      | 214 +++++++++---------
 arch/arm64/boot/dts/renesas/r8a77960.dtsi     | 206 ++++++++---------
 arch/arm64/boot/dts/renesas/r8a77965.dtsi     | 206 ++++++++---------
 arch/arm64/boot/dts/renesas/r8a77970.dtsi     |  52 ++---
 arch/arm64/boot/dts/renesas/r8a77980.dtsi     |  96 ++++----
 .../arm64/boot/dts/renesas/r8a77990-ebisu.dts |   6 +-
 arch/arm64/boot/dts/renesas/r8a77990.dtsi     | 164 +++++++-------
 arch/arm64/boot/dts/renesas/r8a77995.dtsi     |  74 +++---
 .../boot/dts/renesas/salvator-common.dtsi     |   6 +-
 arch/arm64/boot/dts/renesas/ulcb.dtsi         |   3 +-
 41 files changed, 1514 insertions(+), 1543 deletions(-)

-- 
2.17.1

Gr{oetje,eeting}s,

						Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
							    -- Linus Torvalds

^ permalink raw reply	[flat|nested] 12+ messages in thread
* [PATCH] ARM: dts: sh73a0: Add missing clock-frequency for fixed clocks
@ 2019-12-13 16:27 Geert Uytterhoeven
  0 siblings, 0 replies; 12+ messages in thread
From: Geert Uytterhoeven @ 2019-12-13 16:27 UTC (permalink / raw)
  To: Magnus Damm; +Cc: linux-renesas-soc, devicetree, Geert Uytterhoeven

"clock-frequency" is a required property for devices nodes compatible
with "fixed-clock", leading to warnings when running

    $ make dtbs_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/clock/fixed-clock.yaml
    arch/arm/boot/dts/sh73a0-kzm9g.dt.yaml: extcki: 'clock-frequency' is a required property

Fix this by adding the missing "clock-frequency" properties to the various
clocks, to be overridden by the board DTS files when populated.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 arch/arm/boot/dts/sh73a0.dtsi | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/arch/arm/boot/dts/sh73a0.dtsi b/arch/arm/boot/dts/sh73a0.dtsi
index 7efe33c4d041a524..7c67581f57422a26 100644
--- a/arch/arm/boot/dts/sh73a0.dtsi
+++ b/arch/arm/boot/dts/sh73a0.dtsi
@@ -624,19 +624,25 @@
 		extal2_clk: extal2 {
 			compatible = "fixed-clock";
 			#clock-cells = <0>;
+			/* This value must be overridden by the board. */
+			clock-frequency = <0>;
 		};
 		extcki_clk: extcki {
 			compatible = "fixed-clock";
 			#clock-cells = <0>;
+			/* This value can be overridden by the board. */
+			clock-frequency = <0>;
 		};
 		fsiack_clk: fsiack {
 			compatible = "fixed-clock";
 			#clock-cells = <0>;
+			/* This value can be overridden by the board. */
 			clock-frequency = <0>;
 		};
 		fsibck_clk: fsibck {
 			compatible = "fixed-clock";
 			#clock-cells = <0>;
+			/* This value can be overridden by the board. */
 			clock-frequency = <0>;
 		};
 
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 12+ messages in thread
end of thread, other threads:[~2019-12-20 15:51 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-12-13 16:41 [PATCH 0/7] arm: dts: renesas: Group tuples in reg/ranges/dma-ranges/states properties Geert Uytterhoeven
2019-12-13 16:41 ` [PATCH 1/7] ARM: dts: renesas: Group tuples in regulator-gpio states properties Geert Uytterhoeven
2019-12-13 16:41 ` [PATCH] ARM: dts: sh73a0: Add missing clock-frequency for fixed clocks Geert Uytterhoeven
2019-12-13 16:41 ` [PATCH 2/7] ARM: dts: renesas: Group tuples in interrupt properties Geert Uytterhoeven
2019-12-13 16:41 ` [PATCH 3/7] ARM: dts: renesas: Group tuples in pci ranges and dma-ranges properties Geert Uytterhoeven
2019-12-13 16:41 ` [PATCH 4/7] arm64: dts: renesas: Group tuples in regulator-gpio states properties Geert Uytterhoeven
2019-12-13 16:41 ` [PATCH 5/7] arm64: dts: renesas: Group tuples in interrupt properties Geert Uytterhoeven
2019-12-13 16:41 ` [PATCH 6/7] arm64: dts: renesas: Group tuples in pci ranges and dma-ranges properties Geert Uytterhoeven
2019-12-13 16:41 ` [PATCH 7/7] arm64: dts: renesas: r8a77970: Group tuples in thermal reg property Geert Uytterhoeven
2019-12-14  7:08 ` [PATCH 0/7] arm: dts: renesas: Group tuples in reg/ranges/dma-ranges/states properties Ulrich Hecht
2019-12-20 15:51   ` Geert Uytterhoeven
  -- strict thread matches above, loose matches on Subject: below --
2019-12-13 16:27 [PATCH] ARM: dts: sh73a0: Add missing clock-frequency for fixed clocks Geert Uytterhoeven

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).