From: Tony Lindgren <tony@atomide.com>
To: Benoit Cousson <b-cousson@ti.com>
Cc: linux-omap@vger.kernel.org, nm@ti.com
Subject: Re: [RFC v2 0/7] OMAP4: mux: Add the OMAP4430 ES1 & ES2 support
Date: Tue, 19 Oct 2010 16:06:42 -0700 [thread overview]
Message-ID: <20101019230642.GF3038@atomide.com> (raw)
In-Reply-To: <1287526956-21853-1-git-send-email-b-cousson@ti.com>
* Benoit Cousson <b-cousson@ti.com> [101019 15:14]:
> Hi Tony,
>
> Upon Nishanth request, here is the updated version...
>
> It takes into account your proposal to store partition
> information in a partition structure instead of inside every pad entries.
> The mechanism relies on the uniqueness of the pad name in each partition to
> find the correct partition during iteration.
OK, using the offset defines won't be unique necessarily..
> Removed as well some cpu_is_xxx calls from the core code by adding a couple of
> flags during partition init.
That's great.
> Please note that due to low level access to mux api from the RX51 board code,
> I have to disable the mux change to build properly with omap2plus_defconfig.
> Look like you do have some idea to fix that :-)
> I was thinking of:
> - brute force approach that consist of keeping all the data after init, and
> thus allowing removing the __init in the omap_mux_init_signal API
> - or adding some flag in the pad entry to keep them after init .
Well I was thinking we just register a board specific dynamic mux table
and then call that when hitting retention or off and restore after that.
Still need to think about that a bit more, I don't know if we want
the automatic muxing of gpio pins still in addition to that.
> Boot tested on SDP4430 with ES1.0 & ES2.0 with omap2plus_defconfig. Still require
> some real driver to use that mux API in order to validate it.
You can test by echoing wrong values after booting via /sys to
mess up the MMC data lines and then type sync :)
> The series is on top of lo/master (2.6.36-rc8) and is available here:
> git://gitorious.org/omap-pm/linux.git ctrl-wip/mux-omap4-v2
For pulling anything in, the git branches need to be based on something
that's immutable. So preferrably v2.6.35 or some recent -rc tag
in this case. Or rebase it on v2.6.36 after that's been released.
Regards,
Tony
next prev parent reply other threads:[~2010-10-19 23:06 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-10-19 22:22 [RFC v2 0/7] OMAP4: mux: Add the OMAP4430 ES1 & ES2 support Benoit Cousson
2010-10-19 22:22 ` [RFC v2 1/7] OMAP: mux: Replace printk with pr_xxx macros Benoit Cousson
2010-10-19 22:22 ` [RFC v2 2/7] OMAP3: RX-51: Temporary disable dynamic mux change for eMMC Benoit Cousson
2010-10-19 22:22 ` [RFC v2 3/7] OMAP: mux: Add support for control module split in several partitions Benoit Cousson
2010-11-11 16:35 ` Tony Lindgren
2010-11-11 16:50 ` Cousson, Benoit
2010-11-11 16:55 ` Tony Lindgren
2010-10-19 22:22 ` [RFC v2 4/7] OMAP4: mux: Add CBL package data for OMAP4430 ES1 Benoit Cousson
2010-11-11 1:56 ` Tony Lindgren
2010-11-11 12:33 ` Cousson, Benoit
2010-11-11 16:38 ` Tony Lindgren
2010-11-11 16:55 ` Cousson, Benoit
2010-10-19 22:22 ` [RFC v2 5/7] OMAP4: mux: Select CBL package for SDP4430 with ES1 Benoit Cousson
2010-10-19 22:22 ` [RFC v2 6/7] OMAP4: mux: Add CBS package data for OMAP4430 ES2 Benoit Cousson
2010-10-19 22:22 ` [RFC v2 7/7] OMAP4: mux: Select CBS package for SDP4430 with ES2 Benoit Cousson
2010-10-19 23:06 ` Tony Lindgren [this message]
2010-10-20 20:52 ` [RFC v2 0/7] OMAP4: mux: Add the OMAP4430 ES1 & ES2 support Cousson, Benoit
2010-11-11 16:53 ` Tony Lindgren
2010-11-11 17:02 ` 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=20101019230642.GF3038@atomide.com \
--to=tony@atomide.com \
--cc=b-cousson@ti.com \
--cc=linux-omap@vger.kernel.org \
--cc=nm@ti.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).