Re: [PATCH] ARM: tegra: nyan: Mark all USB ports as host

From	Paul Kocialkowski <contact@paulk.fr>
Newsgroups	linux.kernel
Subject	Re: [PATCH] ARM: tegra: nyan: Mark all USB ports as host
Date	2016-11-08 10:20 +0100
Message-ID	<sBaJs-7rN-23@gated-at.bofh.it> (permalink)
References	<siHwd-6V1-7@gated-at.bofh.it> <sAStb-3NV-11@gated-at.bofh.it> <sASWe-4ht-23@gated-at.bofh.it> <sBaq5-76t-23@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

Le mardi 08 novembre 2016 à 10:54 +0200, Peter De Schrijver a écrit :
> On Mon, Nov 07, 2016 at 02:09:31PM +0000, Jon Hunter wrote:
> > 
> > On 07/11/16 13:28, Thierry Reding wrote:
> > > * PGP Signed by an unknown key
> > > 
> > > On Sun, Sep 18, 2016 at 12:28:52PM +0200, Paul Kocialkowski wrote:
> > > > Nyan boards only have host USB ports (2 external, 1 internal), there is
> > > > no OTG-enabled connector.
> > > > 
> > > > Signed-off-by: Paul Kocialkowski <contact@paulk.fr>
> > > > ---
> > > >  arch/arm/boot/dts/tegra124-nyan.dtsi | 2 +-
> > > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > 
> > > Where is this information coming from? I don't have one of the Nyans
> > > myself, but one of the Tegra132 devices I have, which I think was
> > > derived from one of the Nyans uses one of the external host ports as
> > > forced recovery port, for which it would need OTG.
> > > 
> > > I suspect that the way to get U-Boot onto the Nyans is via tegrarcm?
> > > In that case I think one of the ports must be OTG.
> > 
> > It is true that the port on the back on the nyan-big can be used with
> > recovery mode. I was thinking that this is not a true OTG port as it is
> > just a 4-pin type A socket and does not have an ID pin. Thinking some
> > more about this the USB spec does include a "Host Negotiation Protocol
> > (HNP)" that allows a host and device to swap roles and so keeping it as
> > OTG seems valid afterall.
> 
> I don't think the bootrom implements that though. I expect recovery mode
> to just program the controller in device mode, without performing any
> negotiation.

That would make sense.

However, if there's a way (even not implemented yet, but a possible way) to have
the kernel configure this port as USB device instead of host dynamically (e.g.
without changing this bit in the dts), then I think it makes sense to keep the
OTG marking and drop this patch.

After all, switching to USB device mode doesn't necessarily have to come from
the ID pin.

-- 
Paul Kocialkowski, developer of free digital technology at the lower levels

Website: https://www.paulk.fr/
Coding blog: https://code.paulk.fr/
Git repositories: https://git.paulk.fr/ https://git.code.paulk.fr/

Thread

Re: [PATCH] ARM: tegra: nyan: Mark all USB ports as host Thierry Reding <thierry.reding@gmail.com> - 2016-11-07 14:50 +0100
  Re: [PATCH] ARM: tegra: nyan: Mark all USB ports as host Jon Hunter <jonathanh@nvidia.com> - 2016-11-07 15:20 +0100
    Re: [PATCH] ARM: tegra: nyan: Mark all USB ports as host Peter De Schrijver <pdeschrijver@nvidia.com> - 2016-11-08 10:00 +0100
      Re: [PATCH] ARM: tegra: nyan: Mark all USB ports as host Paul Kocialkowski <contact@paulk.fr> - 2016-11-08 10:20 +0100
      Re: [PATCH] ARM: tegra: nyan: Mark all USB ports as host Jon Hunter <jonathanh@nvidia.com> - 2016-11-08 11:00 +0100
        Re: [PATCH] ARM: tegra: nyan: Mark all USB ports as host Jon Hunter <jonathanh@nvidia.com> - 2016-11-08 12:20 +0100
          Re: [PATCH] ARM: tegra: nyan: Mark all USB ports as host Paul Kocialkowski <contact@paulk.fr> - 2016-11-08 14:10 +0100
        Re: [PATCH] ARM: tegra: nyan: Mark all USB ports as host Thierry Reding <thierry.reding@gmail.com> - 2016-11-08 12:30 +0100

csiph-web