Re: [U-Boot] [PATCH 1/2] SPI: Add S25FL064A and S25FL116K flash information

2014-12-18 Thread Jagan Teki
On 17 November 2014 at 20:21, Ben Dooks ben.do...@codethink.co.uk wrote:
 From: Adnan Ali adnan@codethink.co.uk

 Add S25FL064A and S25FL116K flash indentifiers.

 Signed-off-by: Adnan Ali adnan@codethink.co.uk
 ---
  drivers/mtd/spi/sf_params.c | 2 ++
  1 file changed, 2 insertions(+)

 diff --git a/drivers/mtd/spi/sf_params.c b/drivers/mtd/spi/sf_params.c
 index 61545ca..462e5c2 100644
 --- a/drivers/mtd/spi/sf_params.c
 +++ b/drivers/mtd/spi/sf_params.c
 @@ -51,6 +51,8 @@ const struct spi_flash_params spi_flash_params_table[] = {
 {S25FL016A,  0x010214, 0x0,   64 * 1024,32,   0,
 0},
 {S25FL032A,  0x010215, 0x0,   64 * 1024,64,   0,
 0},
 {S25FL064A,  0x010216, 0x0,   64 * 1024,   128,   0,
 0},
 +   {S25FL064A,  0x010216, 0x0,   64 * 1024,   128,   0,
 0},

What is this, already there the same part - any new?

 +   {S25FL116k,  0x014015, 0x0,   64 * 1024,   128,   0,
 0},

Is this tested! can you rebase master and send the same patches.

 {S25FL128P_256K, 0x012018, 0x0300,   256 * 1024,64, RD_FULL,
WR_QPP},
 {S25FL128P_64K,  0x012018, 0x0301,64 * 1024,   256, RD_FULL,
WR_QPP},
 {S25FL032P,  0x010215, 0x4d00,64 * 1024,64, RD_FULL,
WR_QPP},
 --
 2.1.1


thanks!
-- 
Jagan.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 1/2] SPI: Add S25FL064A and S25FL116K flash information

2014-12-18 Thread Ben Dooks
On 18/12/14 12:14, Jagan Teki wrote:
 On 17 November 2014 at 20:21, Ben Dooks ben.do...@codethink.co.uk wrote:
 From: Adnan Ali adnan@codethink.co.uk

 Add S25FL064A and S25FL116K flash indentifiers.

 Signed-off-by: Adnan Ali adnan@codethink.co.uk
 ---
  drivers/mtd/spi/sf_params.c | 2 ++
  1 file changed, 2 insertions(+)

 diff --git a/drivers/mtd/spi/sf_params.c b/drivers/mtd/spi/sf_params.c
 index 61545ca..462e5c2 100644
 --- a/drivers/mtd/spi/sf_params.c
 +++ b/drivers/mtd/spi/sf_params.c
 @@ -51,6 +51,8 @@ const struct spi_flash_params spi_flash_params_table[] = {
 {S25FL016A,  0x010214, 0x0,   64 * 1024,32,   0,   
  0},
 {S25FL032A,  0x010215, 0x0,   64 * 1024,64,   0,   
  0},
 {S25FL064A,  0x010216, 0x0,   64 * 1024,   128,   0,   
  0},
 +   {S25FL064A,  0x010216, 0x0,   64 * 1024,   128,   0,   
  0},
 
 What is this, already there the same part - any new?

I must have missed this when re-basing off the previous u-boot.

 +   {S25FL116k,  0x014015, 0x0,   64 * 1024,   128,   0,   
  0},
 
 Is this tested! can you rebase master and send the same patches.

Yes, it was one of the devices we looked at and tested on a Marvell
88F6281 based system.

 {S25FL128P_256K, 0x012018, 0x0300,   256 * 1024,64, RD_FULL,   
 WR_QPP},
 {S25FL128P_64K,  0x012018, 0x0301,64 * 1024,   256, RD_FULL,   
 WR_QPP},
 {S25FL032P,  0x010215, 0x4d00,64 * 1024,64, RD_FULL,   
 WR_QPP},
 --
 2.1.1

 
 thanks!
 


-- 
Ben Dooks   http://www.codethink.co.uk/
Senior Engineer Codethink - Providing Genius
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 1/2] SPI: Add S25FL064A and S25FL116K flash information

2014-12-18 Thread Jagan Teki
On 18 December 2014 at 18:38, Ben Dooks ben.do...@codethink.co.uk wrote:
 On 18/12/14 12:14, Jagan Teki wrote:
 On 17 November 2014 at 20:21, Ben Dooks ben.do...@codethink.co.uk wrote:
 From: Adnan Ali adnan@codethink.co.uk

 Add S25FL064A and S25FL116K flash indentifiers.

 Signed-off-by: Adnan Ali adnan@codethink.co.uk
 ---
  drivers/mtd/spi/sf_params.c | 2 ++
  1 file changed, 2 insertions(+)

 diff --git a/drivers/mtd/spi/sf_params.c b/drivers/mtd/spi/sf_params.c
 index 61545ca..462e5c2 100644
 --- a/drivers/mtd/spi/sf_params.c
 +++ b/drivers/mtd/spi/sf_params.c
 @@ -51,6 +51,8 @@ const struct spi_flash_params spi_flash_params_table[] = {
 {S25FL016A,  0x010214, 0x0,   64 * 1024,32,   0,  
   0},
 {S25FL032A,  0x010215, 0x0,   64 * 1024,64,   0,  
   0},
 {S25FL064A,  0x010216, 0x0,   64 * 1024,   128,   0,  
   0},
 +   {S25FL064A,  0x010216, 0x0,   64 * 1024,   128,   0,  
   0},

 What is this, already there the same part - any new?

 I must have missed this when re-basing off the previous u-boot.

 +   {S25FL116k,  0x014015, 0x0,   64 * 1024,   128,   0,  
   0},

 Is this tested! can you rebase master and send the same patches.

 Yes, it was one of the devices we looked at and tested on a Marvell
 88F6281 based system.

 {S25FL128P_256K, 0x012018, 0x0300,   256 * 1024,64, RD_FULL,  
  WR_QPP},
 {S25FL128P_64K,  0x012018, 0x0301,64 * 1024,   256, RD_FULL,  
  WR_QPP},
 {S25FL032P,  0x010215, 0x4d00,64 * 1024,64, RD_FULL,  
  WR_QPP},
 --
 2.1.1


Applied to u-boot-spi/master

thanks!
-- 
Jagan.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH 1/2] SPI: Add S25FL064A and S25FL116K flash information

2014-11-17 Thread Ben Dooks
From: Adnan Ali adnan@codethink.co.uk

Add S25FL064A and S25FL116K flash indentifiers.

Signed-off-by: Adnan Ali adnan@codethink.co.uk
---
 drivers/mtd/spi/sf_params.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/mtd/spi/sf_params.c b/drivers/mtd/spi/sf_params.c
index 61545ca..462e5c2 100644
--- a/drivers/mtd/spi/sf_params.c
+++ b/drivers/mtd/spi/sf_params.c
@@ -51,6 +51,8 @@ const struct spi_flash_params spi_flash_params_table[] = {
{S25FL016A,  0x010214, 0x0,   64 * 1024,32,   0,  
  0},
{S25FL032A,  0x010215, 0x0,   64 * 1024,64,   0,  
  0},
{S25FL064A,  0x010216, 0x0,   64 * 1024,   128,   0,  
  0},
+   {S25FL064A,  0x010216, 0x0,   64 * 1024,   128,   0,  
  0},
+   {S25FL116k,  0x014015, 0x0,   64 * 1024,   128,   0,  
  0},
{S25FL128P_256K, 0x012018, 0x0300,   256 * 1024,64, RD_FULL,  
 WR_QPP},
{S25FL128P_64K,  0x012018, 0x0301,64 * 1024,   256, RD_FULL,  
 WR_QPP},
{S25FL032P,  0x010215, 0x4d00,64 * 1024,64, RD_FULL,  
 WR_QPP},
-- 
2.1.1

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot