Re: [PATCH v2 00/23] media: ov5640: Rework the clock tree programming for MIPI

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Tomi,

On Thu, Feb 10, 2022 at 03:00:21PM +0200, Tomi Valkeinen wrote:
> On 10/02/2022 13:04, Jacopo Mondi wrote:
> > v1:
> > https://patchwork.linuxtv.org/project/linux-media/list/?series=7249
>
> You could rather point to lore.kernel.org, so that the intro letter and the
> discussions are also visible.
>
> > A branch for testing based on the most recent media-master is available at
> > https://git.sr.ht/~jmondi_/linux #jmondi/media-master/ov5640-v2
> >
> > If anyone with a DVP setup could verify I have not broken their use case
> > I would very much appreciate that :)
> >
> > v1 -> v2:
> > - rework the modes definition to process the full pixel array
> > - rework get_selection to report the correct BOUND and DEFAULT targets
> > - implement init_cfg
> > - minor style changes as suggested by Laurent
> > - test with 1 data lane
>
> Very nice! I tested this on TI's DRA76 EVM (CSI-2). UYVY and RGB565, with
> the following resolutions: 160 120, 176 144, 320 240, 640 480, 720 480, 720
> 576, 1024 768, 1280 720, 1920 1080.
>
> All work. The only issue I saw was that with 1024x768 and 1280x720 there's a
> thin vertical purple column on the right edge.

Thanks for spotting.

I've sent a v2.1 version of "media: ov5640: Rework analog crop rectangles"
where I have re-introduced the crop rectangles as they were defined in
the register tables for 1024x768 and 1280x720, which are closer to
what the datasheet suggests. The vertical purple column seems to be
gone now.

Thanks
   j
>
>  Tomi



[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux