[U-Boot] [PATCH] sf: winbond: Add support for W25Q64DW

[U-Boot] [PATCH] sf: winbond: Add support for W25Q64DW

[402jagan at gmail.com]

From: Jagannadha Sutradharudu Teki <402jagan@gmail.com>

This patch adds support for Winbond W25Q64DW SPI flash.

Signed-off-by: Jagannadha Sutradharudu Teki <402jagan@gmail.com>
---
 drivers/mtd/spi/winbond.c |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/drivers/mtd/spi/winbond.c b/drivers/mtd/spi/winbond.c
index f6aab3d..4418302 100644
--- a/drivers/mtd/spi/winbond.c
+++ b/drivers/mtd/spi/winbond.c
@@ -67,6 +67,11 @@ static const struct winbond_spi_flash_params winbond_spi_flash_table[] = {
 		.nr_blocks		= 128,
 		.name			= "W25Q80",
 	},
+	{
+		.id			= 0x6017,
+		.nr_blocks		= 128,
+		.name			= "W25Q64DW",
+	},
 };
 
 struct spi_flash *spi_flash_probe_winbond(struct spi_slave *spi, u8 *idcode)
-- 
1.7.0.4

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

[402jagan at gmail.com]

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(-)

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,
-- 
1.7.0.4

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

[Michal Simek]

Hi Mike,

On 09/14/2012 05:16 PM, 402jagan at 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

-- 
Michal Simek, Ing. (M.Eng)
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel 2.6 Microblaze Linux - http://www.monstr.eu/fdt/
Microblaze U-BOOT custodian

[U-Boot] [PATCH] sf: winbond: Add support for W25Q64DW

[Michal Simek]

Hi Mike,

On 09/03/2012 05:17 PM, 402jagan at gmail.com wrote:
> From: Jagannadha Sutradharudu Teki <402jagan@gmail.com>
>
> This patch adds support for Winbond W25Q64DW SPI flash.
>
> Signed-off-by: Jagannadha Sutradharudu Teki <402jagan@gmail.com>
> ---
>   drivers/mtd/spi/winbond.c |    5 +++++
>   1 files changed, 5 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/mtd/spi/winbond.c b/drivers/mtd/spi/winbond.c
> index f6aab3d..4418302 100644
> --- a/drivers/mtd/spi/winbond.c
> +++ b/drivers/mtd/spi/winbond.c
> @@ -67,6 +67,11 @@ static const struct winbond_spi_flash_params winbond_spi_flash_table[] = {
>   		.nr_blocks		= 128,
>   		.name			= "W25Q80",
>   	},
> +	{
> +		.id			= 0x6017,
> +		.nr_blocks		= 128,
> +		.name			= "W25Q64DW",
> +	},
>   };
>
>   struct spi_flash *spi_flash_probe_winbond(struct spi_slave *spi, u8 *idcode)
>

http://www.winbond.com.tw/NR/rdonlyres/9029DFAD-5FDD-4C84-851A-7326DF7389AB/0/W25Q64DW.pdf
(page 18)

Can you please add this patch to your sf branch?

Thanks,
Michal

-- 
Michal Simek, Ing. (M.Eng)
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel 2.6 Microblaze Linux - http://www.monstr.eu/fdt/
Microblaze U-BOOT custodian

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

[Michal Simek]

From: Jagannadha Sutradharudu Teki <jaganna@xilinx.com>

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

Signed-off-by: Jagannadha Sutradharudu Teki <jaganna@xilinx.com>
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---
 drivers/mtd/spi/spansion.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

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,
-- 
1.7.0.4

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

[Marek Vasut]

Dear Michal Simek,

was this ever applied?

> From: Jagannadha Sutradharudu Teki <jaganna@xilinx.com>
> 
> This patch corrected the first byte of idcode1 for S25FL256S SPI flash.
> 
> Signed-off-by: Jagannadha Sutradharudu Teki <jaganna@xilinx.com>
> Signed-off-by: Michal Simek <michal.simek@xilinx.com>
> ---
>  drivers/mtd/spi/spansion.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> 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,

Best regards,
Marek Vasut

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

[Michal Simek]

Hi,

2013/2/4 Marek Vasut <marex@denx.de>:
> Dear Michal Simek,
>
> was this ever applied?

I have sent pull request to Tom and Mike. All patches around cf are here.

http://git.denx.de/?p=u-boot/u-boot-microblaze.git;a=shortlog;h=refs/heads/mainline/sf

Interesting that Tom applied this patch and don't the rest
"sf: stmicro: add support for N25Q064"

Thanks,
Michal


-- 
Michal Simek, Ing. (M.Eng)
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform

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

[Tom Rini]

On Mon, Feb 04, 2013 at 07:11:38PM +0100, Michal Simek wrote:
> Hi,
> 
> 2013/2/4 Marek Vasut <marex@denx.de>:
> > Dear Michal Simek,
> >
> > was this ever applied?
> 
> I have sent pull request to Tom and Mike. All patches around cf are here.
> 
> http://git.denx.de/?p=u-boot/u-boot-microblaze.git;a=shortlog;h=refs/heads/mainline/sf
> 
> Interesting that Tom applied this patch and don't the rest
> "sf: stmicro: add support for N25Q064"

I had grabbed a few things that stood out to me Friday and started a
build.  Took a bit to sort out things and in the interim you made your
pull request.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20130204/5ac30fe8/attachment.pgp>