From: Kevin Hilman <khilman@deeprootsystems.com>
To: Vladimir Pantelic <pan@nt.tu-darmstadt.de>
Cc: linux-omap@vger.kernel.org
Subject: Re: MMC3 Overo
Date: Thu, 06 Aug 2009 08:16:03 -0700 [thread overview]
Message-ID: <873a856ki4.fsf@deeprootsystems.com> (raw)
In-Reply-To: <4A7AEC54.107@nt.tu-darmstadt.de> (Vladimir Pantelic's message of "Thu\, 06 Aug 2009 16\:44\:36 +0200")
Vladimir Pantelic <pan@nt.tu-darmstadt.de> writes:
> Kevin Hilman wrote:
>
>>> AB1_35XXCBB_MMC3_CLK
>>> AF10_35XXCBB_MMC3_CLK
>>> R9_35XXCBC_MMC3_CLK
>>> AB2_35XXCBC_MMC3_CLK
>>> AC1_35XXCUS_MMC3_CLK
>>> We would then have to update mux.c making sure the position matches
>>> and add the proper settings.
>>>
>>> So this is obviously a maintenance nightmare.
>>
>> So why don't we drop the ball (pun intended.) ;)
>>
>> This is what I proposed to Phillip Ballister for his SPI changes for Beagle.
>>
>> Though I haven't looked at the details for each package, I have a hard
>> time imagining that the reg offsets and functionality for each package
>> is different. In fact, I'm pretty sure they're even the same between
>> 34xx and 35xx.
>>
>> IOW, why not just name it OMAP3_MMC3_CLK and have a single entry.
>>
>> Then each board file that cares simply has to call omap_cfg_reg() on
>> that name and not care about the package.
>
> even for 1 package, how will you know whether OMAP3_MMC3_CLK is connected
> to ball AB1 or AF10 as both is possible...
doh! good point. I was thinking of functions that are only availble
on a single ball.
/me resolves to finish waking up before thinking more about mux issues
Kevin
next prev parent reply other threads:[~2009-08-06 15:16 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-07-30 15:49 MMC3 Overo John Sarman
2009-07-30 19:23 ` John Sarman
2009-08-03 1:20 ` Paul Walmsley
2009-08-03 10:33 ` John Sarman
[not found] ` <52AC0DF6-BA95-421B-8C56-5F3DC62279DF@mac.com>
[not found] ` <bb2708720908030356p6d7b1cdegf0bd8c39a5c4ecd9@mail.gmail.com>
2009-08-03 10:57 ` Fwd: " John Sarman
2009-08-03 11:11 ` Paul Walmsley
2009-08-04 12:21 ` John Sarman
2009-08-05 8:32 ` Grazvydas Ignotas
[not found] ` <bb2708720908050901lbc67b1cx5a75a7667fa36fd1@mail.gmail.com>
2009-08-05 16:02 ` Fwd: " John Sarman
2009-08-05 16:25 ` Kevin Hilman
2009-08-05 16:35 ` John Sarman
2009-08-05 17:51 ` Steve Sakoman
2009-08-05 17:56 ` Gadiyar, Anand
2009-08-05 17:58 ` Kevin Hilman
2009-08-05 18:21 ` Steve Sakoman
2009-08-05 18:35 ` Kevin Hilman
2009-08-05 18:55 ` Philip Balister
2009-08-06 2:13 ` Hunyue Yau
[not found] ` <bb2708720908051229x49f10094w9b19b468d4bfaec8@mail.gmail.com>
2009-08-06 2:02 ` John Sarman
2009-08-06 6:44 ` Tony Lindgren
2009-08-06 14:30 ` Kevin Hilman
2009-08-06 14:44 ` Vladimir Pantelic
2009-08-06 15:16 ` Kevin Hilman [this message]
2009-08-06 15:35 ` Steve Sakoman
2009-08-06 15:09 ` Philip Balister
2009-08-06 16:09 ` Peter Barada
2009-08-07 8:21 ` Tony Lindgren
2009-08-07 9:24 ` Vladimir Pantelic
2009-08-07 9:55 ` Tony Lindgren
2009-08-07 13:24 ` Steve Sakoman
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=873a856ki4.fsf@deeprootsystems.com \
--to=khilman@deeprootsystems.com \
--cc=linux-omap@vger.kernel.org \
--cc=pan@nt.tu-darmstadt.de \
/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