From mboxrd@z Thu Jan 1 00:00:00 1970 From: Uwe =?iso-8859-1?Q?Kleine-K=F6nig?= Subject: Re: [PATCH 2/2] serial/imx: add device tree probe support Date: Mon, 4 Jul 2011 11:10:27 +0200 Message-ID: <20110704091027.GG4338@pengutronix.de> References: <1309679760-22796-1-git-send-email-shawn.guo@linaro.org> <1309679760-22796-3-git-send-email-shawn.guo@linaro.org> <20110704065820.GY6069@pengutronix.de> <20110704073826.GC4338@pengutronix.de> <20110704090527.GF10245@S2100-06.ap.freescale.net> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Content-Disposition: inline In-Reply-To: <20110704090527.GF10245@S2100-06.ap.freescale.net> Sender: linux-serial-owner@vger.kernel.org To: Shawn Guo Cc: Sascha Hauer , patches@linaro.org, devicetree-discuss@lists.ozlabs.org, Jason Liu , linux-arm-kernel@lists.infradead.org, linux-serial@vger.kernel.org, Jeremy Kerr , Alan Cox List-Id: devicetree@vger.kernel.org On Mon, Jul 04, 2011 at 05:05:27PM +0800, Shawn Guo wrote: > On Mon, Jul 04, 2011 at 09:38:26AM +0200, Uwe Kleine-K=F6nig wrote: > > On Mon, Jul 04, 2011 at 08:58:20AM +0200, Sascha Hauer wrote: > > > On Sun, Jul 03, 2011 at 03:56:00PM +0800, Shawn Guo wrote: > > > > It adds device tree probe support for imx tty/serial driver. > > > >=20 > > > > Signed-off-by: Jeremy Kerr > > > > Signed-off-by: Jason Liu > > > > Signed-off-by: Shawn Guo > > > > Cc: Sascha Hauer > > > > Cc: Alan Cox > > > > Cc: Grant Likely > > > > --- > > > > .../bindings/tty/serial/fsl-imx-uart.txt | 19 ++++= + > > > > drivers/tty/serial/imx.c | 82 ++++= +++++++++++++--- > > > > 2 files changed, 91 insertions(+), 10 deletions(-) > > > > create mode 100644 Documentation/devicetree/bindings/tty/seria= l/fsl-imx-uart.txt > > > >=20 > > > > diff --git a/Documentation/devicetree/bindings/tty/serial/fsl-i= mx-uart.txt b/Documentation/devicetree/bindings/tty/serial/fsl-imx-uart= =2Etxt > > > > new file mode 100644 > > > > index 0000000..a9c0406 > > > > --- /dev/null > > > > +++ b/Documentation/devicetree/bindings/tty/serial/fsl-imx-uart= =2Etxt > > > > @@ -0,0 +1,19 @@ > > > > +* Freescale i.MX Universal Asynchronous Receiver/Transmitter (= UART) > > > > + > > > > +Required properties: > > > > +- compatible : Should be "fsl,-uart" > > > > +- reg : Address and length of the register set for the device > > > > +- interrupts : Should contain uart interrupt > > > > + > > > > +Optional properties: > > > > +- fsl,uart-has-rtscts : Indicate the uart has rts and cts > > > > +- fsl,irda-mode : Indicate the uart supports irda mode > > > > + > > > > +Example: > > > > + > > > > +uart@73fbc000 { > > > > + compatible =3D "fsl,imx51-uart", "fsl,imx21-uart"; > > > > + reg =3D <0x73fbc000 0x4000>; > > > > + interrupts =3D <31>; > > > > + fsl,uart-has-rtscts; > > > > +}; > > > > diff --git a/drivers/tty/serial/imx.c b/drivers/tty/serial/imx.= c > > > > index 983f3bd..483c447 100644 > > > > --- a/drivers/tty/serial/imx.c > > > > +++ b/drivers/tty/serial/imx.c > > > > @@ -45,6 +45,8 @@ > > > > #include > > > > #include > > > > #include > > > > +#include > > > > +#include > > > > =20 > > > > #include > > > > #include > > > > @@ -239,6 +241,12 @@ static struct platform_device_id imx_uart_= devtype[] =3D { > > > > } > > > > }; > > > > =20 > > > > +static struct of_device_id imx_uart_dt_ids[] =3D { > > > > + { .compatible =3D "fsl,imx1-uart", .data =3D &imx_uart_devdat= a[IMX1_UART], }, > > > > + { .compatible =3D "fsl,imx21-uart", .data =3D &imx_uart_devda= ta[IMX2_UART], }, > > > > + { /* sentinel */ }, > > > > +}; > > >=20 > > > Isn't this table a bit too small? What about the other SoCs? > > I'd expect that using fsl,imx21-uart on imx51 would result in a wor= king > > port, but it feels stange and if that's the intention the example a= bove > > needs fixing. > >=20 > Here is what I have in dts file for uart. The 'compatible' tells > that it's a imx51-uart and compatible with imx21-uart. >=20 > uart0: uart@73fbc000 { > compatible =3D "fsl,imx51-uart", "fsl,imx21-uart"; > reg =3D <0x73fbc000 0x4000>; > interrupts =3D <31>; > fsl,uart-has-rtscts; > }; /me learned something new about dt. Sorry for the noise. Best regards Uwe --=20 Pengutronix e.K. | Uwe Kleine-K=F6nig = | Industrial Linux Solutions | http://www.pengutronix.de/= | -- To unsubscribe from this list: send the line "unsubscribe linux-serial"= in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html