All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/8] Port xf86-video-v4l driver to V4L2 version 2
@ 2011-02-11 15:54 Mauro Carvalho Chehab
  2011-02-11 17:37 ` Alex Deucher
  2011-05-09 13:43 ` Cyril Brulebois
  0 siblings, 2 replies; 6+ messages in thread
From: Mauro Carvalho Chehab @ 2011-02-11 15:54 UTC (permalink / raw)
  To: X.Org Devel List, Linux Media Mailing List; +Cc: Dmitry Butskoy

This is the second version of my backport patches. To avoid spending
people's time on looking at lines that have only whitespace changes,
I broke the patch I submitted two days ago into two patches: 
the first one with just the logical changes, and the second one with
just CodingStyle (whitespace) fixes.

Reworking on a driver with whitespace and identation problems is hard,
as it makes harder for people to work with the source code.

So, the first two patches have basically the same content as the patch
I submitted earlier[1].

[1] http://lists.x.org/archives/xorg-devel/2011-February/019042.html

The fix patch I submited for xserver is still needed[2], otherwise
loading the v4l driver with a video driver that doesn't support overlay
will cause a server crash.

[2] http://lists.x.org/archives/xorg-devel/2011-February/019048.html

The remaining patches on this series fix some bugs found on the tests 
I made with:
	1) a Nvidia FX5200 and the nv driver, and a bttv Encore FM board;
	2) a Nvidia 6600 and nouveau driver, and a Kworld SBTV-D board
	  (to test it on an environment where Xv overlay is not available).
And by some tests made by Dmitry Butskoy with a bttv AVerMedia TVPhone98
board, with a Radeon RV100, using an old version of the Radeon driver.

As I said before, the intention now is to port it to work with textured
video. Help is wanted to point us how to use a textured video from the
v4l driver.

To make easier for people to test and review, the same patches are also
available on a git repository at:
	http://git.linuxtv.org/mchehab/xf86-video-v4l.git

PS.: I'm c/c just this  message to the media devel ML, for people there
     to be aware of the changes. There's not much sense to submit the
     remaining patches there.

Mauro Carvalho Chehab (8):
  Port xf86-video-v4l driver to V4L2
  Coding style Cleanup
  Use the fourcc header instead of redefining it inside the code
  Fix Get/Set Port Attribute logic
  Return BadMatch if a Port Attribute is not found
  Provide a more consistent message if FBUF fails
  Fix arguments for v4l_check_yuv
  Fix standard video size detection

 src/v4l.c       | 1549 ++++++++++++++++++++++++++-------------------
 src/videodev.h  |  254 --------
 src/videodev2.h | 1929 +++++++++++++++++++++++++++++++++++++++++++++++++++++++
 3 files changed, 2831 insertions(+), 901 deletions(-)
 delete mode 100644 src/videodev.h
 create mode 100644 src/videodev2.h


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 0/8] Port xf86-video-v4l driver to V4L2 version 2
  2011-02-11 15:54 [PATCH 0/8] Port xf86-video-v4l driver to V4L2 version 2 Mauro Carvalho Chehab
@ 2011-02-11 17:37 ` Alex Deucher
  2011-02-11 19:22   ` Mauro Carvalho Chehab
  2011-05-09 13:43 ` Cyril Brulebois
  1 sibling, 1 reply; 6+ messages in thread
From: Alex Deucher @ 2011-02-11 17:37 UTC (permalink / raw)
  To: Mauro Carvalho Chehab
  Cc: X.Org Devel List, Linux Media Mailing List, Dmitry Butskoy

On Fri, Feb 11, 2011 at 10:54 AM, Mauro Carvalho Chehab
<mchehab@redhat.com> wrote:
> This is the second version of my backport patches. To avoid spending
> people's time on looking at lines that have only whitespace changes,
> I broke the patch I submitted two days ago into two patches:
> the first one with just the logical changes, and the second one with
> just CodingStyle (whitespace) fixes.

Mauro,  I don't think anyone has maintained or used the v4l module in
ages.  You could be the new maintainer :)
Or at least have commit rights.  Just follow the instructions here:
http://www.freedesktop.org/wiki/AccountRequests

Alex

>
> Reworking on a driver with whitespace and identation problems is hard,
> as it makes harder for people to work with the source code.
>
> So, the first two patches have basically the same content as the patch
> I submitted earlier[1].
>
> [1] http://lists.x.org/archives/xorg-devel/2011-February/019042.html
>
> The fix patch I submited for xserver is still needed[2], otherwise
> loading the v4l driver with a video driver that doesn't support overlay
> will cause a server crash.
>
> [2] http://lists.x.org/archives/xorg-devel/2011-February/019048.html
>
> The remaining patches on this series fix some bugs found on the tests
> I made with:
>        1) a Nvidia FX5200 and the nv driver, and a bttv Encore FM board;
>        2) a Nvidia 6600 and nouveau driver, and a Kworld SBTV-D board
>          (to test it on an environment where Xv overlay is not available).
> And by some tests made by Dmitry Butskoy with a bttv AVerMedia TVPhone98
> board, with a Radeon RV100, using an old version of the Radeon driver.
>
> As I said before, the intention now is to port it to work with textured
> video. Help is wanted to point us how to use a textured video from the
> v4l driver.
>
> To make easier for people to test and review, the same patches are also
> available on a git repository at:
>        http://git.linuxtv.org/mchehab/xf86-video-v4l.git
>
> PS.: I'm c/c just this  message to the media devel ML, for people there
>     to be aware of the changes. There's not much sense to submit the
>     remaining patches there.
>
> Mauro Carvalho Chehab (8):
>  Port xf86-video-v4l driver to V4L2
>  Coding style Cleanup
>  Use the fourcc header instead of redefining it inside the code
>  Fix Get/Set Port Attribute logic
>  Return BadMatch if a Port Attribute is not found
>  Provide a more consistent message if FBUF fails
>  Fix arguments for v4l_check_yuv
>  Fix standard video size detection
>
>  src/v4l.c       | 1549 ++++++++++++++++++++++++++-------------------
>  src/videodev.h  |  254 --------
>  src/videodev2.h | 1929 +++++++++++++++++++++++++++++++++++++++++++++++++++++++
>  3 files changed, 2831 insertions(+), 901 deletions(-)
>  delete mode 100644 src/videodev.h
>  create mode 100644 src/videodev2.h
>
> _______________________________________________
> xorg-devel@lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel
>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 0/8] Port xf86-video-v4l driver to V4L2 version 2
  2011-02-11 17:37 ` Alex Deucher
@ 2011-02-11 19:22   ` Mauro Carvalho Chehab
  0 siblings, 0 replies; 6+ messages in thread
From: Mauro Carvalho Chehab @ 2011-02-11 19:22 UTC (permalink / raw)
  To: Alex Deucher; +Cc: X.Org Devel List, Linux Media Mailing List, Dmitry Butskoy

Hi Alex,

Em 11-02-2011 15:37, Alex Deucher escreveu:
> On Fri, Feb 11, 2011 at 10:54 AM, Mauro Carvalho Chehab
> <mchehab@redhat.com> wrote:
>> This is the second version of my backport patches. To avoid spending
>> people's time on looking at lines that have only whitespace changes,
>> I broke the patch I submitted two days ago into two patches:
>> the first one with just the logical changes, and the second one with
>> just CodingStyle (whitespace) fixes.
> 
> Mauro,  I don't think anyone has maintained or used the v4l module in
> ages.  You could be the new maintainer :)
> Or at least have commit rights.  Just follow the instructions here:
> http://www.freedesktop.org/wiki/AccountRequests

Thanks for the tip! I've filled an account request there.

Thanks!
Mauro

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 0/8] Port xf86-video-v4l driver to V4L2 version 2
  2011-02-11 15:54 [PATCH 0/8] Port xf86-video-v4l driver to V4L2 version 2 Mauro Carvalho Chehab
  2011-02-11 17:37 ` Alex Deucher
@ 2011-05-09 13:43 ` Cyril Brulebois
  1 sibling, 0 replies; 6+ messages in thread
From: Cyril Brulebois @ 2011-05-09 13:43 UTC (permalink / raw)
  To: Mauro Carvalho Chehab
  Cc: X.Org Devel List, Linux Media Mailing List, Dmitry Butskoy

[-- Attachment #1: Type: text/plain, Size: 234 bytes --]

Hi Mauro,

Mauro Carvalho Chehab <mchehab@redhat.com> (11/02/2011):
> Mauro Carvalho Chehab (8):
>   Port xf86-video-v4l driver to V4L2

thanks for your work on it. Maybe it would be nice to have a release
at some point?

Mraw,
KiBi.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 0/8] Port xf86-video-v4l driver to V4L2 version 2
  2011-02-13 19:15 Steven Karatnyk
@ 2011-02-13 21:45 ` Mauro Carvalho Chehab
  0 siblings, 0 replies; 6+ messages in thread
From: Mauro Carvalho Chehab @ 2011-02-13 21:45 UTC (permalink / raw)
  To: Steven Karatnyk; +Cc: Linux-media, alexdeucher

Em 13-02-2011 17:15, Steven Karatnyk escreveu:
> Note: I've added a news item to the main page regarding V4L1's removal from the kernel.  In addition, I've attempted to summarize what the video-v4l  module's purpose, as well as how to use it, by sewing together the info from a bunch of sources (such as Alex's, Mauro comments, etc etc)
> 
> - http://www.linuxtv.org/wiki/index.php/Main_Page
> - http://www.linuxtv.org/wiki/index.php/Xf86-video-v4l

Thank you for adding them to the wiki. The info there seems consistent.

Currently, just some PCI devices implement the V4L2 overlay support,
needed by the driver to work. I'll eventually implement other modes
like mmap and userptr one day, but what I want to do first is to make
it support the textured video. I'm not sure about the other apps, but
xawtv works fine with the v4l driver.

Cheers,
Mauro

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 0/8] Port xf86-video-v4l driver to V4L2 version 2
@ 2011-02-13 19:15 Steven Karatnyk
  2011-02-13 21:45 ` Mauro Carvalho Chehab
  0 siblings, 1 reply; 6+ messages in thread
From: Steven Karatnyk @ 2011-02-13 19:15 UTC (permalink / raw)
  To: Linux-media, mchehab, alexdeucher

Note: I've added a news item to the main page regarding V4L1's removal 
from the kernel.  In addition, I've attempted to summarize what the 
video-v4l  module's purpose, as well as how to use it, by sewing 
together the info from a bunch of sources (such as Alex's, Mauro 
comments, etc etc)

- http://www.linuxtv.org/wiki/index.php/Main_Page
- http://www.linuxtv.org/wiki/index.php/Xf86-video-v4l

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2011-05-09 14:18 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-02-11 15:54 [PATCH 0/8] Port xf86-video-v4l driver to V4L2 version 2 Mauro Carvalho Chehab
2011-02-11 17:37 ` Alex Deucher
2011-02-11 19:22   ` Mauro Carvalho Chehab
2011-05-09 13:43 ` Cyril Brulebois
2011-02-13 19:15 Steven Karatnyk
2011-02-13 21:45 ` Mauro Carvalho Chehab

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.