linux-kernel - Re: [PATCH] arm: dts: stm32: Drop redundant status=okay

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: <651df530-797a-45e1-b199-917deda33222@kernel.org>
Date: Mon, 18 Aug 2025 16:56:10 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Marek Vasut <marek.vasut@...lbox.org>,
 Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
 Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
 Conor Dooley <conor+dt@...nel.org>,
 Maxime Coquelin <mcoquelin.stm32@...il.com>,
 Alexandre Torgue <alexandre.torgue@...s.st.com>,
 Christoph Niedermaier <cniedermaier@...electronics.com>,
 Marek Vasut <marex@...x.de>, devicetree@...r.kernel.org,
 linux-stm32@...md-mailman.stormreply.com,
 linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
 kernel@...electronics.com
Subject: Re: [PATCH] arm: dts: stm32: Drop redundant status=okay

On 18/08/2025 16:51, Krzysztof Kozlowski wrote:
> On 18/08/2025 16:45, Marek Vasut wrote:
>> On 8/18/25 4:37 PM, Krzysztof Kozlowski wrote:
>>> Device nodes are enabled by default, so remove confusing or duplicated
>>> enabling of few nodes.  No practical impact, verified with dtx_diff.
>> I assume the "no practical impact" means DTs are identical before/after 
>> this patch ? If yes,
> 
> 
> No, DTS cannot be identical in this case because one had status, new one
> does not have. Practical impact means... visible impact in practice. How
> to say it more clearly?
To illustrate: this is "no practical impact":


--- dts-old/st/stm32mp157c-dhcom-picoitx.dtb
+++ dts-new/st/stm32mp157c-dhcom-picoitx.dtb
@@ -691,14 +691,12 @@
 					interrupt-controller;
 					interrupts-extended = <0x49 0x00
 					reg = <0x33>;
-					status = "okay";


But this would be a practical impact:


@@ -1124,7 +1121,7 @@
 					dmas = <0x26 0x59 0x400 0x01>;
 					phandle = <0x39>;
 					reg = <0x04 0x20>;
-					status = "okay";
+					status = "disabled";


Best regards,
Krzysztof