diff mbox series

[v3,3/5] dt-bindings: arm: qcom: Document rb8/ride/ride-r3 on QCS9075

Message ID 20241119174954.1219002-4-quic_wasimn@quicinc.com
State Not Applicable
Headers show
Series arm64: qcom: Add support for QCS9075 boards | expand

Checks

Context Check Description
robh/checkpatch success
robh/patch-applied fail build log

Commit Message

Wasim Nazir Nov. 19, 2024, 5:49 p.m. UTC
qcs9075 rb8, ride & ride-r3 boards are based on QCS9075 SoC.

QCS9075 is compatible IoT-industrial grade variant of SA8775p SoC
without safety monitorng feature of SAfety-IsLand subsystem.
This subsystem continues to supports other features like built-in
self-test, error-detection, reset-handling, etc.

Acked-by: Rob Herring (Arm) <robh@kernel.org>
Signed-off-by: Wasim Nazir <quic_wasimn@quicinc.com>
---
 Documentation/devicetree/bindings/arm/qcom.yaml | 9 +++++++++
 1 file changed, 9 insertions(+)

--
2.47.0

Comments

Krzysztof Kozlowski Nov. 20, 2024, 4:43 p.m. UTC | #1
On 19/11/2024 18:49, Wasim Nazir wrote:
> qcs9075 rb8, ride & ride-r3 boards are based on QCS9075 SoC.
> 
> QCS9075 is compatible IoT-industrial grade variant of SA8775p SoC
> without safety monitorng feature of SAfety-IsLand subsystem.

SA or Sa?
Also some typos above.

> This subsystem continues to supports other features like built-in
> self-test, error-detection, reset-handling, etc.

Which subsystem? How does it matter for the board? Drop all marketing
stuff from here and rather explain why this is not related to qcs9100.
We had quite a lot of talks and I am surprised seeing this without any
earlier references.




Best regards,
Krzysztof
Wasim Nazir Dec. 6, 2024, 11 a.m. UTC | #2
On Wed, Nov 20, 2024 at 05:43:12PM +0100, Krzysztof Kozlowski wrote:
> On 19/11/2024 18:49, Wasim Nazir wrote:
> > qcs9075 rb8, ride & ride-r3 boards are based on QCS9075 SoC.
> > 
> > QCS9075 is compatible IoT-industrial grade variant of SA8775p SoC
> > without safety monitorng feature of SAfety-IsLand subsystem.
> 
> SA or Sa?
> Also some typos above.

SAfety-IsLand (SAIL) was intended. I will rectify it along with
other typos.

> 
> > This subsystem continues to supports other features like built-in
> > self-test, error-detection, reset-handling, etc.
> 
> Which subsystem? How does it matter for the board? Drop all marketing
> stuff from here and rather explain why this is not related to qcs9100.
> We had quite a lot of talks and I am surprised seeing this without any
> earlier references.

It is for SAIL subsystem. It does not matter to the board but it is for
the SoC.

> 
> 
> 
> 
> Best regards,
> Krzysztof

Thanks & Regards,
Wasim
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
index 7c8c3a97506a..de5cf9fb28ae 100644
--- a/Documentation/devicetree/bindings/arm/qcom.yaml
+++ b/Documentation/devicetree/bindings/arm/qcom.yaml
@@ -46,6 +46,7 @@  description: |
         qcs8550
         qcm2290
         qcm6490
+        qcs9075
         qcs9100
         qdu1000
         qrb2210
@@ -928,6 +929,14 @@  properties:
               - qcom,sa8775p-ride-r3
           - const: qcom,sa8775p

+      - items:
+          - enum:
+              - qcom,qcs9075-rb8
+              - qcom,qcs9075-ride
+              - qcom,qcs9075-ride-r3
+          - const: qcom,qcs9075
+          - const: qcom,sa8775p
+
       - items:
           - enum:
               - qcom,qcs9100-ride