* [PATCH] dt-bindings: remoteproc: Drop redundant $ref of firmware-name property
@ 2026-07-12 14:57 Krzysztof Kozlowski
2026-07-12 15:58 ` Martin Blumenstingl
2026-07-15 15:33 ` Mathieu Poirier
0 siblings, 2 replies; 3+ messages in thread
From: Krzysztof Kozlowski @ 2026-07-12 14:57 UTC (permalink / raw)
To: Bjorn Andersson, Mathieu Poirier, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Neil Armstrong, Kevin Hilman,
Jerome Brunet, Martin Blumenstingl, Suman Anna, linux-remoteproc,
devicetree, linux-arm-kernel, linux-amlogic, linux-kernel
Cc: Krzysztof Kozlowski
The DT schema core defines the type ($ref) of 'firmware-name' property
as string-array, so individual schemas do not need to. They also should
not redefine it to a single string, but instead just set number of
expected firmware names.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
---
.../devicetree/bindings/remoteproc/amlogic,meson-mx-ao-arc.yaml | 2 +-
.../devicetree/bindings/remoteproc/ti,pru-consumer.yaml | 1 -
2 files changed, 1 insertion(+), 2 deletions(-)
diff --git a/Documentation/devicetree/bindings/remoteproc/amlogic,meson-mx-ao-arc.yaml b/Documentation/devicetree/bindings/remoteproc/amlogic,meson-mx-ao-arc.yaml
index 3f710433e937..4cd5121b9a4e 100644
--- a/Documentation/devicetree/bindings/remoteproc/amlogic,meson-mx-ao-arc.yaml
+++ b/Documentation/devicetree/bindings/remoteproc/amlogic,meson-mx-ao-arc.yaml
@@ -25,7 +25,7 @@ properties:
- const: amlogic,meson-mx-ao-arc
firmware-name:
- $ref: /schemas/types.yaml#/definitions/string
+ maxItems: 1
description:
The name of the firmware which should be loaded for this remote
processor.
diff --git a/Documentation/devicetree/bindings/remoteproc/ti,pru-consumer.yaml b/Documentation/devicetree/bindings/remoteproc/ti,pru-consumer.yaml
index 2811334515d1..591471e5f93a 100644
--- a/Documentation/devicetree/bindings/remoteproc/ti,pru-consumer.yaml
+++ b/Documentation/devicetree/bindings/remoteproc/ti,pru-consumer.yaml
@@ -30,7 +30,6 @@ properties:
maxItems: 1
firmware-name:
- $ref: /schemas/types.yaml#/definitions/string-array
minItems: 1
maxItems: 6
description: |
--
2.53.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] dt-bindings: remoteproc: Drop redundant $ref of firmware-name property
2026-07-12 14:57 [PATCH] dt-bindings: remoteproc: Drop redundant $ref of firmware-name property Krzysztof Kozlowski
@ 2026-07-12 15:58 ` Martin Blumenstingl
2026-07-15 15:33 ` Mathieu Poirier
1 sibling, 0 replies; 3+ messages in thread
From: Martin Blumenstingl @ 2026-07-12 15:58 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Rob Herring, Conor Dooley, Mathieu Poirier, Neil Armstrong,
Kevin Hilman, Bjorn Andersson, linux-remoteproc, linux-kernel,
devicetree, linux-amlogic, Krzysztof Kozlowski, linux-arm-kernel,
Jerome Brunet
On Sun, Jul 12, 2026 at 4:57 PM Krzysztof Kozlowski
<krzysztof.kozlowski@oss.qualcomm.com> wrote:
>
> The DT schema core defines the type ($ref) of 'firmware-name' property
> as string-array, so individual schemas do not need to. They also should
> not redefine it to a single string, but instead just set number of
> expected firmware names.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
Reviewed-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] dt-bindings: remoteproc: Drop redundant $ref of firmware-name property
2026-07-12 14:57 [PATCH] dt-bindings: remoteproc: Drop redundant $ref of firmware-name property Krzysztof Kozlowski
2026-07-12 15:58 ` Martin Blumenstingl
@ 2026-07-15 15:33 ` Mathieu Poirier
1 sibling, 0 replies; 3+ messages in thread
From: Mathieu Poirier @ 2026-07-15 15:33 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Rob Herring, Conor Dooley, Neil Armstrong, Martin Blumenstingl,
Kevin Hilman, Bjorn Andersson, linux-remoteproc, linux-kernel,
devicetree, linux-amlogic, Krzysztof Kozlowski, linux-arm-kernel,
Jerome Brunet
On Sun, Jul 12, 2026 at 04:57:19PM +0200, Krzysztof Kozlowski wrote:
> The DT schema core defines the type ($ref) of 'firmware-name' property
> as string-array, so individual schemas do not need to. They also should
> not redefine it to a single string, but instead just set number of
> expected firmware names.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
> ---
> .../devicetree/bindings/remoteproc/amlogic,meson-mx-ao-arc.yaml | 2 +-
> .../devicetree/bindings/remoteproc/ti,pru-consumer.yaml | 1 -
> 2 files changed, 1 insertion(+), 2 deletions(-)
Applied.
Thanks,
Mathieu
>
> diff --git a/Documentation/devicetree/bindings/remoteproc/amlogic,meson-mx-ao-arc.yaml b/Documentation/devicetree/bindings/remoteproc/amlogic,meson-mx-ao-arc.yaml
> index 3f710433e937..4cd5121b9a4e 100644
> --- a/Documentation/devicetree/bindings/remoteproc/amlogic,meson-mx-ao-arc.yaml
> +++ b/Documentation/devicetree/bindings/remoteproc/amlogic,meson-mx-ao-arc.yaml
> @@ -25,7 +25,7 @@ properties:
> - const: amlogic,meson-mx-ao-arc
>
> firmware-name:
> - $ref: /schemas/types.yaml#/definitions/string
> + maxItems: 1
> description:
> The name of the firmware which should be loaded for this remote
> processor.
> diff --git a/Documentation/devicetree/bindings/remoteproc/ti,pru-consumer.yaml b/Documentation/devicetree/bindings/remoteproc/ti,pru-consumer.yaml
> index 2811334515d1..591471e5f93a 100644
> --- a/Documentation/devicetree/bindings/remoteproc/ti,pru-consumer.yaml
> +++ b/Documentation/devicetree/bindings/remoteproc/ti,pru-consumer.yaml
> @@ -30,7 +30,6 @@ properties:
> maxItems: 1
>
> firmware-name:
> - $ref: /schemas/types.yaml#/definitions/string-array
> minItems: 1
> maxItems: 6
> description: |
> --
> 2.53.0
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2026-07-15 15:33 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-07-12 14:57 [PATCH] dt-bindings: remoteproc: Drop redundant $ref of firmware-name property Krzysztof Kozlowski
2026-07-12 15:58 ` Martin Blumenstingl
2026-07-15 15:33 ` Mathieu Poirier
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox