Messages in this thread | | | Date | Mon, 24 Oct 2022 12:31:55 -0400 | Subject | Re: [PATCH] arm64: dts: qcom: sc8280xp: add TCSR node | From | Krzysztof Kozlowski <> |
| |
On 24/10/2022 10:54, Johan Hovold wrote: >> >>> The binding has been picked up by Lee now so I posted the dts change. >>> Could have added a lore link though. >> >> This also would work and help a lot. >> >> It depends in general on the maintainer - for example Greg does not want >> to deal with individual patches, especially if DTS is just one patch and >> USB would be 10 of them. Our toolset is not good for picking up 10 out >> of 11. For all such cases - please provide link to lore. >> >> If however there are just two patches - one DTS and one for maintainer - >> then having them in one patchset should not cause additional effort for >> the maintainer. > > I'm pretty sure I saw Lee complaining about Bjorn taking also the > binding update through the qcom tree recently when someone did just > that. Apparently it was TCSR related too: > > https://lore.kernel.org/all/Yzbk%2F6SQdpNQTahV@google.com/ > > Heh. That was you. :)
Yep. It was a patchset of 16 patches where one should go to MFD and 15 to Qualcomm SoC.
> >> As you can see on the list, majority of patchsets consist of >> bindings+DTS. Pretty often entire piece - bindings+driver+DTS. > > Yeah, and whatever alternative you go with, someone will get it wrong or > complain it seems.
:)
Best regards, Krzysztof
| |