Hi Miquel, I love your patch! Yet something to improve: [auto build test ERROR on broonie-spi/for-next] [also build test ERROR on mtd/nand/next mtd/mtd/next mtd/mtd/fixes v5.16-rc5 next-20211215] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as documented in https://git-scm.com/docs/git-format-patch] url: https://github.com/0day-ci/linux/commits/Miquel-Raynal/External-ECC-engines-Macronix-support/20211216-191821 base: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.git for-next config: hexagon-randconfig-r045-20211216 (https://download.01.org/0day-ci/archive/20211217/202112170259.JPoGT6na-lkp@xxxxxxxxx/config) compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project dd245bab9fbb364faa1581e4f92ba3119a872fba) reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # https://github.com/0day-ci/linux/commit/bf16b56f7a0cc5aa237129a6b8bd216dc2632c8b git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Miquel-Raynal/External-ECC-engines-Macronix-support/20211216-191821 git checkout bf16b56f7a0cc5aa237129a6b8bd216dc2632c8b # save the config file to linux build tree mkdir build_dir COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=hexagon SHELL=/bin/bash If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp@xxxxxxxxx> All errors (new ones prefixed by >>): >> ld.lld: error: duplicate symbol: spi_mem_no_caps >>> defined at core.c >>> mtd/nand/spi/core.o:(spi_mem_no_caps) in archive drivers/built-in.a >>> defined at gigadevice.c >>> mtd/nand/spi/gigadevice.o:(.rodata+0x35C) in archive drivers/built-in.a -- >> ld.lld: error: duplicate symbol: spi_mem_no_caps >>> defined at core.c >>> mtd/nand/spi/core.o:(spi_mem_no_caps) in archive drivers/built-in.a >>> defined at macronix.c >>> mtd/nand/spi/macronix.o:(.rodata+0x83C) in archive drivers/built-in.a -- >> ld.lld: error: duplicate symbol: spi_mem_no_caps >>> defined at core.c >>> mtd/nand/spi/core.o:(spi_mem_no_caps) in archive drivers/built-in.a >>> defined at sysfs.c >>> mtd/spi-nor/sysfs.o:(.rodata+0x14) in archive drivers/built-in.a -- >> ld.lld: error: duplicate symbol: spi_mem_no_caps >>> defined at core.c >>> mtd/nand/spi/core.o:(spi_mem_no_caps) in archive drivers/built-in.a >>> defined at atmel.c >>> mtd/spi-nor/atmel.o:(.rodata+0x370) in archive drivers/built-in.a -- >> ld.lld: error: duplicate symbol: spi_mem_no_caps >>> defined at core.c >>> mtd/nand/spi/core.o:(spi_mem_no_caps) in archive drivers/built-in.a >>> defined at catalyst.c >>> mtd/spi-nor/catalyst.o:(.rodata+0x178) in archive drivers/built-in.a -- >> ld.lld: error: duplicate symbol: spi_mem_no_caps >>> defined at core.c >>> mtd/nand/spi/core.o:(spi_mem_no_caps) in archive drivers/built-in.a >>> defined at eon.c >>> mtd/spi-nor/eon.o:(.rodata+0x370) in archive drivers/built-in.a -- >> ld.lld: error: duplicate symbol: spi_mem_no_caps >>> defined at core.c >>> mtd/nand/spi/core.o:(spi_mem_no_caps) in archive drivers/built-in.a >>> defined at esmt.c >>> mtd/spi-nor/esmt.o:(.rodata+0xE8) in archive drivers/built-in.a -- >> ld.lld: error: duplicate symbol: spi_mem_no_caps >>> defined at core.c >>> mtd/nand/spi/core.o:(spi_mem_no_caps) in archive drivers/built-in.a >>> defined at everspin.c >>> mtd/spi-nor/everspin.o:(.rodata+0x130) in archive drivers/built-in.a -- >> ld.lld: error: duplicate symbol: spi_mem_no_caps >>> defined at core.c >>> mtd/nand/spi/core.o:(spi_mem_no_caps) in archive drivers/built-in.a >>> defined at fujitsu.c >>> mtd/spi-nor/fujitsu.o:(.rodata+0x58) in archive drivers/built-in.a -- >> ld.lld: error: duplicate symbol: spi_mem_no_caps >>> defined at core.c >>> mtd/nand/spi/core.o:(spi_mem_no_caps) in archive drivers/built-in.a >>> defined at gigadevice.c >>> mtd/spi-nor/gigadevice.o:(.rodata+0x250) in archive drivers/built-in.a -- >> ld.lld: error: duplicate symbol: spi_mem_no_caps >>> defined at core.c >>> mtd/nand/spi/core.o:(spi_mem_no_caps) in archive drivers/built-in.a >>> defined at intel.c >>> mtd/spi-nor/intel.o:(.rodata+0xE8) in archive drivers/built-in.a .. --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx