sailus: yeah, the driver should support at least the non-rotated sensor (plus the rotated one)
mchehab: Should, yes, but currently it only supports rotated operation.
I'll reply to the submitter.
ok, thanks!
please notice that there's another patch on his series, adding something else
https://lore.kernel.org/linux-media/20201028083847.GB32155@kozik-lap/
err... https://lore.kernel.org/linux-media/20201005151559.12003-1-krzk@kernel.org/
Ah, yes. I think I've missed that as the submitted had asked to ignore one of the patches. But not both.
Thanks.
sailus: this one also misses a review upstream: https://lore.kernel.org/linux-media/20210205163819.28439-1-jeanmichel.hautbois@ideasonboard.com/
sailus: can you take a look at these ccs kernel-doc warnings?
drivers/media/i2c/ccs/ccs-quirk.h:50: warning: Function parameter or member 'post_streamoff' not described in 'ccs_quirk'
drivers/media/i2c/ccs/ccs-quirk.h:50: warning: Function parameter or member 'flags' not described in 'ccs_quirk'
Should be easy for you to make a patch fixing those.
let me write this in a proper email, since there are also a bunch of ipu3 warnings.
hverkuil: You seem to have fixed the CCS issues earlier this month. :-)
It's just not yet in Mauro's tree.
I'll send a pull request shortly.
I did? Wow, I'm good :-)
I'll still take this patch to CCS tools since it only changes the file that's in there, not the rest.
@all: I'm granting Stanimir access to patchwork
I'll drop him an e-mail c/c you
mchehab: good
I also delegated the ~80 venus patches to him
Nice!
Number of pending patches per reviewer:
  LinuxTV community                                                     : 257
  Hans Verkuil <hansverk@cisco.com>                                     : 156
  Laurent Pinchart <laurent.pinchart@ideasonboard.com>                  : 136
  Stanimir Varbanov <stanimir.varbanov@linaro.org>                      : 85
  Sakari Ailus <sakari.ailus@iki.fi>                                    : 71
  Sean Young <sean@mess.org>                                            : 21
  Mauro Carvalho Chehab <mchehab@kernel.org>                            : 14
it sounds to me that the non-delegated queue is now on a better shape... 257 patches
I'll do a second round of checks on it, but at least the oldest stuff was almost all handled
pinchartl: not sure what to do with this: https://lore.kernel.org/linux-media/CAO5W59geLtP7kHJkW=ELusAcd9==cEqhsuzznUkfradTkxz+vQ@mail.gmail.com
Patchwork: http://patchwork.linuxtv.org/patch/69532/
there's an ubuntu issue related to it, with a proposed patch to uvc driver itself: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1827452
https://launchpadlibrarian.net/512840042/lp1827452-take2.patch
that patch doesn't seem correct
there can be multiple streaming interfaces in one device
there's a single control interface
a commit message with the rationale for the proposed fix would be useful
but in any case
I have a work in progress series that may fix this issue
if someone wanted to take over development, I can share the WIP patches