lkml.org 
[lkml]   [2014]   [Oct]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH 2/2 v2] SPI: spi-pxa2xx: SPI support for Intel Quark X1000
Date
> > The SPI memory mapped I/O registers supported by Quark are different
> > from the current implementation, and Quark only supports the registers
> > of 'SSCR0', 'SSCR1', 'SSSR', 'SSDR', and 'DDS_RATE'. This patch is to
> > enable the SPI for Intel Quark X1000.
> >
> > This piece of work is derived from Dan O'Donovan's initial work for
> > Intel Quark
> > X1000 SPI enabling.
>
> Minor comments are below.
>
> After addressing them
> Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
>
OK.
> > + case QUARK_X1000_SSP:
> > + return RX_THRESH_QUARK_X1000_DFLT;
> > default:
> > return RX_THRESH_DFLT;
> > }
> > +
>
> Redundant empty line.
>
OK. I will remove it.

> > static void pxa2xx_spi_clear_rx_thre(const struct driver_data *drv_data,
> > - u32 *sccr1_reg)
> > + u32 *sccr1_reg)
>
> Unnecessary change.
>
Improve it in the first patch.
> > static void pxa2xx_spi_set_rx_thre(const struct driver_data *drv_data,
> > - u32 *sccr1_reg, u32 threshold)
> > + u32 *sccr1_reg, u32 threshold)
>
> Ditto.
>
> Or you may do that in the first patch.
>
Improve it in the first patch.
> >
> > pxa2xx_spi_set_rx_thre(drv_data, &sccr1_reg,
> > - rx_thre);
> > + rx_thre);
>
> Ditto.
>
Improve it in the first patch.
> > + if (is_quark_x1000_ssp(drv_data) &&
> > + (read_DDS_RATE(reg) != chip->dds_rate))
>
> Could it be one line?
>
No, it is beyond 80 characters if it is in one line.
\
 
 \ /
  Last update: 2014-10-08 04:01    [W:0.095 / U:0.764 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site