Re: [PATCH 2/2] ioc4: Enable build on non-SN2

From: Randy Dunlap
Date: Tue Oct 10 2006 - 13:38:17 EST


On Tue, 10 Oct 2006 12:11:16 -0500 (CDT) Brent Casavant wrote:

> Index: top/drivers/Kconfig
> ===================================================================
> --- top.orig/drivers/Kconfig 2006-10-09 17:41:25.000000000 -0500
> +++ top/drivers/Kconfig 2006-10-09 17:47:50.415607888 -0500
> @@ -14,6 +14,10 @@
>
> source "drivers/block/Kconfig"
>
> +# misc before ide - BLK_DEV_SGIIOC4 depends on SGI_IOC4
> +
> +source "drivers/misc/Kconfig"
> +
> source "drivers/ide/Kconfig"
>
> source "drivers/scsi/Kconfig"
> @@ -52,8 +56,6 @@
>
> source "drivers/hwmon/Kconfig"
>
> -source "drivers/misc/Kconfig"
> -
> source "drivers/mfd/Kconfig"
>
> source "drivers/media/Kconfig"

Mostly curious: did you observe that this is required?
I always thought that Roman said that unknown config variables
caused a rescan by kconfig. IOW, I thought that it wouldn't
be observable by a user. Just wondering..


> Index: top/drivers/misc/Kconfig
> ===================================================================
> --- top.orig/drivers/misc/Kconfig 2006-10-09 17:41:25.000000000 -0500
> +++ top/drivers/misc/Kconfig 2006-10-09 17:48:56.555631075 -0500
> @@ -28,6 +28,18 @@
>
> If unsure, say N.
>
> +config SGI_IOC4
> + tristate "SGI IOC4 Base IO support"
> + default m
> + ---help---
> + This option enables basic support for the IOC4 chip on certain
> + SGI IO controller cards (IO9, IO10, and PCI-RT). This option
> + does not enable any specific functions on such a card, but provides
> + necessary infrastructure for other drivers to utilize.
> +
> + If you have an SGI Altix with an IOC4-based card say Y.
> + Otherwise say N.

The lines under ---help--- should be indented by 2 spaces (by
convention) (and even though they were not when in the /sn/ subdir).


> config TIFM_CORE
> tristate "TI Flash Media interface support (EXPERIMENTAL)"
> depends on EXPERIMENTAL



---
~Randy
-
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/