Igor Kuznetsov wrote:
>       
> 
> Made support all tuners Beholder. Almost-not yet only support hardware MPEG 
> decoders in a series of M6.
> 
> 
> 
> Patch for all tuners Beholder series 40x, 50x, 60x, M6, and Columbus
> 
> 
> 
> http://www.igk.ru/linux/files/v4l/v4l2-beholder-0.1.patch
> 
> --
> 
> Igor Kuznetsov "IgK"
> 
> Email: [EMAIL PROTECTED] [EMAIL PROTECTED]
> 
> ICQ: 6651879

Igor,

First off, it looks like these devices are analog-only, so it would be more 
appropriate to send this to the video4linux mailing list (cc added)

The patch is large, so I chopped it from the email.  For those interested, 
please see the original email:

http://linuxtv.org/pipermail/linux-dvb/2007-September/020256.html

Secondly, your patch introduces broken whitespace all over.  Tabs should be 
used for leading spacing, not a series of spaces.

We prefer for new cards to be added to the end of the card array-- not 
dispersed randomly throughout.

It looks like your large patch includes work from multiple contributors, based 
on the different names that I see commented within the card array additions.

You did not provide a sign-off on your work.  You should probably also collect 
sign-off's from those other developers involved in this work.

For more information on sign-off, and patch submissions to the v4l/dvb projects 
in general, please see:

http://linuxtv.org/hg/v4l-dvb/file/tip/README.patches

The preferred method would be to clone the latest version of the repository 
from linuxtv.org, apply your patch to that tree, test it, and then generate a 
new diff as follows:

hg diff > new.patch

Then, send in that new patch to the mailing lists.

Again, when you send in the new patch with sign-offs , please be sure to 
include the video4linux mailing list:

Linux and Kernel Video <[EMAIL PROTECTED]>

Cheers,

Mike Krufky

_______________________________________________
linux-dvb mailing list
linux-dvb@linuxtv.org
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb

Reply via email to