linux-kernel - Re: [PATCH 00/14] mips: dts: ralink: mt7621: improve DTS style

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: <5d6c36cb9dd9afda1efb69aa34058517@risingedge.co.za>
Date: Sun, 17 Mar 2024 17:22:39 +0200
From: Justin Swartz <justin.swartz@...ingedge.co.za>
To: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
Cc: Sergio Paracuellos <sergio.paracuellos@...il.com>, Arınç ÜNAL <arinc.unal@...nc9.com>, Rob Herring
 <robh+dt@...nel.org>, Krzysztof Kozlowski
 <krzysztof.kozlowski+dt@...aro.org>, Conor Dooley <conor+dt@...nel.org>,
 Thomas Bogendoerfer <tsbogend@...ha.franken.de>, Matthias Brugger
 <matthias.bgg@...il.com>, AngeloGioacchino Del Regno
 <angelogioacchino.delregno@...labora.com>, linux-mips@...r.kernel.org,
 devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
 linux-arm-kernel@...ts.infradead.org, linux-mediatek@...ts.infradead.org
Subject: Re: [PATCH 00/14] mips: dts: ralink: mt7621: improve DTS style

On 2024-03-17 17:10, Krzysztof Kozlowski wrote:
> On 16/03/2024 16:49, Sergio Paracuellos wrote:
>> On Sat, Mar 16, 2024 at 5:54 AM Justin Swartz
>> <justin.swartz@...ingedge.co.za> wrote:
>>> 
>>> This set of patches was created with the intention of cleaning up
>>> arch/mips/boot/dts/ralink/mt7621.dtsi so that it is aligned with
>>> the Devicetree Sources (DTS) Coding Style [1] [2] guide.
>>> 
>>> [1] Documentation/devicetree/bindings/dts-coding-style.rst
>>> 
>>> [2] https://docs.kernel.org/devicetree/bindings/dts-coding-style.html
>>> 
>>> Justin Swartz (14):
>>>   mips: dts: ralink: mt7621: reorder cpu node attributes
>>>   mips: dts: ralink: mt7621: reorder cpuintc node attributes
>>>   mips: dts: ralink: mt7621: reorder mmc regulator attributes
>>>   mips: dts: ralink: mt7621: reorder sysc node attributes
>>>   mips: dts: ralink: mt7621: reorder gpio node attributes
>>>   mips: dts: ralink: mt7621: reorder i2c node attributes
>>>   mips: dts: ralink: mt7621: reorder spi0 node attributes
>>>   mips: dts: ralink: mt7621: move pinctrl and sort its children
>>>   mips: dts: ralink: mt7621: reorder mmc node attributes
>>>   mips: dts: ralink: mt7621: reorder gic node attributes
>>>   mips: dts: ralink: mt7621: reorder ethernet node attributes and 
>>> kids
>>>   mips: dts: ralink: mt7621: reorder pcie node attributes and 
>>> children
>>>   mips: dts: ralink: mt7621: reorder pci?_phy attributes
> 
> These are all simple cleanups for the same file. It's one patch, not 
> 15.

I agree these are all simple cleanups.

Even though the cleanup pattern was the same, or very similar,
for each node affected, the intention was to isolate each change
to a single node (or a grouping of nodes of that seemed logical
to me) so that if anyone had any objections, the discussion would
be easier to follow in subthreads identifiable by patch names (and
thus subject lines) that clearly indicate the context.

But if there're no objections and it lessens the burden on
maintainers upstream to have less patches to apply, then I have no
problem combining them into a single patch.

Regards
Justin