2005-12-24 13:25:54

by Marc Burkhardt

[permalink] [raw]
Subject: [PATCH] Kconfig option to enable faulty Artec DVB-T USB devices, cleanups, documentation

The following patch adds a Kconfig option in the DVB-T USB section that enables
a user to select wether some faulty Artec devices may be supported due to an
invalid EEPROM. Some googling stated that some people had problems getting the
device to be detected by the driver (though the device was stated to be
supported by the module on various sites). As not anyone is into changing source
code a Kconfig option would be appropriate for these users. Patrick Boettcher
also suggested a Kconfig would make sense. Moreover, one may share the option in
his/her .config accross several kernels instead of uncommenting the #define on
any -git checkout or new kernel release. It was simply annoying. :)

Moreover, there's some cleanup on indentation and such, added some explanation
to the faulty UDB IDs code, re-ordered the list of supported devices in Kconfig
to match out alphabet ;)

The #ifdef-fing of dibusb-mc functions was removed due to Adrian Bunks' hint.

Patrick Boettcher and the Linux DVB Maintainers are CC'ed...

The patch is against the current -git but should cause no problems on the recent
releases.

Regards,
marc


Attachments:
(No filename) (1.10 kB)
dvb-usb.patch (17.98 kB)
Download all attachments

2006-01-08 17:51:21

by Patrick Boettcher

[permalink] [raw]
Subject: Re: [PATCH] Kconfig option to enable faulty Artec DVB-T USB devices, cleanups, documentation

Hi Marc,

I finally found the time to commit some of your cleanups to the
V4L-DVB-CVS.

I only took some of your cleanups. The useful ones :)

Thanks for your help,
Patrick.

--
Mail: [email protected]
WWW: http://www.wi-bw.tfh-wildau.de/~pboettch/

On Sat, 24 Dec 2005, Marc Koschewski wrote:
> The following patch adds a Kconfig option in the DVB-T USB section that enables
> a user to select wether some faulty Artec devices may be supported due to an
> invalid EEPROM. Some googling stated that some people had problems getting the
> device to be detected by the driver (though the device was stated to be
> supported by the module on various sites). As not anyone is into changing source
> code a Kconfig option would be appropriate for these users. Patrick Boettcher
> also suggested a Kconfig would make sense. Moreover, one may share the option in
> his/her .config accross several kernels instead of uncommenting the #define on
> any -git checkout or new kernel release. It was simply annoying. :)
>
> Moreover, there's some cleanup on indentation and such, added some explanation
> to the faulty UDB IDs code, re-ordered the list of supported devices in Kconfig
> to match out alphabet ;)
>
> The #ifdef-fing of dibusb-mc functions was removed due to Adrian Bunks' hint.
>
> Patrick Boettcher and the Linux DVB Maintainers are CC'ed...
>
> The patch is against the current -git but should cause no problems on the recent
> releases.
>
> Regards,
> marc
>