All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sneeker Yeh <sneeker.yeh@gmail.com>
To: Felipe Balbi <balbi@ti.com>
Cc: Kumar Gala <galak@codeaurora.org>,
	linux-usb@vger.kernel.org, Rob Herring <robh+dt@kernel.org>,
	Sneeker Yeh <Sneeker.Yeh@tw.fujitsu.com>,
	Andy Green <andy.green@linaro.org>,
	Ian Campbell <ijc+devicetree@hellion.org.uk>,
	linux-kernel@vger.kernel.org,
	Kishon Vijay Abraham I <kishon@ti.com>,
	Pawel Moll <pawel.moll@arm.com>,
	Mark Rutland <mark.rutland@arm.com>,
	Mathias Nyman <mathias.nyman@intel.com>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Huang Rui <ray.huang@amd.com>,
	Jassi Brar <jaswinder.singh@linaro.org>,
	linux-omap@vger.kernel.org, devicetree@vger.kernel.org,
	Grant Likely <grant.likely@linaro.org>
Subject: Re: [PATCH v4 1/5] xhci: add a quirk for device disconnection errata for Synopsis Designware USB3 core
Date: Wed, 18 Feb 2015 22:39:53 +0800	[thread overview]
Message-ID: <CAJ1gpc3CT8RBZaXDbFz67mG5RcKOEfoxLjh8PEMq1Cme1QW1Vg@mail.gmail.com> (raw)
In-Reply-To: <20150218143345.GC3651@saruman.tx.rr.com>

[-- Attachment #1: Type: text/plain, Size: 1207 bytes --]

Hi,
2015/2/18 下午10:34 於 "Felipe Balbi" <balbi@ti.com> 寫道：
>
> On Wed, Feb 18, 2015 at 10:47:45AM +0200, Mathias Nyman wrote:
> > Hi
> >
> > This looks correct, but if you are still going to make a new series
fixing
> > Felipe's comments then the following tiny nitpicks could be fixed as
well.
> >
> > Otherwise
> >
> > Acked-by: Mathias Nyman <mathias.nyman@linux.intel.com>
> >
> > Felipe, Do you want to take this series through your tree?
>
> I can, no issues :-)
>
> > On 17.02.2015 07:41, Sneeker Yeh wrote:
> > >
> > > +static void xhci_try_to_clear_csc(struct usb_hcd *hcd, int
dev_port_num)
> >
> > Either add a function description explaining something like "Late
> > clearing of connect status.
> > Some quirky hardware will suspend the controller when CSC bit is
> > cleared and leave URBs unhandled"
> >
> > Or change the function name to something like
> > xhci_late_csc_clear_quirk() or xhci_deferred_csc_clear_quirk()
>
> I have a feeling xhci_late_csc_clear_quirk() is a better name, much more
> descriptive than xhci_try_to_clear_csc() :-)

Hmm, I'll use this one, ^^
Thanks for comment it.

Much appreciate,
Sneeker

>
> --
> balbi

[-- Attachment #2: Type: text/html, Size: 1698 bytes --]

  reply	other threads:[~2015-02-18 14:39 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-17  5:41 [PATCH v4 0/5] Add support for Fujitsu USB host controller Sneeker Yeh
2015-02-17  5:41 ` [PATCH v4 1/5] xhci: add a quirk for device disconnection errata for Synopsis Designware USB3 core Sneeker Yeh
2015-02-18  8:47   ` Mathias Nyman
2015-02-18  8:47     ` Mathias Nyman
2015-02-18 14:02     ` Sneeker Yeh
2015-02-18 14:02       ` Sneeker Yeh
2015-02-18 14:33     ` Felipe Balbi
2015-02-18 14:33       ` Felipe Balbi
2015-02-18 14:39       ` Sneeker Yeh [this message]
2015-02-18 14:42         ` Felipe Balbi
2015-02-18 14:42           ` Felipe Balbi
2015-02-17  5:41 ` [PATCH v4 2/5] xhci: Platform: Set Synopsis device disconnection quirk based on platform data Sneeker Yeh
2015-02-17  5:41 ` [PATCH v4 3/5] usb: dwc3: add revision number DWC3_REVISION_290A and DWC3_REVISION_300A Sneeker Yeh
2015-02-17  5:41 ` [PATCH v4 4/5] usb: dwc3: Add quirk for Synopsis device disconnection errata Sneeker Yeh
2015-02-17  5:41 ` [PATCH v4 5/5] usb: dwc3: add Fujitsu Specific Glue layer Sneeker Yeh
2015-02-17 19:26   ` Felipe Balbi
2015-02-17 19:26     ` Felipe Balbi
2015-02-18 13:55     ` Sneeker Yeh

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=CAJ1gpc3CT8RBZaXDbFz67mG5RcKOEfoxLjh8PEMq1Cme1QW1Vg@mail.gmail.com \
    --to=sneeker.yeh@gmail.com \
    --cc=Sneeker.Yeh@tw.fujitsu.com \
    --cc=andy.green@linaro.org \
    --cc=balbi@ti.com \
    --cc=devicetree@vger.kernel.org \
    --cc=galak@codeaurora.org \
    --cc=grant.likely@linaro.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=ijc+devicetree@hellion.org.uk \
    --cc=jaswinder.singh@linaro.org \
    --cc=kishon@ti.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=mathias.nyman@intel.com \
    --cc=pawel.moll@arm.com \
    --cc=ray.huang@amd.com \
    --cc=robh+dt@kernel.org \
    /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.