[MPlayer-dev-eng] [PATCH] fix for v4l1 driver

Vladimir Voroshilov voroshil at gmail.com
Sun Apr 29 01:45:50 CEST 2007


2007/4/29, Trent Piepho <xyzzy at speakeasy.org>:
>
> On Sat, 28 Apr 2007, Diego Biurrun wrote:
>
> > On Sat, Apr 28, 2007 at 03:54:06AM -0700, Trent Piepho wrote:
> > > This patch makes the v4l1 driver work properly.  There were some bugs
> in
> > > the kernel v4l1 compat system, but mplayer had some bugs too.  This
> patch
> > > applies on top of the cosmetic patch I sent earlier.
> >
> > I reindented the file, please resend an updated patch.
>
> Why couldn't you just apply my patch and then convert tabs to
> spaces?  There
> are still indentation errors in the file that I already fixed once before.
>
> Here is a new (tab free) patch that re-fixes the indentation.  I'll wait
> until this is done before generating a new fix patch, as I don't know of
> anyway to generate a new fix patch after a large white-space change other
> than to tediously redo the entire patch by hand, and I'd like to avoid
> doing
> that more than once.


Entire patch except following non-cosmetic change is applied:

>@@ -842,7 +841,7 @@
>         ioctl(priv->video_fd, VIDIOCSWIN, &win);
>     }
>
>-    // initialize video capture
>+    // initialize video sub-capture
>     if (ioctl(priv->video_fd, VIDIOCCAPTURE, &one) == -1)
>     {
>         mp_msg(MSGT_TV, MSGL_ERR, "FATAL: ioctl ccapture failed: %s\n", strerror(errno));

This change will be applied independently.
Thanks for the patch.



-- 
Regards,
Vladimir Voroshilov     mailto:voroshil at gmail.com
JID: voroshil at jabber.ru
ICQ: 95587719



More information about the MPlayer-dev-eng mailing list