On Mon, Oct 31, 2016 at 3:02 PM, Vignesh R <vigne...@ti.com> wrote: > > > On Monday 31 October 2016 02:50 PM, Jagan Teki wrote: >> On Mon, Oct 31, 2016 at 2:24 PM, Jagan Teki <ja...@openedev.com> wrote: >>> On Mon, Oct 31, 2016 at 8:48 AM, Vignesh R <vigne...@ti.com> wrote: >>>> >>>> >>>> On Sunday 30 October 2016 11:16 PM, Jagan Teki wrote: >>>> [...] >>>>> + {"S25FL128P_64K", INFO(0x012018, 0x0301, 64 * 1024, 256, >>>>> RD_FULL | WR_QPP) }, >>>>> + {"S25FL032P", INFO(0x010215, 0x4d00, 64 * 1024, 64, >>>>> RD_FULL | WR_QPP) }, >>>>> + {"S25FL064P", INFO(0x010216, 0x4d00, 64 * 1024, 128, >>>>> RD_FULL | WR_QPP) }, >>>>> + {"S25FL128S_256K", INFO(0x012018, 0x4d00, 256 * 1024, 64, >>>>> RD_FULL | WR_QPP) }, >>>>> + {"S25FL128S_64K", INFO(0x012018, 0x4d01, 64 * 1024, 256, >>>>> RD_FULL | WR_QPP) }, >>>>> + {"S25FL256S_256K", INFO(0x010219, 0x4d00, 256 * 1024, 128, >>>>> RD_FULL | WR_QPP) }, >>>>> + {"S25FL256S_64K", INFO(0x010219, 0x4d01, 64 * 1024, 512, >>>>> RD_FULL | WR_QPP) }, >>>>> + {"S25FS512S", INFO(0x010220, 0x4D00, 128 * 1024, 512, >>>>> RD_FULL | WR_QPP) }, >>>>> + {"S25FL512S_256K", INFO(0x010220, 0x4d00, 256 * 1024, 256, >>>>> RD_FULL | WR_QPP) }, >>>> >>>> Could you change above S25FS512S and S25FL512S_256K to use INFO6 macro? >>>> Right now, S25FL512S_256K gets detected as S25FS512S and sf update fails >>>> due as erasesizes are different. >>> >>> I think this seems to be an existing issue, even before these changes. >>> after this [1] I will prepare a patch to use INFO6 for S25FS512S. >>> >>> [1] >>> http://git.denx.de/?p=u-boot-spi.git;a=commitdiff;h=13c17cad45ad9040fd0f6ebadf76554eb6353381 >> >> Try u-boot-spi/next? >> > > Yes, that works. sf probe correctly detects S25FL512S_256K. Thanks!
Now it's s25fl512s_256k changed to lower-case hope you detect the same? thanks! -- Jagan Teki Free Software Engineer | www.openedev.com U-Boot, Linux | Upstream Maintainer Hyderabad, India. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot