<div class="gmail_quote">On Wed, Sep 29, 2010 at 11:24 AM, Jonathan Martens <span dir="ltr">&lt;<a href="mailto:jonathan@snetram.nl">jonathan@snetram.nl</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">On 29-9-2010 19:53, <a href="mailto:mythtv@cvs.mythtv.org" target="_blank">mythtv@cvs.mythtv.org</a> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
       Author: danielk<br>
         Date: 2010-09-29 17:53:41 +0000 (Wed, 29 Sep 2010)<br>
New Revision: 26576<br>
    Changeset: <a href="http://svn.mythtv.org/trac/changeset/26576" target="_blank">http://svn.mythtv.org/trac/changeset/26576</a><br>
<br>
Log:<br>
<br>
mythtv-rec: Gavin Hurlbut&#39;s hvr2250 changes.. Note: this does not work unless you also comment out<br>
<br>
Modified:<br>
<br>
    branches/mythtv-rec/mythtv/libs/libmythtv/mpegrecorder.cpp<br>
</blockquote>
<br></div>
Daniel, are we missing some information here perhaps?<br><br></blockquote><div><br></div><div>Hehe.  I think I have found the one last niggling issue with this driver, and have sent a (very simple) patch to the driver&#39;s author that should fix it completely, although there will be no VBI support for now as it does it in a different way than mpegrecorder expects.  This shouldn&#39;t be that much of an issue as we can support it later on.</div>
<div><br></div><div>My plan is to have Daniel (and others) look over my changes again tonight after I tidy it up, etc, and then we should have this ready to go for the time being.</div><div><br></div><div>Ongoing, it would be good to use VIDIOC_QUERYCTL to ask the driver what controls it supports, so we can intelligently set only the supported controls.  This could drastically simplify the setup as then we don&#39;t care which driver it is, we just use what it says it can handle.  Likewise, we can likely use VIDIOC_G_SLICED_VBI_CAP to see if the driver does VBI in the IVTV way or not.  This is all post 0.24, I would imagine.  Rather invasive to be committing that to trunk during feature freeze.</div>
</div>