All of lore.kernel.org
 help / color / mirror / Atom feed
From: Maxime Ripard <maxime.ripard@free-electrons.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 12/13] sunxi: A13-Olinuxino: Enable the USB OTG controller
Date: Mon, 14 Sep 2015 23:19:20 +0200	[thread overview]
Message-ID: <20150914211920.GJ4684@lukather> (raw)
In-Reply-To: <55F5B493.4040208@redhat.com>

On Sun, Sep 13, 2015 at 07:38:27PM +0200, Hans de Goede wrote:
> >>The policy I'm thinking of (and which we are currently sorta following
> >>wrt enabling USB_MUSB_HOST) is that on devices with no normal usb
> >>ports we use USB_MUSB_HOST, so that people can interact with u-boot
> >>without needing a serial console (which often requires soldering).
> >>
> >>On boards which do have a normal usb port I fully agree that
> >>USB_MUSB_GADGET is a good default.
> >>
> >>In practice this will likely boil down to use USB_MUSB_HOST on tablets
> >>and USB_MUSB_GADGET everywhere else.
> >
> >Sounds good!
> >
> >If there's a UART gadget, we could also always select USB_MUSB_GADGET,
> >and export a UART there for the tablets, but I haven't seen such a
> >gadget in the code, so I guess your idea makes more sense :)
> 
> Ack. AFAIK u-boot currently lacks a serial gadget, once it gets one
> things get somewhat more complicated, for debugging a serial gadget
> is fine. But for now (no usable tablet ui) I sorta expect people to
> want to have hooked up a usb-keyb to a tablet (which is not running
> android) anyways, at which point the host option makes more sense.
> 
> Ideally we would build in both, and decide which one to use based
> on the id-pin. The sunxi musb-host code already checks the id-pin
> and does not touch the controller if it is not pulled down (iow
> if no host cable is inserted).

Looks like a good plan. I'll update my patch and resend it.

Thanks!
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150914/b99091b2/attachment.sig>

  reply	other threads:[~2015-09-14 21:19 UTC|newest]

Thread overview: 65+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-08-31 14:46 [U-Boot] [PATCH 00/13] Implement fastboot over NAND Maxime Ripard
2015-08-31 14:46 ` [U-Boot] [PATCH 01/13] mtd: uboot: Add meaningful error message Maxime Ripard
2015-09-04 17:20   ` Tom Rini
2015-08-31 14:46 ` [U-Boot] [PATCH 02/13] sparse: Move main header parsing to a function of its own Maxime Ripard
2015-09-04 17:20   ` Tom Rini
2015-08-31 14:46 ` [U-Boot] [PATCH 03/13] sparse: Refactor chunk parsing function Maxime Ripard
2015-09-04 17:20   ` Tom Rini
2015-08-31 14:46 ` [U-Boot] [PATCH 04/13] sparse: Simplify multiple logic Maxime Ripard
2015-09-04 17:20   ` Tom Rini
2015-09-06 11:27     ` Maxime Ripard
2015-09-06 19:28       ` Tom Rini
2015-09-13 17:08         ` Maxime Ripard
2015-08-31 14:46 ` [U-Boot] [PATCH 05/13] sparse: Implement storage abstraction Maxime Ripard
2015-09-04 17:20   ` Tom Rini
2015-08-31 14:46 ` [U-Boot] [PATCH 06/13] fastboot: Move fastboot response functions to fastboot core Maxime Ripard
2015-09-04 17:20   ` Tom Rini
2015-09-06 16:11     ` Maxime Ripard
2015-09-06 19:43       ` Tom Rini
2015-08-31 14:46 ` [U-Boot] [PATCH 07/13] fastboot: Implement NAND backend Maxime Ripard
2015-09-04 17:20   ` Tom Rini
2015-09-06 15:57     ` Maxime Ripard
2015-09-06 19:41       ` Tom Rini
2015-09-10  7:41   ` Boris Brezillon
2015-08-31 14:46 ` [U-Boot] [PATCH 08/13] fastboot: nand: Add pre erase and write hooks Maxime Ripard
2015-09-04 17:20   ` Tom Rini
2015-08-31 14:46 ` [U-Boot] [PATCH 09/13] sparse: Rename the file and header Maxime Ripard
2015-09-04 17:21   ` Tom Rini
2015-09-06 11:28     ` Maxime Ripard
2015-08-31 14:46 ` [U-Boot] [PATCH 10/13] sunxi: Make the fastboot buffer larger Maxime Ripard
2015-08-31 15:01   ` Hans de Goede
2015-08-31 19:17     ` Rob Herring
2015-09-01  7:14       ` Maxime Ripard
2015-09-08 13:00         ` Rob Herring
2015-09-08 15:44           ` Maxime Ripard
2015-09-01  7:05     ` Maxime Ripard
2015-09-01  8:59       ` Hans de Goede
2015-09-03 21:43         ` Maxime Ripard
2015-09-01  8:02     ` Siarhei Siamashka
2015-09-01  7:22   ` Ian Campbell
2015-09-01  7:44     ` Siarhei Siamashka
2015-09-01  8:11       ` Maxime Ripard
2015-09-04 17:02       ` Tom Rini
2015-09-06 11:23         ` Maxime Ripard
2015-09-01  7:57     ` Maxime Ripard
2015-09-04 16:59       ` Tom Rini
2015-09-06 11:22         ` Maxime Ripard
2015-09-07  9:07           ` Ian Campbell
2015-08-31 14:46 ` [U-Boot] [PATCH 11/13] sunxi: Add support for android boot image Maxime Ripard
2015-09-01  7:08   ` Ian Campbell
2015-09-01  7:15     ` Maxime Ripard
2015-09-01  9:00     ` Paul Kocialkowski
2015-09-01 10:46       ` Ian Campbell
2015-09-01 11:29         ` Maxime Ripard
2015-09-01 13:54           ` Ian Campbell
2015-09-01 13:59             ` Paul Kocialkowski
2015-09-01 11:28       ` Maxime Ripard
2015-09-08 13:12     ` Rob Herring
2015-08-31 14:46 ` [U-Boot] [PATCH 12/13] sunxi: A13-Olinuxino: Enable the USB OTG controller Maxime Ripard
2015-09-01  9:01   ` Hans de Goede
2015-09-03 21:41     ` Maxime Ripard
2015-09-10 18:47       ` Hans de Goede
2015-09-13 17:13         ` Maxime Ripard
2015-09-13 17:38           ` Hans de Goede
2015-09-14 21:19             ` Maxime Ripard [this message]
2015-08-31 14:46 ` [U-Boot] [PATCH 13/13] sunxi: cubietruck: " 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=20150914211920.GJ4684@lukather \
    --to=maxime.ripard@free-electrons.com \
    --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.