From: Kevin Hilman <khilman@deeprootsystems.com>
To: "Gadiyar, Anand" <gadiyar@ti.com>
Cc: "Gupta, Ajay Kumar" <ajay.gupta@ti.com>,
"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>,
Tony Lindgren <tony@atomide.com>,
Peter De Schrijver <peter.de-schrijver@nokia.com>
Subject: Re: Padconf not working on pm_defconfig?
Date: Mon, 14 Dec 2009 08:54:19 -0800 [thread overview]
Message-ID: <87fx7dcx8k.fsf@deeprootsystems.com> (raw)
In-Reply-To: <5A47E75E594F054BAF48C5E4FC4B92AB030ADD4F23@dbde02.ent.ti.com> (Anand Gadiyar's message of "Mon\, 14 Dec 2009 15\:53\:44 +0530")
"Gadiyar, Anand" <gadiyar@ti.com> writes:
> Gadiyar, Anand wrote:
>> Gupta, Ajay Kumar wrote:
>> > >
>> > > I'm seeing some strange behavior with linux-omap-pm branch
>> > > and possibly with linux-omap master as well. I booted up
>> > > a 3430 SDP with an image built using omap3_pm_defconfig.
>> > > (I enabled EHCI in the menuconfig).
>> >
>> > Check if CONFIG_OMAP_MUX=y set.
>> >
>>
>> It's set.
>>
>> Also, dumping the registers just after the usb-echi.c makes
>> these mux calls shows they are okay.
>>
>> Something seems to be changing these later - debugging now.
>>
>
>
> Found it - I remember that these lines are muxed with ETK signals
> and with the hwdebug signals.
>
> So I looked at the debobs driver - and that's the culprit!
> (Cc-ing the author of that driver)
>
> How should this be handled? IMO, this driver has should use the
> MUX framework for changing the mux modes.
>
> What say?
Yes, the debobs driver needs to be updated to use the MUX API.
But even with that, AFAICT, the new mux API will still allow
the last caller to "win" as there is no request/free API
to get pins.
Kevin
next prev parent reply other threads:[~2009-12-14 16:54 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-12-14 9:53 Padconf not working on pm_defconfig? Gadiyar, Anand
2009-12-14 10:03 ` Gupta, Ajay Kumar
2009-12-14 10:06 ` Gadiyar, Anand
2009-12-14 10:23 ` Gadiyar, Anand
2009-12-14 16:54 ` Kevin Hilman [this message]
2009-12-14 17:06 ` Gadiyar, Anand
2009-12-14 19:26 ` Tony Lindgren
2009-12-14 19:46 ` Gadiyar, Anand
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=87fx7dcx8k.fsf@deeprootsystems.com \
--to=khilman@deeprootsystems.com \
--cc=ajay.gupta@ti.com \
--cc=gadiyar@ti.com \
--cc=linux-omap@vger.kernel.org \
--cc=peter.de-schrijver@nokia.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 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.