On 23 October 2014 01:26, Marek Vasut <ma...@denx.de> wrote: > Just move the configuration options scattered all over the driver > to the top of the source file. No functional change. > > Signed-off-by: Marek Vasut <ma...@denx.de> > Cc: Chin Liang See <cl...@altera.com> > Cc: Dinh Nguyen <dingu...@altera.com> > Cc: Albert Aribaud <albert.u.b...@aribaud.net> > Cc: Pavel Machek <pa...@denx.de> > Cc: Jagannadha Sutradharudu Teki <jagannadh.t...@gmail.com> > --- > drivers/spi/altera_spi.c | 16 ++++++++-------- > 1 file changed, 8 insertions(+), 8 deletions(-) > > diff --git a/drivers/spi/altera_spi.c b/drivers/spi/altera_spi.c > index 93365c2..8f31507 100644 > --- a/drivers/spi/altera_spi.c > +++ b/drivers/spi/altera_spi.c > @@ -13,6 +13,14 @@ > #include <malloc.h> > #include <spi.h> > > +#ifndef CONFIG_ALTERA_SPI_IDLE_VAL > +#define CONFIG_ALTERA_SPI_IDLE_VAL 0xff > +#endif > + > +#ifndef CONFIG_SYS_ALTERA_SPI_LIST > +#define CONFIG_SYS_ALTERA_SPI_LIST { CONFIG_SYS_SPI_BASE } > +#endif > + > struct altera_spi_regs { > u32 rxdata; > u32 txdata; > @@ -36,10 +44,6 @@ struct altera_spi_regs { > #define ALTERA_SPI_CONTROL_IE_MSK (1 << 8) > #define ALTERA_SPI_CONTROL_SSO_MSK (1 << 10) > > -#ifndef CONFIG_SYS_ALTERA_SPI_LIST > -#define CONFIG_SYS_ALTERA_SPI_LIST { CONFIG_SYS_SPI_BASE } > -#endif > - > static ulong altera_spi_base_list[] = CONFIG_SYS_ALTERA_SPI_LIST; > > struct altera_spi_slave { > @@ -118,10 +122,6 @@ void spi_release_bus(struct spi_slave *slave) > writel(0, &altspi->regs->slave_sel); > } > > -#ifndef CONFIG_ALTERA_SPI_IDLE_VAL > -# define CONFIG_ALTERA_SPI_IDLE_VAL 0xff > -#endif > - > int spi_xfer(struct spi_slave *slave, unsigned int bitlen, const void *dout, > void *din, unsigned long flags) > { > -- > 2.0.0 >
Series missing's - cover letter and v2 change log. Applied to u-boot-spi/master thanks! -- Jagan. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot