devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Question: use nodename clock-<freq> for fixed-clock nodes
@ 2025-07-02 11:47 Peng Fan
  2025-07-02 12:02 ` Krzysztof Kozlowski
  0 siblings, 1 reply; 3+ messages in thread
From: Peng Fan @ 2025-07-02 11:47 UTC (permalink / raw)
  To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Stephen Boyd
  Cc: open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
	Frank Li

Hi All,

I am thinking to write a script to convert all the "fixed-clock"
nodes that have clock-frequency property to use new node
name "clock-<freq>".

Then we could drop this from fixed-clock.yaml
"
      - description: Any name allowed                                                               
        deprecated: true
"

I am not sure whether this work is preferred or this is
an ABI break because of nodename change.

If this work is welcomed, should I use a arch level
treewide patch to use one patch cover one arch,
or use small patch to for each dts[i]?

Thanks,
Peng.


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Question: use nodename clock-<freq> for fixed-clock nodes
  2025-07-02 11:47 Question: use nodename clock-<freq> for fixed-clock nodes Peng Fan
@ 2025-07-02 12:02 ` Krzysztof Kozlowski
  2025-07-02 13:37   ` Peng Fan
  0 siblings, 1 reply; 3+ messages in thread
From: Krzysztof Kozlowski @ 2025-07-02 12:02 UTC (permalink / raw)
  To: Peng Fan, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	Stephen Boyd
  Cc: open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
	Frank Li

On 02/07/2025 13:47, Peng Fan wrote:
> Hi All,
> 
> I am thinking to write a script to convert all the "fixed-clock"
> nodes that have clock-frequency property to use new node
> name "clock-<freq>".
> 
> Then we could drop this from fixed-clock.yaml
> "
>       - description: Any name allowed                                                               
>         deprecated: true
> "
> 
> I am not sure whether this work is preferred or this is
> an ABI break because of nodename change.
> 
Pretty huge churn for little impact, IMO. Considering the node name is
relevant (it will not be ABI break... it will be just runtime break), if
there is no output names, you would need to get extensive testing, which
I think we both know you did not plan to.

So no...

Best regards,
Krzysztof

^ permalink raw reply	[flat|nested] 3+ messages in thread

* RE: Question: use nodename clock-<freq> for fixed-clock nodes
  2025-07-02 12:02 ` Krzysztof Kozlowski
@ 2025-07-02 13:37   ` Peng Fan
  0 siblings, 0 replies; 3+ messages in thread
From: Peng Fan @ 2025-07-02 13:37 UTC (permalink / raw)
  To: Krzysztof Kozlowski, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Stephen Boyd
  Cc: open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
	Frank Li

> Subject: Re: Question: use nodename clock-<freq> for fixed-clock nodes
> 
> On 02/07/2025 13:47, Peng Fan wrote:
> > Hi All,
> >
> > I am thinking to write a script to convert all the "fixed-clock"
> > nodes that have clock-frequency property to use new node name
> > "clock-<freq>".
> >
> > Then we could drop this from fixed-clock.yaml "
> >       - description: Any name allowed
> >         deprecated: true
> > "
> >
> > I am not sure whether this work is preferred or this is an ABI break
> > because of nodename change.
> >
> Pretty huge churn for little impact, IMO. Considering the node name is
> relevant (it will not be ABI break... it will be just runtime break), if there
> is no output names, you would need to get extensive testing, which I
> think we both know you did not plan to.
> 
> So no...

Sure, no problem. Thanks for explaining.

Thanks,
Peng.

> 
> Best regards,
> Krzysztof

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2025-07-02 13:37 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-07-02 11:47 Question: use nodename clock-<freq> for fixed-clock nodes Peng Fan
2025-07-02 12:02 ` Krzysztof Kozlowski
2025-07-02 13:37   ` Peng Fan

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).