]> git.hungrycats.org Git - linux/commit
[PATCH] tuner driver fixes
authorAndrew Morton <akpm@osdl.org>
Wed, 18 Feb 2004 12:59:42 +0000 (04:59 -0800)
committerLinus Torvalds <torvalds@ppc970.osdl.org>
Wed, 18 Feb 2004 12:59:42 +0000 (04:59 -0800)
commit7541c77e4d038366d5642701a511ade2ccd1138d
tree37eb187398cabc9709f2548d21289ab51b2b61ea
parent61ac12a0eb10224867fa820a5639a7b1feca7bf5
[PATCH] tuner driver fixes

From: Gerd Knorr <kraxel@bytesex.org>

"options tuner type=2" is just there for historical reasons and should
only be used/needed if the main driver doesn't allow to configure the
tuner type.  I'm not sure whenever I can remove that altogether without
breaking anything.  There are several users of the tuner module, some
outside the standard kernel tree ...

> >         if (type < TUNERS) {
> > +               t->type = type;
> >                 printk("tuner: type forced to %d (%s) [insmod]\n",
> >                        t->type,tuners[t->type].name);
> >                 set_type(client,type);

That is wrong, it will break in other corner cases, it may cause the
set_type() function not doing the initializations.

The prink can also be dropped because set_type does that too.  The patch
below removes that.  It also makes the kernel messages a bit more verbose
and adds support for two new tuners.
drivers/media/video/tuner.c
include/media/tuner.h