On 09:37-20220530, Krzysztof Kozlowski wrote: > On 30/05/2022 09:34, Rahul T R wrote: > > Add a pattern property for clock, also update the example with > > a clock node > > > > Signed-off-by: Rahul T R <r-ravikumar@xxxxxx> > > --- > > .../bindings/mfd/ti,j721e-system-controller.yaml | 10 ++++++++++ > > 1 file changed, 10 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/mfd/ti,j721e-system-controller.yaml b/Documentation/devicetree/bindings/mfd/ti,j721e-system-controller.yaml > > index fa86691ebf16..290b22cab52f 100644 > > --- a/Documentation/devicetree/bindings/mfd/ti,j721e-system-controller.yaml > > +++ b/Documentation/devicetree/bindings/mfd/ti,j721e-system-controller.yaml > > @@ -48,6 +48,10 @@ patternProperties: > > description: > > This is the SERDES lane control mux. > > > > + "^clock@[0-9a-f]+$": > > + type: object > > + $ref: /schemas/clock/ti,am654-ehrpwm-tbclk.yaml# > > + > > Thanks for the changes, but the question what is this remains. Clock > consumers use "clocks" property. Is this a clock controller (provider) > instead? > > Best regards, > Krzysztof Hi Krzysztof, Yes, this is a clock provider node, I will add the correct description and will also update the example with a clock consumer node that is using a clock from this That should be enough right? Regards Rahul T R