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 09:38:26 +0200 Message-ID: <20110704073826.GC4338@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> 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: <20110704065820.GY6069@pengutronix.de> Sender: linux-serial-owner@vger.kernel.org To: Sascha Hauer Cc: Shawn Guo , patches@linaro.org, devicetree-discuss@lists.ozlabs.org, Jason Liu , Grant Likely , Alan Cox , linux-serial@vger.kernel.org, Jeremy Kerr , linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org 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/serial/fs= l-imx-uart.txt > >=20 > > diff --git a/Documentation/devicetree/bindings/tty/serial/fsl-imx-u= art.txt b/Documentation/devicetree/bindings/tty/serial/fsl-imx-uart.txt > > new file mode 100644 > > index 0000000..a9c0406 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/tty/serial/fsl-imx-uart.txt > > @@ -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_devt= ype[] =3D { > > } > > }; > > =20 > > +static struct of_device_id imx_uart_dt_ids[] =3D { > > + { .compatible =3D "fsl,imx1-uart", .data =3D &imx_uart_devdata[IM= X1_UART], }, > > + { .compatible =3D "fsl,imx21-uart", .data =3D &imx_uart_devdata[I= MX2_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 working port, but it feels stange and if that's the intention the example above needs fixing. 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