[U-Boot] [PATCH] mx28evk: add NAND commands

[U-Boot] [PATCH] mx28evk: add NAND commands

[Eric Bénard]

the mx28evk has a socket which can contains a TSOP48 NAND flash.
So enable the nand command and it's trimffs option.

Tested on MX28EVK with NAND :
NAND:  128 MiB
MX28EVK U-Boot > nand info

Device 0: nand0, sector size 16 KiB
  Page size      2048 b
  OOB size         64 b
  Erase size    16384 b

and without NAND :
NAND:  No NAND device found!!!
0 MiB
MX28EVK U-Boot > nand info

MX28EVK U-Boot >

Signed-off-by: Eric B?nard <eric@eukrea.com>
---
 include/configs/mx28evk.h | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/include/configs/mx28evk.h b/include/configs/mx28evk.h
index 0d918a1..789d980 100644
--- a/include/configs/mx28evk.h
+++ b/include/configs/mx28evk.h
@@ -69,6 +69,8 @@
 #define CONFIG_CMD_USB
 #define CONFIG_CMD_BOOTZ
 #define CONFIG_CMD_I2C
+#define CONFIG_CMD_NAND
+#define CONFIG_CMD_NAND_TRIMFFS
 
 /*
  * Memory configurations
-- 
1.7.11.7

[U-Boot] [PATCH] mx28evk: add NAND commands

[Fabio Estevam]

Hi Eric,

On Mon, Apr 8, 2013 at 10:20 AM, Eric B?nard <eric@eukrea.com> wrote:

> diff --git a/include/configs/mx28evk.h b/include/configs/mx28evk.h
> index 0d918a1..789d980 100644
> --- a/include/configs/mx28evk.h
> +++ b/include/configs/mx28evk.h
> @@ -69,6 +69,8 @@
>  #define CONFIG_CMD_USB
>  #define CONFIG_CMD_BOOTZ
>  #define CONFIG_CMD_I2C
> +#define CONFIG_CMD_NAND

Please check include/configs/mx28evk_nand.h in u-boot.imx. It has this
define already.

> +#define CONFIG_CMD_NAND_TRIMFFS

Then you could add this command there, if you like.

Regards,

Fabio Estevam

[U-Boot] [PATCH] mx28evk: add NAND commands

[Eric Bénard]

Hi Fabio,

Le Mon, 8 Apr 2013 10:24:45 -0300,
Fabio Estevam <festevam@gmail.com> a ?crit :
> On Mon, Apr 8, 2013 at 10:20 AM, Eric B?nard <eric@eukrea.com> wrote:
> 
> > diff --git a/include/configs/mx28evk.h b/include/configs/mx28evk.h
> > index 0d918a1..789d980 100644
> > --- a/include/configs/mx28evk.h
> > +++ b/include/configs/mx28evk.h
> > @@ -69,6 +69,8 @@
> >  #define CONFIG_CMD_USB
> >  #define CONFIG_CMD_BOOTZ
> >  #define CONFIG_CMD_I2C
> > +#define CONFIG_CMD_NAND
> 
> Please check include/configs/mx28evk_nand.h in u-boot.imx. It has this
> define already.
> 
OK I was on master :-(

Eric

[U-Boot] [PATCH] mx28evk: add NAND commands

[Tom Rini]

On Mon, Apr 08, 2013 at 10:24:45AM -0300, Fabio Estevam wrote:
> Hi Eric,
> 
> On Mon, Apr 8, 2013 at 10:20 AM, Eric B??nard <eric@eukrea.com> wrote:
> 
> > diff --git a/include/configs/mx28evk.h b/include/configs/mx28evk.h
> > index 0d918a1..789d980 100644
> > --- a/include/configs/mx28evk.h
> > +++ b/include/configs/mx28evk.h
> > @@ -69,6 +69,8 @@
> >  #define CONFIG_CMD_USB
> >  #define CONFIG_CMD_BOOTZ
> >  #define CONFIG_CMD_I2C
> > +#define CONFIG_CMD_NAND
> 
> Please check include/configs/mx28evk_nand.h in u-boot.imx. It has this
> define already.
> 
> > +#define CONFIG_CMD_NAND_TRIMFFS
> 
> Then you could add this command there, if you like.

And check for UBI related things being on too.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20130408/0f1a050f/attachment.pgp>

[U-Boot] [PATCH] mx28evk: add NAND commands

[Eric Bénard]

Hi Tom,

Le Mon, 8 Apr 2013 09:27:29 -0400,
Tom Rini <trini@ti.com> a ?crit :

> On Mon, Apr 08, 2013 at 10:24:45AM -0300, Fabio Estevam wrote:
> > Hi Eric,
> > 
> > On Mon, Apr 8, 2013 at 10:20 AM, Eric B??nard <eric@eukrea.com> wrote:
> > 
> > > diff --git a/include/configs/mx28evk.h b/include/configs/mx28evk.h
> > > index 0d918a1..789d980 100644
> > > --- a/include/configs/mx28evk.h
> > > +++ b/include/configs/mx28evk.h
> > > @@ -69,6 +69,8 @@
> > >  #define CONFIG_CMD_USB
> > >  #define CONFIG_CMD_BOOTZ
> > >  #define CONFIG_CMD_I2C
> > > +#define CONFIG_CMD_NAND
> > 
> > Please check include/configs/mx28evk_nand.h in u-boot.imx. It has this
> > define already.
> > 
> > > +#define CONFIG_CMD_NAND_TRIMFFS
> > 
> > Then you could add this command there, if you like.
> 
> And check for UBI related things being on too.
> 
that's what I'm actually testing ;-)

Eric