[U-Boot] [PATCH v4 12/25] mtd: ensure MTD and NOR drivers are compiled with ENV_IS_IN_FLASH

[Miquel Raynal <miquel.raynal@bootlin.com>]

Hi Tom,

Tom Rini <trini@konsulko.com> wrote on Mon, 10 Dec 2018 13:10:11 -0500:

> On Mon, Dec 10, 2018 at 07:02:45PM +0100, Miquel Raynal wrote:
> > Hello,
> > 
> > Miquel Raynal <miquel.raynal@bootlin.com> wrote on Sun,  9 Dec 2018
> > 19:07:34 +0100:
> >   
> > > MTD and NOR flash support must be enabled when the environment is in
> > > NOR.
> > > 
> > > Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
> > > ---
> > >  configs/armadillo-800eva_defconfig | 2 ++
> > >  1 file changed, 2 insertions(+)
> > > 
> > > diff --git a/configs/armadillo-800eva_defconfig b/configs/armadillo-800eva_defconfig
> > > index b1d923c069..72758884b4 100644
> > > --- a/configs/armadillo-800eva_defconfig
> > > +++ b/configs/armadillo-800eva_defconfig
> > > @@ -32,6 +32,8 @@ CONFIG_CMD_PING=y
> > >  # CONFIG_CMD_MISC is not set
> > >  CONFIG_ENV_IS_IN_FLASH=y
> > >  # CONFIG_MMC is not set
> > > +CONFIG_MTD=y
> > > +CONFIG_MTD_NOR_FLASH=y
> > >  CONFIG_SH_ETHER=y
> > >  CONFIG_SCIF_CONSOLE=y
> > >  CONFIG_OF_LIBFDT=y  
> > 
> > This change triggered a build failure. This is because, despite
> > declaring an in-flash environment, there was absolutely no MTD driver
> > compiled-in. No driver named after 'flash' or 'nor' or 'mtd' was
> > compiled.
> > 
> > The fix for this issue (the only one reported by Travis for this
> > version of the series) is to just drop this patch. As it has absolutely
> > no dependency and only impacts a single defconfig, I would rather
> > prefer not re-send 24 identical patches in a v5 unless there are
> > comments that I must address.  
> 
> Thanks.  Do you have a patchwork account?  If so you should be able to
> manage the status of your own at least and marking it as "Rejected" or
> "Not Applicable" or anything like that will mean it won't be picked up
> by accident.  If not, I'll get to it soon.
> 

I do have an account on ozlabs' patchwork. I went there to discard this
patch but I saw that someone (probably you) already did it.


Thanks,
Miquèl