On Fri, Jan 15, 2021 at 11:16:13AM +0100, Pawel Dembicki wrote: > This patch adds spi-uart controller to LS1012A-FRDM board dts. > Device is equipped in SC16IS740 from NXP. > > Signed-off-by: Pawel Dembicki <paweldembicki@xxxxxxxxx> > --- > .../boot/dts/freescale/fsl-ls1012a-frdm.dts | 21 +++++++++++++++++++ > 1 file changed, 21 insertions(+) > > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts b/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts > index 67702667ed8a..9473d16336a2 100644 > --- a/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts > @@ -7,6 +7,7 @@ > */ > /dts-v1/; > > +#include <dt-bindings/interrupt-controller/irq.h> > #include "fsl-ls1012a.dtsi" > > / { > @@ -57,6 +58,26 @@ simple-audio-card,codec { > }; > }; > > +&dspi { > + status = "okay"; > + bus-num = <0>; Let's end property list with `status`. > + > + serial@0 { > + reg = <0>; > + compatible = "nxp,sc16is740"; Let's start property list with `compatible`. > + spi-max-frequency = <4000000>; > + clocks = <&sc16is7xx_clk>; > + interrupt-parent = <&gpio1>; > + interrupts = <13 IRQ_TYPE_EDGE_FALLING>; > + > + sc16is7xx_clk: sc16is7xx_clk { clock-sc16is7xx for node name maybe. Shawn > + compatible = "fixed-clock"; > + #clock-cells = <0>; > + clock-frequency = <24000000>; > + }; > + }; > +}; > + > &duart0 { > status = "okay"; > }; > -- > 2.25.1 >