linux-omap.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] usb: musb: omap2430: fix build failure
@ 2011-04-05 17:50 Johan Hovold
       [not found] ` <1302025834-25729-1-git-send-email-jhovold-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
  0 siblings, 1 reply; 5+ messages in thread
From: Johan Hovold @ 2011-04-05 17:50 UTC (permalink / raw)
  To: Felipe Balbi
  Cc: Greg Kroah-Hartman, Hema HK, linux-usb, linux-omap, Johan Hovold

Fix build failure introduced by commit
7acc6197b76edd0b932a7cbcc6cfad0a8a87f026 (usb: musb: Idle path retention
and offmode support for OMAP3) when building without gadget
support.

  CC      drivers/usb/musb/omap2430.o
drivers/usb/musb/omap2430.c: In function ‘musb_otg_notifications’:
drivers/usb/musb/omap2430.c:262: error: ‘struct musb’ has no member named ‘gadget_driver’

Signed-off-by: Johan Hovold <jhovold@gmail.com>
---
 drivers/usb/musb/omap2430.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/drivers/usb/musb/omap2430.c b/drivers/usb/musb/omap2430.c
index 25cb8b0..57a27fa 100644
--- a/drivers/usb/musb/omap2430.c
+++ b/drivers/usb/musb/omap2430.c
@@ -259,9 +259,10 @@ static int musb_otg_notifications(struct notifier_block *nb,
 	case USB_EVENT_VBUS:
 		DBG(4, "VBUS Connect\n");
 
+#ifdef CONFIG_USB_GADGET_MUSB_HDRC
 		if (musb->gadget_driver)
 			pm_runtime_get_sync(musb->controller);
-
+#endif
 		otg_init(musb->xceiv);
 		break;
 
-- 
1.7.4.1

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] usb: musb: omap2430: fix build failure
       [not found] ` <1302025834-25729-1-git-send-email-jhovold-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
@ 2011-04-13 17:01   ` Johan Hovold
  2011-04-13 17:22     ` Felipe Balbi
  0 siblings, 1 reply; 5+ messages in thread
From: Johan Hovold @ 2011-04-13 17:01 UTC (permalink / raw)
  To: Felipe Balbi
  Cc: Greg Kroah-Hartman, Hema HK, linux-usb-u79uwXL29TY76Z2rM5mHXA,
	linux-omap-u79uwXL29TY76Z2rM5mHXA

Hi Filipe, 

On Tue, Apr 05, 2011 at 07:50:34PM +0200, Johan Hovold wrote:
> Fix build failure introduced by commit
> 7acc6197b76edd0b932a7cbcc6cfad0a8a87f026 (usb: musb: Idle path retention
> and offmode support for OMAP3) when building without gadget
> support.
> 
>   CC      drivers/usb/musb/omap2430.o
> drivers/usb/musb/omap2430.c: In function ‘musb_otg_notifications’:
> drivers/usb/musb/omap2430.c:262: error: ‘struct musb’ has no member named ‘gadget_driver’

I noted that this one wasn't included in your pull-request today. Just
thought I'd send a reminder as I haven't received any feedback.

Thanks,
Johan
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] usb: musb: omap2430: fix build failure
  2011-04-13 17:01   ` Johan Hovold
@ 2011-04-13 17:22     ` Felipe Balbi
  2011-04-13 22:27       ` Greg KH
  0 siblings, 1 reply; 5+ messages in thread
From: Felipe Balbi @ 2011-04-13 17:22 UTC (permalink / raw)
  To: Johan Hovold
  Cc: Felipe Balbi, Greg Kroah-Hartman, Hema HK,
	linux-usb-u79uwXL29TY76Z2rM5mHXA,
	linux-omap-u79uwXL29TY76Z2rM5mHXA

Hi,

On Wed, Apr 13, 2011 at 07:01:39PM +0200, Johan Hovold wrote:
> On Tue, Apr 05, 2011 at 07:50:34PM +0200, Johan Hovold wrote:
> > Fix build failure introduced by commit
> > 7acc6197b76edd0b932a7cbcc6cfad0a8a87f026 (usb: musb: Idle path retention
> > and offmode support for OMAP3) when building without gadget
> > support.
> > 
> >   CC      drivers/usb/musb/omap2430.o
> > drivers/usb/musb/omap2430.c: In function ‘musb_otg_notifications’:
> > drivers/usb/musb/omap2430.c:262: error: ‘struct musb’ has no member named ‘gadget_driver’
> 
> I noted that this one wasn't included in your pull-request today. Just
> thought I'd send a reminder as I haven't received any feedback.

Sorry about that, I missed it from my queue. I'll add and send on a
separate pull request. Thanks for the note.

-- 
balbi
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] usb: musb: omap2430: fix build failure
  2011-04-13 17:22     ` Felipe Balbi
@ 2011-04-13 22:27       ` Greg KH
  2011-04-14 10:34         ` Felipe Balbi
  0 siblings, 1 reply; 5+ messages in thread
From: Greg KH @ 2011-04-13 22:27 UTC (permalink / raw)
  To: Felipe Balbi
  Cc: Johan Hovold, Greg Kroah-Hartman, Hema HK, linux-usb, linux-omap

On Wed, Apr 13, 2011 at 08:22:51PM +0300, Felipe Balbi wrote:
> Hi,
> 
> On Wed, Apr 13, 2011 at 07:01:39PM +0200, Johan Hovold wrote:
> > On Tue, Apr 05, 2011 at 07:50:34PM +0200, Johan Hovold wrote:
> > > Fix build failure introduced by commit
> > > 7acc6197b76edd0b932a7cbcc6cfad0a8a87f026 (usb: musb: Idle path retention
> > > and offmode support for OMAP3) when building without gadget
> > > support.
> > > 
> > >   CC      drivers/usb/musb/omap2430.o
> > > drivers/usb/musb/omap2430.c: In function ‘musb_otg_notifications’:
> > > drivers/usb/musb/omap2430.c:262: error: ‘struct musb’ has no member named ‘gadget_driver’
> > 
> > I noted that this one wasn't included in your pull-request today. Just
> > thought I'd send a reminder as I haven't received any feedback.
> 
> Sorry about that, I missed it from my queue. I'll add and send on a
> separate pull request. Thanks for the note.

I can pick it up right now.

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] usb: musb: omap2430: fix build failure
  2011-04-13 22:27       ` Greg KH
@ 2011-04-14 10:34         ` Felipe Balbi
  0 siblings, 0 replies; 5+ messages in thread
From: Felipe Balbi @ 2011-04-14 10:34 UTC (permalink / raw)
  To: Greg KH
  Cc: Felipe Balbi, Johan Hovold, Greg Kroah-Hartman, Hema HK,
	linux-usb, linux-omap

On Wed, Apr 13, 2011 at 03:27:44PM -0700, Greg KH wrote:
> On Wed, Apr 13, 2011 at 08:22:51PM +0300, Felipe Balbi wrote:
> > Hi,
> > 
> > On Wed, Apr 13, 2011 at 07:01:39PM +0200, Johan Hovold wrote:
> > > On Tue, Apr 05, 2011 at 07:50:34PM +0200, Johan Hovold wrote:
> > > > Fix build failure introduced by commit
> > > > 7acc6197b76edd0b932a7cbcc6cfad0a8a87f026 (usb: musb: Idle path retention
> > > > and offmode support for OMAP3) when building without gadget
> > > > support.
> > > > 
> > > >   CC      drivers/usb/musb/omap2430.o
> > > > drivers/usb/musb/omap2430.c: In function ‘musb_otg_notifications’:
> > > > drivers/usb/musb/omap2430.c:262: error: ‘struct musb’ has no member named ‘gadget_driver’
> > > 
> > > I noted that this one wasn't included in your pull-request today. Just
> > > thought I'd send a reminder as I haven't received any feedback.
> > 
> > Sorry about that, I missed it from my queue. I'll add and send on a
> > separate pull request. Thanks for the note.
> 
> I can pick it up right now.

Thanks Greg, here's my SOB line:

Signed-off-by: Felipe Balbi <balbi@ti.com>

-- 
balbi
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2011-04-14 10:34 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-04-05 17:50 [PATCH] usb: musb: omap2430: fix build failure Johan Hovold
     [not found] ` <1302025834-25729-1-git-send-email-jhovold-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2011-04-13 17:01   ` Johan Hovold
2011-04-13 17:22     ` Felipe Balbi
2011-04-13 22:27       ` Greg KH
2011-04-14 10:34         ` 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).