On Thu, May 30, 2013 at 7:19 PM, Jagannadha Sutradharudu Teki <jagannadha.sutradharudu-t...@xilinx.com> wrote: > This commit is based on the patch from Xie Xiaobo <x....@freescale.com> > with commit head title as "sf: spansion: Add support for S25FL128S". > pulled the same code changes into current u-boot tree with little update > on the name field. > > http://patchwork.ozlabs.org/patch/218145/ > > SPANSION recommend S25FL128S supersedes S25FL129P, and the two flash > memory have the same device ID and Memory architecture. So they can > use the same config parameters. > > Signed-off-by: Xie Xiaobo <x....@freescale.com> > Signed-off-by: Jagannadha Sutradharudu Teki <jaga...@xilinx.com> > --- > Changes for v2: > - none > > drivers/mtd/spi/spansion.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/mtd/spi/spansion.c b/drivers/mtd/spi/spansion.c > index 2218e2f..dad4fbb 100644 > --- a/drivers/mtd/spi/spansion.c > +++ b/drivers/mtd/spi/spansion.c > @@ -101,7 +101,7 @@ static const struct spansion_spi_flash_params > spansion_spi_flash_table[] = { > .idcode2 = 0x4d01, > .pages_per_sector = 256, > .nr_sectors = 256, > - .name = "S25FL129P_64K", > + .name = "S25FL129P_64K/S25FL128S_64K",
Any comments on this notation, as the ID's are same for these parts upto 5bytes. if you need to differ then we should read the 6th byte. If i.e the case increase the byte count on probe function for this, due to extra over head only for this, i just noted like this. does it make sense? Request comments. -- Thanks, Jagan. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot