Re: [PATCH]Add support for the ALi m5602 usb bridge

["Erik Andrén" <erik.andren@xxxxxxxxx>]

2008/10/1 Hans de Goede <j.w.r.degoede@xxxxxx>

> Erik Andrén wrote:
>
>> -----BEGIN PGP SIGNED MESSAGE-----
>> Hash: SHA1
>>
>> Hi,
>>
>> This patch adds support for the ALi m5602 usb bridge and is based on the
>> gspca framework.
>> It contains code for communicating with 5 different sensors:
>> OmniVision OV9650, Pixel Plus PO1030, Samsung S5K83A, S5K4AA and finally
>> Micron MT9M111.
>>
>> Regards,
>> Erik Andrén
>>
>>
> I like it, thanks for converting this to a gspca sub driver, I assume this
> helped you to reduce the code size somewhat?

Yes, about 20% give or take.


> If you have any remarks / suggestions to improve the gspca framework please
> say so.
>
It fitted in quite nicely, some more inline comments regarding
implementation would be good.
Perhaps a skeleton driver could be made to ease implementation.

I have some patches which I intend to send in the next couple of days.

Regards,
Erik




>
> Jean François, can you pick this up and feed it to Mauro through the gspca
> tree?
>
> Regards,
>
> Hans
>
>
>
--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@xxxxxxxxxx?subjecthttps://www.redhat.com/mailman/listinfo/video4linux-list