| Message ID | 20220520143502.v4.4.I1318c1ae2ce55ade1d092fc21df846360b15c560@changeid (mailing list archive) |
|---|---|
| State | Accepted |
| Headers | show |
| Series | [v4,1/5] dt-bindings: Document how Chromebooks with depthcharge boot | expand |
On 20/05/2022 23:38, Douglas Anderson wrote: > This copy-pastes compatibles from sc7280-based boards from the device > trees to the yaml file. It also fixes the CRD/IDP bindings which had > gotten stale. > > Signed-off-by: Douglas Anderson <dianders@chromium.org> > Reviewed-by: Matthias Kaehlcke <mka@chromium.org> > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- > It should be noted that these match the sc7280 boards as of the top of > the "for-next" branch of the tree > git.kernel.org/pub/scm/linux/kernel/git/qcom/linux.git > > (no changes since v2) > > Changes in v2: > - Use a "description" instead of a comment for each item. What's the plan for these patches? This is one was reviewed, the 5/5 had comments. Is there going to be resend or pick up? Some other folks work depends on this. Best regards, Krzysztof
Hi, On Wed, Jun 22, 2022 at 1:27 AM Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote: > > On 20/05/2022 23:38, Douglas Anderson wrote: > > This copy-pastes compatibles from sc7280-based boards from the device > > trees to the yaml file. It also fixes the CRD/IDP bindings which had > > gotten stale. > > > > Signed-off-by: Douglas Anderson <dianders@chromium.org> > > Reviewed-by: Matthias Kaehlcke <mka@chromium.org> > > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > > --- > > It should be noted that these match the sc7280 boards as of the top of > > the "for-next" branch of the tree > > git.kernel.org/pub/scm/linux/kernel/git/qcom/linux.git > > > > (no changes since v2) > > > > Changes in v2: > > - Use a "description" instead of a comment for each item. > > What's the plan for these patches? This is one was reviewed, the 5/5 had > comments. Is there going to be resend or pick up? > > Some other folks work depends on this. I thought patch 5/5 was waiting on Bjorn to say whether he wanted a change or was happy the way it was. In general this series, like all of the Qualcomm dts patches at the moment, is blocked waiting for Bjorn to have time to land things. I'm assuming that if Bjorn wants the 5th patch changed then he will still land the first 4 patches and then I can make changes to 5/5 and send it as a standalone patch. -Doug
diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml index 01e40ea40724..3d150d1a93cd 100644 --- a/Documentation/devicetree/bindings/arm/qcom.yaml +++ b/Documentation/devicetree/bindings/arm/qcom.yaml @@ -404,14 +404,40 @@ properties: - const: google,trogdor-sku0 - const: qcom,sc7180 - - items: - - enum: - - qcom,sc7280-crd - - qcom,sc7280-idp - - qcom,sc7280-idp2 - - google,hoglin - - google,piglin - - google,senor + - description: Qualcomm Technologies, Inc. sc7280 CRD platform (rev3 - 4) + items: + - const: qcom,sc7280-crd + - const: google,hoglin-rev3 + - const: google,hoglin-rev4 + - const: google,piglin-rev3 + - const: google,piglin-rev4 + - const: qcom,sc7280 + + - description: Qualcomm Technologies, Inc. sc7280 CRD platform (newest rev) + items: + - const: google,hoglin + - const: qcom,sc7280 + + - description: Qualcomm Technologies, Inc. sc7280 IDP SKU1 platform + items: + - const: qcom,sc7280-idp + - const: google,senor + - const: qcom,sc7280 + + - description: Qualcomm Technologies, Inc. sc7280 IDP SKU2 platform + items: + - const: qcom,sc7280-idp2 + - const: google,piglin + - const: qcom,sc7280 + + - description: Google Herobrine (newest rev) + items: + - const: google,herobrine + - const: qcom,sc7280 + + - description: Google Villager (newest rev) + items: + - const: google,villager - const: qcom,sc7280 - items: