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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <17d1708c-e326-468c-941a-1863dfdc946d@kernel.org>
Date: Wed, 22 Jan 2025 16:08:27 +0100
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Tom Rini <trini@...sulko.com>
Cc: linux-kernel@...r.kernel.org, Aaro Koskinen <aaro.koskinen@....fi>,
 Andreas Kemnade <andreas@...nade.info>, Kevin Hilman <khilman@...libre.com>,
 Roger Quadros <rogerq@...nel.org>, Tony Lindgren <tony@...mide.com>,
 Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
 Conor Dooley <conor+dt@...nel.org>, linux-omap@...r.kernel.org,
 devicetree@...r.kernel.org
Subject: Re: [PATCH] dt-bindings: omap: Add TI Pandaboard A4 variant

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@konsulko.com/
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