* [PATCH] usb: phy-am335x: clarify USB_COMMON dependency
@ 2016-01-20 10:56 Arnd Bergmann
2016-01-20 18:08 ` Bin Liu
0 siblings, 1 reply; 5+ messages in thread
From: Arnd Bergmann @ 2016-01-20 10:56 UTC (permalink / raw)
To: Bin Liu; +Cc: Felipe Balbi, balbif, Greg Kroah-Hartman, linux-usb, linux-kernel
The phy-am335x driver accidentally selects 'USB_COMMON', which is
not correct as that symbol should indicate whether USB host or
target mode is enabled, but it might not:
warning: (AM335X_PHY_USB) selects USB_COMMON which has unmet direct dependencies (USB_SUPPORT && (USB || USB_GADGET))
This changes the 'select' into 'depends on', as it should have
been initially.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Fixes: 59f042f644c5 ("usb: phy: phy-am335x: bypass first VBUS sensing for host-only mode")
diff --git a/drivers/usb/phy/Kconfig b/drivers/usb/phy/Kconfig
index c6904742e2aa..29f063c59e1a 100644
--- a/drivers/usb/phy/Kconfig
+++ b/drivers/usb/phy/Kconfig
@@ -63,10 +63,10 @@ config AM335X_CONTROL_USB
config AM335X_PHY_USB
tristate "AM335x USB PHY Driver"
depends on ARM || COMPILE_TEST
+ depends on USB_COMMON
select USB_PHY
select AM335X_CONTROL_USB
select NOP_USB_XCEIV
- select USB_COMMON
help
This driver provides PHY support for that phy which part for the
AM335x SoC.
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH] usb: phy-am335x: clarify USB_COMMON dependency 2016-01-20 10:56 [PATCH] usb: phy-am335x: clarify USB_COMMON dependency Arnd Bergmann @ 2016-01-20 18:08 ` Bin Liu 2016-02-03 17:54 ` Felipe Balbi 0 siblings, 1 reply; 5+ messages in thread From: Bin Liu @ 2016-01-20 18:08 UTC (permalink / raw) To: Arnd Bergmann Cc: Felipe Balbi, balbif, Greg Kroah-Hartman, linux-usb, linux-kernel Hi, On Wed, Jan 20, 2016 at 11:56:34AM +0100, Arnd Bergmann wrote: > The phy-am335x driver accidentally selects 'USB_COMMON', which is > not correct as that symbol should indicate whether USB host or > target mode is enabled, but it might not: > > warning: (AM335X_PHY_USB) selects USB_COMMON which has unmet direct dependencies (USB_SUPPORT && (USB || USB_GADGET)) Is it possible to have a .config which has either USB nor USB_GADGET set but AM335X_PHY_USB is set to be in the situation as in the warning above? > > This changes the 'select' into 'depends on', as it should have > been initially. Using 'depends on' causes AM335x_PHY_USB driver unable to built-in, since USB_COMMON is an invisible option and its defualt is 'M'. Regards, -Bin. > > Signed-off-by: Arnd Bergmann <arnd@arndb.de> > Fixes: 59f042f644c5 ("usb: phy: phy-am335x: bypass first VBUS sensing for host-only mode") > > diff --git a/drivers/usb/phy/Kconfig b/drivers/usb/phy/Kconfig > index c6904742e2aa..29f063c59e1a 100644 > --- a/drivers/usb/phy/Kconfig > +++ b/drivers/usb/phy/Kconfig > @@ -63,10 +63,10 @@ config AM335X_CONTROL_USB > config AM335X_PHY_USB > tristate "AM335x USB PHY Driver" > depends on ARM || COMPILE_TEST > + depends on USB_COMMON > select USB_PHY > select AM335X_CONTROL_USB > select NOP_USB_XCEIV > - select USB_COMMON > help > This driver provides PHY support for that phy which part for the > AM335x SoC. > ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] usb: phy-am335x: clarify USB_COMMON dependency 2016-01-20 18:08 ` Bin Liu @ 2016-02-03 17:54 ` Felipe Balbi 2016-02-03 21:08 ` Arnd Bergmann 0 siblings, 1 reply; 5+ messages in thread From: Felipe Balbi @ 2016-02-03 17:54 UTC (permalink / raw) To: Bin Liu, Arnd Bergmann Cc: Felipe Balbi, balbif, Greg Kroah-Hartman, linux-usb, linux-kernel [-- Attachment #1: Type: text/plain, Size: 895 bytes --] hi, Bin Liu <b-liu@ti.com> writes: > On Wed, Jan 20, 2016 at 11:56:34AM +0100, Arnd Bergmann wrote: >> The phy-am335x driver accidentally selects 'USB_COMMON', which is >> not correct as that symbol should indicate whether USB host or >> target mode is enabled, but it might not: >> >> warning: (AM335X_PHY_USB) selects USB_COMMON which has unmet direct dependencies (USB_SUPPORT && (USB || USB_GADGET)) > > Is it possible to have a .config which has either USB nor USB_GADGET > set but AM335X_PHY_USB is set to be in the situation as in the warning > above? > >> >> This changes the 'select' into 'depends on', as it should have >> been initially. > > Using 'depends on' causes AM335x_PHY_USB driver unable to built-in, > since USB_COMMON is an invisible option and its defualt is 'M'. yeah, USB_COMMON is supposed to be selected, not depended upon. -- balbi [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 818 bytes --] ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] usb: phy-am335x: clarify USB_COMMON dependency 2016-02-03 17:54 ` Felipe Balbi @ 2016-02-03 21:08 ` Arnd Bergmann 2016-02-03 22:04 ` Felipe Balbi 0 siblings, 1 reply; 5+ messages in thread From: Arnd Bergmann @ 2016-02-03 21:08 UTC (permalink / raw) To: Felipe Balbi Cc: Bin Liu, Felipe Balbi, balbif, Greg Kroah-Hartman, linux-usb, linux-kernel On Wednesday 03 February 2016 19:54:02 Felipe Balbi wrote: > > hi, > > Bin Liu <b-liu@ti.com> writes: > > On Wed, Jan 20, 2016 at 11:56:34AM +0100, Arnd Bergmann wrote: > >> The phy-am335x driver accidentally selects 'USB_COMMON', which is > >> not correct as that symbol should indicate whether USB host or > >> target mode is enabled, but it might not: > >> > >> warning: (AM335X_PHY_USB) selects USB_COMMON which has unmet direct dependencies (USB_SUPPORT && (USB || USB_GADGET)) > > > > Is it possible to have a .config which has either USB nor USB_GADGET > > set but AM335X_PHY_USB is set to be in the situation as in the warning > > above? > > > >> > >> This changes the 'select' into 'depends on', as it should have > >> been initially. > > > > Using 'depends on' causes AM335x_PHY_USB driver unable to built-in, > > since USB_COMMON is an invisible option and its defualt is 'M'. > > > yeah, USB_COMMON is supposed to be selected, not depended upon. That means we should do something like the patch below? Fine with me too, but maybe you can write the changelog for that, as I don't know the background behind the logic. Arnd diff --git a/drivers/phy/Kconfig b/drivers/phy/Kconfig index e7e117d5dbbe..df005f5a0e0c 100644 --- a/drivers/phy/Kconfig +++ b/drivers/phy/Kconfig @@ -249,7 +249,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 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] usb: phy-am335x: clarify USB_COMMON dependency 2016-02-03 21:08 ` Arnd Bergmann @ 2016-02-03 22:04 ` Felipe Balbi 0 siblings, 0 replies; 5+ messages in thread From: Felipe Balbi @ 2016-02-03 22:04 UTC (permalink / raw) To: Arnd Bergmann Cc: Bin Liu, Felipe Balbi, balbif, Greg Kroah-Hartman, linux-usb, linux-kernel [-- Attachment #1: Type: text/plain, Size: 2748 bytes --] Hi, Arnd Bergmann <arnd@arndb.de> writes: >> > On Wed, Jan 20, 2016 at 11:56:34AM +0100, Arnd Bergmann wrote: >> >> The phy-am335x driver accidentally selects 'USB_COMMON', which is >> >> not correct as that symbol should indicate whether USB host or >> >> target mode is enabled, but it might not: >> >> >> >> warning: (AM335X_PHY_USB) selects USB_COMMON which has unmet direct dependencies (USB_SUPPORT && (USB || USB_GADGET)) >> > >> > Is it possible to have a .config which has either USB nor USB_GADGET >> > set but AM335X_PHY_USB is set to be in the situation as in the warning >> > above? >> > >> >> >> >> This changes the 'select' into 'depends on', as it should have >> >> been initially. >> > >> > Using 'depends on' causes AM335x_PHY_USB driver unable to built-in, >> > since USB_COMMON is an invisible option and its defualt is 'M'. >> >> >> yeah, USB_COMMON is supposed to be selected, not depended upon. > > > That means we should do something like the patch below? > > Fine with me too, but maybe you can write the changelog for that, > as I don't know the background behind the logic. > > Arnd > > diff --git a/drivers/phy/Kconfig b/drivers/phy/Kconfig > index e7e117d5dbbe..df005f5a0e0c 100644 > --- a/drivers/phy/Kconfig > +++ b/drivers/phy/Kconfig > @@ -249,7 +249,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 looks great to me. Care to send it in proper patch form ? -- balbi [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 818 bytes --] ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2016-02-03 22:04 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2016-01-20 10:56 [PATCH] usb: phy-am335x: clarify USB_COMMON dependency Arnd Bergmann 2016-01-20 18:08 ` Bin Liu 2016-02-03 17:54 ` Felipe Balbi 2016-02-03 21:08 ` Arnd Bergmann 2016-02-03 22:04 ` Felipe Balbi
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).