| Message ID | 20231025142427.2661-3-quic_sibis@quicinc.com |
|---|---|
| State | Changes Requested, archived |
| Headers | show |
| Series | dts: qcom: Introduce SC8380XP platforms device tree | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | success | |
| robh/patch-applied | success | |
| robh/dtbs-check | warning | build log |
| robh/dt-meta-schema | success |
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
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
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
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 --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: