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

[Adrian Bunk <bunk@kernel.org>]

On Wed, Dec 12, 2007 at 11:00:46AM -0500, mkrufky@linuxtv.org 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@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/