linux-kernel - Re: tvp5150 extension to tvp5151

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <4CEA54C9.4030300@infradead.org>
Date:	Mon, 22 Nov 2010 09:32:25 -0200
From:	Mauro Carvalho Chehab <mchehab@...radead.org>
To:	Raffaele Recalcati <lamiaposta71@...il.com>
CC:	linux-media@...r.kernel.org, linux-kernel@...r.kernel.org
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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/