lkml.org 
[lkml]   [2014]   [Oct]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH 1/2 v2] SPI: spi-pxa2xx: Add helpers for regiseters' accessing
Date

> I'm okay with the current version, though I have few minor comments below.
>
> > Introduce helper functions to access the 'SSCR0' and 'SSCR1'.
> >
>
> Like you said in the summary there are many accessors to many registers, not
> only cr1/cr0. Perhaps, you may extend your commit message.
>
OK.

> In any case
> Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
>
OK.
> > +
> > /*
> > * Read and write LPSS SSP private registers. Caller must first check that
> > * is_lpss_ssp() returns true before these can be called.
> > @@ -234,7 +301,7 @@ static int null_writer(struct driver_data *drv_data)
> > void __iomem *reg = drv_data->ioaddr;
> > u8 n_bytes = drv_data->n_bytes;
> >
> > - if (((read_SSSR(reg) & SSSR_TFL_MASK) == SSSR_TFL_MASK)
> > + if (pxa2xx_spi_txfifo_full(drv_data)
> > || (drv_data->tx == drv_data->tx_end))
>
> Just wondering if those two could fit one line.
>
No, if make the two in one line, it is 84 characters.
\
 
 \ /
  Last update: 2014-10-08 03:22    [W:0.063 / U:0.524 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site