From: "Cousson, Benoit" <b-cousson@ti.com>
To: "R, Sricharan" <r.sricharan@ti.com>
Cc: "linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>
Subject: Re: [PATCH 0/5] OMAP4: mux: Initialise OMAP4 mux pins.
Date: Mon, 15 Nov 2010 23:33:48 +0100 [thread overview]
Message-ID: <4CE1B54C.40306@ti.com> (raw)
In-Reply-To: <1289806685-20688-1-git-send-email-r.sricharan@ti.com>
Hi Sricharan,
On 11/15/2010 8:38 AM, R, Sricharan wrote:
> This series updates the core device drivers to use mux framework
> for OMAP4 SDP and PANDA board. It's generated against the
> linux-omap master branch. It has a dependency on the Benoit's
> omap4 mux data series.
>
> http://www.spinics.net/lists/linux-omap/msg38995.html
I've just realized that this is not the latest one, here it is:
http://www.spinics.net/lists/linux-omap/msg40039.html
Regards,
Benoit
>
> sricharan (5):
> OMAP4: hsmmc: Initialise the mmc mux pins
> OMAP4: usb-musb: Initialise the usb mux pins.
> OMAP4: mcbsp: Initialise the mcbsp mux pins
> OMAP4: board-4430sdp: Initialise the mcspi mux pins
> OMAP4: serial: Initialise the uart mux pins
>
> arch/arm/mach-omap2/board-4430sdp.c | 20 ++++++++
> arch/arm/mach-omap2/devices.c | 83 +++++++++++++++++++++++++++++++++++
> arch/arm/mach-omap2/mcbsp.c | 33 +++++++++++++-
> arch/arm/mach-omap2/serial.c | 38 ++++++++++++++++
> arch/arm/mach-omap2/usb-musb.c | 41 +++++++++++++++++
> 5 files changed, 214 insertions(+), 1 deletions(-)
>
> While doing some internal reviews, there were few debates about existing
> mux framework usage. I am summarising that discussion here and would
> like to hear more on this from the community.
>
> 1. PAD configuration for all pins should be done in a central place(board file)
> Pros:
> a. All pins configured in a central place. Easy to ensure coverage
> and maintenance. Single place to look for all mux related settings
> b. Drivers, unless they have run time pad configuration requirements
> need not worry about muxing.
> Cons:
> a. Adds a lot of duplicate data in different board files assuming
> most of the pins will be connected the same way across different
> boards.
>
> 2. Do PAD configuration independently for each module
> Pros:
> a. Avoids repetition of similar data for different boards.
> b. Gives a knowledge of how pins are configured for a module
> to the respective owners.
> c. Pads are not initialised unless they are really needed.
> Cons:
> a. Can become difficult to maintain if lot of data tend to be
> different across boards.
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
next prev parent reply other threads:[~2010-11-15 22:33 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-11-15 7:38 [PATCH 0/5] OMAP4: mux: Initialise OMAP4 mux pins sricharan
2010-11-15 7:38 ` [PATCH 1/5] OMAP4: hsmmc: Initialise the mmc " sricharan
2010-11-18 19:03 ` Tony Lindgren
2010-11-15 7:38 ` [PATCH 2/5] OMAP4: usb-musb: Initialise the usb " sricharan
2010-11-15 7:38 ` [PATCH 3/5] OMAP4: mcbsp: Initialise the mcbsp " sricharan
2010-11-15 7:38 ` [PATCH 4/5] OMAP4: board-4430sdp: Initialise the mcspi " sricharan
2010-11-15 7:38 ` [PATCH 5/5] OMAP4: serial: Initialise the uart " sricharan
2010-11-15 22:33 ` Cousson, Benoit [this message]
2010-11-18 19:06 ` [PATCH 0/5] OMAP4: mux: Initialise OMAP4 " Tony Lindgren
2010-11-18 21:06 ` Cousson, Benoit
2010-11-18 21:26 ` Tony Lindgren
2010-11-19 8:48 ` R, Sricharan
2010-11-19 16:04 ` Tony Lindgren
[not found] <1289886357-22821-1-git-send-email-r.sricharan@ti.com>
2010-11-16 6:15 ` R, Sricharan
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=4CE1B54C.40306@ti.com \
--to=b-cousson@ti.com \
--cc=linux-omap@vger.kernel.org \
--cc=r.sricharan@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