Hi Mugunthan, On 4 November 2015 at 01:16, Mugunthan V N <mugunthan...@ti.com> wrote: > Prepare driver for DM conversion. > > Signed-off-by: Mugunthan V N <mugunthan...@ti.com> > --- > drivers/spi/ti_qspi.c | 287 > ++++++++++++++++++++++++++++---------------------- > 1 file changed, 161 insertions(+), 126 deletions(-)
This looks OK except for the lack of a change log and one thing below. > > diff --git a/drivers/spi/ti_qspi.c b/drivers/spi/ti_qspi.c > index 44c5762..003df80 100644 > --- a/drivers/spi/ti_qspi.c > +++ b/drivers/spi/ti_qspi.c > @@ -28,6 +28,7 @@ > #define QSPI_3_PIN BIT(18) > #define QSPI_RD_SNGL BIT(16) > #define QSPI_WR_SNGL (2 << 16) > +#define QSPI_RD_DUAL (7 << 16) > #define QSPI_INVAL (4 << 16) > #define QSPI_RD_QUAD (7 << 16) > /* device control */ > @@ -89,46 +90,16 @@ struct ti_qspi_regs { > struct ti_qspi_priv { > struct spi_slave slave; This should not be present when driver model is used. Can you put an #ifdef around it and store anything else you need directly in the structure, rather than using struct spi_slave? > struct ti_qspi_regs *base; > + void *ctrl_mod_mmap; > unsigned int mode; > u32 cmd; > u32 dc; > + u32 num_cs; > + u32 rx_bus_width; > }; > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot