Re: [PATCH 0/6] ARM: dts: qcom: msm8960: cleanup and add missing I2C nodes

[Antony Kurniawan Soemardi]

On 9/16/2025 10:24 PM, Bjorn Andersson wrote:
> On Sun, Sep 14, 2025 at 06:34:38PM +0000, Antony Kurniawan Soemardi wrote:
> > While working on adding support for the Sony Xperia SP (msm8960t), it
> > was pointed out [1] that the msm8960 dtsi could use some general
> > cleanup and reordering. This series applies those suggestions and also
> > fills in missing I2C nodes that are required by several phones based on
> > msm8960.
> >
> > Changes included:
> >
> > 1. Reorders nodes by unit address and sorts properties to comply with
> >     the Devicetree style guide. Cosmetic only.
> > 2. Inlines qcom-msm8960-pins.dtsi into the main SoC dtsi. Most Qualcomm
> >     SoCs embed their TLMM definitions directly, so this removes an
> >     uncommon split file pattern. Cosmetic only.
> > 3. Adds I2C child nodes for GSBI10 and GSBI12, which are present on
> >     MSM8960 and required by devices such as the Sony Xperia SP.
> > 4. Adds I2C child nodes for GSBI1 and GSBI8, which are present on
> >     MSM8960 and required by devices such as the Casio G'zOne. This work
> >     was contributed by Shinjo Park.
> > 5. Updates the binding documentation to rename the GPIO controller node
> >     from "msmgpio" to "tlmm".
> > 5. Renames the GPIO controller node label from "msmgpio" to "tlmm" to
> >     match the convention used by other Qualcomm SoCs. Cosmetic only.
> >
> > [1] https://lore.kernel.org/all/rcazqdcr747ujkba6z2j2oci27ajkqfi4hydgc2kiavm73y37x@g63jga2fcwmo/
> >
> > Thanks,
> >
> > ---
> > Antony Kurniawan Soemardi (6):
> >        ARM: dts: qcom: msm8960: reorder nodes and properties
> >        ARM: dts: qcom: msm8960: inline qcom-msm8960-pins.dtsi
> >        ARM: dts: qcom: msm8960: add I2C nodes for gsbi10 and gsbi12
> >        ARM: dts: qcom: msm8960: add I2C nodes for gsbi1 and gsbi8
> >        dt-bindings: pinctrl: qcom: msm8960: rename msmgpio node to tlmm
> >        ARM: dts: qcom: msm8960: rename msmgpio node to tlmm
> >
> >   .../bindings/pinctrl/qcom,msm8960-pinctrl.yaml     |   4 +-
> >   arch/arm/boot/dts/qcom/qcom-msm8960-cdp.dts        |  10 +-
> >   arch/arm/boot/dts/qcom/qcom-msm8960-pins.dtsi      |  59 --
> >   .../dts/qcom/qcom-msm8960-samsung-expressatt.dts   |  12 +-
> >   .../boot/dts/qcom/qcom-msm8960-sony-huashan.dts    |   2 +-
> >   arch/arm/boot/dts/qcom/qcom-msm8960.dtsi           | 727 ++++++++++++++-------
> >   6 files changed, 491 insertions(+), 323 deletions(-)
> > ---
> > base-commit: 9e410f58e82ea0c0e865f688d84edad1da7824d2
> I unfortunately don't have this commit, and I can't fine a branch where
> patch 1 applies cleanly. Can you please rebase this on top of linux-next
> and resubmit it?
>
> Regards,
> Bjorn
ah sorry, I was using Rudraksha's branch. Should I send the dt-bindings
patch separately now, or grouped together like this again on resubmit?

Thanks,
Antony K. S.