Re: [U-Boot] [PATCH v4 12/25] mtd: ensure MTD and NOR drivers are compiled with ENV_IS_IN_FLASH
Hi Tom, Tom Rini 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 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 > > > --- > > > 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 ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH v4 12/25] mtd: ensure MTD and NOR drivers are compiled with ENV_IS_IN_FLASH
On Mon, Dec 10, 2018 at 07:02:45PM +0100, Miquel Raynal wrote: > Hello, > > Miquel Raynal 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 > > --- > > 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. -- Tom signature.asc Description: PGP signature ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH v4 12/25] mtd: ensure MTD and NOR drivers are compiled with ENV_IS_IN_FLASH
Hello, Miquel Raynal 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 > --- > 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. Travis build for the same series without this patch is there: https://travis-ci.org/miquelraynal/u-boot/builds/466033792 Thanks, Miquèl ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot