| Message ID | 20240904-qcs8300_initial_dtsi-v1-16-d0ea9afdc007@quicinc.com |
|---|---|
| State | Changes Requested |
| Headers | show |
| Series | Add initial support for QCS8300 | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | success | |
| robh/patch-applied | success | |
| robh/dtbs-check | warning | build log |
| robh/dt-meta-schema | success |
On 04/09/2024 10:33, Jingyi Wang wrote: > Document the QCS8275/QCS8300 SoC and its reference board QCS8300 RIDE. > QCS8300 is an Industrial Safe SoC, while QCS8275 is the Industrial > Non-Safe version which can share the same SoC dtsi and board DTS. > > Signed-off-by: Jingyi Wang <quic_jingyw@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 c0529486810f..ccf9a166368f 100644 > --- a/Documentation/devicetree/bindings/arm/qcom.yaml > +++ b/Documentation/devicetree/bindings/arm/qcom.yaml > @@ -42,6 +42,8 @@ description: | > msm8996 > msm8998 > qcs404 > + qcs8275 > + qcs8300 > qcs8550 > qcm2290 > qcm6490 > @@ -895,6 +897,12 @@ properties: > - const: qcom,qcs404-evb > - const: qcom,qcs404 > > + - items: > + - enum: > + - qcom,qcs8300-ride > + - const: qcom,qcs8275 So the qcs8300 ride comes with non-safe SoC? Best regards, Krzysztof
On 9/4/2024 5:38 PM, Krzysztof Kozlowski wrote: > On 04/09/2024 10:33, Jingyi Wang wrote: >> Document the QCS8275/QCS8300 SoC and its reference board QCS8300 RIDE. >> QCS8300 is an Industrial Safe SoC, while QCS8275 is the Industrial >> Non-Safe version which can share the same SoC dtsi and board DTS. >> >> Signed-off-by: Jingyi Wang <quic_jingyw@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 c0529486810f..ccf9a166368f 100644 >> --- a/Documentation/devicetree/bindings/arm/qcom.yaml >> +++ b/Documentation/devicetree/bindings/arm/qcom.yaml >> @@ -42,6 +42,8 @@ description: | >> msm8996 >> msm8998 >> qcs404 >> + qcs8275 >> + qcs8300 >> qcs8550 >> qcm2290 >> qcm6490 >> @@ -895,6 +897,12 @@ properties: >> - const: qcom,qcs404-evb >> - const: qcom,qcs404 >> >> + - items: >> + - enum: >> + - qcom,qcs8300-ride >> + - const: qcom,qcs8275 > > So the qcs8300 ride comes with non-safe SoC? > > Best regards, > Krzysztof > Both QCS8275 and QCS8300 SoC can reference qcs8300 ride board. Could you describe your suggestion in more detail? Thanks, Jingyi
On 05/09/2024 06:42, Jingyi Wang wrote: > > > On 9/4/2024 5:38 PM, Krzysztof Kozlowski wrote: >> On 04/09/2024 10:33, Jingyi Wang wrote: >>> Document the QCS8275/QCS8300 SoC and its reference board QCS8300 RIDE. >>> QCS8300 is an Industrial Safe SoC, while QCS8275 is the Industrial >>> Non-Safe version which can share the same SoC dtsi and board DTS. >>> >>> Signed-off-by: Jingyi Wang <quic_jingyw@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 c0529486810f..ccf9a166368f 100644 >>> --- a/Documentation/devicetree/bindings/arm/qcom.yaml >>> +++ b/Documentation/devicetree/bindings/arm/qcom.yaml >>> @@ -42,6 +42,8 @@ description: | >>> msm8996 >>> msm8998 >>> qcs404 >>> + qcs8275 >>> + qcs8300 >>> qcs8550 >>> qcm2290 >>> qcm6490 >>> @@ -895,6 +897,12 @@ properties: >>> - const: qcom,qcs404-evb >>> - const: qcom,qcs404 >>> >>> + - items: >>> + - enum: >>> + - qcom,qcs8300-ride >>> + - const: qcom,qcs8275 >> >> So the qcs8300 ride comes with non-safe SoC? >> >> Best regards, >> Krzysztof >> > Both QCS8275 and QCS8300 SoC can reference qcs8300 ride board. Could you > describe your suggestion in more detail? I did not suggest anything. I am confused that you claim that every qcs8300 is using the non-safe flavor of the SoC. I am fine with this but I want to understand it and be sure you will not change it next month when you learn what this means. Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml index c0529486810f..ccf9a166368f 100644 --- a/Documentation/devicetree/bindings/arm/qcom.yaml +++ b/Documentation/devicetree/bindings/arm/qcom.yaml @@ -42,6 +42,8 @@ description: | msm8996 msm8998 qcs404 + qcs8275 + qcs8300 qcs8550 qcm2290 qcm6490 @@ -895,6 +897,12 @@ properties: - const: qcom,qcs404-evb - const: qcom,qcs404 + - items: + - enum: + - qcom,qcs8300-ride + - const: qcom,qcs8275 + - const: qcom,qcs8300 + - items: - enum: - qcom,sa8155p-adp
Document the QCS8275/QCS8300 SoC and its reference board QCS8300 RIDE. QCS8300 is an Industrial Safe SoC, while QCS8275 is the Industrial Non-Safe version which can share the same SoC dtsi and board DTS. Signed-off-by: Jingyi Wang <quic_jingyw@quicinc.com> --- Documentation/devicetree/bindings/arm/qcom.yaml | 8 ++++++++ 1 file changed, 8 insertions(+)