Le Wed, Nov 03, 2021 at 04:21:02PM +0100, Hans Verkuil a écrit : > Hi Corentin, > > On 26/10/2021 21:34, Corentin Labbe wrote: > > Hello > > > > The main change of this serie is to fusion all zoran related modules in > > one. > > This fixes the load order problem when everything is built-in. > > I've been testing this series, and while the module load/unload is now working, > I'm running into a lot of other v4l2 compliance issues. > > I've fixed various issues in some follow-up patches available in my tree: > > https://git.linuxtv.org/hverkuil/media_tree.git/log/?h=zoran > > At least some of the worst offenders are now resolved. Note that the patch > dropping read/write support relies on this patch: > > https://patchwork.linuxtv.org/project/linux-media/patch/4f89b139-13b7-eee6-9662-996626b778b0@xxxxxxxxx/ Hello My test branch already included your "zoran: fix various V4L2 compliance errors" I have quickly checked other patch and I am ok with them. I will add and test with them. > > But there is one really major bug that makes me hesitant to merge this: > > This works: > > v4l2-ctl -v pixelformat=MJPG,width=768,height=576 > v4l2-ctl --stream-mmap > > This fails: > > v4l2-ctl -v pixelformat=MJPG,width=768,height=288 > v4l2-ctl --stream-mmap > > It's an immediate lock up with nothing to indicate what is wrong. > As soon as the height is 288 or less, this happens. > > Both with my DC30 and DC30D. Just for curiosity, what is the difference between thoses two ? > > Do you see the same? Any idea what is going on? I would feel much happier > if this is fixed. > > Note that the same problem is present without this patch series, so it's > been there for some time. > I will start on digging this problem and add thoses commands to my CI. And I know there are a huge quantity of problem since origins. A simple example is that just setting MJPEG as default input format does not work. But since it is not related to my serie, can you please merge it. Thanks Regards
