linux-kernel - Re: i.MX6 MIPI-CSI2 OV5640 Camera testing on Mainline Linux

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20181106082727.GW15991@w540>
Date:   Tue, 6 Nov 2018 09:27:27 +0100
From:   jacopo mondi <jacopo@...ndi.org>
To:     Adam Ford <aford173@...il.com>
Cc:     Fabio Estevam <festevam@...il.com>, steve_longerbeam@...tor.com,
        Jagan Teki <jagan@...rulasolutions.com>,
        p.zabel@...gutronix.de, Fabio Estevam <fabio.estevam@....com>,
        gstreamer-devel@...ts.freedesktop.org, linux-media@...r.kernel.org,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: i.MX6 MIPI-CSI2 OV5640 Camera testing on Mainline Linux

Hi Adam,

On Mon, Nov 05, 2018 at 04:32:33PM -0600, Adam Ford wrote:
> On Mon, Oct 29, 2018 at 8:49 AM Fabio Estevam <festevam@...il.com> wrote:
> >
> > Hi Adam,
> >
> > On Sun, Oct 28, 2018 at 3:58 PM Adam Ford <aford173@...il.com> wrote:
> >
> > > Does anyone know when the media branch get's merged into the mainline
> > > kernel?  I assume we're in the merge window with 4.19 just having been
> > > released.  Once these have been merged into the mainline, I'll go
> > > through and start requesting they get pulled into 4.19 and/or 4.14
> > > if/when appropriate.
> >
> > This should happen in 4.20-rc1, which will probably be out next  Sunday.
> I sent an e-mail to stable with a list of a variety of patches for the
> ov5640 to be applied to 4.19.y  So far all looks pretty good, but I
> think I found on minor bug:

Thanks!

>
> If I attempt to change just the resolution, it doesn't take.
>
> Initial read
> media-ctl --get-v4l2 "'ov5640 2-0010':0"
> [fmt:UYVY8_2X8/640x480@...0 field:none colorspace:srgb xfer:srgb
> ycbcr:601 quantization:full-range]
>
> Change resolution
> # media-ctl --set-v4l2 "'ov5640 2-0010':0[fmt:UYVY2X8/720x480 field:none]"
>
> Read it back
> # media-ctl --get-v4l2 "'ov5640 2-0010':0"
> [fmt:UYVY8_2X8/640x480@...0 field:none colorspace:srgb xfer:srgb
> ycbcr:601 quantization:full-range]
>
> However, if I change the resolution AND the format to something other
> than UYVY2x8, the resolution changes. I can then change the format
> back to UYVY and capture and stream video at 1080p and 720x480.
> I can work around this, but I thought I'd mention it.  I was trying to
> figure out a patch to apply to the mailing list myself, but I wasn't
> able to fix it quickly.

Thanks, you are right, this is a known issue :(

There have been patches to address this issue floating around the
mailing list, none of them has yet made it up to inclusion.

Sam has sent a patch to fix this:
https://patchwork.linuxtv.org/patch/52489/

Hugues had something similar I cannot find, if I'm not wrong, and I
carry myself the patch I posted in the patchwork discussion in my
trees.

Feel free to go through the media list and comment. One more report on
this would help to increase chances to have one of those patches
re-sent and then included.

Also, bug reports are always very welcome, but to maximize the chances
they receive the required attention, could you make a new email thread
next time and drop this one please?

Thanks
  j

>
> adam
> adam

Download attachment "signature.asc" of type "application/pgp-signature" (820 bytes)