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@xxxxxxxxxx/ > > 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