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

[Randy Dunlap <rdunlap@xenotime.net>]

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