Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752904AbbGONan (ORCPT ); Wed, 15 Jul 2015 09:30:43 -0400 Received: from comal.ext.ti.com ([198.47.26.152]:38575 "EHLO comal.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751894AbbGONam (ORCPT ); Wed, 15 Jul 2015 09:30:42 -0400 Message-ID: <55A66073.5010800@ti.com> Date: Wed, 15 Jul 2015 16:30:27 +0300 From: Roger Quadros User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 MIME-Version: 1.0 To: Peter Chen CC: , , , , , , , , , , Subject: Re: [PATCH v3 04/11] otg-fsm: move usb_bus_start_enum into otg-fsm->ops References: <1436350777-28056-1-git-send-email-rogerq@ti.com> <1436350777-28056-5-git-send-email-rogerq@ti.com> <20150713015759.GC27710@shlinux2> <55A38F62.8010509@ti.com> <20150714003427.GA31197@shlinux2> In-Reply-To: <20150714003427.GA31197@shlinux2> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3910 Lines: 115 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. :) cheers, -roger >> >>> >>> Peter >>> >>>> Signed-off-by: Roger Quadros >>>> --- >>>> drivers/usb/common/usb-otg-fsm.c | 6 ++++-- >>>> drivers/usb/phy/phy-fsl-usb.c | 2 ++ >>>> include/linux/usb/otg-fsm.h | 1 + >>>> 3 files changed, 7 insertions(+), 2 deletions(-) >>>> >>>> diff --git a/drivers/usb/common/usb-otg-fsm.c b/drivers/usb/common/usb-otg-fsm.c >>>> index 1873eb3..156fd25 100644 >>>> --- a/drivers/usb/common/usb-otg-fsm.c >>>> +++ b/drivers/usb/common/usb-otg-fsm.c >>>> @@ -166,8 +166,10 @@ static int otg_set_state(struct otg_fsm *fsm, enum usb_otg_state new_state) >>>> otg_loc_conn(fsm, 0); >>>> otg_loc_sof(fsm, 1); >>>> otg_set_protocol(fsm, PROTO_HOST); >>>> - usb_bus_start_enum(fsm->otg->host, >>>> - fsm->otg->host->otg_port); >>>> + if (fsm->ops->start_enum) { >>>> + fsm->ops->start_enum(fsm->otg->host, >>>> + fsm->otg->host->otg_port); >>>> + } >>>> break; >>>> case OTG_STATE_A_IDLE: >>>> otg_drv_vbus(fsm, 0); >>>> diff --git a/drivers/usb/phy/phy-fsl-usb.c b/drivers/usb/phy/phy-fsl-usb.c >>>> index ee3f2c2..19541ed 100644 >>>> --- a/drivers/usb/phy/phy-fsl-usb.c >>>> +++ b/drivers/usb/phy/phy-fsl-usb.c >>>> @@ -783,6 +783,8 @@ static struct otg_fsm_ops fsl_otg_ops = { >>>> >>>> .start_host = fsl_otg_start_host, >>>> .start_gadget = fsl_otg_start_gadget, >>>> + >>>> + .start_enum = usb_bus_start_enum, >>>> }; >>>> >>>> /* Initialize the global variable fsl_otg_dev and request IRQ for OTG */ >>>> diff --git a/include/linux/usb/otg-fsm.h b/include/linux/usb/otg-fsm.h >>>> index c631dde..22d8baa 100644 >>>> --- a/include/linux/usb/otg-fsm.h >>>> +++ b/include/linux/usb/otg-fsm.h >>>> @@ -198,6 +198,7 @@ struct otg_fsm_ops { >>>> void (*del_timer)(struct otg_fsm *fsm, enum otg_fsm_timer timer); >>>> int (*start_host)(struct otg_fsm *fsm, int on); >>>> int (*start_gadget)(struct otg_fsm *fsm, int on); >>>> + int (*start_enum)(struct usb_bus *bus, unsigned port_num); >>>> }; >>>> >>>> >>>> -- >>>> 2.1.4 >>>> >>> > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/