Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1363544 > unrolled thread

[PATCH] usb: rework CONFIG_USB_COMMON logic

Started byArnd Bergmann <arnd@arndb.de>
First post2016-03-23 17:50 +0100
Last post2016-03-30 16:00 +0200
Articles 4 — 2 participants

Back to article view | Back to linux.kernel

Contents

  [PATCH] usb: rework CONFIG_USB_COMMON logic Arnd Bergmann <arnd@arndb.de> - 2016-03-23 17:50 +0100
    Re: [PATCH] usb: rework CONFIG_USB_COMMON logic Felipe Balbi <balbi@kernel.org> - 2016-03-30 12:40 +0200
      Re: [PATCH] usb: rework CONFIG_USB_COMMON logic Felipe Balbi <balbi@kernel.org> - 2016-03-30 16:00 +0200
      Re: [PATCH] usb: rework CONFIG_USB_COMMON logic Arnd Bergmann <arnd@arndb.de> - 2016-03-30 16:00 +0200

#1363544 — [PATCH] usb: rework CONFIG_USB_COMMON logic

FromArnd Bergmann <arnd@arndb.de>
Date2016-03-23 17:50 +0100
Subject[PATCH] usb: rework CONFIG_USB_COMMON logic
Message-ID<rfU8P-6dp-27@gated-at.bofh.it>
The phy-am335x driver selects 'USB_COMMON', but all other drivers
use 'depends on' for that symbol, and it depends on USB || USB_GADGET
itself, which causes a Kconfig warning:

warning: (AM335X_PHY_USB) selects USB_COMMON which has unmet direct dependencies (USB_SUPPORT && (USB || USB_GADGET))

As suggested by Felipe Balbi, this turns the logic around, and makes
'USB_COMMON' selected by everything else that needs it, so we can
remove the dependencies.

Fixes: 59f042f644c5 ("usb: phy: phy-am335x: bypass first VBUS sensing for host-only mode")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Acked-by: Felipe Balbi <balbi@kernel.org>
Reviewed-by: Peter Chen <peter.chen@nxp.com>
---
I seem to have dropped the ball on this one after my initial submission
when it wasn't clear who should merge the patch.

Please apply to the USB or the USB gadget tree, whoever gets there
first.
---
 drivers/phy/Kconfig        | 3 ++-
 drivers/usb/Kconfig        | 3 +--
 drivers/usb/gadget/Kconfig | 1 +
 3 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/drivers/phy/Kconfig b/drivers/phy/Kconfig
index 26566db09de0..e92b97cd6056 100644
--- a/drivers/phy/Kconfig
+++ b/drivers/phy/Kconfig
@@ -250,7 +250,8 @@ config PHY_SUN9I_USB
 	tristate "Allwinner sun9i SoC USB PHY driver"
 	depends on ARCH_SUNXI && HAS_IOMEM && OF
 	depends on RESET_CONTROLLER
-	depends on USB_COMMON
+	depends on USB_SUPPORT
+	select USB_COMMON
 	select GENERIC_PHY
 	help
 	  Enable this to support the transceiver that is part of Allwinner
diff --git a/drivers/usb/Kconfig b/drivers/usb/Kconfig
index 8ed451dd651e..8689dcba5201 100644
--- a/drivers/usb/Kconfig
+++ b/drivers/usb/Kconfig
@@ -31,8 +31,6 @@ if USB_SUPPORT
 
 config USB_COMMON
 	tristate
-	default y
-	depends on USB || USB_GADGET
 
 config USB_ARCH_HAS_HCD
 	def_bool y
@@ -41,6 +39,7 @@ config USB_ARCH_HAS_HCD
 config USB
 	tristate "Support for Host-side USB"
 	depends on USB_ARCH_HAS_HCD
+	select USB_COMMON
 	select NLS  # for UTF-8 strings
 	---help---
 	  Universal Serial Bus (USB) is a specification for a serial bus
diff --git a/drivers/usb/gadget/Kconfig b/drivers/usb/gadget/Kconfig
index af5d922a8f5d..2057add439f0 100644
--- a/drivers/usb/gadget/Kconfig
+++ b/drivers/usb/gadget/Kconfig
@@ -15,6 +15,7 @@
 
 menuconfig USB_GADGET
 	tristate "USB Gadget Support"
+	select USB_COMMON
 	select NLS
 	help
 	   USB is a master/slave protocol, organized with one master
-- 
2.7.0

[toc] | [next] | [standalone]

#1367090

FromFelipe Balbi <balbi@kernel.org>
Date2016-03-30 12:40 +0200
Message-ID<rilHB-3WZ-41@gated-at.bofh.it>
In reply to#1363544

[Multipart message — attachments visible in raw view] — view raw

Arnd Bergmann <arnd@arndb.de> writes:
> [ text/plain ]
> The phy-am335x driver selects 'USB_COMMON', but all other drivers
> use 'depends on' for that symbol, and it depends on USB || USB_GADGET
> itself, which causes a Kconfig warning:
>
> warning: (AM335X_PHY_USB) selects USB_COMMON which has unmet direct dependencies (USB_SUPPORT && (USB || USB_GADGET))
>
> As suggested by Felipe Balbi, this turns the logic around, and makes
> 'USB_COMMON' selected by everything else that needs it, so we can
> remove the dependencies.
>
> Fixes: 59f042f644c5 ("usb: phy: phy-am335x: bypass first VBUS sensing for host-only mode")
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> Acked-by: Felipe Balbi <balbi@kernel.org>
> Reviewed-by: Peter Chen <peter.chen@nxp.com>
> ---
> I seem to have dropped the ball on this one after my initial submission
> when it wasn't clear who should merge the patch.
>
> Please apply to the USB or the USB gadget tree, whoever gets there
> first.

seems like this is one of those "can wait for next merge window" right ?
I'll take it through my tree. No issues.

-- 
balbi

[toc] | [prev] | [next] | [standalone]

#1367223

FromFelipe Balbi <balbi@kernel.org>
Date2016-03-30 16:00 +0200
Message-ID<rioP8-6bC-3@gated-at.bofh.it>
In reply to#1367090

[Multipart message — attachments visible in raw view] — view raw

Arnd Bergmann <arnd@arndb.de> writes:

> [ text/plain ]
> On Wednesday 30 March 2016 13:31:01 Felipe Balbi wrote:
>> Arnd Bergmann <arnd@arndb.de> writes:
>> > [ text/plain ]
>> > The phy-am335x driver selects 'USB_COMMON', but all other drivers
>> > use 'depends on' for that symbol, and it depends on USB || USB_GADGET
>> > itself, which causes a Kconfig warning:
>> >
>> > warning: (AM335X_PHY_USB) selects USB_COMMON which has unmet direct dependencies (USB_SUPPORT && (USB || USB_GADGET))
>> >
>> > As suggested by Felipe Balbi, this turns the logic around, and makes
>> > 'USB_COMMON' selected by everything else that needs it, so we can
>> > remove the dependencies.
>> >
>> > Fixes: 59f042f644c5 ("usb: phy: phy-am335x: bypass first VBUS sensing for host-only mode")
>> > Signed-off-by: Arnd Bergmann <arnd@arndb.de>
>> > Acked-by: Felipe Balbi <balbi@kernel.org>
>> > Reviewed-by: Peter Chen <peter.chen@nxp.com>
>> > ---
>> > I seem to have dropped the ball on this one after my initial submission
>> > when it wasn't clear who should merge the patch.
>> >
>> > Please apply to the USB or the USB gadget tree, whoever gets there
>> > first.
>> 
>> seems like this is one of those "can wait for next merge window" right ?
>> I'll take it through my tree. No issues.
>
> Correct, it's not urgent at all, I just want it to disappear from
> my backlog of unapplied patches.

https://git.kernel.org/cgit/linux/kernel/git/balbi/usb.git/commit/?h=testing/next&id=5412212bf636f75cba8a0e3637e666b279e7067a

-- 
balbi

[toc] | [prev] | [next] | [standalone]

#1367231

FromArnd Bergmann <arnd@arndb.de>
Date2016-03-30 16:00 +0200
Message-ID<rioP8-6bC-5@gated-at.bofh.it>
In reply to#1367090
On Wednesday 30 March 2016 13:31:01 Felipe Balbi wrote:
> Arnd Bergmann <arnd@arndb.de> writes:
> > [ text/plain ]
> > The phy-am335x driver selects 'USB_COMMON', but all other drivers
> > use 'depends on' for that symbol, and it depends on USB || USB_GADGET
> > itself, which causes a Kconfig warning:
> >
> > warning: (AM335X_PHY_USB) selects USB_COMMON which has unmet direct dependencies (USB_SUPPORT && (USB || USB_GADGET))
> >
> > As suggested by Felipe Balbi, this turns the logic around, and makes
> > 'USB_COMMON' selected by everything else that needs it, so we can
> > remove the dependencies.
> >
> > Fixes: 59f042f644c5 ("usb: phy: phy-am335x: bypass first VBUS sensing for host-only mode")
> > Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> > Acked-by: Felipe Balbi <balbi@kernel.org>
> > Reviewed-by: Peter Chen <peter.chen@nxp.com>
> > ---
> > I seem to have dropped the ball on this one after my initial submission
> > when it wasn't clear who should merge the patch.
> >
> > Please apply to the USB or the USB gadget tree, whoever gets there
> > first.
> 
> seems like this is one of those "can wait for next merge window" right ?
> I'll take it through my tree. No issues.

Correct, it's not urgent at all, I just want it to disappear from
my backlog of unapplied patches.

	Arnd

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web