diff mbox series

[1/2] dt-bindings: arm: qcom: Adjust LTE SKUs for sc7280-villager

Message ID 20220829084732.1.I9ef7f8b909a7afbef9ff2251a98c67033f37b516@changeid (mailing list archive)
State Accepted
Headers show
Series [1/2] dt-bindings: arm: qcom: Adjust LTE SKUs for sc7280-villager | expand

Commit Message

Doug Anderson Aug. 29, 2022, 3:48 p.m. UTC
There have been a few changes since the patch ("dt-bindings: arm:
qcom: document sc7280 and villager board").
* New firmware reports LTE boards as "SKU 512" now. Old firmware will
  still report "SKU 0", but that's all pre-production and everyone
  will update.
* It's been relaized that no "-rev0" boards were ever built that were
  WiFi-only. Thus we don't two entries for -rev0.

Signed-off-by: Douglas Anderson <dianders@chromium.org>
---
This builds upon the patch ("dt-bindings: arm: qcom: document sc7280
and villager board") which hasn't landed yet. In theory this could be
squashed into that patch.

 Documentation/devicetree/bindings/arm/qcom.yaml | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

Comments

Krzysztof Kozlowski Aug. 30, 2022, 9:44 a.m. UTC | #1
On 29/08/2022 18:48, Douglas Anderson wrote:
> There have been a few changes since the patch ("dt-bindings: arm:
> qcom: document sc7280 and villager board").
> * New firmware reports LTE boards as "SKU 512" now. Old firmware will
>   still report "SKU 0", but that's all pre-production and everyone
>   will update.
> * It's been relaized that no "-rev0" boards were ever built that were
>   WiFi-only. Thus we don't two entries for -rev0.
> 
> Signed-off-by: Douglas Anderson <dianders@chromium.org>
> ---
> This builds upon the patch ("dt-bindings: arm: qcom: document sc7280
> and villager board") which hasn't landed yet. In theory this could be
> squashed into that patch.

Unless Bjorn already merged it, this should be squashed.

Best regards,
Krzysztof
Doug Anderson Aug. 30, 2022, 6:31 p.m. UTC | #2
Hi,

On Tue, Aug 30, 2022 at 2:44 AM Krzysztof Kozlowski
<krzysztof.kozlowski@linaro.org> wrote:
>
> On 29/08/2022 18:48, Douglas Anderson wrote:
> > There have been a few changes since the patch ("dt-bindings: arm:
> > qcom: document sc7280 and villager board").
> > * New firmware reports LTE boards as "SKU 512" now. Old firmware will
> >   still report "SKU 0", but that's all pre-production and everyone
> >   will update.
> > * It's been relaized that no "-rev0" boards were ever built that were
> >   WiFi-only. Thus we don't two entries for -rev0.
> >
> > Signed-off-by: Douglas Anderson <dianders@chromium.org>
> > ---
> > This builds upon the patch ("dt-bindings: arm: qcom: document sc7280
> > and villager board") which hasn't landed yet. In theory this could be
> > squashed into that patch.
>
> Unless Bjorn already merged it, this should be squashed.

Sure. Squashed as v9 of Jimmy's patches.

https://lore.kernel.org/r/20220830182923.3720287-1-dianders@chromium.org

-Doug
Bjorn Andersson Sept. 15, 2022, 3:36 a.m. UTC | #3
On Mon, 29 Aug 2022 08:48:23 -0700, Douglas Anderson wrote:
> There have been a few changes since the patch ("dt-bindings: arm:
> qcom: document sc7280 and villager board").
> * New firmware reports LTE boards as "SKU 512" now. Old firmware will
>   still report "SKU 0", but that's all pre-production and everyone
>   will update.
> * It's been relaized that no "-rev0" boards were ever built that were
>   WiFi-only. Thus we don't two entries for -rev0.
> 
> [...]

Applied, thanks!

[1/2] dt-bindings: arm: qcom: Adjust LTE SKUs for sc7280-villager
      commit: ff0ea86a538e80879243364bcf1a42e2d0eb6254
[2/2] arm64: dts: qcom: sc7280-villager: Adjust LTE SKUs
      commit: 68aa834823e09a540a23374129326a51efe877a3

Best regards,
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
index 8e46a3c5160f..40705e2d56de 100644
--- a/Documentation/devicetree/bindings/arm/qcom.yaml
+++ b/Documentation/devicetree/bindings/arm/qcom.yaml
@@ -582,14 +582,9 @@  properties:
           - const: google,villager
           - const: qcom,sc7280
 
-      - description: Google Villager with LTE (rev0)
-        items:
-          - const: google,villager-rev0-sku0
-          - const: qcom,sc7280
-
       - description: Google Villager with LTE (newest rev)
         items:
-          - const: google,villager-sku0
+          - const: google,villager-sku512
           - const: qcom,sc7280
 
       - items: