On 18/02/2023 23:39, Marek Vasut wrote: > The UART IP must be connected to clock, document the properties in DT bindings. > Update example to match Linux arch/arm/boot/dts/imx51.dtsi . > > Signed-off-by: Marek Vasut <marex@xxxxxxx> > --- > Cc: Fabio Estevam <festevam@xxxxxxxxx> > Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> > Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@xxxxxxxxxx> > Cc: NXP Linux Team <linux-imx@xxxxxxx> > Cc: Pengutronix Kernel Team <kernel@xxxxxxxxxxxxxx> > Cc: Rob Herring <robh+dt@xxxxxxxxxx> > Cc: Sascha Hauer <s.hauer@xxxxxxxxxxxxxx> > Cc: Shawn Guo <shawnguo@xxxxxxxxxx> > Cc: devicetree@xxxxxxxxxxxxxxx > Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx > Cc: linux-serial@xxxxxxxxxxxxxxx > --- > .../bindings/serial/fsl-imx-uart.yaml | 19 +++++++++++++++++++ > 1 file changed, 19 insertions(+) > > diff --git a/Documentation/devicetree/bindings/serial/fsl-imx-uart.yaml b/Documentation/devicetree/bindings/serial/fsl-imx-uart.yaml > index c22aab8c55f83..d42061331aba3 100644 > --- a/Documentation/devicetree/bindings/serial/fsl-imx-uart.yaml > +++ b/Documentation/devicetree/bindings/serial/fsl-imx-uart.yaml > @@ -49,6 +49,18 @@ properties: > reg: > maxItems: 1 > > + assigned-clocks: true > + assigned-clock-rates: true > + assigned-clock-parents: true You shouldn't need these three. > + > + clocks: > + maxItems: 2 > + Best regards, Krzysztof