public inbox for devicetree@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2] dt-bindings: bus: renesas-bsc: allow additional properties
@ 2025-10-09  7:08 Wolfram Sang
  2025-10-09  7:19 ` Geert Uytterhoeven
  2025-10-09 18:34 ` Wolfram Sang
  0 siblings, 2 replies; 3+ messages in thread
From: Wolfram Sang @ 2025-10-09  7:08 UTC (permalink / raw)
  To: linux-renesas-soc
  Cc: Wolfram Sang, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	Geert Uytterhoeven, Magnus Damm, devicetree

Allow additional properties to enable devices attached to the bus.
Fixes warnings like these:

arch/arm/boot/dts/renesas/sh73a0-kzm9g.dtb: bus@fec10000 (renesas,bsc-sh73a0): Unevaluated properties are not allowed ('ethernet@10000000' was unexpected)
arch/arm/boot/dts/renesas/r8a73a4-ape6evm.dtb: bus@fec10000 (renesas,bsc-r8a73a4): Unevaluated properties are not allowed ('ethernet@8000000', 'flash@0' were unexpected)

Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
---

Changes since v1:
* don't restrict node names, the core already does it
* drop A-F from unit address pattern

Thanks to Geert and Rob for the input!

 .../devicetree/bindings/bus/renesas,bsc.yaml         | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/Documentation/devicetree/bindings/bus/renesas,bsc.yaml b/Documentation/devicetree/bindings/bus/renesas,bsc.yaml
index f53a37785413..48b9d02b65ed 100644
--- a/Documentation/devicetree/bindings/bus/renesas,bsc.yaml
+++ b/Documentation/devicetree/bindings/bus/renesas,bsc.yaml
@@ -41,6 +41,18 @@ properties:
   interrupts:
     maxItems: 1
 
+patternProperties:
+  # All other properties should be child nodes with unit-address and 'reg'
+  "^.*@[0-9a-f]+$":
+    type: object
+    additionalProperties: true
+    properties:
+      reg:
+        maxItems: 1
+
+    required:
+      - reg
+
 required:
   - reg
 
-- 
2.47.2


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH v2] dt-bindings: bus: renesas-bsc: allow additional properties
  2025-10-09  7:08 [PATCH v2] dt-bindings: bus: renesas-bsc: allow additional properties Wolfram Sang
@ 2025-10-09  7:19 ` Geert Uytterhoeven
  2025-10-09 18:34 ` Wolfram Sang
  1 sibling, 0 replies; 3+ messages in thread
From: Geert Uytterhoeven @ 2025-10-09  7:19 UTC (permalink / raw)
  To: Wolfram Sang
  Cc: linux-renesas-soc, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	Magnus Damm, devicetree

On Thu, 9 Oct 2025 at 09:10, Wolfram Sang
<wsa+renesas@sang-engineering.com> wrote:
> Allow additional properties to enable devices attached to the bus.
> Fixes warnings like these:
>
> arch/arm/boot/dts/renesas/sh73a0-kzm9g.dtb: bus@fec10000 (renesas,bsc-sh73a0): Unevaluated properties are not allowed ('ethernet@10000000' was unexpected)
> arch/arm/boot/dts/renesas/r8a73a4-ape6evm.dtb: bus@fec10000 (renesas,bsc-r8a73a4): Unevaluated properties are not allowed ('ethernet@8000000', 'flash@0' were unexpected)
>
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> ---
>
> Changes since v1:
> * don't restrict node names, the core already does it
> * drop A-F from unit address pattern

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

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] 3+ messages in thread

* Re: [PATCH v2] dt-bindings: bus: renesas-bsc: allow additional properties
  2025-10-09  7:08 [PATCH v2] dt-bindings: bus: renesas-bsc: allow additional properties Wolfram Sang
  2025-10-09  7:19 ` Geert Uytterhoeven
@ 2025-10-09 18:34 ` Wolfram Sang
  1 sibling, 0 replies; 3+ messages in thread
From: Wolfram Sang @ 2025-10-09 18:34 UTC (permalink / raw)
  To: linux-renesas-soc
  Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	Geert Uytterhoeven, Magnus Damm, devicetree

[-- Attachment #1: Type: text/plain, Size: 177 bytes --]


> +patternProperties:
> +  # All other properties should be child nodes with unit-address and 'reg'
> +  "^.*@[0-9a-f]+$":

Per Rob's comment in another thread, '^.*' can go.


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2025-10-09 18:34 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-10-09  7:08 [PATCH v2] dt-bindings: bus: renesas-bsc: allow additional properties Wolfram Sang
2025-10-09  7:19 ` Geert Uytterhoeven
2025-10-09 18:34 ` Wolfram Sang

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox