[U-Boot] [v2 0/6] stv0991: enable cadence qspi controller & spi flash

[Vikas MANOCHA]

Hi Jagan,

> -----Original Message-----
> From: Jagan Teki [mailto:jteki at openedev.com]
> Sent: Monday, June 29, 2015 11:15 AM
> To: Vikas MANOCHA
> Cc: u-boot at lists.denx.de
> Subject: Re: [U-Boot] [v2 0/6] stv0991: enable cadence qspi controller & spi
> flash
> 
> On 29 June 2015 at 23:14, Vikas MANOCHA <vikas.manocha at st.com> wrote:
> > Thanks Jagan,
> >
> >> -----Original Message-----
> >> From: Jagan Teki [mailto:jteki at openedev.com]
> >> Sent: Sunday, June 28, 2015 3:57 AM
> >> To: Vikas MANOCHA
> >> Cc: u-boot at lists.denx.de
> >> Subject: Re: [U-Boot] [v2 0/6] stv0991: enable cadence qspi
> >> controller & spi flash
> >>
> >> On 25 June 2015 at 04:36, Vikas Manocha <vikas.manocha at st.com>
> wrote:
> >> > This patchset enables cadence qspi controller for stv0991 soc,
> >> > saves environment in spi flash & moves CONFIG_OF in the defconfig
> file.
> >> >
> >> > In fact it clubs two patchsets submitted earlier:
> >> > - [PATCH 0/3] stv0991: enable cadence qspi controller & spi flash
> >> > - [PATCH 0/3] stv0991: spi env & configs related board changes
> >>
> >> I'm OK, with this series and commented about v2, 6/6
> >>
> >> Please rebase all these 6 patches
> >
> > Ok, I will rebase these 6 on u-boot-spi/next-dev, add doc as per your v2,
> 6/6 comment & send the v3.
> >
> >> + 3 patches on to u-boot-spi/next-dev [1]
> >> [PATCH v2 0/3] spi: cadence_qspi: sram depth from DT & fix for FIFO
> >
> > I checked these 3 patches & can be applied to spi-next-dev without any
> modification.
> 
> I wasn't able to apply these 3 at-lest - please rebase all 9 against this branch
> and send.

You are right, I will rebase these 3 also & send the v3 for them.
Also about your comment on stv0991 patchset v2, 6/6 of adding binding doc : It is more relevant for the cadence_qspi.
Do you mind if I add this binding document as 4th patch in this patchset ?

Rgds,
Vikas

> 
> >
> >>
> >> So send all 9 patches with fix on v2, 6/6 at once.
> >>
> >> [1] http://git.denx.de/?p=u-boot/u-boot-
> >> spi.git;a=shortlog;h=refs/heads/next-dev
> >>
> >> >
> >> > Changed in v2:
> >> > - remove checkpatch.pl error from patch "stv0991: configure clock..."
> >> > - clubbed two patchsets mentioned above.
> >> >
> >> > Vikas Manocha (6):
> >> >   stv0991: enable saving enrironment in spi flash
> >> >   stv0991: move OF_CONTROL config to defconfig
> >> >   stv0991: remove define CONFIG_OF_SEPARATE from board file
> >> >   stv0991: configure clock & pad muxing for qspi
> >> >   stv0991: enable cadence qspi controller & spi flash
> >> >   stv0991: configure device tree for cadence qspi & flash
> >> >
> >> >  arch/arm/cpu/armv7/stv0991/clock.c                 |    4 ++-
> >> >  arch/arm/cpu/armv7/stv0991/pinmux.c                |    5 +++
> >> >  arch/arm/dts/stv0991.dts                           |   34 ++++++++++++++++++++
> >> >  arch/arm/include/asm/arch-stv0991/stv0991_cgu.h    |   15 +++++++++
> >> >  arch/arm/include/asm/arch-stv0991/stv0991_creg.h   |    9 ++++++
> >> >  arch/arm/include/asm/arch-stv0991/stv0991_periph.h |    2 ++
> >> >  board/st/stv0991/stv0991.c                         |    8 +++++
> >> >  configs/stv0991_defconfig                          |    1 +
> >> >  include/configs/stv0991.h                          |   25 +++++++++++---
> >> >  9 files changed, 98 insertions(+), 5 deletions(-)
> 
> thanks!
> --
> Jagan | openedev.