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. -- 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
