[U-Boot] [PATCH v2 2/8] sf: spansion: Add support for S25FL128S_64K

[Jagan Teki]

On Thu, May 30, 2013 at 7:19 PM, Jagannadha Sutradharudu Teki
<jagannadha.sutradharudu-teki at xilinx.com> wrote:
> This commit is based on the patch from Xie Xiaobo <X.Xie at 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.Xie at freescale.com>
> Signed-off-by: Jagannadha Sutradharudu Teki <jaganna at 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.