[PATCH v6 2/5] media: platform: visconti: Add Toshiba Visconti Video Input Interface driver

[yuji2.ishikawa at toshiba.co.jp]

> -----Original Message-----
> From: Krzysztof Kozlowski <krzysztof.kozlowski at linaro.org>
> Sent: Monday, July 10, 2023 3:05 PM
> To: ishikawa yuji(石川 悠司 ○ＲＤＣ□ＡＩＴＣ○ＥＡ開)
> <yuji2.ishikawa at toshiba.co.jp>; lkp at intel.com; hverkuil at xs4all.nl;
> sakari.ailus at iki.fi; laurent.pinchart at ideasonboard.com; mchehab at kernel.org;
> robh+dt at kernel.org; krzk at kernel.org; conor+dt at kernel.org; iwamatsu
> nobuhiro(岩松 信洋 ○ＤＩＴＣ□ＤＩＴ○ＯＳＴ)
> <nobuhiro1.iwamatsu at toshiba.co.jp>
> Cc: llvm at lists.linux.dev; oe-kbuild-all at lists.linux.dev;
> linux-media at vger.kernel.org; devicetree at vger.kernel.org;
> linux-arm-kernel at lists.infradead.org; linux-kernel at vger.kernel.org
> Subject: Re: [PATCH v6 2/5] media: platform: visconti: Add Toshiba Visconti
> Video Input Interface driver
> 
> On 10/07/2023 02:16, yuji2.ishikawa at toshiba.co.jp wrote:
> >> -----Original Message-----
> >> From: kernel test robot <lkp at intel.com>
> >> Sent: Tuesday, June 20, 2023 9:58 PM
> >> To: ishikawa yuji(石川 悠司 ○ＲＤＣ□ＡＩＴＣ○ＥＡ開)
> >> <yuji2.ishikawa at toshiba.co.jp>; Hans Verkuil <hverkuil at xs4all.nl>;
> >> Sakari Ailus <sakari.ailus at iki.fi>; Laurent Pinchart
> >> <laurent.pinchart at ideasonboard.com>; Mauro Carvalho Chehab
> >> <mchehab at kernel.org>; Rob Herring <robh+dt at kernel.org>; Krzysztof
> >> Kozlowski <krzk at kernel.org>; Conor Dooley <conor+dt at kernel.org>;
> >> iwamatsu nobuhiro(岩松 信洋 ○ＤＩＴＣ□ＤＩＴ○ＯＳＴ)
> >> <nobuhiro1.iwamatsu at toshiba.co.jp>
> >> Cc: llvm at lists.linux.dev; oe-kbuild-all at lists.linux.dev;
> >> linux-media at vger.kernel.org; devicetree at vger.kernel.org;
> >> linux-arm-kernel at lists.infradead.org; linux-kernel at vger.kernel.org
> >> Subject: Re: [PATCH v6 2/5] media: platform: visconti: Add Toshiba
> >> Visconti Video Input Interface driver
> >>
> >> Hi Yuji,
> >>
> >> kernel test robot noticed the following build errors:
> >>
> >> [auto build test ERROR on media-tree/master] [also build test ERROR
> >> on linus/master sailus-media-tree/streams v6.4-rc7 next-20230620] [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#_base_tree_information]
> >>
> >> url:
> >> https://github.com/intel-lab-lkp/linux/commits/Yuji-Ishikawa/dt-bindi
> >> ngs-me
> >> dia-platform-visconti-Add-Toshiba-Visconti-Video-Input-Interface-bind
> >> ings/20
> >> 230620-120839
> >> base:   git://linuxtv.org/media_tree.git master
> >> patch link:
> >> https://lore.kernel.org/r/20230620031111.3776-3-yuji2.ishikawa%40tosh
> >> iba.co.j
> >> p
> >> patch subject: [PATCH v6 2/5] media: platform: visconti: Add Toshiba
> >> Visconti Video Input Interface driver
> >> config: x86_64-buildonly-randconfig-r001-20230620
> >> (https://download.01.org/0day-ci/archive/20230620/202306202006.FUjZfd
> >> 9O-l
> >> kp at intel.com/config)
> >> compiler: clang version 15.0.7
> >> (https://github.com/llvm/llvm-project.git
> >> 8dfdcc7b7bf66834a761bd8de445840ef68e4d1a)
> >> reproduce:
> >> (https://download.01.org/0day-ci/archive/20230620/202306202006.FUjZfd
> >> 9O-l
> >> kp at intel.com/reproduce)
> >>
> >> If you fix the issue in a separate patch/commit (i.e. not just a new
> >> version of the same patch/commit), kindly add following tags
> >> | Reported-by: kernel test robot <lkp at intel.com>
> >> | Closes:
> >> | https://lore.kernel.org/oe-kbuild-all/202306202006.FUjZfd9O-lkp@intel.
> >> | com/
> >>
> >> All errors (new ones prefixed by >>):
> >>
> >>    In file included from <built-in>:1:
> >>    In file included from ./usr/include/linux/visconti_viif.h:12:
> >>>> usr/include/linux/videodev2.h:2464:20: error: field has incomplete
> >>>> type
> >> 'struct timespec'
> >>            struct timespec                 timestamp;
> >>                                            ^
> >>    usr/include/linux/videodev2.h:2464:9: note: forward declaration of
> >> 'struct timespec'
> >>            struct timespec                 timestamp;
> >>                   ^
> >>    1 error generated.
> >>
> >> --
> >> 0-DAY CI Kernel Test Service
> >> https://github.com/intel/lkp-tests/wiki
> >
> > I could not reproduce the error with the steps had been specified.
> >
> > * archived config file didn't compile Visconti VIIF driver (at least,
> > COMPILE_TEST is needed)
> 
> That's not true. The config has enabled CONFIG_VIDEO_VISCONTI_VIIF, so your
> reproduction steps were not correct (e.g. wrong arch, compiler etc).
> 
> > * the error message didn't appear even if I add some options to config file.
> > * Environment:
> >   * OS: Ubuntu 22.04
> >   * compiler: Ubuntu clang version 15.07
> 
> Different compilers can have have or not have warnings, but it does not allow to
> ignore this warning. You can analyze the code even without reproducing it, if the
> reproduction is a problem.
> 
> Best regards,
> Krzysztof

Thank you for your advice, I finally reproduced the error message.
The message is issued by HDRTEST process (not by CC or AR), therefore, we don't need to set Visconti related options to config.
The fix will be included in v7 patch.

Best regards,
Yuji Ishikawa