2010-11-19 18:26:16

by Raffaele Recalcati

[permalink] [raw]
Subject: tvp5150 extension to tvp5151

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


2010-11-22 11:32:52

by Mauro Carvalho Chehab

[permalink] [raw]
Subject: Re: tvp5150 extension to tvp5151

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

2010-11-22 14:44:22

by Raffaele Recalcati

[permalink] [raw]
Subject: Re: tvp5150 extension to tvp5151

On Mon, Nov 22, 2010 at 12:32 PM, Mauro Carvalho Chehab
<[email protected]> 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