public inbox for linux-kbuild@vger.kernel.org
 help / color / mirror / Atom feed
From: Richard Kennedy <richard@rsk.demon.co.uk>
To: Randy Dunlap <randy.dunlap@oracle.com>
Cc: David Vrabel <david.vrabel@csr.com>,
	Roman Zippel <zippel@linux-m68k.org>,
	linux-usb <linux-usb@vger.kernel.org>,
	linux-kbuild <linux-kbuild@vger.kernel.org>,
	lkml <linux-kernel@vger.kernel.org>
Subject: Re: UWB Kconfig: cannot set UWB to N in gconfig
Date: Fri, 11 Sep 2009 15:00:30 +0100	[thread overview]
Message-ID: <1252677630.2282.11.camel@castor> (raw)
In-Reply-To: <20090910090806.e8db2300.randy.dunlap@oracle.com>

On Thu, 2009-09-10 at 09:08 -0700, Randy Dunlap wrote:
> On Thu, 10 Sep 2009 11:05:54 +0100 Richard Kennedy wrote:
> 
> > I sometimes use 'make gconfig' and I noticed that I cannot disable the
> > UWB option -- it just won't let me set it to N.
> 
> Not specific to 'gconfig'.
> 
> > AFAICT this is because the CONFIG USB_WUSB selects UWB which forces it
> > on.
> 
> Yes.
> 
> > applying this patch fixes the issue but I'm not sure if this is the
> > right way to go about it.
> > 
> > any thoughts?
> 
> UWB can be disabled by disabling both of these options that are several
> entries below it in the same menu:
> 
> USB_WHCI_HCD and USB_HWA_HCD
> 
> and then returning to UWB and disabling it.
> 
> Should users have to figure that out?  no.
> 
thanks,
although it's not easy to figure out how to disable this in gconfig.
USB_WHCI_HCD & USB_HWA_HCD end up under a different menu
(drivers/usb/host). 
If you don't know that they exist, all you see is the options under UWB
set to M and gconfig doesn't let you turn them off.  

Here's another version of the patch that changes them all to 'depends
on'. This lets gconfig do the right thing :)

regards
Richard

diff --git a/drivers/usb/host/Kconfig b/drivers/usb/host/Kconfig
index 1a920c7..5b6924b 100644
--- a/drivers/usb/host/Kconfig
+++ b/drivers/usb/host/Kconfig
@@ -347,8 +347,8 @@ config USB_WHCI_HCD
 	tristate "Wireless USB Host Controller Interface (WHCI) driver (EXPERIMENTAL)"
 	depends on EXPERIMENTAL
 	depends on PCI && USB
-	select USB_WUSB
-	select UWB_WHCI
+	depends on  USB_WUSB
+	depends on UWB_WHCI
 	help
 	  A driver for PCI-based Wireless USB Host Controllers that are
 	  compliant with the WHCI specification.
@@ -360,8 +360,8 @@ config USB_HWA_HCD
 	tristate "Host Wire Adapter (HWA) driver (EXPERIMENTAL)"
 	depends on EXPERIMENTAL
 	depends on USB
-	select USB_WUSB
-	select UWB_HWA
+	depends on USB_WUSB
+	depends on UWB_HWA
 	help
 	  This driver enables you to connect Wireless USB devices to
 	  your system using a Host Wire Adaptor USB dongle. This is an
diff --git a/drivers/usb/wusbcore/Kconfig b/drivers/usb/wusbcore/Kconfig
index eb09a0a..8209da9 100644
--- a/drivers/usb/wusbcore/Kconfig
+++ b/drivers/usb/wusbcore/Kconfig
@@ -5,7 +5,7 @@ config USB_WUSB
 	tristate "Enable Wireless USB extensions (EXPERIMENTAL)"
 	depends on EXPERIMENTAL
 	depends on USB
-        select UWB
+	depends on UWB
         select CRYPTO
         select CRYPTO_BLKCIPHER
         select CRYPTO_CBC




  reply	other threads:[~2009-09-11 14:00 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-09-10 10:05 UWB Kconfig: cannot set UWB to N in gconfig Richard Kennedy
2009-09-10 16:08 ` Randy Dunlap
2009-09-11 14:00   ` Richard Kennedy [this message]
2009-09-14 10:34 ` David Vrabel

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1252677630.2282.11.camel@castor \
    --to=richard@rsk.demon.co.uk \
    --cc=david.vrabel@csr.com \
    --cc=linux-kbuild@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=randy.dunlap@oracle.com \
    --cc=zippel@linux-m68k.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox