All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paul Kocialkowski <contact@paulk.fr>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/2] sunxi: musb: Fix "usb reset" handling
Date: Fri, 19 Jun 2015 11:11:26 +0200	[thread overview]
Message-ID: <1434705086.2453.1.camel@collins> (raw)
In-Reply-To: <558178AD.8070408@redhat.com>

Le mercredi 17 juin 2015 ? 15:39 +0200, Hans de Goede a ?crit :
> Hi,
> 
> On 15-06-15 21:21, Paul Kocialkowski wrote:
> > Le dimanche 14 juin 2015 ? 12:40 +0200, Hans de Goede a ?crit :
> >> Hi Ian, Paul,
> >>
> >> Here is a patch to fix the problems where most usb devices will no longer
> >> work after a "usb reset ", when connected to the otg controller in host
> >> mode + a related cleanup patch.
> >
> > This works fine for an USB storage device and an USB2 keyboard but does
> > not work with an USB1 keyboard, with error:
> >
> > sunxi# usb reset
> > resetting USB...
> > USB0:   scanning bus 0 for devices... USB device descriptor short read
> > (expected 8, got 0)
> > No USB Device found
> 
> Hmm, did you test my sunxi-wip branch perhaps? This bug does exist there,
> but it is the result of me refactoring things so that the musb code can
> use the device-model when build in host mode, which will allow enabling
> both the otg port in host mode and regular usb hosts in a single build,
> which is esp. useful for boards which have the otg hooked up in host-only
> mode (e.g. connected to an usb-a receptacle, or usb <-> sata bridge).

This is nice, thanks a lot for all your work!

I did however test on top of the regular U-Boot tree. I'll try again in
a few days to see if it still happens.

-- 
Paul Kocialkowski, Replicant developer

Replicant is a fully free Android distribution running on several
devices, a free software mobile operating system putting the emphasis on
freedom and privacy/security.

Website: http://www.replicant.us/
Blog: http://blog.replicant.us/
Wiki/tracker/forums: http://redmine.replicant.us/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150619/27357958/attachment.sig>

  reply	other threads:[~2015-06-19  9:11 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-06-14 10:40 [U-Boot] [PATCH 0/2] sunxi: musb: Fix "usb reset" handling Hans de Goede
2015-06-14 10:40 ` [U-Boot] [PATCH 1/2] sunxi: musb: Do not fully reset the controler from sunxi_musb_disable Hans de Goede
2015-06-14 11:46   ` Ian Campbell
2015-06-14 17:21     ` Hans de Goede
2015-06-14 10:40 ` [U-Boot] [PATCH 2/2] sunxi: musb: Remove unused sunxi_musb_exit method Hans de Goede
2015-06-14 11:47   ` Ian Campbell
2015-06-15 19:21 ` [U-Boot] [PATCH 0/2] sunxi: musb: Fix "usb reset" handling Paul Kocialkowski
2015-06-17 13:39   ` Hans de Goede
2015-06-19  9:11     ` Paul Kocialkowski [this message]
2015-06-19 12:35 ` Marek Vasut
2015-06-19 13:07   ` Hans de Goede
2015-06-19 13:08     ` Marek Vasut
2015-06-26 10:20 ` Maxime Ripard
2015-06-26 11:01   ` Hans de Goede
2015-06-28 20:42     ` Maxime Ripard
2015-06-26 11:02   ` Hans de Goede
2015-06-26 11:11     ` Maxime Ripard

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=1434705086.2453.1.camel@collins \
    --to=contact@paulk.fr \
    --cc=u-boot@lists.denx.de \
    /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.