Re: [PATCH] dt-bindings: omap: Add TI Pandaboard A4 variant

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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?

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.


Best regards,
Krzysztof




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux