Re: [v5 12/14] iio: imu: add BNO055 serdev driver

[Andrea Merello]

Il giorno mer 27 apr 2022 alle ore 15:41 Andy Shevchenko
<andy.shevchenko@xxxxxxxxx> ha scritto:
>
> On Wed, Apr 27, 2022 at 10:11 AM kernel test robot <lkp@xxxxxxxxx> wrote:
> >
> > Hi Andrea,
> >
> > Thank you for the patch! Yet something to improve:
> >
> > [auto build test ERROR on jic23-iio/togreg]
> > [also build test ERROR on linus/master linux/master v5.18-rc4 next-20220426]
> > [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/intel-lab-lkp/linux/commits/Andrea-Merello/Add-support-for-Bosch-BNO055-IMU/20220426-212132
> > base:   https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git togreg
> > config: arc-allyesconfig (https://download.01.org/0day-ci/archive/20220427/202204271554.EiuIRNPI-lkp@xxxxxxxxx/config)
> > compiler: arceb-elf-gcc (GCC) 11.3.0
> > 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/intel-lab-lkp/linux/commit/675ca9cd13af45cc5943dd15caad5e866fd7c971
> >         git remote add linux-review https://github.com/intel-lab-lkp/linux
> >         git fetch --no-tags linux-review Andrea-Merello/Add-support-for-Bosch-BNO055-IMU/20220426-212132
> >         git checkout 675ca9cd13af45cc5943dd15caad5e866fd7c971
> >         # save the config file
> >         mkdir build_dir && cp config build_dir/.config
> >         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.3.0 make.cross W=1 O=build_dir ARCH=arc 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 >>):
> >
> > >> make[5]: *** No rule to make target 'drivers/iio/imu/bno055/bno055_ser_trace.o', needed by 'drivers/iio/imu/bno055/built-in.a'.
> >    make[5]: Target '__build' not remade because of errors.
>
> You need to add a C-file with the only line
>
> #include <..._trace.h>
>
> And drop that include from the _core.c.

Hum, I'm a bit confused here: the bno055_ser_core.c file explicitly
looks for that tracepoints (e.g. it calls trace_send_chunks() and
friends); dropping the include prevents build here because there would
be no definition for those tracepoints.

There is already a C file bno055_ser_trace.c that just contains the
said include and it defines CREATE_TRACE_POINTS; I see other drivers
like dwc3 do the same..

But my problem is that I cannot reproduce the issue found by the bot:
the compiler that is downloaded by the script doesn't run on my build
box because it wants a newer libc (I was hoping that those compilers
were statically linked, but they aren't), while any other attempt I
did with other older compilers resulted in either successful build or
failed with other weird, apparently unrelated, errors about relocation
issues (of course I tried with the arch and config used by the build
bot).

Is there any build farm publicly available or something like that?

> --
> With Best Regards,
> Andy Shevchenko