* [U-Boot] [PATCH] nand: add ID for Micron MT29F8G08A
@ 2012-09-27 21:49 Lucas Stach
2012-09-27 21:53 ` Scott Wood
0 siblings, 1 reply; 7+ messages in thread
From: Lucas Stach @ 2012-09-27 21:49 UTC (permalink / raw)
To: u-boot
As found on the Colibri T20 engineering sample board.
Signed-off-by: Lucas Stach <dev@lynxeye.de>
---
drivers/mtd/nand/nand_ids.c | 1 +
1 Datei ge?ndert, 1 Zeile hinzugef?gt(+)
diff --git a/drivers/mtd/nand/nand_ids.c b/drivers/mtd/nand/nand_ids.c
index 3953549..ae300fe 100644
--- a/drivers/mtd/nand/nand_ids.c
+++ b/drivers/mtd/nand/nand_ids.c
@@ -105,6 +105,7 @@ const struct nand_flash_dev nand_flash_ids[] = {
{"NAND 512MiB 3,3V 16-bit", 0xCC, 0, 512, 0, LP_OPTIONS16},
/* 8 Gigabit */
+ {"NAND 1GiB 3,3V 8-bit", 0x38, 0, 1024, 0, LP_OPTIONS},
{"NAND 1GiB 1,8V 8-bit", 0xA3, 0, 1024, 0, LP_OPTIONS},
{"NAND 1GiB 3,3V 8-bit", 0xD3, 0, 1024, 0, LP_OPTIONS},
{"NAND 1GiB 1,8V 16-bit", 0xB3, 0, 1024, 0, LP_OPTIONS16},
--
1.7.11.4
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH] nand: add ID for Micron MT29F8G08A
2012-09-27 21:49 [U-Boot] [PATCH] nand: add ID for Micron MT29F8G08A Lucas Stach
@ 2012-09-27 21:53 ` Scott Wood
2012-09-27 22:08 ` Lucas Stach
0 siblings, 1 reply; 7+ messages in thread
From: Scott Wood @ 2012-09-27 21:53 UTC (permalink / raw)
To: u-boot
On 09/27/2012 04:49:31 PM, Lucas Stach wrote:
> As found on the Colibri T20 engineering sample board.
>
> Signed-off-by: Lucas Stach <dev@lynxeye.de>
> ---
> drivers/mtd/nand/nand_ids.c | 1 +
> 1 Datei ge?ndert, 1 Zeile hinzugef?gt(+)
>
> diff --git a/drivers/mtd/nand/nand_ids.c b/drivers/mtd/nand/nand_ids.c
> index 3953549..ae300fe 100644
> --- a/drivers/mtd/nand/nand_ids.c
> +++ b/drivers/mtd/nand/nand_ids.c
> @@ -105,6 +105,7 @@ const struct nand_flash_dev nand_flash_ids[] = {
> {"NAND 512MiB 3,3V 16-bit", 0xCC, 0, 512, 0, LP_OPTIONS16},
>
> /* 8 Gigabit */
> + {"NAND 1GiB 3,3V 8-bit", 0x38, 0, 1024, 0, LP_OPTIONS},
> {"NAND 1GiB 1,8V 8-bit", 0xA3, 0, 1024, 0, LP_OPTIONS},
> {"NAND 1GiB 3,3V 8-bit", 0xD3, 0, 1024, 0, LP_OPTIONS},
> {"NAND 1GiB 1,8V 16-bit", 0xB3, 0, 1024, 0, LP_OPTIONS16},
> --
> 1.7.11.4
Does this chip support ONFI? If so, we don't use the ID table.
-Scott
^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH] nand: add ID for Micron MT29F8G08A
2012-09-27 21:53 ` Scott Wood
@ 2012-09-27 22:08 ` Lucas Stach
2012-09-27 22:13 ` Scott Wood
0 siblings, 1 reply; 7+ messages in thread
From: Lucas Stach @ 2012-09-27 22:08 UTC (permalink / raw)
To: u-boot
Am Donnerstag, den 27.09.2012, 16:53 -0500 schrieb Scott Wood:
> On 09/27/2012 04:49:31 PM, Lucas Stach wrote:
> > As found on the Colibri T20 engineering sample board.
> >
> > Signed-off-by: Lucas Stach <dev@lynxeye.de>
> > ---
> > drivers/mtd/nand/nand_ids.c | 1 +
> > 1 Datei ge?ndert, 1 Zeile hinzugef?gt(+)
> >
> > diff --git a/drivers/mtd/nand/nand_ids.c b/drivers/mtd/nand/nand_ids.c
> > index 3953549..ae300fe 100644
> > --- a/drivers/mtd/nand/nand_ids.c
> > +++ b/drivers/mtd/nand/nand_ids.c
> > @@ -105,6 +105,7 @@ const struct nand_flash_dev nand_flash_ids[] = {
> > {"NAND 512MiB 3,3V 16-bit", 0xCC, 0, 512, 0, LP_OPTIONS16},
> >
> > /* 8 Gigabit */
> > + {"NAND 1GiB 3,3V 8-bit", 0x38, 0, 1024, 0, LP_OPTIONS},
> > {"NAND 1GiB 1,8V 8-bit", 0xA3, 0, 1024, 0, LP_OPTIONS},
> > {"NAND 1GiB 3,3V 8-bit", 0xD3, 0, 1024, 0, LP_OPTIONS},
> > {"NAND 1GiB 1,8V 16-bit", 0xB3, 0, 1024, 0, LP_OPTIONS16},
> > --
> > 1.7.11.4
>
> Does this chip support ONFI? If so, we don't use the ID table.
>
I just rechecked, the datasheet claims the chip supports ONFI 2.0, but
it does not get detected by the u-boot NAND code without the entry in
the id table. CONFIG_SYS_NAND_ONFI_DETECTION is set.
Lucas
^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH] nand: add ID for Micron MT29F8G08A
2012-09-27 22:08 ` Lucas Stach
@ 2012-09-27 22:13 ` Scott Wood
2012-09-27 22:19 ` Lucas Stach
0 siblings, 1 reply; 7+ messages in thread
From: Scott Wood @ 2012-09-27 22:13 UTC (permalink / raw)
To: u-boot
On 09/27/2012 05:08:03 PM, Lucas Stach wrote:
> Am Donnerstag, den 27.09.2012, 16:53 -0500 schrieb Scott Wood:
> > On 09/27/2012 04:49:31 PM, Lucas Stach wrote:
> > > As found on the Colibri T20 engineering sample board.
> > >
> > > Signed-off-by: Lucas Stach <dev@lynxeye.de>
> > > ---
> > > drivers/mtd/nand/nand_ids.c | 1 +
> > > 1 Datei ge?ndert, 1 Zeile hinzugef?gt(+)
> > >
> > > diff --git a/drivers/mtd/nand/nand_ids.c
> b/drivers/mtd/nand/nand_ids.c
> > > index 3953549..ae300fe 100644
> > > --- a/drivers/mtd/nand/nand_ids.c
> > > +++ b/drivers/mtd/nand/nand_ids.c
> > > @@ -105,6 +105,7 @@ const struct nand_flash_dev nand_flash_ids[]
> = {
> > > {"NAND 512MiB 3,3V 16-bit", 0xCC, 0, 512, 0, LP_OPTIONS16},
> > >
> > > /* 8 Gigabit */
> > > + {"NAND 1GiB 3,3V 8-bit", 0x38, 0, 1024, 0, LP_OPTIONS},
> > > {"NAND 1GiB 1,8V 8-bit", 0xA3, 0, 1024, 0, LP_OPTIONS},
> > > {"NAND 1GiB 3,3V 8-bit", 0xD3, 0, 1024, 0, LP_OPTIONS},
> > > {"NAND 1GiB 1,8V 16-bit", 0xB3, 0, 1024, 0, LP_OPTIONS16},
> > > --
> > > 1.7.11.4
> >
> > Does this chip support ONFI? If so, we don't use the ID table.
> >
> I just rechecked, the datasheet claims the chip supports ONFI 2.0, but
> it does not get detected by the u-boot NAND code without the entry in
> the id table. CONFIG_SYS_NAND_ONFI_DETECTION is set.
Could you debug it? Could there be a problem in the NAND controller
(or the driver)?
Are you using a recent U-Boot tree?
-Scott
^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH] nand: add ID for Micron MT29F8G08A
2012-09-27 22:13 ` Scott Wood
@ 2012-09-27 22:19 ` Lucas Stach
2012-09-27 22:27 ` Scott Wood
0 siblings, 1 reply; 7+ messages in thread
From: Lucas Stach @ 2012-09-27 22:19 UTC (permalink / raw)
To: u-boot
Am Donnerstag, den 27.09.2012, 17:13 -0500 schrieb Scott Wood:
> On 09/27/2012 05:08:03 PM, Lucas Stach wrote:
> > Am Donnerstag, den 27.09.2012, 16:53 -0500 schrieb Scott Wood:
> > > On 09/27/2012 04:49:31 PM, Lucas Stach wrote:
> > > > As found on the Colibri T20 engineering sample board.
> > > >
> > > > Signed-off-by: Lucas Stach <dev@lynxeye.de>
> > > > ---
> > > > drivers/mtd/nand/nand_ids.c | 1 +
> > > > 1 Datei ge?ndert, 1 Zeile hinzugef?gt(+)
> > > >
> > > > diff --git a/drivers/mtd/nand/nand_ids.c
> > b/drivers/mtd/nand/nand_ids.c
> > > > index 3953549..ae300fe 100644
> > > > --- a/drivers/mtd/nand/nand_ids.c
> > > > +++ b/drivers/mtd/nand/nand_ids.c
> > > > @@ -105,6 +105,7 @@ const struct nand_flash_dev nand_flash_ids[]
> > = {
> > > > {"NAND 512MiB 3,3V 16-bit", 0xCC, 0, 512, 0, LP_OPTIONS16},
> > > >
> > > > /* 8 Gigabit */
> > > > + {"NAND 1GiB 3,3V 8-bit", 0x38, 0, 1024, 0, LP_OPTIONS},
> > > > {"NAND 1GiB 1,8V 8-bit", 0xA3, 0, 1024, 0, LP_OPTIONS},
> > > > {"NAND 1GiB 3,3V 8-bit", 0xD3, 0, 1024, 0, LP_OPTIONS},
> > > > {"NAND 1GiB 1,8V 16-bit", 0xB3, 0, 1024, 0, LP_OPTIONS16},
> > > > --
> > > > 1.7.11.4
> > >
> > > Does this chip support ONFI? If so, we don't use the ID table.
> > >
> > I just rechecked, the datasheet claims the chip supports ONFI 2.0, but
> > it does not get detected by the u-boot NAND code without the entry in
> > the id table. CONFIG_SYS_NAND_ONFI_DETECTION is set.
>
> Could you debug it? Could there be a problem in the NAND controller
> (or the driver)?
>
Haven't done much in tracking this down much further. Could quite
possible be a bug in the Tegra nand driver, this driver isn't in tree
for a long time and it seems no Tegra board so far tried using ONFI.
> Are you using a recent U-Boot tree?
>
Recent master tree as of today.
Lucas
^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH] nand: add ID for Micron MT29F8G08A
2012-09-27 22:19 ` Lucas Stach
@ 2012-09-27 22:27 ` Scott Wood
2012-09-28 15:20 ` Lucas Stach
0 siblings, 1 reply; 7+ messages in thread
From: Scott Wood @ 2012-09-27 22:27 UTC (permalink / raw)
To: u-boot
On 09/27/2012 05:19:23 PM, Lucas Stach wrote:
> Am Donnerstag, den 27.09.2012, 17:13 -0500 schrieb Scott Wood:
> > On 09/27/2012 05:08:03 PM, Lucas Stach wrote:
> > > Am Donnerstag, den 27.09.2012, 16:53 -0500 schrieb Scott Wood:
> > > > On 09/27/2012 04:49:31 PM, Lucas Stach wrote:
> > > > > As found on the Colibri T20 engineering sample board.
> > > > >
> > > > > Signed-off-by: Lucas Stach <dev@lynxeye.de>
> > > > > ---
> > > > > drivers/mtd/nand/nand_ids.c | 1 +
> > > > > 1 Datei ge?ndert, 1 Zeile hinzugef?gt(+)
> > > > >
> > > > > diff --git a/drivers/mtd/nand/nand_ids.c
> > > b/drivers/mtd/nand/nand_ids.c
> > > > > index 3953549..ae300fe 100644
> > > > > --- a/drivers/mtd/nand/nand_ids.c
> > > > > +++ b/drivers/mtd/nand/nand_ids.c
> > > > > @@ -105,6 +105,7 @@ const struct nand_flash_dev
> nand_flash_ids[]
> > > = {
> > > > > {"NAND 512MiB 3,3V 16-bit", 0xCC, 0, 512, 0,
> LP_OPTIONS16},
> > > > >
> > > > > /* 8 Gigabit */
> > > > > + {"NAND 1GiB 3,3V 8-bit", 0x38, 0, 1024, 0,
> LP_OPTIONS},
> > > > > {"NAND 1GiB 1,8V 8-bit", 0xA3, 0, 1024, 0,
> LP_OPTIONS},
> > > > > {"NAND 1GiB 3,3V 8-bit", 0xD3, 0, 1024, 0,
> LP_OPTIONS},
> > > > > {"NAND 1GiB 1,8V 16-bit", 0xB3, 0, 1024, 0,
> LP_OPTIONS16},
> > > > > --
> > > > > 1.7.11.4
> > > >
> > > > Does this chip support ONFI? If so, we don't use the ID table.
> > > >
> > > I just rechecked, the datasheet claims the chip supports ONFI
> 2.0, but
> > > it does not get detected by the u-boot NAND code without the
> entry in
> > > the id table. CONFIG_SYS_NAND_ONFI_DETECTION is set.
> >
> > Could you debug it? Could there be a problem in the NAND controller
> > (or the driver)?
> >
> Haven't done much in tracking this down much further. Could quite
> possible be a bug in the Tegra nand driver, this driver isn't in tree
> for a long time and it seems no Tegra board so far tried using ONFI.
I don't see the Tegra READID code using column, which is a problem.
Hopefully this is a driver limitation and not hardware, unlike i.MX.
-Scott
^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH] nand: add ID for Micron MT29F8G08A
2012-09-27 22:27 ` Scott Wood
@ 2012-09-28 15:20 ` Lucas Stach
0 siblings, 0 replies; 7+ messages in thread
From: Lucas Stach @ 2012-09-28 15:20 UTC (permalink / raw)
To: u-boot
Hi Scott,
Am Donnerstag, den 27.09.2012, 17:27 -0500 schrieb Scott Wood:
> On 09/27/2012 05:19:23 PM, Lucas Stach wrote:
> > Am Donnerstag, den 27.09.2012, 17:13 -0500 schrieb Scott Wood:
> > > On 09/27/2012 05:08:03 PM, Lucas Stach wrote:
> > > > Am Donnerstag, den 27.09.2012, 16:53 -0500 schrieb Scott Wood:
> > > > > On 09/27/2012 04:49:31 PM, Lucas Stach wrote:
> > > > > > As found on the Colibri T20 engineering sample board.
> > > > > >
> > > > > > Signed-off-by: Lucas Stach <dev@lynxeye.de>
> > > > > > ---
> > > > > > drivers/mtd/nand/nand_ids.c | 1 +
> > > > > > 1 Datei ge?ndert, 1 Zeile hinzugef?gt(+)
> > > > > >
> > > > > > diff --git a/drivers/mtd/nand/nand_ids.c
> > > > b/drivers/mtd/nand/nand_ids.c
> > > > > > index 3953549..ae300fe 100644
> > > > > > --- a/drivers/mtd/nand/nand_ids.c
> > > > > > +++ b/drivers/mtd/nand/nand_ids.c
> > > > > > @@ -105,6 +105,7 @@ const struct nand_flash_dev
> > nand_flash_ids[]
> > > > = {
> > > > > > {"NAND 512MiB 3,3V 16-bit", 0xCC, 0, 512, 0,
> > LP_OPTIONS16},
> > > > > >
> > > > > > /* 8 Gigabit */
> > > > > > + {"NAND 1GiB 3,3V 8-bit", 0x38, 0, 1024, 0,
> > LP_OPTIONS},
> > > > > > {"NAND 1GiB 1,8V 8-bit", 0xA3, 0, 1024, 0,
> > LP_OPTIONS},
> > > > > > {"NAND 1GiB 3,3V 8-bit", 0xD3, 0, 1024, 0,
> > LP_OPTIONS},
> > > > > > {"NAND 1GiB 1,8V 16-bit", 0xB3, 0, 1024, 0,
> > LP_OPTIONS16},
> > > > > > --
> > > > > > 1.7.11.4
> > > > >
> > > > > Does this chip support ONFI? If so, we don't use the ID table.
> > > > >
> > > > I just rechecked, the datasheet claims the chip supports ONFI
> > 2.0, but
> > > > it does not get detected by the u-boot NAND code without the
> > entry in
> > > > the id table. CONFIG_SYS_NAND_ONFI_DETECTION is set.
> > >
> > > Could you debug it? Could there be a problem in the NAND controller
> > > (or the driver)?
> > >
> > Haven't done much in tracking this down much further. Could quite
> > possible be a bug in the Tegra nand driver, this driver isn't in tree
> > for a long time and it seems no Tegra board so far tried using ONFI.
>
> I don't see the Tegra READID code using column, which is a problem.
> Hopefully this is a driver limitation and not hardware, unlike i.MX.
>
Please disregard this patch. I've got ONFI working properly on the Tegra
and will send out a patch to enable this shortly.
Thanks,
Lucas
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2012-09-28 15:20 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-09-27 21:49 [U-Boot] [PATCH] nand: add ID for Micron MT29F8G08A Lucas Stach
2012-09-27 21:53 ` Scott Wood
2012-09-27 22:08 ` Lucas Stach
2012-09-27 22:13 ` Scott Wood
2012-09-27 22:19 ` Lucas Stach
2012-09-27 22:27 ` Scott Wood
2012-09-28 15:20 ` Lucas Stach
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox