On 16/07/15 03:54, Peter Chen wrote: > On Wed, Jul 15, 2015 at 04:30:27PM +0300, Roger Quadros wrote: >> On 14/07/15 03:34, Peter Chen wrote: >>> On Mon, Jul 13, 2015 at 01:13:54PM +0300, Roger Quadros wrote: >>>> Peter, >>>> >>>> On 13/07/15 04:58, Peter Chen wrote: >>>>> On Wed, Jul 08, 2015 at 01:19:30PM +0300, Roger Quadros wrote: >>>>>> This is to prevent missing symbol build error if OTG is >>>>>> enabled (built-in) and HCD core (CONFIG_USB) is module. >>>>>> >>>>> >>>>> We may let the OTG-DRD/OTG-FSM depends on CONFIG_USB to fix it. >>>> >>>> CONFIG_OTG already depends on CONFIG_USB as it is a sub-option of >>>> CONFIG_USB. It doesn't depend on CONFIG_USB_GADGET and that can >>>> be fixed. >>>> >>>> But dependency is not the problem here. Symbols not available to >>>> OTG driver when USB/GADGET is 'm' is the problem. >>>> >>>> e.g. >>>> CONFIG_USB_OTG is always built-in. >>>> we need to work if CONFIG_USB is 'm'/'y' >>>> _and_ if CONFIG_USB_GADGET is 'm'/'y' >>>> >>> >>> below should fix this issue, but we may need to make some >>> changes for code which are defined by CONFIG_USB_OTG. >>> >>> diff --git a/drivers/usb/core/Kconfig b/drivers/usb/core/Kconfig >>> index a99c89e..5e374ad 100644 >>> --- a/drivers/usb/core/Kconfig >>> +++ b/drivers/usb/core/Kconfig >>> @@ -42,8 +42,9 @@ config USB_DYNAMIC_MINORS >>> If you are unsure about this, say N here. >>> >>> config USB_OTG >>> - bool "OTG support" >>> + tristate "OTG support" >>> depends on PM >>> + depends on USB && USB_GADGET >>> default n >>> help >>> The most notable feature of >>> USB OTG is support for a >> >> With this USB_OTG will become 'm' when either USB or USB_GADGET is m >> and will break if either USB or USB_GADGET is made y as all OTG core >> API symbols won't be available. :) >> > > Ok, after thinking more, seems we can't handle properly if USB_OTG as > 'm', your idea that using host/gadget/fsm->ops to call hcd/gadget API > and the controller driver will defines these ops (due to it will use > hcd/gadget function) is proper way currently. >
Can I take this as your Ack for this patch? :) cheers, -roger -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
