* [PATCH] dt-bindings: remoteproc: qcom: Drop types for firmware-name
@ 2026-03-06 14:03 Shawn Guo
2026-03-06 14:12 ` Krzysztof Kozlowski
0 siblings, 1 reply; 2+ messages in thread
From: Shawn Guo @ 2026-03-06 14:03 UTC (permalink / raw)
To: Bjorn Andersson, Rob Herring
Cc: Mathieu Poirier, Krzysztof Kozlowski, Conor Dooley,
Stephan Gerhold, Sibi Sankar, Bartosz Golaszewski,
Manivannan Sadhasivam, linux-arm-msm, linux-remoteproc,
devicetree, linux-kernel, Shawn Guo
The type of firmware-name is already defined by core schemas. Some of
qcom remoteproc bindings define it redundantly, while others do not.
Drop it to be consistent and avoid it being copied over to new files.
Signed-off-by: Shawn Guo <shengchao.guo@oss.qualcomm.com>
---
.../devicetree/bindings/remoteproc/qcom,msm8916-mss-pil.yaml | 1 -
.../devicetree/bindings/remoteproc/qcom,msm8996-mss-pil.yaml | 1 -
.../devicetree/bindings/remoteproc/qcom,sa8775p-pas.yaml | 1 -
.../devicetree/bindings/remoteproc/qcom,sc7180-mss-pil.yaml | 1 -
.../devicetree/bindings/remoteproc/qcom,sc7280-mss-pil.yaml | 1 -
.../devicetree/bindings/remoteproc/qcom,sc8280xp-pas.yaml | 1 -
Documentation/devicetree/bindings/remoteproc/qcom,sdx55-pas.yaml | 1 -
.../devicetree/bindings/remoteproc/qcom,sm8550-pas.yaml | 1 -
8 files changed, 8 deletions(-)
diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,msm8916-mss-pil.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,msm8916-mss-pil.yaml
index c179b560572b..1e7c165f925e 100644
--- a/Documentation/devicetree/bindings/remoteproc/qcom,msm8916-mss-pil.yaml
+++ b/Documentation/devicetree/bindings/remoteproc/qcom,msm8916-mss-pil.yaml
@@ -137,7 +137,6 @@ properties:
- description: MPSS reserved region
firmware-name:
- $ref: /schemas/types.yaml#/definitions/string-array
items:
- description: Name of MBA firmware
- description: Name of modem firmware
diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,msm8996-mss-pil.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,msm8996-mss-pil.yaml
index 4d2055f283ac..1b65813cc8ad 100644
--- a/Documentation/devicetree/bindings/remoteproc/qcom,msm8996-mss-pil.yaml
+++ b/Documentation/devicetree/bindings/remoteproc/qcom,msm8996-mss-pil.yaml
@@ -126,7 +126,6 @@ properties:
- description: Metadata reserved region
firmware-name:
- $ref: /schemas/types.yaml#/definitions/string-array
items:
- description: Name of MBA firmware
- description: Name of modem firmware
diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,sa8775p-pas.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,sa8775p-pas.yaml
index 188a25194000..bcd2bcf96e24 100644
--- a/Documentation/devicetree/bindings/remoteproc/qcom,sa8775p-pas.yaml
+++ b/Documentation/devicetree/bindings/remoteproc/qcom,sa8775p-pas.yaml
@@ -51,7 +51,6 @@ properties:
description: Reference to the AOSS side-channel message RAM.
firmware-name:
- $ref: /schemas/types.yaml#/definitions/string-array
items:
- description: Firmware name of the Hexagon core
diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,sc7180-mss-pil.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,sc7180-mss-pil.yaml
index b1402bef0ebe..7c9accac92d0 100644
--- a/Documentation/devicetree/bindings/remoteproc/qcom,sc7180-mss-pil.yaml
+++ b/Documentation/devicetree/bindings/remoteproc/qcom,sc7180-mss-pil.yaml
@@ -98,7 +98,6 @@ properties:
- description: metadata reserved region
firmware-name:
- $ref: /schemas/types.yaml#/definitions/string-array
items:
- description: Name of MBA firmware
- description: Name of modem firmware
diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,sc7280-mss-pil.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,sc7280-mss-pil.yaml
index 005cb21732af..f349c303fa07 100644
--- a/Documentation/devicetree/bindings/remoteproc/qcom,sc7280-mss-pil.yaml
+++ b/Documentation/devicetree/bindings/remoteproc/qcom,sc7280-mss-pil.yaml
@@ -98,7 +98,6 @@ properties:
- description: metadata reserved region
firmware-name:
- $ref: /schemas/types.yaml#/definitions/string-array
items:
- description: Name of MBA firmware
- description: Name of modem firmware
diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,sc8280xp-pas.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,sc8280xp-pas.yaml
index 5dbda3a55047..63ae3a30f626 100644
--- a/Documentation/devicetree/bindings/remoteproc/qcom,sc8280xp-pas.yaml
+++ b/Documentation/devicetree/bindings/remoteproc/qcom,sc8280xp-pas.yaml
@@ -42,7 +42,6 @@ properties:
description: Reference to the reserved-memory for the Hexagon core
firmware-name:
- $ref: /schemas/types.yaml#/definitions/string
description: Firmware name for the Hexagon core
required:
diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,sdx55-pas.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,sdx55-pas.yaml
index 5d463272165f..7e67eda13240 100644
--- a/Documentation/devicetree/bindings/remoteproc/qcom,sdx55-pas.yaml
+++ b/Documentation/devicetree/bindings/remoteproc/qcom,sdx55-pas.yaml
@@ -56,7 +56,6 @@ properties:
smd-edge: false
firmware-name:
- $ref: /schemas/types.yaml#/definitions/string
description: Firmware name for the Hexagon core
required:
diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,sm8550-pas.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,sm8550-pas.yaml
index 11b056d6a480..27b8c127d74f 100644
--- a/Documentation/devicetree/bindings/remoteproc/qcom,sm8550-pas.yaml
+++ b/Documentation/devicetree/bindings/remoteproc/qcom,sm8550-pas.yaml
@@ -52,7 +52,6 @@ properties:
smd-edge: false
firmware-name:
- $ref: /schemas/types.yaml#/definitions/string-array
items:
- description: Firmware name of the Hexagon core
- description: Firmware name of the Hexagon Devicetree
--
2.43.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] dt-bindings: remoteproc: qcom: Drop types for firmware-name
2026-03-06 14:03 [PATCH] dt-bindings: remoteproc: qcom: Drop types for firmware-name Shawn Guo
@ 2026-03-06 14:12 ` Krzysztof Kozlowski
0 siblings, 0 replies; 2+ messages in thread
From: Krzysztof Kozlowski @ 2026-03-06 14:12 UTC (permalink / raw)
To: Shawn Guo, Bjorn Andersson, Rob Herring
Cc: Mathieu Poirier, Krzysztof Kozlowski, Conor Dooley,
Stephan Gerhold, Sibi Sankar, Bartosz Golaszewski,
Manivannan Sadhasivam, linux-arm-msm, linux-remoteproc,
devicetree, linux-kernel
On 06/03/2026 15:03, Shawn Guo wrote:
> The type of firmware-name is already defined by core schemas. Some of
> qcom remoteproc bindings define it redundantly, while others do not.
> Drop it to be consistent and avoid it being copied over to new files.
>
> Signed-off-by: Shawn Guo <shengchao.guo@oss.qualcomm.com>
> ---
> .../devicetree/bindings/remoteproc/qcom,msm8916-mss-pil.yaml | 1 -
> .../devicetree/bindings/remoteproc/qcom,msm8996-mss-pil.yaml | 1 -
> .../devicetree/bindings/remoteproc/qcom,sa8775p-pas.yaml | 1 -
> .../devicetree/bindings/remoteproc/qcom,sc7180-mss-pil.yaml | 1 -
> .../devicetree/bindings/remoteproc/qcom,sc7280-mss-pil.yaml | 1 -
> .../devicetree/bindings/remoteproc/qcom,sc8280xp-pas.yaml | 1 -
> Documentation/devicetree/bindings/remoteproc/qcom,sdx55-pas.yaml | 1 -
> .../devicetree/bindings/remoteproc/qcom,sm8550-pas.yaml | 1 -
I was fixing all remoteprocs here:
https://lore.kernel.org/all/20240115182031.1610088-1-krzysztof.kozlowski@linaro.org/#r
so I am a bit confused how I missed these. Maybe at that time the
dtschema did not have definition and I postponed fixing these for later?
And then more wrong addons like sa8775p appeared...
> - $ref: /schemas/types.yaml#/definitions/string-array
> items:
> - description: Name of MBA firmware
> - description: Name of modem firmware
> diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,sc8280xp-pas.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,sc8280xp-pas.yaml
> index 5dbda3a55047..63ae3a30f626 100644
> --- a/Documentation/devicetree/bindings/remoteproc/qcom,sc8280xp-pas.yaml
> +++ b/Documentation/devicetree/bindings/remoteproc/qcom,sc8280xp-pas.yaml
> @@ -42,7 +42,6 @@ properties:
> description: Reference to the reserved-memory for the Hexagon core
>
> firmware-name:
> - $ref: /schemas/types.yaml#/definitions/string
> description: Firmware name for the Hexagon core
This and other needs fixes, look at my commit.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2026-03-06 14:12 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-03-06 14:03 [PATCH] dt-bindings: remoteproc: qcom: Drop types for firmware-name Shawn Guo
2026-03-06 14:12 ` Krzysztof Kozlowski
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox