mail archive of the barebox mailing list
 help / color / mirror / Atom feed
* [PATCH] i.mx35pdk add nand identification
@ 2012-09-06 12:48 Alex Gershgorin
  2012-09-06 13:27 ` Eric Bénard
  2012-09-06 13:47 ` Sascha Hauer
  0 siblings, 2 replies; 9+ messages in thread
From: Alex Gershgorin @ 2012-09-06 12:48 UTC (permalink / raw)
  To: Sascha Hauer; +Cc: barebox, Alex Gershgorin

Without this patch I get following messages

Board: Freescale MX35 3Stack   
NAND type unknown: ec,d7
No NAND device found (-19)!

Signed-off-by: Alex Gershgorin <alexg@meprolight.com>
---
 drivers/mtd/nand/nand_ids.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/drivers/mtd/nand/nand_ids.c b/drivers/mtd/nand/nand_ids.c
index cb53fc6..5f184bd 100644
--- a/drivers/mtd/nand/nand_ids.c
+++ b/drivers/mtd/nand/nand_ids.c
@@ -103,6 +103,7 @@ struct nand_flash_dev nand_flash_ids[] = {
 	/* 4 Gigabit */
 	{__NANDSTR("NAND 512MiB 1,8V 8-bit"),	0xAC, 0, 512, 0, LP_OPTIONS},
 	{__NANDSTR("NAND 512MiB 3,3V 8-bit"),	0xDC, 0, 512, 0, LP_OPTIONS},
+	{__NANDSTR("NAND 512MiB 3,3V 8-bit"),	0xD7, 0, 512, 0, LP_OPTIONS},
 	{__NANDSTR("NAND 512MiB 1,8V 16-bit"),	0xBC, 0, 512, 0, LP_OPTIONS16},
 	{__NANDSTR("NAND 512MiB 3,3V 16-bit"),	0xCC, 0, 512, 0, LP_OPTIONS16},
 
-- 
1.7.0.4


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] i.mx35pdk add nand identification
  2012-09-06 12:48 [PATCH] i.mx35pdk add nand identification Alex Gershgorin
@ 2012-09-06 13:27 ` Eric Bénard
  2012-09-06 13:39   ` Alex Gershgorin
  2012-09-06 13:47 ` Sascha Hauer
  1 sibling, 1 reply; 9+ messages in thread
From: Eric Bénard @ 2012-09-06 13:27 UTC (permalink / raw)
  To: Alex Gershgorin; +Cc: barebox

Le Thu, 6 Sep 2012 15:48:30 +0300,
Alex Gershgorin <alexg@meprolight.com> a écrit :

> Without this patch I get following messages
> 
> Board: Freescale MX35 3Stack   
> NAND type unknown: ec,d7
> No NAND device found (-19)!
> 
what is the part number of the NAND ?

Eric

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 9+ messages in thread

* RE: [PATCH] i.mx35pdk add nand identification
  2012-09-06 13:27 ` Eric Bénard
@ 2012-09-06 13:39   ` Alex Gershgorin
  2012-09-06 13:54     ` Eric Bénard
  0 siblings, 1 reply; 9+ messages in thread
From: Alex Gershgorin @ 2012-09-06 13:39 UTC (permalink / raw)
  To: Eric Bénard; +Cc: barebox

Hi Eric,

> Without this patch I get following messages
>
> Board: Freescale MX35 3Stack
> NAND type unknown: ec,d7
> No NAND device found (-19)!
>
what is the part number of the NAND ?

If I'm not mistaken
 K9LBG08U0D-PCB0
 
Regards,
Alex
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] i.mx35pdk add nand identification
  2012-09-06 12:48 [PATCH] i.mx35pdk add nand identification Alex Gershgorin
  2012-09-06 13:27 ` Eric Bénard
@ 2012-09-06 13:47 ` Sascha Hauer
  2012-09-06 13:51   ` Alex Gershgorin
  1 sibling, 1 reply; 9+ messages in thread
From: Sascha Hauer @ 2012-09-06 13:47 UTC (permalink / raw)
  To: Alex Gershgorin; +Cc: barebox

On Thu, Sep 06, 2012 at 03:48:30PM +0300, Alex Gershgorin wrote:
> Without this patch I get following messages
> 
> Board: Freescale MX35 3Stack   
> NAND type unknown: ec,d7
> No NAND device found (-19)!

Please replace the subject with something like:

"mtd nand: Add <blachip> to flash ids"

Having 'i.mx35pdk' seems to imply this affects only i.MX users.

Sascha

> 
> Signed-off-by: Alex Gershgorin <alexg@meprolight.com>
> ---
>  drivers/mtd/nand/nand_ids.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/mtd/nand/nand_ids.c b/drivers/mtd/nand/nand_ids.c
> index cb53fc6..5f184bd 100644
> --- a/drivers/mtd/nand/nand_ids.c
> +++ b/drivers/mtd/nand/nand_ids.c
> @@ -103,6 +103,7 @@ struct nand_flash_dev nand_flash_ids[] = {
>  	/* 4 Gigabit */
>  	{__NANDSTR("NAND 512MiB 1,8V 8-bit"),	0xAC, 0, 512, 0, LP_OPTIONS},
>  	{__NANDSTR("NAND 512MiB 3,3V 8-bit"),	0xDC, 0, 512, 0, LP_OPTIONS},
> +	{__NANDSTR("NAND 512MiB 3,3V 8-bit"),	0xD7, 0, 512, 0, LP_OPTIONS},
>  	{__NANDSTR("NAND 512MiB 1,8V 16-bit"),	0xBC, 0, 512, 0, LP_OPTIONS16},
>  	{__NANDSTR("NAND 512MiB 3,3V 16-bit"),	0xCC, 0, 512, 0, LP_OPTIONS16},
>  
> -- 
> 1.7.0.4
> 
> 

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 9+ messages in thread

* RE: [PATCH] i.mx35pdk add nand identification
  2012-09-06 13:47 ` Sascha Hauer
@ 2012-09-06 13:51   ` Alex Gershgorin
  0 siblings, 0 replies; 9+ messages in thread
From: Alex Gershgorin @ 2012-09-06 13:51 UTC (permalink / raw)
  To: Sascha Hauer; +Cc: barebox

Hi Sascha,

Thanks for your comments.
 
On Thu, Sep 06, 2012 at 03:48:30PM +0300, Alex Gershgorin wrote:
> Without this patch I get following messages
>
> Board: Freescale MX35 3Stack
> NAND type unknown: ec,d7
> No NAND device found (-19)!

> >Please replace the subject with something like:

> >"mtd nand: Add <blachip> to flash ids"

> >Having 'i.mx35pdk' seems to imply this affects only i.MX users.

You are absolutely right, I will correct it. 

Regards,
Alex

>
> Signed-off-by: Alex Gershgorin <alexg@meprolight.com>
> ---
>  drivers/mtd/nand/nand_ids.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/mtd/nand/nand_ids.c b/drivers/mtd/nand/nand_ids.c
> index cb53fc6..5f184bd 100644
> --- a/drivers/mtd/nand/nand_ids.c
> +++ b/drivers/mtd/nand/nand_ids.c
> @@ -103,6 +103,7 @@ struct nand_flash_dev nand_flash_ids[] = {
>       /* 4 Gigabit */
>       {__NANDSTR("NAND 512MiB 1,8V 8-bit"),   0xAC, 0, 512, 0, LP_OPTIONS},
>       {__NANDSTR("NAND 512MiB 3,3V 8-bit"),   0xDC, 0, 512, 0, LP_OPTIONS},
> +     {__NANDSTR("NAND 512MiB 3,3V 8-bit"),   0xD7, 0, 512, 0, LP_OPTIONS},
>       {__NANDSTR("NAND 512MiB 1,8V 16-bit"),  0xBC, 0, 512, 0, LP_OPTIONS16},
>       {__NANDSTR("NAND 512MiB 3,3V 16-bit"),  0xCC, 0, 512, 0, LP_OPTIONS16},
>
> --
> 1.7.0.4
>
>

--
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] i.mx35pdk add nand identification
  2012-09-06 13:39   ` Alex Gershgorin
@ 2012-09-06 13:54     ` Eric Bénard
  2012-09-06 14:20       ` Alex Gershgorin
  0 siblings, 1 reply; 9+ messages in thread
From: Eric Bénard @ 2012-09-06 13:54 UTC (permalink / raw)
  To: Alex Gershgorin; +Cc: barebox

Hi Alex,

Le Thu, 6 Sep 2012 16:39:33 +0300,
Alex Gershgorin <alexg@meprolight.com> a écrit :
> > Without this patch I get following messages
> >
> > Board: Freescale MX35 3Stack
> > NAND type unknown: ec,d7
> > No NAND device found (-19)!
> >
> what is the part number of the NAND ?
> 
> If I'm not mistaken
>  K9LBG08U0D-PCB0
>  
this seems to be a 4GiB flash with 4k page + 218 OOB, not a 512MiB

Eric

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 9+ messages in thread

* RE: [PATCH] i.mx35pdk add nand identification
  2012-09-06 13:54     ` Eric Bénard
@ 2012-09-06 14:20       ` Alex Gershgorin
  2012-09-06 14:37         ` Alex Gershgorin
  2012-09-06 14:39         ` Eric Bénard
  0 siblings, 2 replies; 9+ messages in thread
From: Alex Gershgorin @ 2012-09-06 14:20 UTC (permalink / raw)
  To: Eric Bénard; +Cc: barebox

 
> > >Hi Alex,

Le Thu, 6 Sep 2012 16:39:33 +0300,
Alex Gershgorin <alexg@meprolight.com> a écrit :
> > Without this patch I get following messages
> >
> > Board: Freescale MX35 3Stack
> > NAND type unknown: ec,d7
> > No NAND device found (-19)!
> >
> what is the part number of the NAND ?
>
> If I'm not mistaken
>  K9LBG08U0D-PCB0
>
> > > this seems to be a 4GiB flash with 4k page + 218 OOB, not a 512MiB

If I understand correctly it should look like

{__NANDSTR("NAND 4GiB 3,3V 8-bit"), 0xD7, 0, 2048, 0, LP_OPTIONS}


Right?
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 9+ messages in thread

* RE: [PATCH] i.mx35pdk add nand identification
  2012-09-06 14:20       ` Alex Gershgorin
@ 2012-09-06 14:37         ` Alex Gershgorin
  2012-09-06 14:39         ` Eric Bénard
  1 sibling, 0 replies; 9+ messages in thread
From: Alex Gershgorin @ 2012-09-06 14:37 UTC (permalink / raw)
  To: Alex Gershgorin, Eric Bénard; +Cc: barebox



________________________________________
From: Alex Gershgorin
Sent: Thursday, September 06, 2012 5:20 PM
To: Eric Bénard
Cc: Sascha Hauer; barebox@lists.infradead.org
Subject: RE: [PATCH] i.mx35pdk add nand identification

> > >Hi Alex,

Le Thu, 6 Sep 2012 16:39:33 +0300,
Alex Gershgorin <alexg@meprolight.com> a écrit :
> > Without this patch I get following messages
> >
> > Board: Freescale MX35 3Stack
> > NAND type unknown: ec,d7
> > No NAND device found (-19)!
> >
> what is the part number of the NAND ?
>
> If I'm not mistaken
>  K9LBG08U0D-PCB0
>
> > > this seems to be a 4GiB flash with 4k page + 218 OOB, not a 512MiB

Sorry my mistake in the previous mail :-(

{__NANDSTR("NAND 4GiB 3,3V 8-bit"), 0xD7, 0, 4096, 0, LP_OPTIONS}


Right?
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] i.mx35pdk add nand identification
  2012-09-06 14:20       ` Alex Gershgorin
  2012-09-06 14:37         ` Alex Gershgorin
@ 2012-09-06 14:39         ` Eric Bénard
  1 sibling, 0 replies; 9+ messages in thread
From: Eric Bénard @ 2012-09-06 14:39 UTC (permalink / raw)
  To: Alex Gershgorin; +Cc: barebox

Hi,

Le Thu, 6 Sep 2012 17:20:41 +0300,
Alex Gershgorin <alexg@meprolight.com> a écrit :

>  
> > > >Hi Alex,
> 
> Le Thu, 6 Sep 2012 16:39:33 +0300,
> Alex Gershgorin <alexg@meprolight.com> a écrit :
> > > Without this patch I get following messages
> > >
> > > Board: Freescale MX35 3Stack
> > > NAND type unknown: ec,d7
> > > No NAND device found (-19)!
> > >
> > what is the part number of the NAND ?
> >
> > If I'm not mistaken
> >  K9LBG08U0D-PCB0
> >
> > > > this seems to be a 4GiB flash with 4k page + 218 OOB, not a 512MiB
> 
> If I understand correctly it should look like
> 
> {__NANDSTR("NAND 4GiB 3,3V 8-bit"), 0xD7, 0, 2048, 0, LP_OPTIONS}
> 
> 
> Right?
no 2048 means 2k page, this nand has 4k page.

kernel has :
{"NAND 4GiB 3,3V 8-bit",	0xD7, 0, 4096, 0, LP_OPTIONS},

which should be the right configuration.

Eric

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2012-09-06 14:39 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-09-06 12:48 [PATCH] i.mx35pdk add nand identification Alex Gershgorin
2012-09-06 13:27 ` Eric Bénard
2012-09-06 13:39   ` Alex Gershgorin
2012-09-06 13:54     ` Eric Bénard
2012-09-06 14:20       ` Alex Gershgorin
2012-09-06 14:37         ` Alex Gershgorin
2012-09-06 14:39         ` Eric Bénard
2012-09-06 13:47 ` Sascha Hauer
2012-09-06 13:51   ` Alex Gershgorin

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox