On Mon, Apr 29, 2024 at 7:11 PM Stephen Boyd <sboyd@xxxxxxxxxx> wrote: > > Quoting Rob Herring (Arm) (2024-04-25 11:38:09) > > diff --git a/Documentation/devicetree/bindings/clock/fixed-clock.yaml b/Documentation/devicetree/bindings/clock/fixed-clock.yaml > > index b0a4fb8256e2..d287bd092054 100644 > > --- a/Documentation/devicetree/bindings/clock/fixed-clock.yaml > > +++ b/Documentation/devicetree/bindings/clock/fixed-clock.yaml > > @@ -11,6 +11,13 @@ maintainers: > > - Stephen Boyd <sboyd@xxxxxxxxxx> > > > > properties: > > + $nodename: > > + anyOf: > > + - description: Preferred name is 'clock-<freq>' > > Is the preferred value of <freq> the clock-frequency property? Should > say that explicitly somehow so that it's clear. Yes, will add. > > + pattern: "^clock-([0-9]+|[a-z0-9-]+)$" > > + - description: Any name allowed > > + deprecated: true > > + > > compatible: > > const: fixed-clock > > > > diff --git a/Documentation/devicetree/bindings/clock/fixed-factor-clock.yaml b/Documentation/devicetree/bindings/clock/fixed-factor-clock.yaml > > index 8f71ab300470..962a1fe85416 100644 > > --- a/Documentation/devicetree/bindings/clock/fixed-factor-clock.yaml > > +++ b/Documentation/devicetree/bindings/clock/fixed-factor-clock.yaml > > @@ -11,6 +11,14 @@ maintainers: > > - Stephen Boyd <sboyd@xxxxxxxxxx> > > > > properties: > > + $nodename: > > + anyOf: > > + - description: > > + Preferred name is 'clock-<freq>' if the input frequency is fixed > > Similar question here. Is <freq> supposed to be the output clock frequency? Yes. > > > + pattern: "^clock-([0-9]+|[0-9a-z-]+)$" > > + - description: Any name allowed > > + deprecated: true > > I hope that deprecating this doesn't cause folks to try to clean things > up and then break code that's depending on the node name to be the name > of the clk. We don't want that string name to be important but it is > sometimes. Right. Leaving things as-is is the intent. We won't be enabling any "no deprecated" mode by default when it is added. Rob
