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

[Doug Anderson]

Hi,

On Tue, Aug 30, 2022 at 11:29 AM Douglas Anderson <dianders@xxxxxxxxxxxx> 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@xxxxxxxxxxxxxx/
> [2] https://lore.kernel.org/all/d3d4d90b-85b5-5ad9-78e6-5a074c21af4f@xxxxxxxxxx/
> [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