* [U-Boot] [PATCH v2 1/8] sf: spansion: Update the name for S25FL256S flash
[not found] <1369921773-8871-1-git-send-email-jaganna@xilinx.com>
@ 2013-05-30 13:49 ` Jagannadha Sutradharudu Teki
2013-06-13 18:54 ` Jagan Teki
2013-05-30 13:49 ` [U-Boot] [PATCH v2 2/8] sf: spansion: Add support for S25FL128S_64K Jagannadha Sutradharudu Teki
` (6 subsequent siblings)
7 siblings, 1 reply; 21+ messages in thread
From: Jagannadha Sutradharudu Teki @ 2013-05-30 13:49 UTC (permalink / raw)
To: u-boot
As the per the ID tabl the flash is under Uniform 64-kB sector
architecture, hence updated with proper name.
Signed-off-by: Jagannadha Sutradharudu Teki <jaganna@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 307b852..2218e2f 100644
--- a/drivers/mtd/spi/spansion.c
+++ b/drivers/mtd/spi/spansion.c
@@ -108,7 +108,7 @@ static const struct spansion_spi_flash_params spansion_spi_flash_table[] = {
.idcode2 = 0x4d01,
.pages_per_sector = 256,
.nr_sectors = 512,
- .name = "S25FL256S",
+ .name = "S25FL256S_64K",
},
};
--
1.8.3
^ permalink raw reply related [flat|nested] 21+ messages in thread
* [U-Boot] [PATCH v2 1/8] sf: spansion: Update the name for S25FL256S flash
2013-05-30 13:49 ` [U-Boot] [PATCH v2 1/8] sf: spansion: Update the name for S25FL256S flash Jagannadha Sutradharudu Teki
@ 2013-06-13 18:54 ` Jagan Teki
0 siblings, 0 replies; 21+ messages in thread
From: Jagan Teki @ 2013-06-13 18:54 UTC (permalink / raw)
To: u-boot
On 30-05-2013 19:19, Jagannadha Sutradharudu Teki wrote:
> As the per the ID tabl the flash is under Uniform 64-kB sector
> architecture, hence updated with proper name.
>
> Signed-off-by: Jagannadha Sutradharudu Teki <jaganna@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 307b852..2218e2f 100644
> --- a/drivers/mtd/spi/spansion.c
> +++ b/drivers/mtd/spi/spansion.c
> @@ -108,7 +108,7 @@ static const struct spansion_spi_flash_params spansion_spi_flash_table[] = {
> .idcode2 = 0x4d01,
> .pages_per_sector = 256,
> .nr_sectors = 512,
> - .name = "S25FL256S",
> + .name = "S25FL256S_64K",
> },
> };
>
>
Applied to u-boot-spi/master
--
Thanks,
Jagan.
^ permalink raw reply [flat|nested] 21+ messages in thread
* [U-Boot] [PATCH v2 2/8] sf: spansion: Add support for S25FL128S_64K
[not found] <1369921773-8871-1-git-send-email-jaganna@xilinx.com>
2013-05-30 13:49 ` [U-Boot] [PATCH v2 1/8] sf: spansion: Update the name for S25FL256S flash Jagannadha Sutradharudu Teki
@ 2013-05-30 13:49 ` Jagannadha Sutradharudu Teki
2013-06-02 19:18 ` Jagan Teki
2013-05-30 13:49 ` [U-Boot] [PATCH v2 3/8] sf: winbond: Add support for W25Q80BW Jagannadha Sutradharudu Teki
` (5 subsequent siblings)
7 siblings, 1 reply; 21+ messages in thread
From: Jagannadha Sutradharudu Teki @ 2013-05-30 13:49 UTC (permalink / raw)
To: u-boot
This commit is based on the patch from Xie Xiaobo <X.Xie@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@freescale.com>
Signed-off-by: Jagannadha Sutradharudu Teki <jaganna@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",
},
{
.idcode1 = 0x0219,
--
1.8.3
^ permalink raw reply related [flat|nested] 21+ messages in thread
* [U-Boot] [PATCH v2 2/8] sf: spansion: Add support for S25FL128S_64K
2013-05-30 13:49 ` [U-Boot] [PATCH v2 2/8] sf: spansion: Add support for S25FL128S_64K Jagannadha Sutradharudu Teki
@ 2013-06-02 19:18 ` Jagan Teki
0 siblings, 0 replies; 21+ messages in thread
From: Jagan Teki @ 2013-06-02 19:18 UTC (permalink / raw)
To: u-boot
On Thu, May 30, 2013 at 7:19 PM, Jagannadha Sutradharudu Teki
<jagannadha.sutradharudu-teki@xilinx.com> wrote:
> This commit is based on the patch from Xie Xiaobo <X.Xie@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@freescale.com>
> Signed-off-by: Jagannadha Sutradharudu Teki <jaganna@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.
^ permalink raw reply [flat|nested] 21+ messages in thread
* [U-Boot] [PATCH v2 3/8] sf: winbond: Add support for W25Q80BW
[not found] <1369921773-8871-1-git-send-email-jaganna@xilinx.com>
2013-05-30 13:49 ` [U-Boot] [PATCH v2 1/8] sf: spansion: Update the name for S25FL256S flash Jagannadha Sutradharudu Teki
2013-05-30 13:49 ` [U-Boot] [PATCH v2 2/8] sf: spansion: Add support for S25FL128S_64K Jagannadha Sutradharudu Teki
@ 2013-05-30 13:49 ` Jagannadha Sutradharudu Teki
2013-06-13 18:55 ` Jagan Teki
2013-05-30 13:49 ` [U-Boot] [PATCH v2 4/8] sf: winbond: Correct the nr_blocks used for W25Q32DW Jagannadha Sutradharudu Teki
` (4 subsequent siblings)
7 siblings, 1 reply; 21+ messages in thread
From: Jagannadha Sutradharudu Teki @ 2013-05-30 13:49 UTC (permalink / raw)
To: u-boot
Add support for Winbond W25Q80BW SPI flash.
This patch corrected the flash name, nr_blocks and
also commit message header from below patch.
"sf: winbond: add W25Q32"
(sha1: c969abc47033d6f810d3c9dbdb994ea9d691d038)
Signed-off-by: Jagannadha Sutradharudu Teki <jaganna@xilinx.com>
---
Changes for v2:
- none
drivers/mtd/spi/winbond.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/mtd/spi/winbond.c b/drivers/mtd/spi/winbond.c
index cb57ce0..a7cd51f 100644
--- a/drivers/mtd/spi/winbond.c
+++ b/drivers/mtd/spi/winbond.c
@@ -84,8 +84,8 @@ static const struct winbond_spi_flash_params winbond_spi_flash_table[] = {
},
{
.id = 0x5014,
- .nr_blocks = 128,
- .name = "W25Q80",
+ .nr_blocks = 16,
+ .name = "W25Q80BW",
},
{
.id = 0x6016,
--
1.8.3
^ permalink raw reply related [flat|nested] 21+ messages in thread
* [U-Boot] [PATCH v2 3/8] sf: winbond: Add support for W25Q80BW
2013-05-30 13:49 ` [U-Boot] [PATCH v2 3/8] sf: winbond: Add support for W25Q80BW Jagannadha Sutradharudu Teki
@ 2013-06-13 18:55 ` Jagan Teki
0 siblings, 0 replies; 21+ messages in thread
From: Jagan Teki @ 2013-06-13 18:55 UTC (permalink / raw)
To: u-boot
On 30-05-2013 19:19, Jagannadha Sutradharudu Teki wrote:
> Add support for Winbond W25Q80BW SPI flash.
>
> This patch corrected the flash name, nr_blocks and
> also commit message header from below patch.
> "sf: winbond: add W25Q32"
> (sha1: c969abc47033d6f810d3c9dbdb994ea9d691d038)
>
> Signed-off-by: Jagannadha Sutradharudu Teki <jaganna@xilinx.com>
> ---
> Changes for v2:
> - none
>
> drivers/mtd/spi/winbond.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/mtd/spi/winbond.c b/drivers/mtd/spi/winbond.c
> index cb57ce0..a7cd51f 100644
> --- a/drivers/mtd/spi/winbond.c
> +++ b/drivers/mtd/spi/winbond.c
> @@ -84,8 +84,8 @@ static const struct winbond_spi_flash_params winbond_spi_flash_table[] = {
> },
> {
> .id = 0x5014,
> - .nr_blocks = 128,
> - .name = "W25Q80",
> + .nr_blocks = 16,
> + .name = "W25Q80BW",
> },
> {
> .id = 0x6016,
>
Applied to u-boot-spi/master
--
Thanks,
Jagan.
^ permalink raw reply [flat|nested] 21+ messages in thread
* [U-Boot] [PATCH v2 4/8] sf: winbond: Correct the nr_blocks used for W25Q32DW
[not found] <1369921773-8871-1-git-send-email-jaganna@xilinx.com>
` (2 preceding siblings ...)
2013-05-30 13:49 ` [U-Boot] [PATCH v2 3/8] sf: winbond: Add support for W25Q80BW Jagannadha Sutradharudu Teki
@ 2013-05-30 13:49 ` Jagannadha Sutradharudu Teki
2013-06-13 18:56 ` Jagan Teki
2013-05-30 13:49 ` [U-Boot] [PATCH v2 5/8] sf: winbond: Update the names for W25Q 0x40XX ID's flash parts Jagannadha Sutradharudu Teki
` (3 subsequent siblings)
7 siblings, 1 reply; 21+ messages in thread
From: Jagannadha Sutradharudu Teki @ 2013-05-30 13:49 UTC (permalink / raw)
To: u-boot
This patch corrected the nr_blocks used for W25Q32DW SPI flash.
nr_blcoks are incorrectly assigned on below patch
"sf: winbond: add W25Q32DW"
(sha1: 772ba15474f73adc942e817cc072b6e9750836cc)
Signed-off-by: Jagannadha Sutradharudu Teki <jaganna@xilinx.com>
---
Changes for v2:
- none
drivers/mtd/spi/winbond.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mtd/spi/winbond.c b/drivers/mtd/spi/winbond.c
index a7cd51f..8457808 100644
--- a/drivers/mtd/spi/winbond.c
+++ b/drivers/mtd/spi/winbond.c
@@ -89,7 +89,7 @@ static const struct winbond_spi_flash_params winbond_spi_flash_table[] = {
},
{
.id = 0x6016,
- .nr_blocks = 512,
+ .nr_blocks = 64,
.name = "W25Q32DW",
},
{
--
1.8.3
^ permalink raw reply related [flat|nested] 21+ messages in thread
* [U-Boot] [PATCH v2 4/8] sf: winbond: Correct the nr_blocks used for W25Q32DW
2013-05-30 13:49 ` [U-Boot] [PATCH v2 4/8] sf: winbond: Correct the nr_blocks used for W25Q32DW Jagannadha Sutradharudu Teki
@ 2013-06-13 18:56 ` Jagan Teki
0 siblings, 0 replies; 21+ messages in thread
From: Jagan Teki @ 2013-06-13 18:56 UTC (permalink / raw)
To: u-boot
On 30-05-2013 19:19, Jagannadha Sutradharudu Teki wrote:
> This patch corrected the nr_blocks used for W25Q32DW SPI flash.
>
> nr_blcoks are incorrectly assigned on below patch
> "sf: winbond: add W25Q32DW"
> (sha1: 772ba15474f73adc942e817cc072b6e9750836cc)
>
> Signed-off-by: Jagannadha Sutradharudu Teki <jaganna@xilinx.com>
> ---
> Changes for v2:
> - none
>
> drivers/mtd/spi/winbond.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/mtd/spi/winbond.c b/drivers/mtd/spi/winbond.c
> index a7cd51f..8457808 100644
> --- a/drivers/mtd/spi/winbond.c
> +++ b/drivers/mtd/spi/winbond.c
> @@ -89,7 +89,7 @@ static const struct winbond_spi_flash_params winbond_spi_flash_table[] = {
> },
> {
> .id = 0x6016,
> - .nr_blocks = 512,
> + .nr_blocks = 64,
> .name = "W25Q32DW",
> },
> {
>
Applied to u-boot-spi/master
--
Thanks,
Jagan.
^ permalink raw reply [flat|nested] 21+ messages in thread
* [U-Boot] [PATCH v2 5/8] sf: winbond: Update the names for W25Q 0x40XX ID's flash parts
[not found] <1369921773-8871-1-git-send-email-jaganna@xilinx.com>
` (3 preceding siblings ...)
2013-05-30 13:49 ` [U-Boot] [PATCH v2 4/8] sf: winbond: Correct the nr_blocks used for W25Q32DW Jagannadha Sutradharudu Teki
@ 2013-05-30 13:49 ` Jagannadha Sutradharudu Teki
2013-06-02 19:23 ` Jagan Teki
2013-05-30 13:49 ` [U-Boot] [PATCH v2 6/8] sf: winbond: Add support for W25Q128FW Jagannadha Sutradharudu Teki
` (2 subsequent siblings)
7 siblings, 1 reply; 21+ messages in thread
From: Jagannadha Sutradharudu Teki @ 2013-05-30 13:49 UTC (permalink / raw)
To: u-boot
Use the exact names for W25Q 0x40XX ID's flash parts, as the same
sizes of flashes comes with different ID's. so-that the distinguishes
becomes easy with this change.
Signed-off-by: Jagannadha Sutradharudu Teki <jaganna@xilinx.com>
---
Changes for v2:
- none
drivers/mtd/spi/winbond.c | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/mtd/spi/winbond.c b/drivers/mtd/spi/winbond.c
index 8457808..3c0ca88 100644
--- a/drivers/mtd/spi/winbond.c
+++ b/drivers/mtd/spi/winbond.c
@@ -55,27 +55,27 @@ static const struct winbond_spi_flash_params winbond_spi_flash_table[] = {
{
.id = 0x4014,
.nr_blocks = 16,
- .name = "W25Q80BL",
+ .name = "W25Q80BL/W25Q80BV",
},
{
.id = 0x4015,
.nr_blocks = 32,
- .name = "W25Q16",
+ .name = "W25Q16CL/W25Q16DV",
},
{
.id = 0x4016,
.nr_blocks = 64,
- .name = "W25Q32",
+ .name = "W25Q32BV",
},
{
.id = 0x4017,
.nr_blocks = 128,
- .name = "W25Q64",
+ .name = "W25Q64CV",
},
{
.id = 0x4018,
.nr_blocks = 256,
- .name = "W25Q128",
+ .name = "W25Q128BV",
},
{
.id = 0x4019,
--
1.8.3
^ permalink raw reply related [flat|nested] 21+ messages in thread
* [U-Boot] [PATCH v2 5/8] sf: winbond: Update the names for W25Q 0x40XX ID's flash parts
2013-05-30 13:49 ` [U-Boot] [PATCH v2 5/8] sf: winbond: Update the names for W25Q 0x40XX ID's flash parts Jagannadha Sutradharudu Teki
@ 2013-06-02 19:23 ` Jagan Teki
2013-06-04 6:02 ` Jagan Teki
0 siblings, 1 reply; 21+ messages in thread
From: Jagan Teki @ 2013-06-02 19:23 UTC (permalink / raw)
To: u-boot
Hi,
On Thu, May 30, 2013 at 7:19 PM, Jagannadha Sutradharudu Teki
<jagannadha.sutradharudu-teki@xilinx.com> wrote:
> Use the exact names for W25Q 0x40XX ID's flash parts, as the same
> sizes of flashes comes with different ID's. so-that the distinguishes
> becomes easy with this change.
>
> Signed-off-by: Jagannadha Sutradharudu Teki <jaganna@xilinx.com>
> ---
> Changes for v2:
> - none
>
> drivers/mtd/spi/winbond.c | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/mtd/spi/winbond.c b/drivers/mtd/spi/winbond.c
> index 8457808..3c0ca88 100644
> --- a/drivers/mtd/spi/winbond.c
> +++ b/drivers/mtd/spi/winbond.c
> @@ -55,27 +55,27 @@ static const struct winbond_spi_flash_params winbond_spi_flash_table[] = {
> {
> .id = 0x4014,
> .nr_blocks = 16,
> - .name = "W25Q80BL",
> + .name = "W25Q80BL/W25Q80BV",
> },
> {
> .id = 0x4015,
> .nr_blocks = 32,
> - .name = "W25Q16",
> + .name = "W25Q16CL/W25Q16DV",
The id codes were same for these two parts, I think there is no way to
detect the separately.
Any idea?
Thanks,
Jagan.
> },
> {
> .id = 0x4016,
> .nr_blocks = 64,
> - .name = "W25Q32",
> + .name = "W25Q32BV",
> },
> {
> .id = 0x4017,
> .nr_blocks = 128,
> - .name = "W25Q64",
> + .name = "W25Q64CV",
> },
> {
> .id = 0x4018,
> .nr_blocks = 256,
> - .name = "W25Q128",
> + .name = "W25Q128BV",
> },
> {
> .id = 0x4019,
> --
> 1.8.3
>
>
^ permalink raw reply [flat|nested] 21+ messages in thread
* [U-Boot] [PATCH v2 5/8] sf: winbond: Update the names for W25Q 0x40XX ID's flash parts
2013-06-02 19:23 ` Jagan Teki
@ 2013-06-04 6:02 ` Jagan Teki
2013-06-04 17:11 ` Syed Hussain
0 siblings, 1 reply; 21+ messages in thread
From: Jagan Teki @ 2013-06-04 6:02 UTC (permalink / raw)
To: u-boot
Hi Syed,
On Mon, Jun 3, 2013 at 12:53 AM, Jagan Teki <jagannadh.teki@gmail.com> wrote:
> Hi,
>
> On Thu, May 30, 2013 at 7:19 PM, Jagannadha Sutradharudu Teki
> <jagannadha.sutradharudu-teki@xilinx.com> wrote:
>> Use the exact names for W25Q 0x40XX ID's flash parts, as the same
>> sizes of flashes comes with different ID's. so-that the distinguishes
>> becomes easy with this change.
>>
>> Signed-off-by: Jagannadha Sutradharudu Teki <jaganna@xilinx.com>
>> ---
>> Changes for v2:
>> - none
>>
>> drivers/mtd/spi/winbond.c | 10 +++++-----
>> 1 file changed, 5 insertions(+), 5 deletions(-)
>>
>> diff --git a/drivers/mtd/spi/winbond.c b/drivers/mtd/spi/winbond.c
>> index 8457808..3c0ca88 100644
>> --- a/drivers/mtd/spi/winbond.c
>> +++ b/drivers/mtd/spi/winbond.c
>> @@ -55,27 +55,27 @@ static const struct winbond_spi_flash_params winbond_spi_flash_table[] = {
>> {
>> .id = 0x4014,
>> .nr_blocks = 16,
>> - .name = "W25Q80BL",
>> + .name = "W25Q80BL/W25Q80BV",
>> },
>> {
>> .id = 0x4015,
>> .nr_blocks = 32,
>> - .name = "W25Q16",
>> + .name = "W25Q16CL/W25Q16DV",
>
> The id codes were same for these two parts, I think there is no way to
> detect the separately.
> Any idea?
Any comments on this.
--
Thanks,
Jagan.
^ permalink raw reply [flat|nested] 21+ messages in thread
* [U-Boot] [PATCH v2 5/8] sf: winbond: Update the names for W25Q 0x40XX ID's flash parts
2013-06-04 6:02 ` Jagan Teki
@ 2013-06-04 17:11 ` Syed Hussain
2013-06-04 17:19 ` MCHEN4 at winbond.com
0 siblings, 1 reply; 21+ messages in thread
From: Syed Hussain @ 2013-06-04 17:11 UTC (permalink / raw)
To: u-boot
Copying Mike, Technical Marketing Director
-----Original Message-----
From: Jagan Teki [mailto:jagannadh.teki at gmail.com]
Sent: Monday, June 03, 2013 11:03 PM
To: US30 Syed Hussain
Cc: u-boot at lists.denx.de; Michal Simek; Tom Rini; Simon Glass; Jagannadha Sutradharudu Teki
Subject: Re: [PATCH v2 5/8] sf: winbond: Update the names for W25Q 0x40XX ID's flash parts
Hi Syed,
On Mon, Jun 3, 2013 at 12:53 AM, Jagan Teki <jagannadh.teki@gmail.com> wrote:
> Hi,
>
> On Thu, May 30, 2013 at 7:19 PM, Jagannadha Sutradharudu Teki
> <jagannadha.sutradharudu-teki@xilinx.com> wrote:
>> Use the exact names for W25Q 0x40XX ID's flash parts, as the same
>> sizes of flashes comes with different ID's. so-that the distinguishes
>> becomes easy with this change.
>>
>> Signed-off-by: Jagannadha Sutradharudu Teki <jaganna@xilinx.com>
>> ---
>> Changes for v2:
>> - none
>>
>> drivers/mtd/spi/winbond.c | 10 +++++-----
>> 1 file changed, 5 insertions(+), 5 deletions(-)
>>
>> diff --git a/drivers/mtd/spi/winbond.c b/drivers/mtd/spi/winbond.c
>> index 8457808..3c0ca88 100644
>> --- a/drivers/mtd/spi/winbond.c
>> +++ b/drivers/mtd/spi/winbond.c
>> @@ -55,27 +55,27 @@ static const struct winbond_spi_flash_params winbond_spi_flash_table[] = {
>> {
>> .id = 0x4014,
>> .nr_blocks = 16,
>> - .name = "W25Q80BL",
>> + .name = "W25Q80BL/W25Q80BV",
>> },
>> {
>> .id = 0x4015,
>> .nr_blocks = 32,
>> - .name = "W25Q16",
>> + .name = "W25Q16CL/W25Q16DV",
>
> The id codes were same for these two parts, I think there is no way to
> detect the separately.
> Any idea?
Any comments on this.
--
Thanks,
Jagan.
The privileged confidential information contained in this email is intended for use only by the addressees as indicated by the original sender of this email. If you are not the addressee indicated in this email or are not responsible for delivery of the email to such a person, please kindly reply to the sender indicating this fact and delete all copies of it from your computer and network server immediately. Your cooperation is highly appreciated. It is advised that any unauthorized use of confidential information of Winbond is strictly prohibited; and any information in this email irrelevant to the official business of Winbond shall be deemed as neither given nor endorsed by Winbond.
^ permalink raw reply [flat|nested] 21+ messages in thread
* [U-Boot] [PATCH v2 5/8] sf: winbond: Update the names for W25Q 0x40XX ID's flash parts
2013-06-04 17:11 ` Syed Hussain
@ 2013-06-04 17:19 ` MCHEN4 at winbond.com
2013-06-04 17:26 ` Jagan Teki
0 siblings, 1 reply; 21+ messages in thread
From: MCHEN4 at winbond.com @ 2013-06-04 17:19 UTC (permalink / raw)
To: u-boot
Jagan,
So far, our major customer request is to have the same ID for backward compatibility. Therefore the IDs are the same for W25Q16C and W25Q16D. As for the 2.5V parts (W25Q80BL/W25Q16CL), they are the same die as the 3V (W25Q80BV/W25Q16CV), so the IDs are the same as well.
The differences between the pairs are not that significant. We consider them as drop-in compatible.
Best Regards,
Mike Chen
Technical Marketing
Winbond Electronics Corp. America
(408)544-2667
-----Original Message-----
From: US30 Syed Hussain
Sent: Tuesday, June 04, 2013 10:12 AM
To: Jagan Teki; US20 Hui (Mike) Chen
Cc: u-boot at lists.denx.de; Michal Simek; Tom Rini; Simon Glass; Jagannadha Sutradharudu Teki
Subject: RE: [PATCH v2 5/8] sf: winbond: Update the names for W25Q 0x40XX ID's flash parts
Copying Mike, Technical Marketing Director
-----Original Message-----
From: Jagan Teki [mailto:jagannadh.teki at gmail.com]
Sent: Monday, June 03, 2013 11:03 PM
To: US30 Syed Hussain
Cc: u-boot at lists.denx.de; Michal Simek; Tom Rini; Simon Glass; Jagannadha Sutradharudu Teki
Subject: Re: [PATCH v2 5/8] sf: winbond: Update the names for W25Q 0x40XX ID's flash parts
Hi Syed,
On Mon, Jun 3, 2013 at 12:53 AM, Jagan Teki <jagannadh.teki@gmail.com> wrote:
> Hi,
>
> On Thu, May 30, 2013 at 7:19 PM, Jagannadha Sutradharudu Teki
> <jagannadha.sutradharudu-teki@xilinx.com> wrote:
>> Use the exact names for W25Q 0x40XX ID's flash parts, as the same
>> sizes of flashes comes with different ID's. so-that the distinguishes
>> becomes easy with this change.
>>
>> Signed-off-by: Jagannadha Sutradharudu Teki <jaganna@xilinx.com>
>> ---
>> Changes for v2:
>> - none
>>
>> drivers/mtd/spi/winbond.c | 10 +++++-----
>> 1 file changed, 5 insertions(+), 5 deletions(-)
>>
>> diff --git a/drivers/mtd/spi/winbond.c b/drivers/mtd/spi/winbond.c
>> index 8457808..3c0ca88 100644
>> --- a/drivers/mtd/spi/winbond.c
>> +++ b/drivers/mtd/spi/winbond.c
>> @@ -55,27 +55,27 @@ static const struct winbond_spi_flash_params winbond_spi_flash_table[] = {
>> {
>> .id = 0x4014,
>> .nr_blocks = 16,
>> - .name = "W25Q80BL",
>> + .name = "W25Q80BL/W25Q80BV",
>> },
>> {
>> .id = 0x4015,
>> .nr_blocks = 32,
>> - .name = "W25Q16",
>> + .name = "W25Q16CL/W25Q16DV",
>
> The id codes were same for these two parts, I think there is no way to
> detect the separately.
> Any idea?
Any comments on this.
--
Thanks,
Jagan.
The privileged confidential information contained in this email is intended for use only by the addressees as indicated by the original sender of this email. If you are not the addressee indicated in this email or are not responsible for delivery of the email to such a person, please kindly reply to the sender indicating this fact and delete all copies of it from your computer and network server immediately. Your cooperation is highly appreciated. It is advised that any unauthorized use of confidential information of Winbond is strictly prohibited; and any information in this email irrelevant to the official business of Winbond shall be deemed as neither given nor endorsed by Winbond.
^ permalink raw reply [flat|nested] 21+ messages in thread
* [U-Boot] [PATCH v2 5/8] sf: winbond: Update the names for W25Q 0x40XX ID's flash parts
2013-06-04 17:19 ` MCHEN4 at winbond.com
@ 2013-06-04 17:26 ` Jagan Teki
2013-06-05 1:01 ` MCHEN4 at winbond.com
[not found] ` <CAD6G_RQ6efJce77bvSUrVsb9aBoJorE+czunUoVZ3Ad7wLeoJQ@mail.gmail.com>
0 siblings, 2 replies; 21+ messages in thread
From: Jagan Teki @ 2013-06-04 17:26 UTC (permalink / raw)
To: u-boot
On Tue, Jun 4, 2013 at 10:49 PM, MCHEN4 at winbond.com <MCHEN4@winbond.com> wrote:
> Jagan,
>
> So far, our major customer request is to have the same ID for backward compatibility. Therefore the IDs are the same for W25Q16C and W25Q16D. As for the 2.5V parts (W25Q80BL/W25Q16CL), they are the same die as the 3V (W25Q80BV/W25Q16CV), so the IDs are the same as well.
>
> The differences between the pairs are not that significant. We consider them as drop-in compatible.
Ok, thanks for your response.
And are your ok with below representation for common id's parts ?
+ .name = "W25Q80BL/W25Q80BV",
+ .name = "W25Q16CL/W25Q16DV",
+ .name = "W25Q32BV/W25Q32FV_SPI",
+ .name = "W25Q64CV/W25Q64FV_SPI",
+ .name = "W25Q128BV/W25Q128FV_SPI",
+ .name = "W25Q32DW/W25Q32FV_QPI",
+ .name = "W25Q64DW/W25Q64FV_QPI",
+ .name = "W25Q128FW/W25Q128FV_QPI",
--
Thanks,
Jagan.
^ permalink raw reply [flat|nested] 21+ messages in thread
* [U-Boot] [PATCH v2 5/8] sf: winbond: Update the names for W25Q 0x40XX ID's flash parts
2013-06-04 17:26 ` Jagan Teki
@ 2013-06-05 1:01 ` MCHEN4 at winbond.com
[not found] ` <CAD6G_RQ6efJce77bvSUrVsb9aBoJorE+czunUoVZ3Ad7wLeoJQ@mail.gmail.com>
1 sibling, 0 replies; 21+ messages in thread
From: MCHEN4 at winbond.com @ 2013-06-05 1:01 UTC (permalink / raw)
To: u-boot
Jagan,
Yes, it is OK. Of course, the 3V QPI IDs of W25Q32FV/64FV/128FV are identical to that of the 1.8V devices (W25Q32DW/64DW/128FW). It may not be a good idea but it's too late for us to change.
Best Regards,
Mike Chen
Technical Marketing
Winbond Electronics Corp. America
(408)544-2667
-----Original Message-----
From: Jagan Teki [mailto:jagannadh.teki at gmail.com]
Sent: Tuesday, June 04, 2013 10:26 AM
To: US20 Hui (Mike) Chen
Cc: US30 Syed Hussain; u-boot at lists.denx.de; Michal Simek; Tom Rini; Simon Glass; Jagannadha Sutradharudu Teki
Subject: Re: [PATCH v2 5/8] sf: winbond: Update the names for W25Q 0x40XX ID's flash parts
On Tue, Jun 4, 2013 at 10:49 PM, MCHEN4 at winbond.com <MCHEN4@winbond.com> wrote:
> Jagan,
>
> So far, our major customer request is to have the same ID for backward compatibility. Therefore the IDs are the same for W25Q16C and W25Q16D. As for the 2.5V parts (W25Q80BL/W25Q16CL), they are the same die as the 3V (W25Q80BV/W25Q16CV), so the IDs are the same as well.
>
> The differences between the pairs are not that significant. We consider them as drop-in compatible.
Ok, thanks for your response.
And are your ok with below representation for common id's parts ?
+ .name = "W25Q80BL/W25Q80BV",
+ .name = "W25Q16CL/W25Q16DV",
+ .name = "W25Q32BV/W25Q32FV_SPI",
+ .name = "W25Q64CV/W25Q64FV_SPI",
+ .name = "W25Q128BV/W25Q128FV_SPI",
+ .name = "W25Q32DW/W25Q32FV_QPI",
+ .name = "W25Q64DW/W25Q64FV_QPI",
+ .name = "W25Q128FW/W25Q128FV_QPI",
--
Thanks,
Jagan.
The privileged confidential information contained in this email is intended for use only by the addressees as indicated by the original sender of this email. If you are not the addressee indicated in this email or are not responsible for delivery of the email to such a person, please kindly reply to the sender indicating this fact and delete all copies of it from your computer and network server immediately. Your cooperation is highly appreciated. It is advised that any unauthorized use of confidential information of Winbond is strictly prohibited; and any information in this email irrelevant to the official business of Winbond shall be deemed as neither given nor endorsed by Winbond.
^ permalink raw reply [flat|nested] 21+ messages in thread
[parent not found: <CAD6G_RQ6efJce77bvSUrVsb9aBoJorE+czunUoVZ3Ad7wLeoJQ@mail.gmail.com>]
* [U-Boot] [PATCH v2 6/8] sf: winbond: Add support for W25Q128FW
[not found] <1369921773-8871-1-git-send-email-jaganna@xilinx.com>
` (4 preceding siblings ...)
2013-05-30 13:49 ` [U-Boot] [PATCH v2 5/8] sf: winbond: Update the names for W25Q 0x40XX ID's flash parts Jagannadha Sutradharudu Teki
@ 2013-05-30 13:49 ` Jagannadha Sutradharudu Teki
2013-05-30 13:49 ` [U-Boot] [PATCH v2 7/8] sf: winbond: Add support for W25Q16DW Jagannadha Sutradharudu Teki
2013-05-30 13:49 ` [U-Boot] [PATCH v2 8/8] sf: winbond: Add support for W25QXXXFV Jagannadha Sutradharudu Teki
7 siblings, 0 replies; 21+ messages in thread
From: Jagannadha Sutradharudu Teki @ 2013-05-30 13:49 UTC (permalink / raw)
To: u-boot
Add support for Winbond W25Q128FW SPI flash.
Signed-off-by: Jagannadha Sutradharudu Teki <jaganna@xilinx.com>
---
Changes for v2:
- none
drivers/mtd/spi/winbond.c | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/drivers/mtd/spi/winbond.c b/drivers/mtd/spi/winbond.c
index 3c0ca88..e07480b 100644
--- a/drivers/mtd/spi/winbond.c
+++ b/drivers/mtd/spi/winbond.c
@@ -97,6 +97,11 @@ static const struct winbond_spi_flash_params winbond_spi_flash_table[] = {
.nr_blocks = 128,
.name = "W25Q64DW",
},
+ {
+ .id = 0x6018,
+ .nr_blocks = 256,
+ .name = "W25Q128FW",
+ },
};
struct spi_flash *spi_flash_probe_winbond(struct spi_slave *spi, u8 *idcode)
--
1.8.3
^ permalink raw reply related [flat|nested] 21+ messages in thread
* [U-Boot] [PATCH v2 7/8] sf: winbond: Add support for W25Q16DW
[not found] <1369921773-8871-1-git-send-email-jaganna@xilinx.com>
` (5 preceding siblings ...)
2013-05-30 13:49 ` [U-Boot] [PATCH v2 6/8] sf: winbond: Add support for W25Q128FW Jagannadha Sutradharudu Teki
@ 2013-05-30 13:49 ` Jagannadha Sutradharudu Teki
2013-05-30 13:49 ` [U-Boot] [PATCH v2 8/8] sf: winbond: Add support for W25QXXXFV Jagannadha Sutradharudu Teki
7 siblings, 0 replies; 21+ messages in thread
From: Jagannadha Sutradharudu Teki @ 2013-05-30 13:49 UTC (permalink / raw)
To: u-boot
Add support for Winbond W25Q16DW SPI flash.
Signed-off-by: Jagannadha Sutradharudu Teki <jaganna@xilinx.com>
---
Changes for v2:
- none
drivers/mtd/spi/winbond.c | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/drivers/mtd/spi/winbond.c b/drivers/mtd/spi/winbond.c
index e07480b..9fb4910 100644
--- a/drivers/mtd/spi/winbond.c
+++ b/drivers/mtd/spi/winbond.c
@@ -88,6 +88,11 @@ static const struct winbond_spi_flash_params winbond_spi_flash_table[] = {
.name = "W25Q80BW",
},
{
+ .id = 0x6015,
+ .nr_blocks = 32,
+ .name = "W25Q16DW",
+ },
+ {
.id = 0x6016,
.nr_blocks = 64,
.name = "W25Q32DW",
--
1.8.3
^ permalink raw reply related [flat|nested] 21+ messages in thread
* [U-Boot] [PATCH v2 8/8] sf: winbond: Add support for W25QXXXFV
[not found] <1369921773-8871-1-git-send-email-jaganna@xilinx.com>
` (6 preceding siblings ...)
2013-05-30 13:49 ` [U-Boot] [PATCH v2 7/8] sf: winbond: Add support for W25Q16DW Jagannadha Sutradharudu Teki
@ 2013-05-30 13:49 ` Jagannadha Sutradharudu Teki
7 siblings, 0 replies; 21+ messages in thread
From: Jagannadha Sutradharudu Teki @ 2013-05-30 13:49 UTC (permalink / raw)
To: u-boot
Add support for Winbond W25QXXXFV SPI flash.
Signed-off-by: Jagannadha Sutradharudu Teki <jaganna@xilinx.com>
---
Changes for v2:
- none
drivers/mtd/spi/winbond.c | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/mtd/spi/winbond.c b/drivers/mtd/spi/winbond.c
index 9fb4910..c399bf1 100644
--- a/drivers/mtd/spi/winbond.c
+++ b/drivers/mtd/spi/winbond.c
@@ -65,17 +65,17 @@ static const struct winbond_spi_flash_params winbond_spi_flash_table[] = {
{
.id = 0x4016,
.nr_blocks = 64,
- .name = "W25Q32BV",
+ .name = "W25Q32BV/W25Q32FV_SPI",
},
{
.id = 0x4017,
.nr_blocks = 128,
- .name = "W25Q64CV",
+ .name = "W25Q64CV/W25Q64FV_SPI",
},
{
.id = 0x4018,
.nr_blocks = 256,
- .name = "W25Q128BV",
+ .name = "W25Q128BV/W25Q128FV_SPI",
},
{
.id = 0x4019,
@@ -95,17 +95,17 @@ static const struct winbond_spi_flash_params winbond_spi_flash_table[] = {
{
.id = 0x6016,
.nr_blocks = 64,
- .name = "W25Q32DW",
+ .name = "W25Q32DW/W25Q32FV_QPI",
},
{
.id = 0x6017,
.nr_blocks = 128,
- .name = "W25Q64DW",
+ .name = "W25Q64DW/W25Q64FV_QPI",
},
{
.id = 0x6018,
.nr_blocks = 256,
- .name = "W25Q128FW",
+ .name = "W25Q128FW/W25Q128FV_QPI",
},
};
--
1.8.3
^ permalink raw reply related [flat|nested] 21+ messages in thread