| Message ID | 20250305-topic-sm8x50-iris-v10-v2-1-bd65a3fc099e@linaro.org |
|---|---|
| State | New |
| Headers | show |
| Series | [v2,1/7] dt-bindings: media: qcom,sm8550-iris: document SM8650 IRIS accelerator | expand |
On 05/03/2025 19:05, Neil Armstrong wrote: > Document the IRIS video decoder and encoder accelerator found in the > SM8650 platform, it requires 2 more reset lines in addition to the > properties required for the SM8550 platform. > > Reviewed-by: Rob Herring (Arm) <robh@kernel.org> > Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org> > --- > .../bindings/media/qcom,sm8550-iris.yaml | 33 ++++++++++++++++++---- > 1 file changed, 28 insertions(+), 5 deletions(-) > > diff --git a/Documentation/devicetree/bindings/media/qcom,sm8550-iris.yaml b/Documentation/devicetree/bindings/media/qcom,sm8550-iris.yaml > index e424ea84c211f473a799481fd5463a16580187ed..536cf458dcb08141e5a1ec8c3df964196e599a57 100644 > --- a/Documentation/devicetree/bindings/media/qcom,sm8550-iris.yaml > +++ b/Documentation/devicetree/bindings/media/qcom,sm8550-iris.yaml > @@ -14,12 +14,11 @@ description: > The iris video processing unit is a video encode and decode accelerator > present on Qualcomm platforms. > > -allOf: > - - $ref: qcom,venus-common.yaml# > - > properties: > compatible: > - const: qcom,sm8550-iris > + enum: > + - qcom,sm8550-iris > + - qcom,sm8650-iris > > power-domains: > maxItems: 4 > @@ -49,11 +48,15 @@ properties: > - const: video-mem > > resets: > - maxItems: 1 > + minItems: 1 > + maxItems: 3 > > reset-names: > + minItems: 1 > items: > - const: bus > + - const: xo > + - const: core > > iommus: > maxItems: 2 > @@ -75,6 +78,26 @@ required: > - iommus > - dma-coherent > > +allOf: > + - $ref: qcom,venus-common.yaml# > + - if: > + properties: > + compatible: > + enum: > + - qcom,sm8650-iris > + then: > + properties: > + resets: > + minItems: 3 > + reset-names: > + minItems: 3 > + else: > + properties: > + resets: > + maxItems: 1 > + reset-names: > + maxItems: 1 > + > unevaluatedProperties: false > > examples: > > -- > 2.34.1 > > Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
On 3/6/2025 12:35 AM, Neil Armstrong wrote: > Document the IRIS video decoder and encoder accelerator found in the > SM8650 platform, it requires 2 more reset lines in addition to the > properties required for the SM8550 platform. > > Reviewed-by: Rob Herring (Arm) <robh@kernel.org> > Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org> > --- > .../bindings/media/qcom,sm8550-iris.yaml | 33 ++++++++++++++++++---- > 1 file changed, 28 insertions(+), 5 deletions(-) > > diff --git a/Documentation/devicetree/bindings/media/qcom,sm8550-iris.yaml b/Documentation/devicetree/bindings/media/qcom,sm8550-iris.yaml > index e424ea84c211f473a799481fd5463a16580187ed..536cf458dcb08141e5a1ec8c3df964196e599a57 100644 > --- a/Documentation/devicetree/bindings/media/qcom,sm8550-iris.yaml > +++ b/Documentation/devicetree/bindings/media/qcom,sm8550-iris.yaml > @@ -14,12 +14,11 @@ description: > The iris video processing unit is a video encode and decode accelerator > present on Qualcomm platforms. > > -allOf: > - - $ref: qcom,venus-common.yaml# > - > properties: > compatible: > - const: qcom,sm8550-iris > + enum: > + - qcom,sm8550-iris > + - qcom,sm8650-iris > > power-domains: > maxItems: 4 > @@ -49,11 +48,15 @@ properties: > - const: video-mem > > resets: > - maxItems: 1 > + minItems: 1 > + maxItems: 3 > > reset-names: > + minItems: 1 > items: > - const: bus > + - const: xo > + - const: core > > iommus: > maxItems: 2 > @@ -75,6 +78,26 @@ required: > - iommus > - dma-coherent > > +allOf: > + - $ref: qcom,venus-common.yaml# > + - if: > + properties: > + compatible: > + enum: > + - qcom,sm8650-iris > + then: > + properties: > + resets: > + minItems: 3 > + reset-names: > + minItems: 3 > + else: > + properties: > + resets: > + maxItems: 1 > + reset-names: > + maxItems: 1 > + > unevaluatedProperties: false > > examples: > Reviewed-by: Vikash Garodia <quic_vgarodia@quicinc.com>
diff --git a/Documentation/devicetree/bindings/media/qcom,sm8550-iris.yaml b/Documentation/devicetree/bindings/media/qcom,sm8550-iris.yaml index e424ea84c211f473a799481fd5463a16580187ed..536cf458dcb08141e5a1ec8c3df964196e599a57 100644 --- a/Documentation/devicetree/bindings/media/qcom,sm8550-iris.yaml +++ b/Documentation/devicetree/bindings/media/qcom,sm8550-iris.yaml @@ -14,12 +14,11 @@ description: The iris video processing unit is a video encode and decode accelerator present on Qualcomm platforms. -allOf: - - $ref: qcom,venus-common.yaml# - properties: compatible: - const: qcom,sm8550-iris + enum: + - qcom,sm8550-iris + - qcom,sm8650-iris power-domains: maxItems: 4 @@ -49,11 +48,15 @@ properties: - const: video-mem resets: - maxItems: 1 + minItems: 1 + maxItems: 3 reset-names: + minItems: 1 items: - const: bus + - const: xo + - const: core iommus: maxItems: 2 @@ -75,6 +78,26 @@ required: - iommus - dma-coherent +allOf: + - $ref: qcom,venus-common.yaml# + - if: + properties: + compatible: + enum: + - qcom,sm8650-iris + then: + properties: + resets: + minItems: 3 + reset-names: + minItems: 3 + else: + properties: + resets: + maxItems: 1 + reset-names: + maxItems: 1 + unevaluatedProperties: false examples: