Re: [U-Boot] [Uboot][PATCH] drivers: spi: macronix: Add MX66L51235L data

2013-10-07 Thread Sourav Poddar 

On Thursday 03 October 2013 09:19 PM, Jagan Teki wrote:

Hi Sourav,

Can you test this macronix part w.r.t latest code base on
u-boot-spi.git with master-probe branch.
Please let me know about any issues/concerns.


Regarding the number of blocks, it should be 1024 only.

I will do other testing asap and let you know. This particular flash
is on another SOC so I will send that SOC specific qspi data after 
testing it

with the subject patch.

On Tue, Aug 27, 2013 at 2:48 PM, Jagan Tekijagannadh.t...@gmail.com  wrote:

Hi

On Tue, Aug 27, 2013 at 11:15 AM, Sourav Poddarsourav.pod...@ti.com  wrote:

Add support for macronix MX66L51235L flash device.

Signed-off-by: Sourav Poddarsourav.pod...@ti.com
---
  drivers/mtd/spi/macronix.c |5 +
  1 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/drivers/mtd/spi/macronix.c b/drivers/mtd/spi/macronix.c
index 70435eb..5a528c3 100644
--- a/drivers/mtd/spi/macronix.c
+++ b/drivers/mtd/spi/macronix.c
@@ -61,6 +61,11 @@ static const struct macronix_spi_flash_params 
macronix_spi_flash_table[] = {
 .nr_blocks = 256,
 .name = MX25L12855E,
 },
+   {
+   .idcode = 0x201a,
+   .nr_blocks = 512,
+   .name = MX66L51235L,
+   },
  };


Can you check the similar patch
http://patchwork.ozlabs.org/patch/262928/

Please confirm about nr_blocks?
Also have u tested with latest spi_flash code, since ur part is 64MB
must require a BAR to access entire memory.?
http://patchwork.ozlabs.org/patch/262929/

Let me know for any concerns.

--
Thanks,
Jagan.





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

Re: [U-Boot] [Uboot][PATCH] drivers: spi: macronix: Add MX66L51235L data

2013-10-07 Thread Jagan Teki 
On Mon, Oct 7, 2013 at 8:50 PM, Sourav Poddar sourav.pod...@ti.com wrote:
 On Thursday 03 October 2013 09:19 PM, Jagan Teki wrote:

 Hi Sourav,

 Can you test this macronix part w.r.t latest code base on
 u-boot-spi.git with master-probe branch.
 Please let me know about any issues/concerns.

 Regarding the number of blocks, it should be 1024 only.

 I will do other testing asap and let you know. This particular flash
 is on another SOC so I will send that SOC specific qspi data after testing
 it
 with the subject patch.
OK, thanks.
Please use u-boot-spi.git master branch.


 On Tue, Aug 27, 2013 at 2:48 PM, Jagan Tekijagannadh.t...@gmail.com
 wrote:

 Hi

 On Tue, Aug 27, 2013 at 11:15 AM, Sourav Poddarsourav.pod...@ti.com
 wrote:

 Add support for macronix MX66L51235L flash device.

 Signed-off-by: Sourav Poddarsourav.pod...@ti.com
 ---
   drivers/mtd/spi/macronix.c |5 +
   1 files changed, 5 insertions(+), 0 deletions(-)

 diff --git a/drivers/mtd/spi/macronix.c b/drivers/mtd/spi/macronix.c
 index 70435eb..5a528c3 100644
 --- a/drivers/mtd/spi/macronix.c
 +++ b/drivers/mtd/spi/macronix.c
 @@ -61,6 +61,11 @@ static const struct macronix_spi_flash_params
 macronix_spi_flash_table[] = {
  .nr_blocks = 256,
  .name = MX25L12855E,
  },
 +   {
 +   .idcode = 0x201a,
 +   .nr_blocks = 512,
 +   .name = MX66L51235L,
 +   },
   };

 Can you check the similar patch
 http://patchwork.ozlabs.org/patch/262928/

 Please confirm about nr_blocks?
 Also have u tested with latest spi_flash code, since ur part is 64MB
 must require a BAR to access entire memory.?
 http://patchwork.ozlabs.org/patch/262929/

 Let me know for any concerns.

 --
 Thanks,
 Jagan.







-- 
Thanks,
Jagan.

Jagannadha Sutradharudu Teki,
E: jagannadh.t...@gmail.com, P: +91-9676773388
Engineer - System Software Hacker
U-boot - SPI Custodian and Zynq APSOC
Ln: http://www.linkedin.com/in/jaganteki
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [Uboot][PATCH] drivers: spi: macronix: Add MX66L51235L data

2013-10-03 Thread Jagan Teki 
Hi Sourav,

Can you test this macronix part w.r.t latest code base on
u-boot-spi.git with master-probe branch.
Please let me know about any issues/concerns.

On Tue, Aug 27, 2013 at 2:48 PM, Jagan Teki jagannadh.t...@gmail.com wrote:
 Hi

 On Tue, Aug 27, 2013 at 11:15 AM, Sourav Poddar sourav.pod...@ti.com wrote:
 Add support for macronix MX66L51235L flash device.

 Signed-off-by: Sourav Poddar sourav.pod...@ti.com
 ---
  drivers/mtd/spi/macronix.c |5 +
  1 files changed, 5 insertions(+), 0 deletions(-)

 diff --git a/drivers/mtd/spi/macronix.c b/drivers/mtd/spi/macronix.c
 index 70435eb..5a528c3 100644
 --- a/drivers/mtd/spi/macronix.c
 +++ b/drivers/mtd/spi/macronix.c
 @@ -61,6 +61,11 @@ static const struct macronix_spi_flash_params 
 macronix_spi_flash_table[] = {
 .nr_blocks = 256,
 .name = MX25L12855E,
 },
 +   {
 +   .idcode = 0x201a,
 +   .nr_blocks = 512,
 +   .name = MX66L51235L,
 +   },
  };


 Can you check the similar patch
 http://patchwork.ozlabs.org/patch/262928/

 Please confirm about nr_blocks?
 Also have u tested with latest spi_flash code, since ur part is 64MB
 must require a BAR to access entire memory.?
 http://patchwork.ozlabs.org/patch/262929/

 Let me know for any concerns.

 --
 Thanks,
 Jagan.



-- 
Thanks,
Jagan.

Jagannadha Sutradharudu Teki,
E: jagannadh.t...@gmail.com, P: +91-9676773388
Engineer - System Software Hacker
U-boot - SPI Custodian and Zynq APSOC
Ln: http://www.linkedin.com/in/jaganteki
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [Uboot][PATCH] drivers: spi: macronix: Add MX66L51235L data

2013-08-27 Thread Jagan Teki 
Hi

On Tue, Aug 27, 2013 at 11:15 AM, Sourav Poddar sourav.pod...@ti.com wrote:
 Add support for macronix MX66L51235L flash device.

 Signed-off-by: Sourav Poddar sourav.pod...@ti.com
 ---
  drivers/mtd/spi/macronix.c |5 +
  1 files changed, 5 insertions(+), 0 deletions(-)

 diff --git a/drivers/mtd/spi/macronix.c b/drivers/mtd/spi/macronix.c
 index 70435eb..5a528c3 100644
 --- a/drivers/mtd/spi/macronix.c
 +++ b/drivers/mtd/spi/macronix.c
 @@ -61,6 +61,11 @@ static const struct macronix_spi_flash_params 
 macronix_spi_flash_table[] = {
 .nr_blocks = 256,
 .name = MX25L12855E,
 },
 +   {
 +   .idcode = 0x201a,
 +   .nr_blocks = 512,
 +   .name = MX66L51235L,
 +   },
  };


Can you check the similar patch
http://patchwork.ozlabs.org/patch/262928/

Please confirm about nr_blocks?
Also have u tested with latest spi_flash code, since ur part is 64MB
must require a BAR to access entire memory.?
http://patchwork.ozlabs.org/patch/262929/

Let me know for any concerns.

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

[U-Boot] [Uboot][PATCH] drivers: spi: macronix: Add MX66L51235L data

2013-08-26 Thread Sourav Poddar 
Add support for macronix MX66L51235L flash device.

Signed-off-by: Sourav Poddar sourav.pod...@ti.com
---
 drivers/mtd/spi/macronix.c |5 +
 1 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/drivers/mtd/spi/macronix.c b/drivers/mtd/spi/macronix.c
index 70435eb..5a528c3 100644
--- a/drivers/mtd/spi/macronix.c
+++ b/drivers/mtd/spi/macronix.c
@@ -61,6 +61,11 @@ static const struct macronix_spi_flash_params 
macronix_spi_flash_table[] = {
.nr_blocks = 256,
.name = MX25L12855E,
},
+   {
+   .idcode = 0x201a,
+   .nr_blocks = 512,
+   .name = MX66L51235L,
+   },
 };
 
 struct spi_flash *spi_flash_probe_macronix(struct spi_slave *spi, u8 *idcode)
-- 
1.7.1

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