public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Hans de Goede <hdegoede@redhat.com>
To: Linux Media Mailing List <linux-media@vger.kernel.org>
Subject: RFC: howto handle driver changes which require libv4l > x.y ?
Date: Tue, 07 Jul 2009 15:41:49 +0200	[thread overview]
Message-ID: <4A53509D.8060503@redhat.com> (raw)

Hi All,

So recently I've hit 2 issues where kernel side fixes need
to go hand in hand with libv4l updates to not cause regressions.

First lets discuss the 2 cases:
1) The pac207 driver currently limits the framerate (and thus
    the minimum exposure time) because at higher framerate the
    cam starts using a higher compression and we could not
    decompress this. Thanks to Bertrik Sikken we can now handle
    the higher decompression.

    So no I really want to enable the higher framerates as those
    are needed to make the cam work properly in full daylight.

    But if I do this, things will regress for people with an
    older libv4l, as that won't be able to decompress the frames

2) Several zc3xxx cams have a timing issue between the bridge and
    the sensor (the windows drivers have the same issue) which
    makes them do only 320x236 instead of 320x240. Currently
    we report their resolution to userspace as 320x240, leading to
    a bar of noise at the bottom of the screen.

    The fix here obviously is to report the real effective resoltion
    to userspace, but this will cause regressions for apps which blindly
    assume 320x240 is available (such as skype). The latest libv4l will
    make the apps happy again by giving them 320x240 by adding small
    black borders.


Now I see 2 solutions here:

a) Just make the changes, seen from the kernel side these are most
    certainly bugfixes. I tend towards this for case 2)

b) Come up with an API to tell the libv4l version to the kernel and
    make these changes in the drivers conditional on the libv4l version


So this is my dilemma, your input is greatly appreciated.

Regards,

Hans

             reply	other threads:[~2009-07-07 13:41 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-07-07 13:41 Hans de Goede [this message]
2009-07-07 13:55 ` RFC: howto handle driver changes which require libv4l > x.y ? Erik Andrén
2009-07-07 14:35   ` Mauro Carvalho Chehab
2009-07-09  9:05     ` Hans de Goede

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=4A53509D.8060503@redhat.com \
    --to=hdegoede@redhat.com \
    --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