Re: Linux 2.6.3-rc4

[Roman Zippel]

Hi,

On Tue, 17 Feb 2004, Adrian Bunk wrote:

> @@ -548,7 +544,8 @@
>
>  config FB_MATROX_I2C
>  	tristate "Matrox I2C support"
> -	depends on FB_MATROX && I2C
> +	depends on FB_MATROX
> +	select I2C
>  	select I2C_ALGOBIT
>  	---help---
>  	  This drivers creates I2C buses which are needed for accessing the

This was okay, this is a tristate and limited by I2C, so it will select
I2C_ALGOBIT correctly.

>  config FB_RADEON_I2C
>  	bool "DDC/I2C for ATI Radeon support"
> -	depends on FB_RADEON && I2C
> +	depends on FB_RADEON
> +	select I2C
>  	select I2C_ALGOBIT
>  	default y
>  	help

Linus, I think that's the best solution for now. I have to think a bit
more about the problem, how a boolean symbol should select a tristate
symbol.

bye, Roman
-
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/