lkml.org 
[lkml]   [2013]   [Jan]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [v3 PATCH 05/12] spi/atmel_spi: update the dt support
Date


> -----Original Message-----
> From: Joe Perches [mailto:joe@perches.com]
> Sent: 2013年1月7日 10:06
> To: Yang, Wenyou
> Cc: linux-arm-kernel@lists.infradead.org; linux-kernel@vger.kernel.org; Ferre,
> Nicolas; plagnioj@jcrosoft.com; richard.genoud@gmail.com; Lin, JM
> Subject: Re: [v3 PATCH 05/12] spi/atmel_spi: update the dt support
>
> On Mon, 2013-01-07 at 09:50 +0800, Wenyou Yang wrote:
> > To meet the different spi IP version of atmel SoC,
> > add more compatible "atmel,at91rm9200-spi", "atmel,at91sam9260-spi"
> > "atmel,at91sam9g45-spi", "atmel,at91sam9x5-spi" with different
> > config and devtype.
>
> trivial comment:
>
> > diff --git a/drivers/spi/spi-atmel.c b/drivers/spi/spi-atmel.c
> []
> > @@ -230,11 +302,15 @@ struct atmel_spi_device {
> > * register, but I haven't checked that it exists on all chips, and
> > * this is cheaper anyway.
> > */
> > -static bool atmel_spi_is_v2(void)
> > +static bool atmel_spi_is_v2(struct atmel_spi *as)
>
> There's a comment that needs updating above this now
>
> > {
> > - return !cpu_is_at91rm9200();
> > + if (as->pdata->version == 2)
> > + return true;
> > + else
> > + return false;
>
> return as->pdata->version == 2;
>
> is rather more readable to me

Thanks a lot for quick feedback.
I will change it next version.
>

\
 
 \ /
  Last update: 2013-01-08 02:41    [W:0.090 / U:0.248 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site