[U-Boot,v2] sf: update sst25wf040b flash params

Message ID	1435569352-13329-1-git-send-email-haikun.wang@freescale.com
State	Accepted
Delegated to:	Jagannadha Sutradharudu Teki
Headers	show Return-Path: <u-boot-bounces@lists.denx.de> Received-SPF: Fail (protection.outlook.com: domain of freescale.com does not designate 192.88.158.2 as permitted sender) receiver=protection.outlook.com; client-ip=192.88.158.2; helo=az84smr01.freescale.net; From: Haikun Wang <haikun.wang@freescale.com> To: u-boot@lists.denx.de, jagannadh.teki@gmail.com Date: Mon, 29 Jun 2015 17:15:52 +0800 Message-ID: <1435569352-13329-1-git-send-email-haikun.wang@freescale.com> MIME-Version: 1.0 X-Microsoft-Exchange-Diagnostics: 1; BLUPR03MB391; 2:9qpLc87q8/LSBgwp0HALQpjrn0/Ou9nwa+eod3Dh9eqKkc0bZbAWU+SJ3Tbq7IlE; 3:qzTsFVNTnB+DScpeo/27L6GvRyQWesolRLzMfTxCVdHtuBlMJmXL9KX7CE4BMc1g78HiDCEOp1MIweCeItPh/JPvgZg23QaqASDC26LL/wChnDO2iqsr3Cxe0zuaD8dIZ9I51J+igvtqtbZRsKei2d/sJgILMQpmpMftZ8VI64Ywd5AJQTyh2hNdyoNx+wiCOo3k9FsQoPZWW04t42QP9XV94hhCcliHk3HM4MEHRFY=; 25:maFW8HC93/iKXs039Su3weB+95+bPmu8m6pbFG1qP+EIbOgoMaayEjbSERSTQ0Fv596Z8dK4VWhoj4xLHLbRD/64ai/o1lqMgh1K67eRf5FIGjpL23nH6z3HMpCam3MHHtDd8/0/YOf/ROllzMBrtXGP4xvIpQPrF8yuHs3blCH5ankPFVpjBlKcp03r74qw6qhiar+UdclLfNLhxILQVrYcWEd7PdZTELjrc5OD0YyDg2lxavE8N8oM/3M+NATNmKag92eWtUrwfMWIM2YzTA==; 20:7N3DiKIr8lBwpl8n8i7LCM4WzwVTFjSomFg+f1OHYi+MJb0xcWLgp3/zA1QpgaCoGOnYdgPaPPpNNwqapoCP5jtsoyK0r5BZRn71DpkGW8yZBYqTU5/TkuCyFbRZXi03QcmgexaJwklqGGTjCc/XIBvpOJrxmWZOPfRRUbKeUE/+T9TAnuuCqnvNFkDUMiuTrqBkZOkj9wUD0OoIdp1XpyZGV/R8Bbl7EOHZCt1cbLZugdJko5oXZs80KPreZq+w8kSvz0D1qvdnbaLKnmMogyx2wR0hMq/VOuq/UVf8wiWI76Yo1bqBJjRJP/XDGZMl+XwI6f08ZAR0VfhmJ1xM6PZmBlkRO6iZOu+E OxjNQcY= Cc: Haikun Wang <B53464@freescale.com> Subject: [U-Boot] [PATCH v2] sf: update sst25wf040b flash params Precedence: list Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" <u-boot-bounces@lists.denx.de>

Message ID

1435569352-13329-1-git-send-email-haikun.wang@freescale.com

State

Accepted

Delegated to:

Jagannadha Sutradharudu Teki

Headers

Received-SPF: Fail (protection.outlook.com: domain of freescale.com does not
	designate 192.88.158.2 as permitted sender)
	receiver=protection.outlook.com; 
	client-ip=192.88.158.2; helo=az84smr01.freescale.net;
From: Haikun Wang <haikun.wang@freescale.com>
To: u-boot@lists.denx.de, jagannadh.teki@gmail.com
Date: Mon, 29 Jun 2015 17:15:52 +0800
Message-ID: <1435569352-13329-1-git-send-email-haikun.wang@freescale.com>
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 Jun 2015 09:20:24.7575
	(UTC)
X-MS-Exchange-CrossTenant-Id: 710a03f5-10f6-4d38-9ff4-a80b81da590d
X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=710a03f5-10f6-4d38-9ff4-a80b81da590d;
	Ip=[192.88.158.2]; 
	Helo=[az84smr01.freescale.net]
X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BLUPR03MB391
Cc: Haikun Wang <B53464@freescale.com>
Subject: [U-Boot] [PATCH v2] sf: update sst25wf040b flash params
X-BeenThere: u-boot@lists.denx.de
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: U-Boot discussion <u-boot.lists.denx.de>
List-Unsubscribe: <http://lists.denx.de/mailman/options/u-boot>,
	<mailto:u-boot-request@lists.denx.de?subject=unsubscribe>
List-Archive: <http://lists.denx.de/pipermail/u-boot/>
List-Post: <mailto:u-boot@lists.denx.de>
List-Help: <mailto:u-boot-request@lists.denx.de?subject=help>
List-Subscribe: <http://lists.denx.de/mailman/listinfo/u-boot>,
	<mailto:u-boot-request@lists.denx.de?subject=subscribe>
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Errors-To: u-boot-bounces@lists.denx.de
Sender: "U-Boot" <u-boot-bounces@lists.denx.de>

Commit Message

Haikun.Wang@freescale.com June 29, 2015, 9:15 a.m. UTC

From: Haikun Wang <Haikun.Wang@freescale.com>

sst25wf040b doesn't support Auto Address Increment Programming command.
Remove SST_WR flag.

Signed-off-by: Haikun Wang <B53464@freescale.com>
---
Changes in V2:
- Remove extra line

 drivers/mtd/spi/sf_params.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Jagan Teki June 29, 2015, 9:26 a.m. UTC | #1

On 29 June 2015 at 14:45, Haikun Wang <haikun.wang@freescale.com> wrote:
> From: Haikun Wang <Haikun.Wang@freescale.com>
>
> sst25wf040b doesn't support Auto Address Increment Programming command.
> Remove SST_WR flag.
>
> Signed-off-by: Haikun Wang <B53464@freescale.com>
> ---
> Changes in V2:
> - Remove extra line
>
>  drivers/mtd/spi/sf_params.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/mtd/spi/sf_params.c b/drivers/mtd/spi/sf_params.c
> index c12e8c6..4a4a3af 100644
> --- a/drivers/mtd/spi/sf_params.c
> +++ b/drivers/mtd/spi/sf_params.c
> @@ -100,7 +100,7 @@ const struct spi_flash_params spi_flash_params_table[] = {
>         {"SST25WF010",     0xbf2502, 0x0,       64 * 1024,     2, RD_NORM,          SECT_4K | SST_WR},
>         {"SST25WF020",     0xbf2503, 0x0,       64 * 1024,     4, RD_NORM,          SECT_4K | SST_WR},
>         {"SST25WF040",     0xbf2504, 0x0,       64 * 1024,     8, RD_NORM,          SECT_4K | SST_WR},
> -       {"SST25WF040B",    0x621613, 0x0,       64 * 1024,     8, RD_NORM,          SECT_4K | SST_WR},
> +       {"SST25WF040B",    0x621613, 0x0,       64 * 1024,     8, RD_NORM,                   SECT_4K},

Tested-by: ?

>         {"SST25WF080",     0xbf2505, 0x0,       64 * 1024,    16, RD_NORM,          SECT_4K | SST_WR},
>  #endif
>  #ifdef CONFIG_SPI_FLASH_WINBOND                /* WINBOND */
> --
> 2.1.0.27.g96db324
>

thanks!

Haikun.Wang@freescale.com June 29, 2015, 9:36 a.m. UTC | #2

On 6/29/2015 5:26 PM, Jagan Teki wrote:
> On 29 June 2015 at 14:45, Haikun Wang <haikun.wang@freescale.com> wrote:
>> From: Haikun Wang <Haikun.Wang@freescale.com>
>>
>> sst25wf040b doesn't support Auto Address Increment Programming command.
>> Remove SST_WR flag.
>>
>> Signed-off-by: Haikun Wang <B53464@freescale.com>
>> ---
>> Changes in V2:
>> - Remove extra line
>>
>>   drivers/mtd/spi/sf_params.c | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/mtd/spi/sf_params.c b/drivers/mtd/spi/sf_params.c
>> index c12e8c6..4a4a3af 100644
>> --- a/drivers/mtd/spi/sf_params.c
>> +++ b/drivers/mtd/spi/sf_params.c
>> @@ -100,7 +100,7 @@ const struct spi_flash_params spi_flash_params_table[] = {
>>          {"SST25WF010",     0xbf2502, 0x0,       64 * 1024,     2, RD_NORM,          SECT_4K | SST_WR},
>>          {"SST25WF020",     0xbf2503, 0x0,       64 * 1024,     4, RD_NORM,          SECT_4K | SST_WR},
>>          {"SST25WF040",     0xbf2504, 0x0,       64 * 1024,     8, RD_NORM,          SECT_4K | SST_WR},
>> -       {"SST25WF040B",    0x621613, 0x0,       64 * 1024,     8, RD_NORM,          SECT_4K | SST_WR},
>> +       {"SST25WF040B",    0x621613, 0x0,       64 * 1024,     8, RD_NORM,                   SECT_4K},
>
> Tested-by: ?
>
>>          {"SST25WF080",     0xbf2505, 0x0,       64 * 1024,    16, RD_NORM,          SECT_4K | SST_WR},
>>   #endif
>>   #ifdef CONFIG_SPI_FLASH_WINBOND                /* WINBOND */
>> --
>> 2.1.0.27.g96db324
>>
>
> thanks!
>
Verified on LS2085AQDS board.

Tested-by: Haikun Wang <haikun.wang@freescale.com>

Jagan Teki June 29, 2015, 7:53 p.m. UTC | #3

On 29 June 2015 at 15:06, Wang Haikun <Haikun.Wang@freescale.com> wrote:
> On 6/29/2015 5:26 PM, Jagan Teki wrote:
>> On 29 June 2015 at 14:45, Haikun Wang <haikun.wang@freescale.com> wrote:
>>> From: Haikun Wang <Haikun.Wang@freescale.com>
>>>
>>> sst25wf040b doesn't support Auto Address Increment Programming command.
>>> Remove SST_WR flag.
>>>
>>> Signed-off-by: Haikun Wang <B53464@freescale.com>
>>> ---
>>> Changes in V2:
>>> - Remove extra line
>>>
>>>   drivers/mtd/spi/sf_params.c | 2 +-
>>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/drivers/mtd/spi/sf_params.c b/drivers/mtd/spi/sf_params.c
>>> index c12e8c6..4a4a3af 100644
>>> --- a/drivers/mtd/spi/sf_params.c
>>> +++ b/drivers/mtd/spi/sf_params.c
>>> @@ -100,7 +100,7 @@ const struct spi_flash_params spi_flash_params_table[] = {
>>>          {"SST25WF010",     0xbf2502, 0x0,       64 * 1024,     2, RD_NORM,          SECT_4K | SST_WR},
>>>          {"SST25WF020",     0xbf2503, 0x0,       64 * 1024,     4, RD_NORM,          SECT_4K | SST_WR},
>>>          {"SST25WF040",     0xbf2504, 0x0,       64 * 1024,     8, RD_NORM,          SECT_4K | SST_WR},
>>> -       {"SST25WF040B",    0x621613, 0x0,       64 * 1024,     8, RD_NORM,          SECT_4K | SST_WR},
>>> +       {"SST25WF040B",    0x621613, 0x0,       64 * 1024,     8, RD_NORM,                   SECT_4K},
>>
>> Tested-by: ?
>>
>>>          {"SST25WF080",     0xbf2505, 0x0,       64 * 1024,    16, RD_NORM,          SECT_4K | SST_WR},
>>>   #endif
>>>   #ifdef CONFIG_SPI_FLASH_WINBOND                /* WINBOND */
>>> --
>>> 2.1.0.27.g96db324
>>>
>>
>> thanks!
>>
> Verified on LS2085AQDS board.
>
> Tested-by: Haikun Wang <haikun.wang@freescale.com>

Applied to u-boot-spi/master

thanks!

diff --git a/drivers/mtd/spi/sf_params.c b/drivers/mtd/spi/sf_params.c
index c12e8c6..4a4a3af 100644
--- a/drivers/mtd/spi/sf_params.c
+++ b/drivers/mtd/spi/sf_params.c
@@ -100,7 +100,7 @@  const struct spi_flash_params spi_flash_params_table[] = {
 	{"SST25WF010",	   0xbf2502, 0x0,	64 * 1024,     2, RD_NORM,          SECT_4K | SST_WR},
 	{"SST25WF020",	   0xbf2503, 0x0,	64 * 1024,     4, RD_NORM,	    SECT_4K | SST_WR},
 	{"SST25WF040",	   0xbf2504, 0x0,	64 * 1024,     8, RD_NORM,	    SECT_4K | SST_WR},
-	{"SST25WF040B",	   0x621613, 0x0,	64 * 1024,     8, RD_NORM,	    SECT_4K | SST_WR},
+	{"SST25WF040B",	   0x621613, 0x0,	64 * 1024,     8, RD_NORM,		     SECT_4K},
 	{"SST25WF080",	   0xbf2505, 0x0,	64 * 1024,    16, RD_NORM,	    SECT_4K | SST_WR},
 #endif
 #ifdef CONFIG_SPI_FLASH_WINBOND		/* WINBOND */

[U-Boot,v2] sf: update sst25wf040b flash params

Commit Message

Comments

Patch