Re: [PATCH 4/4] dt-bindings: clock: ti: add ti,autoidle.yaml reference

Sukrut Bellary
2025-04-09T08:43:43Z

On Sat, Apr 05, 2025 at 09:49:04PM +0200, Andreas Kemnade wrote:
> Am Thu, 3 Apr 2025 18:45:00 -0700
> schrieb Sukrut Bellary <sbellary@baylibre.com>:
>
> > ti,divider-clock uses properties from ti,autoidle.
> >
> > As we are converting autoidle binding to ti,autoidle.yaml,
> > fix the reference here.
> >
> > Add dual license.
> >
> > Signed-off-by: Sukrut Bellary <sbellary@baylibre.com>
> > ---
> > .../bindings/clock/ti/ti,divider-clock.yaml | 24 ++++---------------
> > 1 file changed, 5 insertions(+), 19 deletions(-)
> >
> > diff --git a/Documentation/devicetree/bindings/clock/ti/ti,divider-clock.yaml b/Documentation/devicetree/bindings/clock/ti/ti,divider-clock.yaml
> > index 3fbe236eb565..aba879ae302d 100644
> > --- a/Documentation/devicetree/bindings/clock/ti/ti,divider-clock.yaml
> > +++ b/Documentation/devicetree/bindings/clock/ti/ti,divider-clock.yaml
> > @@ -1,4 +1,4 @@
> > -# SPDX-License-Identifier: GPL-2.0-only
> > +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
> > %YAML 1.2
> > ---
> > $id: http://devicetree.org/schemas/clock/ti/ti,divider-clock.yaml#
> > @@ -55,9 +55,10 @@ description: |
> > is missing it is the same as supplying a zero shift.
> >
> > This binding can also optionally provide support to the hardware autoidle
> > - feature, see [1].
> > + feature.
> >
> > - [1] Documentation/devicetree/bindings/clock/ti/autoidle.txt
> > +allOf:
> > + - $ref: /schemas/clock/ti/ti,autoidle.yaml#
> >
> > properties:
> > compatible:
> > @@ -97,7 +98,6 @@ properties:
> > minimum: 1
> > default: 1
> >
> > -
> > ti,max-div:
> > $ref: /schemas/types.yaml#/definitions/uint32
> > description:
> > @@ -116,20 +116,6 @@ properties:
> > valid divisor programming must be a power of two,
> > only valid if ti,dividers is not defined.
> >
> > - ti,autoidle-shift:
> > - $ref: /schemas/types.yaml#/definitions/uint32
> > - description:
> > - bit shift of the autoidle enable bit for the clock,
> > - see [1].
> > - maximum: 31
> > - default: 0
> > -
> now you make ti,autoidle-shift required. That makes so sense. A clock
> does not necessarily support autoidle.

Thanks for the review.
No, we are removing the required from yaml bindings and keeping it as
one of the properties. As all clock don't support autoidle, required is
not applicable here.

> Regards,
> Andreas