linux-omap.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Tim Nordell <tim.nordell@logicpd.com>
To: "Gadiyar, Anand" <gadiyar@ti.com>
Cc: Benoit Cousson <b-cousson@ti.com>,
	linux-omap@vger.kernel.org, Tony Lindgren <tony@atomide.com>,
	Paul Walmsley <paul@pwsan.com>,
	Kevin Hilman <khilman@deeprootsystems.com>
Subject: Re: [RFC 2/5] OMAP: mux: Make low level function private
Date: Fri, 24 Sep 2010 18:50:26 -0500	[thread overview]
Message-ID: <4C9D3942.8080600@logicpd.com> (raw)
In-Reply-To: <AANLkTikN8M+6Q0cDdp29mnQdtqSwL4Zp5Kpoz8TL+WHu@mail.gmail.com>

On 09/24/10 18:09, Gadiyar, Anand wrote:
> On Fri, Sep 24, 2010 at 2:45 PM, Benoit Cousson <b-cousson@ti.com> wrote:
>> omap_mux_read / omap_mux_write should not be accessed directly
>> outside the mux framework.
>> Do we really have use case that require dynamic mux change beside
>> GPIO?
>>
> 
> Only case I can think of is for any workarounds for issues that turn up.
> No such cases exist today on OMAP3 at least, and none are likely to
> appear in future (I hope). So your assumption is valid.
> 

We have a use-case here - granted the code currently is a complete hack
and not something clean enough for the mainline kernel.

We recently rediscovered the USB3320 PHY suspend issue that is in the
errata on the OMAP35x platform (Advisory 3.1.1.193) and we came up with
a workaround for it (despite the errata saying there isn't any) where
essentially we do:

1) Allow the EHCI controller to suspend the PHY as normal
2) Cut the power to the USB host controller using power domains.  This
resets the logic states in the USB host controller so that it is usable
again.
3) Remux all the pins talking to the PHY into GPIO mode
4) Monitor said pins for changes in status to know when a remote wakeup
request occurs
5) Remux pins back to USB PHY mode
6) Reenable power to the USB host controller and reinitialize registers

- Tim

  reply	other threads:[~2010-09-24 23:50 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-24  9:15 [RFC 0/5] OMAP4: mux: Add the OMAP4430 ES1 support Benoit Cousson
2010-09-24  9:15 ` [RFC 1/5] OMAP: mux: Add support for control module split in several partitions Benoit Cousson
2010-09-25  0:22   ` Tony Lindgren
2010-09-27 15:46     ` Tony Lindgren
2010-09-27 17:24       ` Cousson, Benoit
2010-09-27 17:36         ` Tony Lindgren
2010-09-27 20:03           ` Cousson, Benoit
2010-09-27 20:06             ` Tony Lindgren
2010-09-24  9:15 ` [RFC 2/5] OMAP: mux: Make low level function private Benoit Cousson
2010-09-24 23:09   ` Gadiyar, Anand
2010-09-24 23:50     ` Tim Nordell [this message]
2010-09-25  0:07       ` Anand Gadiyar
2010-09-25  6:48         ` Tim Nordell
2010-09-24  9:15 ` [RFC 3/5] OMAP4: mux: Add data for OMAP4430 ES1 Benoit Cousson
2010-09-24 23:18   ` Anand Gadiyar
2010-09-27  9:31     ` Cousson, Benoit
2010-09-24  9:15 ` [RFC 4/5] OMAP4: mux: Select CBL package for SDP4430 with ES1 Benoit Cousson
2010-09-24 23:14   ` Anand Gadiyar
2010-09-27  7:24     ` Cousson, Benoit
2010-09-24  9:15 ` [RFC 5/5] OMAP4: mux: Temporary initial SDP4430 mux settings Benoit Cousson
2010-10-18 18:09 ` [RFC 0/5] OMAP4: mux: Add the OMAP4430 ES1 support Menon, Nishanth
2010-10-18 20:51   ` Cousson, Benoit
2010-10-18 20:53     ` Menon, Nishanth
2010-10-18 20:57       ` Cousson, Benoit
2010-10-18 21:08         ` Menon, Nishanth
2010-10-18 23:00     ` Tony Lindgren
2010-10-18 23:12       ` Cousson, Benoit

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=4C9D3942.8080600@logicpd.com \
    --to=tim.nordell@logicpd.com \
    --cc=b-cousson@ti.com \
    --cc=gadiyar@ti.com \
    --cc=khilman@deeprootsystems.com \
    --cc=linux-omap@vger.kernel.org \
    --cc=paul@pwsan.com \
    --cc=tony@atomide.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).