<!-- Some styling for better description lists --><style type='text/css'>dt { font-weight: bold;float: left;display:inline;margin-right: 1em} dd { display:block; margin-left: 2em}</style>

   sailus: I'd suggest to moving forward with these, and address the license header problem later.
   ***: ChanServ sets mode: +v hverkuil
   mchehab: <u>sailus</u>: ok, I'll proceed handling it, fixing the license header on a followup patch when applying it
   <br> (assuming that the license is already there somehow - otherwise, the doc file author will need to submit the patch)
   <br> and we would need to reject the PR
   <br> this one doesn't sound a false positive to me, but easy to fix while applying it:
   <br> CHECK:SPACING: spaces preferred around that '/' (ctx:VxV)
   <br> #711: FILE: drivers/media/i2c/ov5675.c:619:
   <br> +			       OV5675_EXPOSURE_MAX_MARGIN)/2;
   <br> <u>sailus</u>: that license header issue is actually a false positive :-)
   <br> SPDX currently laks proper support for GFDL
   <br> there's an open issue there related to it
   <br> https://github.com/spdx/license-list-XML/issues/686#
   sailus: How did you get that warnings from checkpatck.pl btw.?
   <br> Mine doesn't produce it.
   mchehab: --strict
   sailus: Right. I'll add that in the future.
   <br> Thanks!
   mchehab: anytime. PR applied.
   sailus: Hello!
   mchehab: hi all
   hverkuil: hi
   pinchartl: hello
   mchehab: anything for today's discussion?
   hverkuil: Not from me.
   pinchartl: not from me either
   mchehab: sailus, syoung: anything for today's meeting?
   sailus: Hmm. Nothing today.
   mchehab: from my side: the builder.linuxtv.org is now in production... it is building all projects at linuxtv.org, except for ZBar (with uses TravisCI already)
   <br> please let me know if you find any issues
   <br> in the case of the patchwork-based builder, it is now verifying also GPG signatures
   <br> we got an issue yesterday related to some pull requests that were using ssh:// url
   <br> the builder is independent of the main infra, and doesn't have any sort of account nor any other type of trustee with linuxtv.org
   <br> (I opted to do that, in order to make it safer)
   sailus: Sounds good.
   mchehab: please let me know if you also want to place some other thing at the builder
   pinchartl: does that mean it compiles libcamera too ?
   mchehab: yes
   <br> https://builder.linuxtv.org/job/libcamera/
   pinchartl: what distribution does it build it in ?
   mchehab: Debian 10
   pinchartl: that should work fine
   <br> if you want to extend build coverage, you can add the -Dandroid=true option to meson
   mchehab: when calling meson or ninja?
   pinchartl: meson
   mchehab: https://builder.linuxtv.org/job/libcamera/3/console
   <br> failed
   <br> 09:15:11 ../src/android/camera_hal_manager.cpp:89:9: error: comparison of unsigned expression &lt; 0 is always false [-Werror=type-limits]
   <br> as I said before, building with -Werror by default is a very bad idea... it is very likely to fail on a different distro than the one you're using
   <br> is there a -D option (or whatever other option) to disable -Werror?
   pinchartl: weird, I thought we had fixed that
   mchehab: <u>pinchartl</u>: how do you want me to proceed? keep the -Dandroid=true there, remove it or add some other build flag?
   pinchartl: let me push a fix :-)
   mchehab: ok, so I'll keep it as-is
   <br> the build failures (and failure -&gt; normal transitions) will be emailed to linux-media
   <br> anything else?
   syoung: hi
   <br> <u>mchehab</u>: I was thinking this https://patchwork.linuxtv.org/patch/57608/ is a bad idea. What do you think?
   mchehab: (btw, we may need to install some packages at the builder machine in order to fully build some packages - for example, right now v4l-utils doesn't detect SDL support)
   <br> <u>syoung</u>: that's a good point...
   <br> yeah, I think that using devm_*() in general is a bad idea
   <br> yet, it might have some valid usecase
   <br> for some platform-specific driver
   <br> so, I won't object myself if you want to push for it - or if you want to deny such new ABI calls
   <br> but I'm against using those for USB devices
   syoung: that makes sense. I've been checking rc drivers that use usb devm_* :)
   pinchartl: <u>mchehab</u>: libcamera compilation issue fixed
   mchehab: ok, starting a new build
   <br> ok, passed now
   <br> I fixed SDL dependencies for fully v4l-utils build
   <br> now, just xawtv4 doesn't build everything
   <br> (probably not worth fixing it)
   syoung: does it build v4l-utils with HAVE_BPF?
   mchehab: no, it just builds with make
   <br> the current build script is pretty simple:
   <br> ./bootstrap.sh
   <br> ./configure
   <br> make
   <br> does it need any parameter to build BPF?
   syoung: no parameter, it needs some packages. The last line of the output of ./configure should be:
   <br> BPF IR Decoders:           : yes