* [PATCH] arm64: dts: xilinx: Drop undocumented "always-on" properties
@ 2025-08-28 21:30 Rob Herring (Arm)
2025-08-29 5:46 ` Michal Simek
0 siblings, 1 reply; 4+ messages in thread
From: Rob Herring (Arm) @ 2025-08-28 21:30 UTC (permalink / raw)
To: Krzysztof Kozlowski, Conor Dooley, Michal Simek
Cc: devicetree, linux-arm-kernel, linux-kernel
The "always-on" properties are not documented for "silabs,si5341"
binding nor used by the Linux driver, so drop them.
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
---
arch/arm64/boot/dts/xilinx/zynqmp-zcu102-revA.dts | 8 --------
arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts | 6 ------
arch/arm64/boot/dts/xilinx/zynqmp-zcu111-revA.dts | 6 ------
3 files changed, 20 deletions(-)
diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-zcu102-revA.dts b/arch/arm64/boot/dts/xilinx/zynqmp-zcu102-revA.dts
index e172a30e7b21..6bee21d4fa4d 100644
--- a/arch/arm64/boot/dts/xilinx/zynqmp-zcu102-revA.dts
+++ b/arch/arm64/boot/dts/xilinx/zynqmp-zcu102-revA.dts
@@ -552,42 +552,34 @@ si5341: clock-generator@36 { /* SI5341 - u69 */
si5341_0: out@0 {
/* refclk0 for PS-GT, used for DP */
reg = <0>;
- always-on;
};
si5341_2: out@2 {
/* refclk2 for PS-GT, used for USB3 */
reg = <2>;
- always-on;
};
si5341_3: out@3 {
/* refclk3 for PS-GT, used for SATA */
reg = <3>;
- always-on;
};
si5341_4: out@4 {
/* refclk4 for PS-GT, used for PCIE slot */
reg = <4>;
- always-on;
};
si5341_5: out@5 {
/* refclk5 for PS-GT, used for PCIE */
reg = <5>;
- always-on;
};
si5341_6: out@6 {
/* refclk6 PL CLK125 */
reg = <6>;
- always-on;
};
si5341_7: out@7 {
/* refclk7 PL CLK74 */
reg = <7>;
- always-on;
};
si5341_9: out@9 {
/* refclk9 used for PS_REF_CLK 33.3 MHz */
reg = <9>;
- always-on;
};
};
};
diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts b/arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts
index cd132abf6e00..f450fad0edf8 100644
--- a/arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts
+++ b/arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts
@@ -551,32 +551,26 @@ si5341: clock-generator@36 { /* SI5341 - u69 */
si5341_0: out@0 {
/* refclk0 for PS-GT, used for DP */
reg = <0>;
- always-on;
};
si5341_2: out@2 {
/* refclk2 for PS-GT, used for USB3 */
reg = <2>;
- always-on;
};
si5341_3: out@3 {
/* refclk3 for PS-GT, used for SATA */
reg = <3>;
- always-on;
};
si5341_6: out@6 {
/* refclk6 PL CLK125 */
reg = <6>;
- always-on;
};
si5341_7: out@7 {
/* refclk7 PL CLK74 */
reg = <7>;
- always-on;
};
si5341_9: out@9 {
/* refclk9 used for PS_REF_CLK 33.3 MHz */
reg = <9>;
- always-on;
};
};
diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-zcu111-revA.dts b/arch/arm64/boot/dts/xilinx/zynqmp-zcu111-revA.dts
index 428b5558fbba..b4768f615182 100644
--- a/arch/arm64/boot/dts/xilinx/zynqmp-zcu111-revA.dts
+++ b/arch/arm64/boot/dts/xilinx/zynqmp-zcu111-revA.dts
@@ -439,32 +439,26 @@ si5341: clock-generator@36 { /* SI5341 - u46 */
si5341_0: out@0 {
/* refclk0 for PS-GT, used for DP */
reg = <0>;
- always-on;
};
si5341_2: out@2 {
/* refclk2 for PS-GT, used for USB3 */
reg = <2>;
- always-on;
};
si5341_3: out@3 {
/* refclk3 for PS-GT, used for SATA */
reg = <3>;
- always-on;
};
si5341_5: out@5 {
/* refclk5 PL CLK100 */
reg = <5>;
- always-on;
};
si5341_6: out@6 {
/* refclk6 PL CLK125 */
reg = <6>;
- always-on;
};
si5341_9: out@9 {
/* refclk9 used for PS_REF_CLK 33.3 MHz */
reg = <9>;
- always-on;
};
};
};
--
2.50.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] arm64: dts: xilinx: Drop undocumented "always-on" properties
2025-08-28 21:30 [PATCH] arm64: dts: xilinx: Drop undocumented "always-on" properties Rob Herring (Arm)
@ 2025-08-29 5:46 ` Michal Simek
2025-08-29 6:25 ` Mike Looijmans
0 siblings, 1 reply; 4+ messages in thread
From: Michal Simek @ 2025-08-29 5:46 UTC (permalink / raw)
To: Rob Herring (Arm), Krzysztof Kozlowski, Conor Dooley,
Mike Looijmans
Cc: devicetree, linux-arm-kernel, linux-kernel
Hi,
On 8/28/25 23:30, Rob Herring (Arm) wrote:
> The "always-on" properties are not documented for "silabs,si5341"
> binding nor used by the Linux driver, so drop them.
Binding
Documentation/devicetree/bindings/clock/silabs,si5341.txt
doesn't contain it
but driver drivers/clk/clk-si5341.c is reading it.
1345 config[num].always_on =
1346 of_property_read_bool(child, "always-on");
It is there from 2019 when driver has been added.
If it is not standard clock property it should be moved to standard one.
And DT binding should be converted to yaml with all properties used today.
Thanks,
Michal
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] arm64: dts: xilinx: Drop undocumented "always-on" properties
2025-08-29 5:46 ` Michal Simek
@ 2025-08-29 6:25 ` Mike Looijmans
2025-08-29 7:02 ` Krzysztof Kozlowski
0 siblings, 1 reply; 4+ messages in thread
From: Mike Looijmans @ 2025-08-29 6:25 UTC (permalink / raw)
To: Michal Simek, Rob Herring (Arm), Krzysztof Kozlowski,
Conor Dooley
Cc: devicetree, linux-arm-kernel, linux-kernel
On 29-08-2025 07:46, Michal Simek wrote:
> Hi,
>
> On 8/28/25 23:30, Rob Herring (Arm) wrote:
>> The "always-on" properties are not documented for "silabs,si5341"
>> binding nor used by the Linux driver, so drop them.
>
> Binding
> Documentation/devicetree/bindings/clock/silabs,si5341.txt
> doesn't contain it
> but driver drivers/clk/clk-si5341.c is reading it.
>
> 1345 config[num].always_on =
> 1346 of_property_read_bool(child, "always-on");
>
> It is there from 2019 when driver has been added.
>
> If it is not standard clock property it should be moved to standard one.
Agree - either "always-on" should become a standard property, or it should be
removed from the driver.
As a standard property, akin to "regulator-always-on", maybe it should
probably be renamed to "clock-always-on"?
I'd be happy to provide a patch either way.
> And DT binding should be converted to yaml with all properties used today.
>
> Thanks,
> Michal
>
>
Mike.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] arm64: dts: xilinx: Drop undocumented "always-on" properties
2025-08-29 6:25 ` Mike Looijmans
@ 2025-08-29 7:02 ` Krzysztof Kozlowski
0 siblings, 0 replies; 4+ messages in thread
From: Krzysztof Kozlowski @ 2025-08-29 7:02 UTC (permalink / raw)
To: Mike Looijmans, Michal Simek, Rob Herring (Arm),
Krzysztof Kozlowski, Conor Dooley
Cc: devicetree, linux-arm-kernel, linux-kernel
On 29/08/2025 08:25, Mike Looijmans wrote:
> On 29-08-2025 07:46, Michal Simek wrote:
>> Hi,
>>
>> On 8/28/25 23:30, Rob Herring (Arm) wrote:
>>> The "always-on" properties are not documented for "silabs,si5341"
>>> binding nor used by the Linux driver, so drop them.
>>
>> Binding
>> Documentation/devicetree/bindings/clock/silabs,si5341.txt
>> doesn't contain it
>> but driver drivers/clk/clk-si5341.c is reading it.
>>
>> 1345 config[num].always_on =
>> 1346 of_property_read_bool(child, "always-on");
>>
>> It is there from 2019 when driver has been added.
>>
>> If it is not standard clock property it should be moved to standard one.
>
> Agree - either "always-on" should become a standard property, or it should be
> removed from the driver.
>
> As a standard property, akin to "regulator-always-on", maybe it should
> probably be renamed to "clock-always-on"?
>
> I'd be happy to provide a patch either way.
That's an implied ABI now, so you cannot remove it, but about
"clock-always-on" - it does not look like a property we want in DT. If
on one board this is always connected (but on other not), you should
take the proper clock reference.
>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2025-08-29 7:05 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-08-28 21:30 [PATCH] arm64: dts: xilinx: Drop undocumented "always-on" properties Rob Herring (Arm)
2025-08-29 5:46 ` Michal Simek
2025-08-29 6:25 ` Mike Looijmans
2025-08-29 7:02 ` Krzysztof Kozlowski
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).