public inbox for linux-omap@vger.kernel.org
 help / color / mirror / Atom feed
From: Tony Lindgren <tony@atomide.com>
To: Grant Likely <grant.likely@secretlab.ca>
Cc: Olof Johansson <olof@lixom.net>,
	Peter Barada <peterb@logicpd.com>,
	Mika Westerberg <ext-mika.1.westerberg@nokia.com>,
	linux-omap@vger.kernel.org
Subject: Re: Preventing OMAP3 serial driver to take control of all UARTs
Date: Wed, 2 Dec 2009 16:59:21 -0800	[thread overview]
Message-ID: <20091203005921.GO4348@atomide.com> (raw)
In-Reply-To: <fa686aa40912020924i6079275n5ad31fa91e52b8a@mail.gmail.com>

* Grant Likely <grant.likely@secretlab.ca> [091202 09:24]:
> On Wed, Dec 2, 2009 at 9:16 AM, Olof Johansson <olof@lixom.net> wrote:
> > On Wed, Dec 02, 2009 at 09:04:49AM -0700, Grant Likely wrote:
> >
> >> Ah, you're talking about pin muxing configuration, right?  Yes, that
> >> GPIO binding deals with controllers, not pin mux.  Pin mux is very
> >> much an SoC specific thing that isn't mapped easily to a generic
> >> binding.
> >
> > Yep.
> >
> >> On the 5200, I haven't attempted to describe pin-mux in the device
> >> tree at all, and have either expected firmware to set it up correctly,
> >> or fixed it up in the platform code.
> >
> > Yeah. And it's one of the things Tony commented on that firmware tends
> > to get wrong, seems like people aren't doing complete mux configs in
> > u-boot, etc.
> >
> > So, if it needs to be fixed up in platform code, there will (likely) be
> > need for board-specific code there anyway. A bummer, since the device
> > tree would otherwise make it real easy to bring up new boards without
> > kernel code changes.
> 
> I didn't create a binding on the 5200 because I actually see very
> little buggy firmware in that regard (partially because I kept telling
> people to go fix their firmware).  :-)  If it ends up being the norm
> that the kernel has to fix it for a given SoC, then I would create an
> SoC-specific binding for pin mux configuration in the device tree so
> that some degree of common code can still fix it up.
> 
> It should be feasible for board-specific code to be the exception, not the rule.

The problem with pin multiplexing is that development boards such as
Beagle don't know the desired configuration in the bootloader.
There are pins available for connecting external hardware, such as
I2C, SPI, or whatever to the 16-bit GPMC bus. Also, the bootloaders
tend not to care about pin multiplexing for power management.

We now have kernel cmdline override for pin multiplexing, but
that does not help with adding platform_data for custom I2C or
SPI devices.

Just something to keep in mind, I still think we can benefit
from the open firmware support in various ways :)

Regards,

Tony
--
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

  reply	other threads:[~2009-12-03  0:59 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-11-30  8:46 Preventing OMAP3 serial driver to take control of all UARTs Mika Westerberg
2009-11-30 16:36 ` Peter Barada
2009-11-30 17:01   ` Grant Likely
2009-11-30 19:40     ` Tony Lindgren
2009-11-30 20:31       ` Peter Barada
2009-11-30 21:09         ` Tony Lindgren
2009-12-01 11:02           ` Mika Westerberg
2009-12-09 22:43             ` Kevin Hilman
2009-12-10 10:33               ` [PATCH] OMAP3: serial - allow platforms specify which UARTs to initialize Mika Westerberg
2009-12-11 22:27                 ` [APPLIED] [PATCH] OMAP3: serial - allow platforms specify which UARTs to Tony Lindgren
2009-11-30 20:52       ` Preventing OMAP3 serial driver to take control of all UARTs Tony Lindgren
2009-12-02 15:07       ` Grant Likely
2009-12-02 15:53         ` Olof Johansson
2009-12-02 16:04           ` Grant Likely
2009-12-02 16:16             ` Olof Johansson
2009-12-02 17:24               ` Grant Likely
2009-12-03  0:59                 ` Tony Lindgren [this message]
2009-12-03  1:00         ` Tony Lindgren
2009-12-03  6:56           ` Mika Westerberg
2009-12-03  8:46             ` Artem Bityutskiy
2009-12-03 19:52               ` Tony Lindgren
2009-12-07 10:44                 ` Mika Westerberg

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=20091203005921.GO4348@atomide.com \
    --to=tony@atomide.com \
    --cc=ext-mika.1.westerberg@nokia.com \
    --cc=grant.likely@secretlab.ca \
    --cc=linux-omap@vger.kernel.org \
    --cc=olof@lixom.net \
    --cc=peterb@logicpd.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