diff mbox

[U-Boot] mx28evk: add NAND commands

Message ID 1365427216-15220-1-git-send-email-eric@eukrea.com
State Changes Requested
Headers show

Commit Message

Eric Benard April 8, 2013, 1:20 p.m. UTC
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(+)

Comments

Fabio Estevam April 8, 2013, 1:24 p.m. UTC | #1
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
Eric Benard April 8, 2013, 1:26 p.m. UTC | #2
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
Tom Rini April 8, 2013, 1:27 p.m. UTC | #3
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.
Eric Benard April 8, 2013, 1:28 p.m. UTC | #4
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
diff mbox

Patch

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