On Thu, Dec 19, 2013 at 2:17 PM, Sourav Poddar <sourav.pod...@ti.com> wrote: > On Thursday 19 December 2013 12:49 PM, Jagan Teki wrote: >> >> On Thu, Dec 19, 2013 at 12:37 PM, Sourav Poddar<sourav.pod...@ti.com> >> wrote: >>> >>> On Thursday 19 December 2013 12:24 PM, Jagan Teki wrote: >>>> >>>> On Mon, Nov 25, 2013 at 4:28 PM, Sourav Poddar<sourav.pod...@ti.com> >>>> wrote: >>>>> >>>>> Hi Jagan, >>>>> >>>>> On Thursday 14 November 2013 09:09 PM, Sourav Poddar wrote: >>>>>> >>>>>> The patch series add support for enabling qspi >>>>>> on AM43xx at uboot. >>>>>> >>>>>> Testing done: >>>>>> ------------- >>>>>> Wrote a uImage to the flash, read it back and boot the >>>>>> kernel. >>>>>> >>>>>> Jagannadha Sutradharudu Teki (1): >>>>>> sf: macronix: Add support for MX25L51235F >>>>>> >>>>>> Sourav Poddar (4): >>>>>> am43xx: add qspi support >>>>>> am437x_epos_evm: add SPL API, QSPI, and serial flash support >>>>>> qspi/spi: Add AM43xx specifics changes >>>>>> am43xx: add delay before xfer >>>>>> >>>>>> arch/arm/cpu/armv7/am33xx/clock_am43xx.c | 1 + >>>>>> arch/arm/include/asm/arch-am33xx/cpu.h | 4 +++- >>>>>> arch/arm/include/asm/arch-am33xx/omap.h | 1 + >>>>>> board/ti/am43xx/mux.c | 11 +++++++++++ >>>>>> drivers/mtd/spi/sf_probe.c | 2 ++ >>>>>> drivers/spi/ti_qspi.c | 29 >>>>>> ++++++++++++++++++++++++++--- >>>>>> include/configs/am43xx_evm.h | 20 >>>>>> ++++++++++++++++++++ >>>>>> 7 files changed, 64 insertions(+), 4 deletions(-) >>>>>> >>>>> If these patches seems fine to you, can these be picked? >>>> >>>> Please send the next version patches - fix the comments as we discussed >>>> so >>>> far. >>>> >>> I will send the next version. >>> But, what about the delay patch, I tried other stuffs but could not see >>> any >>> thing >>> that helps. >>> So, I will send the delay patch also as of now, to make it work out of >>> mainline. >> >> I am some how OK with that as that change is in ur driver.. >> >> Just try this change >> /* FIXME: comments here... */ >> +#ifdef CONFIG_AM43XX >> + udelay(100); >> +#endif >> >> And also write the bug log on commit message(means with out delay) >> >> Try to ass the testing logs on doc/ - for each time you tested a new >> flash or new driver. >> > Jagan, > > Just realised that this patches depends on AM43x base patches which are in > ti-uboot mainline tree. So, what should I do in that case. ?
Send all of them together..! -- Thanks, Jagan. -------- Jagannadha Sutradharudu Teki, E: jagannadh.t...@gmail.com, P: +91-9676773388 Engineer - System Software Hacker U-boot - SPI Custodian and Zynq APSOC Ln: http://www.linkedin.com/in/jaganteki _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot