Re: Linux 2.6.3-rc4

From: Roman Zippel
Date: Tue Feb 17 2004 - 21:46:41 EST


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/