diff mbox series

[v2,2/2] dt-bindings: remoteproc: qcom: adsp: Add ADSP on MSM8953

Message ID 20221006044745.286264-3-sireeshkodali1@gmail.com
State Changes Requested, archived
Headers show
Series remoteproc: qcom: Add support for MSM8953 ADSP | expand

Checks

Context Check Description
robh/checkpatch success
robh/patch-applied success
robh/dtbs-check warning build log
robh/dt-meta-schema success

Commit Message

Sireesh Kodali Oct. 6, 2022, 4:47 a.m. UTC
Add support for booting the Audio DSP on the MSM8953 platform. This is
used by SoCs like SDM450, SDM625, SDM626, APQ8053, etc. Since the
configuration is the same on all SoCs, a single compatible string is
used.

Signed-off-by: Sireesh Kodali <sireeshkodali1@gmail.com>
---
 Documentation/devicetree/bindings/remoteproc/qcom,adsp.yaml | 5 +++++
 1 file changed, 5 insertions(+)

Comments

Krzysztof Kozlowski Oct. 6, 2022, 7:12 a.m. UTC | #1
On 06/10/2022 06:47, Sireesh Kodali wrote:
> Add support for booting the Audio DSP on the MSM8953 platform. This is
> used by SoCs like SDM450, SDM625, SDM626, APQ8053, etc. Since the
> configuration is the same on all SoCs, a single compatible string is
> used.
> 
> Signed-off-by: Sireesh Kodali <sireeshkodali1@gmail.com>
> ---
>  Documentation/devicetree/bindings/remoteproc/qcom,adsp.yaml | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,adsp.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,adsp.yaml
> index 3072af5f9d79..c9d69f6160a1 100644
> --- a/Documentation/devicetree/bindings/remoteproc/qcom,adsp.yaml
> +++ b/Documentation/devicetree/bindings/remoteproc/qcom,adsp.yaml
> @@ -18,6 +18,7 @@ properties:
>      enum:
>        - qcom,msm8226-adsp-pil
>        - qcom,msm8974-adsp-pil
> +      - qcom,msm8953-adsp-pil

Alphanumeric order.

Same in other places.

Best regards,
Krzysztof
Sireesh Kodali Oct. 6, 2022, 4:26 p.m. UTC | #2
On Thu Oct 6, 2022 at 12:42 PM IST, Krzysztof Kozlowski wrote:
> On 06/10/2022 06:47, Sireesh Kodali wrote:
> > Add support for booting the Audio DSP on the MSM8953 platform. This is
> > used by SoCs like SDM450, SDM625, SDM626, APQ8053, etc. Since the
> > configuration is the same on all SoCs, a single compatible string is
> > used.
> > 
> > Signed-off-by: Sireesh Kodali <sireeshkodali1@gmail.com>
> > ---
> >  Documentation/devicetree/bindings/remoteproc/qcom,adsp.yaml | 5 +++++
> >  1 file changed, 5 insertions(+)
> > 
> > diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,adsp.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,adsp.yaml
> > index 3072af5f9d79..c9d69f6160a1 100644
> > --- a/Documentation/devicetree/bindings/remoteproc/qcom,adsp.yaml
> > +++ b/Documentation/devicetree/bindings/remoteproc/qcom,adsp.yaml
> > @@ -18,6 +18,7 @@ properties:
> >      enum:
> >        - qcom,msm8226-adsp-pil
> >        - qcom,msm8974-adsp-pil
> > +      - qcom,msm8953-adsp-pil
>
> Alphanumeric order.

Ok, will be fixed in v3

Regards,
Sireesh Kodali
>
> Same in other places.
>
> Best regards,
> Krzysztof
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,adsp.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,adsp.yaml
index 3072af5f9d79..c9d69f6160a1 100644
--- a/Documentation/devicetree/bindings/remoteproc/qcom,adsp.yaml
+++ b/Documentation/devicetree/bindings/remoteproc/qcom,adsp.yaml
@@ -18,6 +18,7 @@  properties:
     enum:
       - qcom,msm8226-adsp-pil
       - qcom,msm8974-adsp-pil
+      - qcom,msm8953-adsp-pil
       - qcom,msm8996-adsp-pil
       - qcom,msm8996-slpi-pil
       - qcom,msm8998-adsp-pas
@@ -179,6 +180,7 @@  allOf:
             enum:
               - qcom,msm8226-adsp-pil
               - qcom,msm8974-adsp-pil
+              - qcom,msm8953-adsp-pil
               - qcom,msm8996-adsp-pil
               - qcom,msm8996-slpi-pil
               - qcom,msm8998-adsp-pas
@@ -298,6 +300,7 @@  allOf:
             enum:
               - qcom,msm8226-adsp-pil
               - qcom,msm8974-adsp-pil
+              - qcom,msm8953-adsp-pil
               - qcom,msm8996-adsp-pil
               - qcom,msm8996-slpi-pil
               - qcom,msm8998-adsp-pas
@@ -369,6 +372,7 @@  allOf:
           contains:
             enum:
               - qcom,msm8226-adsp-pil
+              - qcom,msm8953-adsp-pil
               - qcom,msm8996-adsp-pil
               - qcom,msm8998-adsp-pas
               - qcom,sm8150-adsp-pas
@@ -559,6 +563,7 @@  allOf:
             enum:
               - qcom,msm8226-adsp-pil
               - qcom,msm8974-adsp-pil
+              - qcom,msm8953-adsp-pil
               - qcom,msm8996-adsp-pil
               - qcom,msm8996-slpi-pil
               - qcom,msm8998-adsp-pas