2012-02-10 07:08:00

by Mike Galbraith

[permalink] [raw]
Subject: [patch] usb: add missing Kconfig UWB select to USB_WHCI_HCD and USB_HWA_HCD

usb: add missing Kconfig UWB select to USB_WHCI_HCD and USB_HWA_HCD

cripts/kconfig/conf --oldconfig Kconfig
warning: (USB_WHCI_HCD && USB_HWA_HCD) selects USB_WUSB which has unmet direct dependencies (USB_SUPPORT && EXPERIMENTAL && USB && PCI && UWB)
warning: (USB_WHCI_HCD) selects UWB_WHCI which has unmet direct dependencies (UWB && PCI)
warning: (USB_WHCI_HCD && USB_HWA_HCD) selects USB_WUSB which has unmet direct dependencies (USB_SUPPORT && EXPERIMENTAL && USB && PCI && UWB)
warning: (USB_WHCI_HCD) selects UWB_WHCI which has unmet direct dependencies (UWB && PCI)

Signed-off-by: Mike Galbraith <[email protected]>

diff --git a/drivers/usb/host/Kconfig b/drivers/usb/host/Kconfig
index 91413ca..b224959 100644
--- a/drivers/usb/host/Kconfig
+++ b/drivers/usb/host/Kconfig
@@ -547,6 +547,7 @@ config USB_WHCI_HCD
tristate "Wireless USB Host Controller Interface (WHCI) driver (EXPERIMENTAL)"
depends on EXPERIMENTAL
depends on PCI && USB
+ select UWB
select USB_WUSB
select UWB_WHCI
help
@@ -560,6 +561,7 @@ config USB_HWA_HCD
tristate "Host Wire Adapter (HWA) driver (EXPERIMENTAL)"
depends on EXPERIMENTAL
depends on USB
+ select UWB
select USB_WUSB
select UWB_HWA
help


2012-02-10 16:11:54

by Randy Dunlap

[permalink] [raw]
Subject: Re: [patch] usb: add missing Kconfig UWB select to USB_WHCI_HCD and USB_HWA_HCD

On 02/09/2012 11:07 PM, Mike Galbraith wrote:
> usb: add missing Kconfig UWB select to USB_WHCI_HCD and USB_HWA_HCD
>
> cripts/kconfig/conf --oldconfig Kconfig
> warning: (USB_WHCI_HCD && USB_HWA_HCD) selects USB_WUSB which has unmet direct dependencies (USB_SUPPORT && EXPERIMENTAL && USB && PCI && UWB)
> warning: (USB_WHCI_HCD) selects UWB_WHCI which has unmet direct dependencies (UWB && PCI)
> warning: (USB_WHCI_HCD && USB_HWA_HCD) selects USB_WUSB which has unmet direct dependencies (USB_SUPPORT && EXPERIMENTAL && USB && PCI && UWB)
> warning: (USB_WHCI_HCD) selects UWB_WHCI which has unmet direct dependencies (UWB && PCI)
>
> Signed-off-by: Mike Galbraith <[email protected]>

I have 2 patches in linux-next that fix the UWB/WUSB problems by using "depends on"
instead...


>
> diff --git a/drivers/usb/host/Kconfig b/drivers/usb/host/Kconfig
> index 91413ca..b224959 100644
> --- a/drivers/usb/host/Kconfig
> +++ b/drivers/usb/host/Kconfig
> @@ -547,6 +547,7 @@ config USB_WHCI_HCD
> tristate "Wireless USB Host Controller Interface (WHCI) driver (EXPERIMENTAL)"
> depends on EXPERIMENTAL
> depends on PCI && USB
> + select UWB
> select USB_WUSB
> select UWB_WHCI
> help
> @@ -560,6 +561,7 @@ config USB_HWA_HCD
> tristate "Host Wire Adapter (HWA) driver (EXPERIMENTAL)"
> depends on EXPERIMENTAL
> depends on USB
> + select UWB
> select USB_WUSB
> select UWB_HWA
> help
>
>
> --


--
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***

2012-02-10 18:14:44

by Greg Kroah-Hartman

[permalink] [raw]
Subject: Re: [patch] usb: add missing Kconfig UWB select to USB_WHCI_HCD and USB_HWA_HCD

On Fri, Feb 10, 2012 at 08:11:26AM -0800, Randy Dunlap wrote:
> On 02/09/2012 11:07 PM, Mike Galbraith wrote:
> > usb: add missing Kconfig UWB select to USB_WHCI_HCD and USB_HWA_HCD
> >
> > cripts/kconfig/conf --oldconfig Kconfig
> > warning: (USB_WHCI_HCD && USB_HWA_HCD) selects USB_WUSB which has unmet direct dependencies (USB_SUPPORT && EXPERIMENTAL && USB && PCI && UWB)
> > warning: (USB_WHCI_HCD) selects UWB_WHCI which has unmet direct dependencies (UWB && PCI)
> > warning: (USB_WHCI_HCD && USB_HWA_HCD) selects USB_WUSB which has unmet direct dependencies (USB_SUPPORT && EXPERIMENTAL && USB && PCI && UWB)
> > warning: (USB_WHCI_HCD) selects UWB_WHCI which has unmet direct dependencies (UWB && PCI)
> >
> > Signed-off-by: Mike Galbraith <[email protected]>
>
> I have 2 patches in linux-next that fix the UWB/WUSB problems by using "depends on"
> instead...

Yes, this should all now be resolved, Mike can you verify this please?

thanks,

greg k-h

2012-02-11 05:43:29

by Mike Galbraith

[permalink] [raw]
Subject: Re: [patch] usb: add missing Kconfig UWB select to USB_WHCI_HCD and USB_HWA_HCD

On Fri, 2012-02-10 at 10:11 -0800, Greg Kroah-Hartman wrote:
> On Fri, Feb 10, 2012 at 08:11:26AM -0800, Randy Dunlap wrote:
> > On 02/09/2012 11:07 PM, Mike Galbraith wrote:
> > > usb: add missing Kconfig UWB select to USB_WHCI_HCD and USB_HWA_HCD
> > >
> > > cripts/kconfig/conf --oldconfig Kconfig
> > > warning: (USB_WHCI_HCD && USB_HWA_HCD) selects USB_WUSB which has unmet direct dependencies (USB_SUPPORT && EXPERIMENTAL && USB && PCI && UWB)
> > > warning: (USB_WHCI_HCD) selects UWB_WHCI which has unmet direct dependencies (UWB && PCI)
> > > warning: (USB_WHCI_HCD && USB_HWA_HCD) selects USB_WUSB which has unmet direct dependencies (USB_SUPPORT && EXPERIMENTAL && USB && PCI && UWB)
> > > warning: (USB_WHCI_HCD) selects UWB_WHCI which has unmet direct dependencies (UWB && PCI)
> > >
> > > Signed-off-by: Mike Galbraith <[email protected]>
> >
> > I have 2 patches in linux-next that fix the UWB/WUSB problems by using "depends on"
> > instead...
>
> Yes, this should all now be resolved, Mike can you verify this please?

Nah, Randy surely did, and it would be hard to mess it up ;-)

-Mike

2012-02-11 19:08:34

by Randy Dunlap

[permalink] [raw]
Subject: Re: [patch] usb: add missing Kconfig UWB select to USB_WHCI_HCD and USB_HWA_HCD

On 02/10/2012 09:43 PM, Mike Galbraith wrote:
> On Fri, 2012-02-10 at 10:11 -0800, Greg Kroah-Hartman wrote:
>> On Fri, Feb 10, 2012 at 08:11:26AM -0800, Randy Dunlap wrote:
>>> On 02/09/2012 11:07 PM, Mike Galbraith wrote:
>>>> usb: add missing Kconfig UWB select to USB_WHCI_HCD and USB_HWA_HCD
>>>>
>>>> cripts/kconfig/conf --oldconfig Kconfig
>>>> warning: (USB_WHCI_HCD && USB_HWA_HCD) selects USB_WUSB which has unmet direct dependencies (USB_SUPPORT && EXPERIMENTAL && USB && PCI && UWB)
>>>> warning: (USB_WHCI_HCD) selects UWB_WHCI which has unmet direct dependencies (UWB && PCI)
>>>> warning: (USB_WHCI_HCD && USB_HWA_HCD) selects USB_WUSB which has unmet direct dependencies (USB_SUPPORT && EXPERIMENTAL && USB && PCI && UWB)
>>>> warning: (USB_WHCI_HCD) selects UWB_WHCI which has unmet direct dependencies (UWB && PCI)
>>>>
>>>> Signed-off-by: Mike Galbraith <[email protected]>
>>>
>>> I have 2 patches in linux-next that fix the UWB/WUSB problems by using "depends on"
>>> instead...
>>
>> Yes, this should all now be resolved, Mike can you verify this please?
>
> Nah, Randy surely did, and it would be hard to mess it up ;-)

Hah. It took me two simple patches to get it right AFAIK. :(

--
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***