[U-Boot] [PATCH v4 12/25] mtd: ensure MTD and NOR drivers are compiled with ENV_IS_IN_FLASH
Miquel Raynal
miquel.raynal at bootlin.com
Tue Dec 11 07:49:56 UTC 2018
Hi Tom,
Tom Rini <trini at 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 at 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 at 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
More information about the U-Boot
mailing list