All of lore.kernel.org
 help / color / mirror / Atom feed
From: Daniel Mack <zonque@gmail.com>
To: George Cherian <george.cherian@ti.com>,
	linux-kernel@vger.kernel.org, linux-usb@vger.kernel.org,
	linux-omap@vger.kernel.org
Cc: balbi@ti.com, gregkh@linuxfoundation.org
Subject: Re: [PATCH v3 0/5] Add support for SW babble Control
Date: Tue, 13 May 2014 15:30:36 +0200	[thread overview]
Message-ID: <53721E7C.3090406@gmail.com> (raw)
In-Reply-To: <53721D24.4090704@ti.com>

On 05/13/2014 03:24 PM, George Cherian wrote:
> Basically, there are 2 types of babble conditions.
> 1) Transient babble condition - which could be recovered from without an 
> IP reset .
> 2) Babble condition - which could be recovered from only by doing an IP 
> reset.

Ok, thanks for the explanation.

> Looks like you are always hitting case 2 (Most times am also hitting the 
> same).

Seems like, yes.

> There is no 100% reliable method to force it to happen. Following is
> my setup ,
> I have a HUB with 4 devices connected , which gives me a Babble interrupt
> on both connects and disconnects ( Not always though).

I also get them at disconnects, but only with one specific USB device.

But as I don't ever see case 1) above, I can't say if your approach
works. What I can say, though, is that your patches don't break the
recovery from babble conditions that I experienced :)


Daniel

  reply	other threads:[~2014-05-13 13:30 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-13  8:31 [PATCH v3 0/5] Add support for SW babble Control George Cherian
2014-05-13  8:31 ` George Cherian
2014-05-13  8:31 ` [PATCH v3 1/5] usb: musb: core: Convert babble recover work to delayed work George Cherian
2014-05-13  8:31   ` George Cherian
2014-05-13  8:31 ` [PATCH v3 2/5] usb: musb: dsps: Call usb_phy(_shutdown/_init) during musb_platform_reset() George Cherian
2014-05-13  8:31   ` George Cherian
2014-05-13  8:31 ` [PATCH v3 3/5] usb: musb: core: Convert the musb_platform_reset to have a return value George Cherian
2014-05-13  8:31   ` George Cherian
2014-05-13  8:31 ` [PATCH v3 4/5] usb: musb: dsps: Add the sw_babble_control() George Cherian
2014-05-13  8:31   ` George Cherian
2014-05-13  8:31 ` [PATCH v3 5/5] usb: musb: dsps: Enable sw babble control for newer silicon George Cherian
2014-05-13  8:31   ` George Cherian
2014-05-13  9:46 ` [PATCH v3 0/5] Add support for SW babble Control Daniel Mack
2014-05-13 11:57   ` George Cherian
2014-05-13 11:57     ` George Cherian
2014-05-13 12:20     ` Daniel Mack
2014-05-13 12:57       ` George Cherian
2014-05-13 12:57         ` George Cherian
2014-05-13 13:14         ` Daniel Mack
2014-05-13 13:24           ` George Cherian
2014-05-13 13:24             ` George Cherian
2014-05-13 13:30             ` Daniel Mack [this message]
2014-05-13 18:37             ` Bin Liu
2014-05-14  5:37               ` George Cherian
2014-05-14  5:37                 ` George Cherian
2014-05-14 14:34                 ` Bin Liu
2014-05-14 16:43                   ` Bin Liu
2014-05-15  6:28                     ` George Cherian
2014-05-15  6:28                       ` George Cherian
2014-05-15 15:19                       ` Bin Liu
2014-05-19  8:40                         ` George Cherian
2014-05-19  8:40                           ` George Cherian
2014-05-19 13:53                           ` Bin Liu

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=53721E7C.3090406@gmail.com \
    --to=zonque@gmail.com \
    --cc=balbi@ti.com \
    --cc=george.cherian@ti.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=linux-usb@vger.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.