On 30 June 2015 at 00:26, Vikas MANOCHA <vikas.mano...@st.com> wrote: > Hi Jagan, > >> -----Original Message----- >> From: Jagan Teki [mailto:jt...@openedev.com] >> Sent: Monday, June 29, 2015 11:15 AM >> To: Vikas MANOCHA >> Cc: u-boot@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.mano...@st.com> wrote: >> > Thanks Jagan, >> > >> >> -----Original Message----- >> >> From: Jagan Teki [mailto:jt...@openedev.com] >> >> Sent: Sunday, June 28, 2015 3:57 AM >> >> To: Vikas MANOCHA >> >> Cc: u-boot@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.mano...@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 ?
Sure, send them all at once. >> > >> >> >> >> 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 U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot