Re: [PATCH v3 0/4] mtd: rawnand: Improve compile-test coverage

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

Hello,

Miquel Raynal <miquel.raynal@bootlin.com> wrote on Wed, 18 Jul 2018
23:40:10 +0200:

> Hi Boris,
> 
> Boris Brezillon <boris.brezillon@bootlin.com> wrote on Wed, 18 Jul 2018
> 10:56:48 +0200:
> 
> > Hello,
> > 
> > This is an attempt at adding "depends || COMPILE_TEST" to all NAND
> > drivers that have no compile-time dependencies on arch
> > features/headers.
> > 
> > This will hopefully help us (NAND/MTD maintainers) in detecting build
> > issues earlier. Unfortunately we still have a few drivers that can't
> > easily be modified to be arch independent.
> > 
> > Regards,
> > 
> > Boris
> > 
> > Changes in v2:
> > - Fix a few problems reported by kbuild robots and Stephen Rothwell
> > 
> > Changes in v3:
> > - Drop depends on !IA64 since Arnd has added {read,write}s{b,w,l}()
> >   accessors to this arch
> > - Make sure the orion driver compiles when __LINUX_ARM_ARCH__ is not
> >   defined
> > 
> > Boris Brezillon (4):
> >   mtd: rawnand: s3c2410: Allow selection of this driver when
> >     COMPILE_TEST=y
> >   mtd: rawnand: orion: Avoid direct inclusion of asm headers
> >   mtd: rawnand: orion: Handle cases where __LINUX_ARM_ARCH__ is not
> >     defined
> >   mtd: rawnand: orion: Allow selection of this driver when
> >     COMPILE_TEST=y
> > 
> >  drivers/mtd/nand/raw/Kconfig      | 6 ++++--
> >  drivers/mtd/nand/raw/orion_nand.c | 6 +++---
> >  2 files changed, 7 insertions(+), 5 deletions(-)
> >   
> 
> Series applied to nand/next, thanks for this work that will speed up
> our local tests.

As we received build robots complaints about both orion and s3c
drivers, I removed both patches to avoid build issues:

mtd: rawnand: s3c2410: Allow selection of this driver when COMPILE_TEST=y
mtd: rawnand: orion: Allow selection of this driver when COMPILE_TEST=y

Let's fix the issues and include them in the next release.

Thanks,
Miquèl