Messages in this thread | ![/](/images/icornerl.gif) | | From | Daniel Baluta <> | Date | Tue, 23 Apr 2019 09:16:26 +0300 | Subject | Re: [PATCH v2 1/2] arm64: dts: imx8mm: Add SAI nodes |
| |
On Tue, Apr 23, 2019 at 8:21 AM Marco Felsch <m.felsch@pengutronix.de> wrote: > > Hi Daniel, > > On 19-04-22 19:35, Daniel Baluta wrote: > > i.MX8MM has 5 SAI instances with the following base > > addresses according to RM. > > > > SAI1 base address: 3001_0000h > > SAI2 base address: 3002_0000h > > SAI3 base address: 3003_0000h > > SAI5 base address: 3005_0000h > > SAI6 base address: 3006_0000h > > > > Signed-off-by: Daniel Baluta <daniel.baluta@nxp.com> > > --- > > arch/arm64/boot/dts/freescale/imx8mm.dtsi | 71 +++++++++++++++++++++++ > > 1 file changed, 71 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > index de3498c2dd44..4d080dc47216 100644 > > --- a/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > +++ b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > @@ -171,6 +171,77 @@ > > #size-cells = <1>; > > ranges; > > > > + sai1: sai@30010000 { > > + compatible = "fsl,imx8mm-sai", > > + "fsl,imx8mq-sai"; > > Just a nitpick but I would not break this line here. This applies to the > other sai nodes too.
Yes, I thought about this and I think I will fix it as you say also I get past the 80 chars limit.
| ![\](/images/icornerr.gif) |