Hi Jagan, > -----Original Message----- > From: Jagan Teki [mailto:[email protected]] > Sent: Monday, June 29, 2015 11:15 AM > To: Vikas MANOCHA > Cc: [email protected] > Subject: Re: [U-Boot] [v2 0/6] stv0991: enable cadence qspi controller & spi > flash > > On 29 June 2015 at 23:14, Vikas MANOCHA <[email protected]> wrote: > > Thanks Jagan, > > > >> -----Original Message----- > >> From: Jagan Teki [mailto:[email protected]] > >> Sent: Sunday, June 28, 2015 3:57 AM > >> To: Vikas MANOCHA > >> Cc: [email protected] > >> Subject: Re: [U-Boot] [v2 0/6] stv0991: enable cadence qspi > >> controller & spi flash > >> > >> On 25 June 2015 at 04:36, Vikas Manocha <[email protected]> > 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. _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
