Re: [PATCH v3 1/4] ARM: dts: qcom: msm8960: expressatt: Sort node references and includes

[Rudraksha Gupta]

On 4/7/26 14:04, Krzysztof Kozlowski wrote:
> On 07/04/2026 22:39, Rudraksha Gupta wrote:
> > On 4/7/26 12:59, Krzysztof Kozlowski wrote:
> > > On 01/04/2026 22:32, Rudraksha Gupta via B4 Relay wrote:
> > > > From: Rudraksha Gupta <guptarud@xxxxxxxxx>
> > > >
> > > > Reorganize the DTS file for consistency with other msm8960 board files.
> > > >
> > > > Assisted-by: Claude:claude-opus-4.6
> > > > Signed-off-by: Rudraksha Gupta <guptarud@xxxxxxxxx>
> > > > ---
> > > >    .../dts/qcom/qcom-msm8960-samsung-expressatt.dts   | 408 +++++++++++----------
> > > >    1 file changed, 207 insertions(+), 201 deletions(-)
> > > Sorry, but no. We are not taking Claude as one determining coding style.
> > > Are we going to do the work again the moment we come with proper tool?
> > There is no tool currently to auto format DTS, and doesn't seem to be
> > coming for a while:
> >
> > https://www.youtube.com/watch?v=cvoIbTL_ZQA
> >
> >
> > Claude didn't determine the coding style. I did based on sony-huashan,
> > which is already upstream:
> >
> > https://github.com/torvalds/linux/blob/master/arch/arm/boot/dts/qcom/qcom-msm8960-sony-huashan.dts
> >
> >
> > I just used Claude to do the manual work for me. In v2, I made sure the
> > diff before and after the change was nill. v3 included additional
> > changes requested by Konrad and some comments that I remembered during
> > prior attempts mainlining patch series for this device.
> IMO, it is just too risky to let Claude reorganize the nodes, but I
> assume reviewers of your code did run dtx_diff.
>
> If they did not, then it is on them :(

That's fair. I also re-ran my tests after DTS reorganization and they 
all work:

https://wiki.postmarketos.org/wiki/Samsung_Galaxy_Express_SGH-I437_(samsung-expressatt)#Testing


Also, I think I'm the only oddball using this device too, so minimal 
impact. Someone listed the I8730 model in the wiki page, but I believe 
that's a MSM8930 device.


Hopefully that reassures you. :)


Thanks,

Rudraksha


> Best regards,
> Krzysztof