mbox series

[v9,0/3] arm64: dts: qcom: Add new board revision and LTE SKUs for sc7280-villager family

Message ID 20220830182923.3720287-1-dianders@chromium.org
Headers show
Series arm64: dts: qcom: Add new board revision and LTE SKUs for sc7280-villager family | expand

Message

Doug Anderson Aug. 30, 2022, 6:29 p.m. UTC 
These patches add a new board revision for device Villager, and add
new LTE sku for both board revisions.

yaml issue has been clarified in [1] and [2], and 'status' has been
reordeded last since v4.

v9 is me squashing changes from my patches [3] [4] in. For patches I
touched I removed my Reviewed-by. I added my Signed-off-by to all
patches.

[1] https://lore.kernel.org/all/CAD=FV=WtKRFQr5jSQvsr08x9dgHrvenUWWtX_SKuCLuSvSH7WQ@mail.gmail.com/
[2] https://lore.kernel.org/all/d3d4d90b-85b5-5ad9-78e6-5a074c21af4f@linaro.org/
[3] https://lore.kernel.org/r/20220829084732.1.I9ef7f8b909a7afbef9ff2251a98c67033f37b516@changeid
[4] https://lore.kernel.org/r/20220829084732.2.I22e256d1ebac577a91fac44d1d12919be7111cd4@changeid/

Changes in v9:
- Squash https://lore.kernel.org/r/20220829084732.1.I9ef7f8b909a7afbef9ff2251a98c67033f37b516@changeid
- Squash https://lore.kernel.org/r/20220829084732.2.I22e256d1ebac577a91fac44d1d12919be7111cd4@changeid/

Changes in v7:
- Revise typo 'ARCG' to 'ARCH' in Makefile

Changes in v6:
- Remove v5 accidentally added sc7280-herobrine-herobrine-r1-lte.dts

Changes in v5:
- Revise rev0+ to rev0
- Add -r1 line to the Makefile
- Reorder '.dtb' in Makefile
- Put the "interconnects" line back

Changes in v4:
- ("...: Add herobrine-villager-r1") new for v4
- Reorder 'status' last

Changes in v2:
- ("...: document sc7280 and villager board") new for v2.

Jimmy Chen (3):
  dt-bindings: arm: qcom: document sc7280 and villager board
  arm64: dts: qcom: sc7280: Add herobrine-villager-r1
  arm64: dts: qcom: Add LTE SKUs for sc7280-villager family

 .../devicetree/bindings/arm/qcom.yaml         |  10 +
 arch/arm64/boot/dts/qcom/Makefile             |   2 +
 .../boot/dts/qcom/sc7280-chrome-common.dtsi   |  11 -
 .../boot/dts/qcom/sc7280-herobrine-crd.dts    |   1 +
 .../qcom/sc7280-herobrine-herobrine-r1.dts    |   1 +
 .../dts/qcom/sc7280-herobrine-lte-sku.dtsi    |  17 +
 .../dts/qcom/sc7280-herobrine-villager-r0.dts | 325 +----------------
 .../qcom/sc7280-herobrine-villager-r1-lte.dts |  14 +
 .../dts/qcom/sc7280-herobrine-villager-r1.dts |  16 +
 .../dts/qcom/sc7280-herobrine-villager.dtsi   | 326 ++++++++++++++++++
 arch/arm64/boot/dts/qcom/sc7280-idp.dts       |   1 +
 11 files changed, 392 insertions(+), 332 deletions(-)
 create mode 100644 arch/arm64/boot/dts/qcom/sc7280-herobrine-lte-sku.dtsi
 create mode 100644 arch/arm64/boot/dts/qcom/sc7280-herobrine-villager-r1-lte.dts
 create mode 100644 arch/arm64/boot/dts/qcom/sc7280-herobrine-villager-r1.dts
 create mode 100644 arch/arm64/boot/dts/qcom/sc7280-herobrine-villager.dtsi

Comments

Doug Anderson Sept. 6, 2022, 10 p.m. UTC | #1 
Hi,

On Tue, Aug 30, 2022 at 11:29 AM Douglas Anderson <dianders@chromium.org> wrote:
>
> These patches add a new board revision for device Villager, and add
> new LTE sku for both board revisions.
>
> yaml issue has been clarified in [1] and [2], and 'status' has been
> reordeded last since v4.
>
> v9 is me squashing changes from my patches [3] [4] in. For patches I
> touched I removed my Reviewed-by. I added my Signed-off-by to all
> patches.
>
> [1] https://lore.kernel.org/all/CAD=FV=WtKRFQr5jSQvsr08x9dgHrvenUWWtX_SKuCLuSvSH7WQ@mail.gmail.com/
> [2] https://lore.kernel.org/all/d3d4d90b-85b5-5ad9-78e6-5a074c21af4f@linaro.org/
> [3] https://lore.kernel.org/r/20220829084732.1.I9ef7f8b909a7afbef9ff2251a98c67033f37b516@changeid
> [4] https://lore.kernel.org/r/20220829084732.2.I22e256d1ebac577a91fac44d1d12919be7111cd4@changeid/
>
> Changes in v9:
> - Squash https://lore.kernel.org/r/20220829084732.1.I9ef7f8b909a7afbef9ff2251a98c67033f37b516@changeid
> - Squash https://lore.kernel.org/r/20220829084732.2.I22e256d1ebac577a91fac44d1d12919be7111cd4@changeid/
>
> Changes in v7:
> - Revise typo 'ARCG' to 'ARCH' in Makefile
>
> Changes in v6:
> - Remove v5 accidentally added sc7280-herobrine-herobrine-r1-lte.dts
>
> Changes in v5:
> - Revise rev0+ to rev0
> - Add -r1 line to the Makefile
> - Reorder '.dtb' in Makefile
> - Put the "interconnects" line back
>
> Changes in v4:
> - ("...: Add herobrine-villager-r1") new for v4
> - Reorder 'status' last
>
> Changes in v2:
> - ("...: document sc7280 and villager board") new for v2.
>
> Jimmy Chen (3):
>   dt-bindings: arm: qcom: document sc7280 and villager board
>   arm64: dts: qcom: sc7280: Add herobrine-villager-r1
>   arm64: dts: qcom: Add LTE SKUs for sc7280-villager family
>
>  .../devicetree/bindings/arm/qcom.yaml         |  10 +
>  arch/arm64/boot/dts/qcom/Makefile             |   2 +
>  .../boot/dts/qcom/sc7280-chrome-common.dtsi   |  11 -
>  .../boot/dts/qcom/sc7280-herobrine-crd.dts    |   1 +
>  .../qcom/sc7280-herobrine-herobrine-r1.dts    |   1 +
>  .../dts/qcom/sc7280-herobrine-lte-sku.dtsi    |  17 +
>  .../dts/qcom/sc7280-herobrine-villager-r0.dts | 325 +----------------
>  .../qcom/sc7280-herobrine-villager-r1-lte.dts |  14 +
>  .../dts/qcom/sc7280-herobrine-villager-r1.dts |  16 +
>  .../dts/qcom/sc7280-herobrine-villager.dtsi   | 326 ++++++++++++++++++
>  arch/arm64/boot/dts/qcom/sc7280-idp.dts       |   1 +
>  11 files changed, 392 insertions(+), 332 deletions(-)
>  create mode 100644 arch/arm64/boot/dts/qcom/sc7280-herobrine-lte-sku.dtsi
>  create mode 100644 arch/arm64/boot/dts/qcom/sc7280-herobrine-villager-r1-lte.dts
>  create mode 100644 arch/arm64/boot/dts/qcom/sc7280-herobrine-villager-r1.dts
>  create mode 100644 arch/arm64/boot/dts/qcom/sc7280-herobrine-villager.dtsi

Is there anything blocking these changes from landing? I just tried
applying them and they now give a (tiny) Makefile context conflict
with the evoker dts that just landed, but "git am -3" handled it fine
for me.

-Doug