Patchwork [U-Boot] sf: spansion: Correct the first byte of idcode1 for S25FL256S part

login
register
mail settings
Submitter 402jagan@gmail.com
Date Sept. 14, 2012, 3:16 p.m.
Message ID <1347635771-1680-2-git-send-email-402jagan@gmail.com>
Download mbox | patch
Permalink /patch/183943/
State Accepted
Delegated to: Mike Frysinger
Headers show

Comments

402jagan@gmail.com - Sept. 14, 2012, 3:16 p.m.
From: Jagannadha Sutradharudu Teki <402jagan@gmail.com>

This patch corrected the first byte of idcode1 for S25FL256S SPI flash.

Signed-off-by: Jagannadha Sutradharudu Teki <402jagan@gmail.com>
---
 drivers/mtd/spi/spansion.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
Michal Simek - Oct. 4, 2012, 5:55 a.m.
Hi Mike,

On 09/14/2012 05:16 PM, 402jagan@gmail.com wrote:
> From: Jagannadha Sutradharudu Teki <402jagan@gmail.com>
>
> This patch corrected the first byte of idcode1 for S25FL256S SPI flash.
>
> Signed-off-by: Jagannadha Sutradharudu Teki <402jagan@gmail.com>
> ---
>   drivers/mtd/spi/spansion.c |    2 +-
>   1 files changed, 1 insertions(+), 1 deletions(-)

Can you please add this patch to your sf branch?

I have checked it at http://www.spansion.com/Support/Datasheets/S25FL128S_256S_00.pdf (page 133)
and 02 is correct for 256Mb.

Thanks,
Michal

Patch

diff --git a/drivers/mtd/spi/spansion.c b/drivers/mtd/spi/spansion.c
index 32b76e0..9288672 100644
--- a/drivers/mtd/spi/spansion.c
+++ b/drivers/mtd/spi/spansion.c
@@ -97,7 +97,7 @@  static const struct spansion_spi_flash_params spansion_spi_flash_table[] = {
 		.name = "S25FL129P_64K",
 	},
 	{
-		.idcode1 = 0x2019,
+		.idcode1 = 0x0219,
 		.idcode2 = 0x4d01,
 		.pages_per_sector = 256,
 		.nr_sectors = 512,