[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