netdev - Re: [PATCH v2] dt-bindings: net: dsa: mediatek,mt7530: Minor wording fixes

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] [day] [month] [year] [list]

Date: Wed, 26 Jun 2024 09:38:22 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Conor Dooley <conor@...nel.org>,
 Chris Packham <chris.packham@...iedtelesis.co.nz>
Cc: andrew@...n.ch, f.fainelli@...il.com, olteanv@...il.com,
 netdev@...r.kernel.org, devicetree@...r.kernel.org,
 linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
 linux-mediatek@...ts.infradead.org, "David S. Miller" <davem@...emloft.net>,
 Eric Dumazet <edumazet@...gle.com>, Jakub Kicinski <kuba@...nel.org>,
 Paolo Abeni <pabeni@...hat.com>, Rob Herring <robh@...nel.org>,
 Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
 <conor+dt@...nel.org>, Matthias Brugger <matthias.bgg@...il.com>,
 AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>,
 Arınç ÜNAL <arinc.unal@...nc9.com>,
 Landen Chao <Landen.Chao@...iatek.com>, DENG Qingfang <dqfext@...il.com>,
 Sean Wang <sean.wang@...iatek.com>, Daniel Golle <daniel@...rotopia.org>
Subject: Re: [PATCH v2] dt-bindings: net: dsa: mediatek,mt7530: Minor wording
 fixes

On 25/06/2024 18:00, Conor Dooley wrote:
> On Tue, Jun 25, 2024 at 09:18:57AM +1200, Chris Packham wrote:
>> Update the mt7530 binding with some minor updates that make the document
>> easier to read.
>>
>> Signed-off-by: Chris Packham <chris.packham@...iedtelesis.co.nz>
>> ---
>>
>> Notes:
>>     I was referring to this dt binding and found a couple of places where
>>     the wording could be improved. I'm not exactly a techical writer but
>>     hopefully I've made things a bit better.
>>     
>>     Changes in v2:
>>     - Update title, this is not just fixing grammar
>>     - Add missing The instead of changing has to have
>>
> 
> I don't really want to ack this, a 4th ack for some wording that has
> no impact on the binding itself just seems so utterly silly to me...
> Instead I've spent more time writing how silly I think it is than
> hitting the ack macro would take :)

But look how many reviews/acks it got :). If you acked this, the patch
would have like 4 tags!

If only complex patches get so much attention... :)

Best regards,
Krzysztof