From: Borut Lampe <nude.bird@gmail.com>
To: Jacopo Mondi <jacopo@jmondi.org>
Cc: linux-media@vger.kernel.org
Subject: Re: OV5640 720p@60fps
Date: Wed, 6 Oct 2021 14:02:29 +0200 [thread overview]
Message-ID: <CAHfbzRsO65KNfHXTF-Psn3Hq0qkPoAz2ZgFSjHDS5suATnCiaw@mail.gmail.com> (raw)
In-Reply-To: <20210928160249.xcgpay5x2kaccsdi@uno.localdomain>
Jacopo,
Thanks for the info. I will look into it and get back to you if I have
any additional questions, hopefully I can get it working.
Kind regards,
V V tor., 28. sep. 2021 ob 18:02 je oseba Jacopo Mondi
<jacopo@jmondi.org> napisala:
>
> Hi Borut,
>
> On Tue, Sep 28, 2021 at 04:31:11PM +0200, Borut Lampe wrote:
> > Hi Jacopo,
> >
> > I am using MIPI CSI-2 with the Qualcomm Camera Subsystem driver on a
> > custom 820 dragonboard.
>
> 2 lanes, right ?
>
> I don't have an easy solution but I can provide a bit of context.
>
> It started with
> aa2882481cad ("media: ov5640: Adjust the clock based on the expected rate")
>
> where we moved to compute the clock tree at runtime instead of relying
> on the hardcoded configuration in the register tables:
>
> c14d107e7417 ("media: ov5640: Remove the clocks registers initialization")
>
> From there the ov5640_set_mipi_pclk() went in and we never really got
> to a version that works for all the modes supported by the sensor.
>
> There were a few attempts to improve it, Tomi has found a discrepancy
> in the H/V TOT values, I tried to re-implement the CSI-2 clock tree
> calculation, but we never get to a set of results that was globally
> satisfying and I guess we all dropped the ball
>
> You can find more about that here:
> https://patchwork.linuxtv.org/project/linux-media/cover/20201028225706.110078-1-jacopo+renesas@jmondi.org/
>
> And if you're willing to test those patches and see if the situation
> improves it would be very useful.
>
> Thanks and good luck!
>
> >
> > Kind regards,
> > Borut
> >
> > V V tor., 28. sep. 2021 ob 11:49 je oseba Jacopo Mondi
> > <jacopo@jmondi.org> napisala:
> > >
> > > Hi Borut,
> > > which kind of setup are you working with, parallel or CSI-2 ?
> > >
> > > On Thu, Sep 23, 2021 at 12:37:01PM +0200, Borut Lampe wrote:
> > > > Hi,
> > > >
> > > > I noticed that for now only VGA resolution supports 60fps even though
> > > > the documentation says that 60fps is possible also for 1280x720.
> > > > Is there a reason this hasn't been implemented yet (e.g hardware
> > > > restraints)? Are there any plans to implement this? Can anyone give me
> > > > some pointers on what would have to be changed in the ov5640 driver to
> > > > achieve this (starting point would be linaro kernel qcomlt-4.14)?
> > > >
> > > > Kind regards,
> > > > Borut
prev parent reply other threads:[~2021-10-06 12:02 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-09-23 10:37 OV5640 720p@60fps Borut Lampe
2021-09-28 9:50 ` Jacopo Mondi
2021-09-28 14:31 ` Borut Lampe
2021-09-28 16:02 ` Jacopo Mondi
2021-10-06 12:02 ` Borut Lampe [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAHfbzRsO65KNfHXTF-Psn3Hq0qkPoAz2ZgFSjHDS5suATnCiaw@mail.gmail.com \
--to=nude.bird@gmail.com \
--cc=jacopo@jmondi.org \
--cc=linux-media@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).