[U-Boot] [PATCH 3/7] sunxi: Enable UBI and NAND support

[Boris Brezillon]

On Wed, 9 Nov 2016 15:32:37 +0100
Maxime Ripard <maxime.ripard at free-electrons.com> wrote:

> Hi Boris,
> 
> On Tue, Nov 08, 2016 at 05:27:48PM +0100, Boris Brezillon wrote:
> > On Tue,  8 Nov 2016 17:21:13 +0100
> > Maxime Ripard <maxime.ripard at free-electrons.com> wrote:
> >   
> > > From: Hans de Goede <hdegoede at redhat.com>
> > > 
> > > Enable the NAND and UBI support in the configuration header so that we can
> > > (finally) use it.
> > > 
> > > Signed-off-by: Hans de Goede <hdegoede at redhat.com>
> > > Signed-off-by: Maxime Ripard <maxime.ripard at free-electrons.com>
> > > ---
> > >  include/configs/sunxi-common.h | 26 ++++++++++++++++++++++----
> > >  1 file changed, 22 insertions(+), 4 deletions(-)
> > > 
> > > diff --git a/include/configs/sunxi-common.h b/include/configs/sunxi-common.h
> > > index 8363414828fa..1733767ba53b 100644
> > > --- a/include/configs/sunxi-common.h
> > > +++ b/include/configs/sunxi-common.h
> > > @@ -129,9 +129,23 @@
> > >  #define CONFIG_SERIAL_TAG
> > >  
> > >  #ifdef CONFIG_NAND_SUNXI
> > > +#define CONFIG_SYS_NAND_U_BOOT_OFFS	(8 << 20) /* 8 MiB */  
> > 
> > Can we make this configurable through Kconfig?  
> 
> Is there a need for that?

Yes, I think so, since each board has a different kind of NAND (with
possibly different eraseblock size) and I'm not sure this is always
acceptable to lose 8MiB of storage for the SPL images (especially for
small SLC NANDs).

> AFAIK, everyone in U-Boot defines it there
> (and converting everyone to Kconfig for all the CONFIG_SYS_NAND
> options is going to be painful).

It already exists: enable the SYS_NAND_U_BOOT_LOCATIONS Kconfig
option, and you should see the SYS_NAND_U_BOOT_OFFS and
SYS_NAND_U_BOOT_OFFS_REDUND ones.

> 
> Thanks!
> Maxime
>