Hi Chiranjeevi,

On Thu, Aug 10, 2017 at 6:59 AM, Chiranjeevi Rapolu
<chiranjeevi.rap...@intel.com> wrote:
> Previously, pixel-rate/(pixels-per-line * lines-per-frame) was
> yielding incorrect frame timing for the user.
> OV sensor is using internal timing and this requires
> conversion (internal timing -> PPL) for correct HBLANK calculation.
> Now, change pixels-per-line domain from internal sensor clock to
> pixels domain. Set HBLANK read-only because fixed PPL is used for all
> resolutions. And, use more accurate link-frequency 422.4MHz instead of
> rounding down to 420MHz.
> Signed-off-by: Chiranjeevi Rapolu <chiranjeevi.rap...@intel.com>
> ---
> Changes in v2:
>         - Change subject to reflect frame timing info.
>         - Change OV5670_DEF_PPL so that it doesn't convey a register default
>           value. And, add more comments to it.
>  drivers/media/i2c/ov5670.c | 45 +++++++++++++++++++++++----------------------
>  1 file changed, 23 insertions(+), 22 deletions(-)

Okay, the numbers in this version finally make sense. Thanks for
figuring this out.

Reviewed-by: Tomasz Figa <tf...@chromium.org>

Best regards,

Reply via email to