Re: [v4l-dvb-maintainer] build #340 failed for 2.6.24-rc5-g75b8c13in?linux/drivers/media/video/tuner.ko

From: Adrian Bunk
Date: Wed Dec 12 2007 - 12:57:40 EST


On Wed, Dec 12, 2007 at 11:00:46AM -0500, mkrufky@xxxxxxxxxxx wrote:
> Mauro Carvalho Chehab wrote:
>...
> What we really need is to make the Kconfig selection behave as if it
> depends on VIDEO_TUNER, but only if VIDEO_TUNER is selected. I know
> that sounds redundant, but that would actually solve the problem.
>
> If VIDEO_TUNER is not selected, then the driver can be built as either Y
> or M, If, however, VIDEO_TUNER is selected, then the driver must be
> built as whatever VIDEO_TUNER was selected as.
>...

The problem is not how to express this (which is not a big problem), the
problem is how to keep these options maintainable...

I'll take a look at it (but I'm not sure whether this will be this year).

> remove the bit " if !VIDEO_TUNER_CUSTOMIZE" , such that the select lines
> will read:
>
> select TUNER_MT20XX
> select TUNER_TDA8290
> select TUNER_TEA5761
> select TUNER_TEA5767
> select TUNER_SIMPLE
> select TUNER_TDA9887
>
> instead of:
>
> select TUNER_MT20XX if !VIDEO_TUNER_CUSTOMIZE
> select TUNER_TDA8290 if !VIDEO_TUNER_CUSTOMIZE
> select TUNER_TEA5761 if !VIDEO_TUNER_CUSTOMIZE
> select TUNER_TEA5767 if !VIDEO_TUNER_CUSTOMIZE
> select TUNER_SIMPLE if !VIDEO_TUNER_CUSTOMIZE
> select TUNER_TDA9887 if !VIDEO_TUNER_CUSTOMIZE
>
> If this fixes the problem, then I think it would be a good temporary fix
> for 2.6.24, until we can make a better, more drastic fix for 2.6.25

select sets a minimum value for a variable, but our problem is that the
value is too high...

> Thanks for testing.
>
> Regards,
> Mike Krufky

cu
Adrian

--

"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/