[RFC PATCH 10/10] mx5: Add MX53 EVK board support
Herring Robert-RA7055
ra7055 at freescale.com
Mon Apr 19 16:47:12 EDT 2010
Timur,
> -----Original Message-----
> From: Timur Tabi [mailto:timur.tabi at gmail.com]
> Sent: Monday, April 19, 2010 3:40 PM
> To: Herring Robert-RA7055
> Cc: linux-arm-kernel at lists.infradead.org;
> s.hauer at pengutronix.de; amit.kucheria at canonical.com
> Subject: Re: [RFC PATCH 10/10] mx5: Add MX53 EVK board support
>
> On Fri, Apr 16, 2010 at 2:35 PM, Rob Herring
> <r.herring at freescale.com> wrote:
>
> > +/* Serial ports */
> > +#if defined(CONFIG_SERIAL_IMX) || defined(CONFIG_SERIAL_IMX_MODULE)
> > +static struct imxuart_platform_data uart_pdata = {
> > +// .flags = IMXUART_HAVE_RTSCTS,
>
> You forgot to run checkpatch.pl. C++ comments are not allowed.
Yes I know. I'm only soliciting comments on the way I have structured adding MX53 to MX51 codebase.
>
> > +#else /* !SERIAL_IMX */
> > +static inline void mxc_init_imx_uart(void)
> > +{
> > +}
> > +#endif /* SERIAL_IMX */
>
> How about:
>
> #define mxc_init_imx_uart() do { } while (0)
Just following existing code style in board-mx51_babbage.c.
Rob
More information about the linux-arm-kernel
mailing list