Re: tvp5150 extension to tvp5151

2010-11-22 Thread Mauro Carvalho Chehab
Hi Raffaele,

Em 19-11-2010 16:26, Raffaele Recalcati escreveu:
 I need to support fully tvp5151.
 So I'm trying to understand your driver, that is ready for VBI and not
 for video acquisition.
 I also take sometimes a look at tvp514x.c, for instance trying to add
 VIDIOC_ENUM_FMT and other ioctls.
 I think we can move from tvp5150.c to tvp515x.c, maybe...
 I don't think is good to have tvp51xx.c because tvp514x.c family is
 more complex (more inputs...).
 By now I'm using tvp5150.c with some modifications and video acquisition 
 works.
 I need to complete the support in order to have gstreamer fully running.
 I'm working on 2.6.32, but I have planned to port it to mainline.

Renaming it to tvp515x.c and adding support for tvp5051 seems a good way.
With tvp5150, video acquisition works fine with several devices with em28xx,
although we may need to add more video formats at the media bus, depending
on what you're doing. I didn't test VBI support on it, because the devices
I have are based on em28xx, and I didn't find any way to capture the VBI
decoded packages and sent to userspace with that design. So, the basic stuff
is there, but maybe some adjusts may be needed for VBI.

 Do you have any suggestion for my work?

The better is to submit the patches you have, for us to apply upstream.

Cheers,
Mauro
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: tvp5150 extension to tvp5151

2010-11-22 Thread Raffaele Recalcati
On Mon, Nov 22, 2010 at 12:32 PM, Mauro Carvalho Chehab
mche...@infradead.org wrote:
 Hi Raffaele,

 Em 19-11-2010 16:26, Raffaele Recalcati escreveu:
 I need to support fully tvp5151.
 So I'm trying to understand your driver, that is ready for VBI and not
 for video acquisition.
 I also take sometimes a look at tvp514x.c, for instance trying to add
 VIDIOC_ENUM_FMT and other ioctls.
 I think we can move from tvp5150.c to tvp515x.c, maybe...
 I don't think is good to have tvp51xx.c because tvp514x.c family is
 more complex (more inputs...).
 By now I'm using tvp5150.c with some modifications and video acquisition 
 works.
 I need to complete the support in order to have gstreamer fully running.
 I'm working on 2.6.32, but I have planned to port it to mainline.

 Renaming it to tvp515x.c and adding support for tvp5051 seems a good way.
 With tvp5150, video acquisition works fine with several devices with em28xx,
 although we may need to add more video formats at the media bus, depending
 on what you're doing. I didn't test VBI support on it, because the devices
 I have are based on em28xx, and I didn't find any way to capture the VBI
 decoded packages and sent to userspace with that design. So, the basic stuff
 is there, but maybe some adjusts may be needed for VBI.

 Do you have any suggestion for my work?

 The better is to submit the patches you have, for us to apply upstream.

Thx for your reply..
I have added many v4l2 support similar to tvp514x.
I need to test them, hoping they work.
When all we'll be ok I'll send patches.
So I go on renaming all strings from tvp5150 to tvp515x.

Thx,
Raffaele
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


tvp5150 extension to tvp5151

2010-11-19 Thread Raffaele Recalcati
I need to support fully tvp5151.
So I'm trying to understand your driver, that is ready for VBI and not
for video acquisition.
I also take sometimes a look at tvp514x.c, for instance trying to add
VIDIOC_ENUM_FMT and other ioctls.
I think we can move from tvp5150.c to tvp515x.c, maybe...
I don't think is good to have tvp51xx.c because tvp514x.c family is
more complex (more inputs...).
By now I'm using tvp5150.c with some modifications and video acquisition works.
I need to complete the support in order to have gstreamer fully running.
I'm working on 2.6.32, but I have planned to port it to mainline.
Do you have any suggestion for my work?

Thx,
Raffaele
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html