On Wed, Jan 22, 2025 at 04:08:27PM +0100, Krzysztof Kozlowski wrote: > On 22/01/2025 15:17, Tom Rini wrote: > > On Wed, Jan 22, 2025 at 09:36:29AM +0100, Krzysztof Kozlowski wrote: > >> On 22/01/2025 01:12, Tom Rini wrote: > >>> Document the ti,omap4-panda-a4 compatible string in the appropriate > >>> place within the omap family binding file. > >> > >> Why? Where is any user of this? Your commit msg should explain this, > >> because it's not obvious. Obvious is to send binding with the user, but > >> the second patch is missing. > > > > You were cc'd on > > https://lore.kernel.org/all/20250121200749.4131923-1-trini@xxxxxxxxxxxx/ > I am cc-ed on 200 patches per day and it means nothing. Do you expect me > to look for missing user in 1000 patches per week, for every 1000 > patches? Or how does it supposed to work? Being an overwhelmed maintainer sucks, I really do get that, sorry for my short reply. > But regardless, the majority of review is done via patchwork and that > DTS patch was not in that thread. It's missing. > > You are supposed to send the binding and the user DTS in the same > patchset. Separate makes little sense in case of kernel. > > If you need the binding for other projects, then of course above changes > into: always reference the other project submission. It's fixing a 10 year old regression where the equally viable option is to just delete the dts file as I'm 99.9% sure no one has even tried using it in that time. -- Tom
