Hi Geert, Thank you for the patch.
On Tuesday 24 December 2013 10:49:33 Geert Uytterhoeven wrote: > Make it more obvious that this value is dummy data. > > Signed-off-by: Geert Uytterhoeven <[email protected]> > --- > drivers/spi/spi-rspi.c | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/drivers/spi/spi-rspi.c b/drivers/spi/spi-rspi.c > index 4e376db1ca4f..51fe3bdacb18 100644 > --- a/drivers/spi/spi-rspi.c > +++ b/drivers/spi/spi-rspi.c > @@ -149,6 +149,8 @@ > #define SPBFCR_TXRST 0x80 /* qspi only */ > #define SPBFCR_RXRST 0x40 /* qspi only */ > > +#define DUMMY_DATA 0x00 > + > struct rspi_data { > void __iomem *addr; > u32 max_speed_hz; > @@ -541,7 +543,7 @@ static int rspi_receive_pio(struct rspi_data *rspi, > struct spi_message *mesg, return -ETIMEDOUT; > } > /* dummy write for generate clock */ > - rspi_write16(rspi, 0x00, RSPI_SPDR); > + rspi_write16(rspi, DUMMY_DATA, RSPI_SPDR); Doesn't the comment make it obvious already ? > if (rspi_wait_for_interrupt(rspi, SPSR_SPRF, SPCR_SPRIE) < 0) { > dev_err(&rspi->master->dev, > @@ -586,7 +588,7 @@ static int qspi_receive_pio(struct rspi_data *rspi, > struct spi_message *mesg, return -ETIMEDOUT; > } > /* dummy write for generate clock */ > - rspi_write8(rspi, 0x00, RSPI_SPDR); > + rspi_write8(rspi, DUMMY_DATA, RSPI_SPDR); > > if (rspi_wait_for_interrupt(rspi, SPSR_SPRF, SPCR_SPRIE) < 0) { > dev_err(&rspi->master->dev, -- Regards, Laurent Pinchart -- To unsubscribe from this list: send the line "unsubscribe linux-spi" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
