All of lore.kernel.org
 help / color / mirror / Atom feed
From: Greg KH <gregkh@linuxfoundation.org>
To: Stephen Rothwell <sfr@canb.auug.org.au>, tomoya.rohm@gmail.com
Cc: Paul Gortmaker <paul.gortmaker@windriver.com>,
	Felipe Balbi <balbi@ti.com>,
	linux-next@vger.kernel.org
Subject: Re: "usb: gadget: pch_udc: Detecting VBUS through GPIO with interrupt" breaks sparc
Date: Mon, 5 Mar 2012 08:43:08 -0800	[thread overview]
Message-ID: <20120305164308.GE2459@kroah.com> (raw)
In-Reply-To: <20120303104142.9615e7a2d9330a5443e0e027@canb.auug.org.au>

On Sat, Mar 03, 2012 at 10:41:42AM +1100, Stephen Rothwell wrote:
> [Just cc'ing Greg as this commit is in the usb tree]
> 
> On Fri, 2 Mar 2012 17:01:16 -0500 Paul Gortmaker <paul.gortmaker@windriver.com> wrote:
> >
> > This commit:
> > 
> > ---------------
> > commit 637b78eb31e0b167ed913f1750bb645dfeda38f0
> > Author: Tomoya MORINAGA <tomoya.rohm@gmail.com>
> > Date:   Fri Feb 3 16:14:18 2012 +0900
> > 
> >     usb: gadget: pch_udc: Detecting VBUS through GPIO with interrupt
> >     
> >     Problem:
> >      pch_udc continues operation even if VBUS becomes Low.
> >      pch_udc performs D+ pulling up before VBUS becomes High.
> >      USB device should be controlled according to VBUS state.
> >     
> >     Root cause:
> >      The current pch_udc is not always monitoring VBUS.
> >     
> >     Solution:
> >      The change of VBUS is detected using an interrupt of GPIO.
> >      If VBUS became Low, pch_udc handles 'disconnect'.
> >      After VBUS became High, a pull improves D+, and pch_udc
> >      handles 'connect'.
> >     
> >     [ balbi@ti.com : make it actually compile ]
> >     
> >     Signed-off-by: Tomoya MORINAGA <tomoya.rohm@gmail.com>
> >     Signed-off-by: Felipe Balbi <balbi@ti.com>
> > ---------------
> > 
> > breaks the sparc builds in next:
> > 
> > http://kisskb.ellerman.id.au/kisskb/buildresult/5771890/
> > http://kisskb.ellerman.id.au/kisskb/buildresult/5771905/

Ick, not good.

Tomoya, can you please send me a patch to fix this?  Felipe is off-line
for a few days and can't do this himself.


greg k-h

  reply	other threads:[~2012-03-05 16:53 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-02 22:01 "usb: gadget: pch_udc: Detecting VBUS through GPIO with interrupt" breaks sparc Paul Gortmaker
2012-03-02 23:41 ` Stephen Rothwell
2012-03-05 16:43   ` Greg KH [this message]
2012-03-06  0:07     ` Tomoya MORINAGA
2012-03-06  2:35       ` [PATCH][linux-next] usb/gadget/pch_udc: Fix compile error Tomoya MORINAGA
2012-03-06  2:49     ` [PATCH][linux-next][RESEND] " Tomoya MORINAGA
2012-03-06  6:39       ` Tomoya MORINAGA
2012-03-06 14:58         ` Greg Kroah-Hartman
2012-03-07  0:44           ` Tomoya MORINAGA

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120305164308.GE2459@kroah.com \
    --to=gregkh@linuxfoundation.org \
    --cc=balbi@ti.com \
    --cc=linux-next@vger.kernel.org \
    --cc=paul.gortmaker@windriver.com \
    --cc=sfr@canb.auug.org.au \
    --cc=tomoya.rohm@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.