diff mbox series

[2/5] dt-bindings: arm: qcom: Document SC8380XP SoC and boards

Message ID 20231025142427.2661-3-quic_sibis@quicinc.com (mailing list archive)
State Changes Requested
Headers show
Series dts: qcom: Introduce SC8380XP platforms device tree | expand

Commit Message

Sibi Sankar Oct. 25, 2023, 2:24 p.m. UTC
From: Rajendra Nayak <quic_rjendra@quicinc.com>

Document the SC8380XP SoC binding and also the boards using it.
Also document the new board id qcp (Qualcomm Compute Platform).

Signed-off-by: Rajendra Nayak <quic_rjendra@quicinc.com>
Signed-off-by: Sibi Sankar <quic_sibis@quicinc.com>
---
 Documentation/devicetree/bindings/arm/qcom.yaml | 8 ++++++++
 1 file changed, 8 insertions(+)

Comments

Konrad Dybcio Oct. 26, 2023, 10:24 a.m. UTC | #1
On 10/25/23 16:24, Sibi Sankar wrote:
> From: Rajendra Nayak <quic_rjendra@quicinc.com>
> 
> Document the SC8380XP SoC binding and also the boards using it.
> Also document the new board id qcp (Qualcomm Compute Platform).
> 
> Signed-off-by: Rajendra Nayak <quic_rjendra@quicinc.com>
> Signed-off-by: Sibi Sankar <quic_sibis@quicinc.com>
> ---
Looks good, but can you elaborate a bit more on QCP? Is it going
to be a new fuse setting for devices that fall into the "compute"
category, but aren't necessarily CRDs?

Konrad
Krzysztof Kozlowski Oct. 27, 2023, 8:16 a.m. UTC | #2
On 25/10/2023 16:24, Sibi Sankar wrote:
> From: Rajendra Nayak <quic_rjendra@quicinc.com>
> 
> Document the SC8380XP SoC binding and also the boards using it.
> Also document the new board id qcp (Qualcomm Compute Platform).
> 
> Signed-off-by: Rajendra Nayak <quic_rjendra@quicinc.com>
> Signed-off-by: Sibi Sankar <quic_sibis@quicinc.com>
> ---
>  Documentation/devicetree/bindings/arm/qcom.yaml | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
> index 88b84035e7b1..6145e9ad340c 100644
> --- a/Documentation/devicetree/bindings/arm/qcom.yaml
> +++ b/Documentation/devicetree/bindings/arm/qcom.yaml
> @@ -62,6 +62,7 @@ description: |
>          sc7280
>          sc8180x
>          sc8280xp
> +        sc8380xp
>          sda660
>          sdm450
>          sdm630
> @@ -110,6 +111,7 @@ description: |
>          liquid
>          rdp432-c2
>          mtp
> +        qcp
>          qrd
>          rb2
>          ride
> @@ -1044,6 +1046,12 @@ properties:
>                - qcom,sm8550-qrd
>            - const: qcom,sm8550
>  
> +      - items:
> +          - enum:
> +              - qcom,sc8380xp-crd
> +              - qcom,sc8380xp-qcp
> +          - const: qcom,sc8380xp

This is some odd placement. Please keep things properly ordered.

Best regards,
Krzysztof
Sibi Sankar Nov. 17, 2023, 4:05 a.m. UTC | #3
Hey Konrad,

Thanks for taking time to review the series.

On 10/26/23 15:54, Konrad Dybcio wrote:
> 
> 
> On 10/25/23 16:24, Sibi Sankar wrote:
>> From: Rajendra Nayak <quic_rjendra@quicinc.com>
>>
>> Document the SC8380XP SoC binding and also the boards using it.
>> Also document the new board id qcp (Qualcomm Compute Platform).
>>
>> Signed-off-by: Rajendra Nayak <quic_rjendra@quicinc.com>
>> Signed-off-by: Sibi Sankar <quic_sibis@quicinc.com>
>> ---
> Looks good, but can you elaborate a bit more on QCP? Is it going
> to be a new fuse setting for devices that fall into the "compute"
> category, but aren't necessarily CRDs?

QCP and CRD have different form factors, the former looks like a laptop
while the latter isn't. The fuse settings probably won't have any diffs
between them but the board schematics will have diffs.

-Sibi

> 
> Konrad
Sibi Sankar Nov. 17, 2023, 4:07 a.m. UTC | #4
Hey Krzysztof,

Thanks for taking time to review the series.

On 10/27/23 13:46, Krzysztof Kozlowski wrote:
> On 25/10/2023 16:24, Sibi Sankar wrote:
>> From: Rajendra Nayak <quic_rjendra@quicinc.com>
>>
>> Document the SC8380XP SoC binding and also the boards using it.
>> Also document the new board id qcp (Qualcomm Compute Platform).
>>
>> Signed-off-by: Rajendra Nayak <quic_rjendra@quicinc.com>
>> Signed-off-by: Sibi Sankar <quic_sibis@quicinc.com>
>> ---
>>   Documentation/devicetree/bindings/arm/qcom.yaml | 8 ++++++++
>>   1 file changed, 8 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
>> index 88b84035e7b1..6145e9ad340c 100644
>> --- a/Documentation/devicetree/bindings/arm/qcom.yaml
>> +++ b/Documentation/devicetree/bindings/arm/qcom.yaml
>> @@ -62,6 +62,7 @@ description: |
>>           sc7280
>>           sc8180x
>>           sc8280xp
>> +        sc8380xp
>>           sda660
>>           sdm450
>>           sdm630
>> @@ -110,6 +111,7 @@ description: |
>>           liquid
>>           rdp432-c2
>>           mtp
>> +        qcp
>>           qrd
>>           rb2
>>           ride
>> @@ -1044,6 +1046,12 @@ properties:
>>                 - qcom,sm8550-qrd
>>             - const: qcom,sm8550
>>   
>> +      - items:
>> +          - enum:
>> +              - qcom,sc8380xp-crd
>> +              - qcom,sc8380xp-qcp
>> +          - const: qcom,sc8380xp
> 
> This is some odd placement. Please keep things properly ordered.

will fix this in the next re-spin.

-Sibi

> 
> Best regards,
> Krzysztof
>
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
index 88b84035e7b1..6145e9ad340c 100644
--- a/Documentation/devicetree/bindings/arm/qcom.yaml
+++ b/Documentation/devicetree/bindings/arm/qcom.yaml
@@ -62,6 +62,7 @@  description: |
         sc7280
         sc8180x
         sc8280xp
+        sc8380xp
         sda660
         sdm450
         sdm630
@@ -110,6 +111,7 @@  description: |
         liquid
         rdp432-c2
         mtp
+        qcp
         qrd
         rb2
         ride
@@ -1044,6 +1046,12 @@  properties:
               - qcom,sm8550-qrd
           - const: qcom,sm8550
 
+      - items:
+          - enum:
+              - qcom,sc8380xp-crd
+              - qcom,sc8380xp-qcp
+          - const: qcom,sc8380xp
+
   # Board compatibles go above
 
   qcom,msm-id: