Message-ID: <4915B769.6090403@s5r6.in-berlin.de>
Date: Sat, 08 Nov 2008 16:59:37 +0100
From: Stefan Richter <stefanr@s5r6.in-berlin.de>
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1.17) Gecko/20081027 SeaMonkey/1.1.12
MIME-Version: 1.0
To: Greg KH <greg@kroah.com>
CC: linux-kernel@vger.kernel.org, Greg Kroah-Hartman <gregkh@suse.de>
Subject: Re: [PATCH 1/2] Staging: only build the tree if we really want to
References: <20081108054511.GA28846@kroah.com> <1226123143-28859-1-git-send-email-greg@kroah.com>
In-Reply-To: <1226123143-28859-1-git-send-email-greg@kroah.com>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1057
Lines: 37

Greg KH wrote:
> --- a/drivers/staging/Kconfig
> +++ b/drivers/staging/Kconfig
> @@ -21,7 +21,23 @@ menuconfig STAGING
>  
>  	  If in doubt, say N here.
>  
> -if STAGING
> +
> +config STAGING_EXCLUDE_BUILD
> +	bool "Exclude Staging drivers from being built"
> +	default y
> +	---help---
...
> +if !STAGING_EXCLUDE_BUILD
>  
>  source "drivers/staging/et131x/Kconfig"
>  
> @@ -45,4 +61,4 @@ source "drivers/staging/at76_usb/Kconfig"
>  
>  source "drivers/staging/poch/Kconfig"
>  
> -endif # STAGING
> +endif # !STAGING_EXCLUDE_BUILD

Which purpose is the STAGING kconfig variable serving after this?  It
controls the visibility of the menu, but switching it off won't switch
off the staging drivers if they were previously enabled, would it?
-- 
Stefan Richter
-=====-==--- =-== -=---
http://arcgraph.de/sr/
--
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/