Re: Upstreaming syntek driver

[Hans de Goede <hdegoede@redhat.com>]

Hi,

On 01/20/2011 12:35 PM, Luca Tettamanti wrote:
> On Tue, Jan 18, 2011 at 11:45 PM, Hans Verkuil<hverkuil@xs4all.nl>  wrote:
> [...]
>> After a quick scan through the sources in svn I found the following (in no
>> particular order):
>>
>> - Supports easycap model with ID 05e1:0408: a driver for this model is now
>>   in driver/staging/easycap.
>
> Can you elaborate? Is this the same hardware?
>
>> - format conversion must be moved to libv4lconvert (if that can't already be
>>   used out of the box). Ditto for software brightness correction.
>>
>> - kill off the sysfs bits
>>
>> - kill off V4L1
>>
>> - use the new control framework for the control handling
>>
>> - use video_ioctl2 instead of the current ioctl function
>>
>> - use unlocked_ioctl instead of ioctl
>
> Ok, major surgery then :)
>
>> But probably the first step should be to see if this can't be made part of the
>> gspca driver. I can't help thinking that that would be the best approach. But
>> I guess the gspca developers can give a better idea of how hard that is.
>
> I've looked at the framework provided by gspca, it would probably
> allow to drop most of the USB support code from the driver.

Yeah, that is the whole idea :) I give a big +1 to the Hans' suggestion
to convert this to a gspca driver!

> I'm looking into frame handling.

Let me know if you need any help / explanation about how certain things
are done in gspca.

Regards,

Hans G (aka the other Hans).