Thanks for the patch. On 15 July 2014 06:26, Simon Glass <s...@chromium.org> wrote: > Some of the #defines in spi.h are not bracketed. To avoid future mistakes > add brackets. Also add an explanatory comment for SPI_CONN_DUAL_... > > Signed-off-by: Simon Glass <s...@chromium.org> > --- > > include/spi.h | 24 ++++++++++++------------ > 1 file changed, 12 insertions(+), 12 deletions(-) > > diff --git a/include/spi.h b/include/spi.h > index ffd6647..b673be2 100644 > --- a/include/spi.h > +++ b/include/spi.h > @@ -30,24 +30,24 @@ > #define SPI_XFER_MMAP 0x08 /* Memory Mapped start */ > #define SPI_XFER_MMAP_END 0x10 /* Memory Mapped End */ > #define SPI_XFER_ONCE (SPI_XFER_BEGIN | SPI_XFER_END) > -#define SPI_XFER_U_PAGE (1 << 5) > +#define SPI_XFER_U_PAGE (1 << 5) > > /* SPI TX operation modes */ > -#define SPI_OPM_TX_QPP 1 << 0 > +#define SPI_OPM_TX_QPP (1 << 0) > > /* SPI RX operation modes */ > -#define SPI_OPM_RX_AS 1 << 0 > -#define SPI_OPM_RX_DOUT 1 << 1 > -#define SPI_OPM_RX_DIO 1 << 2 > -#define SPI_OPM_RX_QOF 1 << 3 > -#define SPI_OPM_RX_QIOF 1 << 4 > -#define SPI_OPM_RX_EXTN SPI_OPM_RX_AS | SPI_OPM_RX_DOUT | \ > +#define SPI_OPM_RX_AS (1 << 0) > +#define SPI_OPM_RX_DOUT (1 << 1) > +#define SPI_OPM_RX_DIO (1 << 2) > +#define SPI_OPM_RX_QOF (1 << 3) > +#define SPI_OPM_RX_QIOF (1 << 4) > +#define SPI_OPM_RX_EXTN (SPI_OPM_RX_AS | SPI_OPM_RX_DOUT | \ > SPI_OPM_RX_DIO | SPI_OPM_RX_QOF | \ > - SPI_OPM_RX_QIOF > + SPI_OPM_RX_QIOF) > > -/* SPI bus connection options */ > -#define SPI_CONN_DUAL_SHARED 1 << 0 > -#define SPI_CONN_DUAL_SEPARATED 1 << 1 > +/* SPI bus connection options - see enum spi_dual_flash */ > +#define SPI_CONN_DUAL_SHARED (1 << 0) > +#define SPI_CONN_DUAL_SEPARATED (1 << 1) > > /* Header byte that marks the start of the message */ > #define SPI_PREAMBLE_END_BYTE 0xec > -- > 2.0.0.526.g5318336 >
Reviewed-by: Jagannadha Sutradharudu Teki <jaga...@xilinx.com> thanks! -- Jagan. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
