[PATCH] arm: dts: stm32: Drop redundant status=okay

[Krzysztof Kozlowski]

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