From: Tony Lindgren <tony@atomide.com>
To: David Brownell <david-b@pacbell.net>
Cc: Madhusudhan Chikkature <madhu.cr@ti.com>, linux-omap@vger.kernel.org
Subject: Re: [PATCH]Add 3430 SDP board level support for HSMMC2
Date: Wed, 10 Sep 2008 16:59:14 -0700 [thread overview]
Message-ID: <20080910235913.GW21163@atomide.com> (raw)
In-Reply-To: <200809081226.18098.david-b@pacbell.net>
* David Brownell <david-b@pacbell.net> [080908 12:26]:
> On Wednesday 13 August 2008, Madhusudhan Chikkature wrote:
> > > A better way to do this would be to let the boards say exactly
> > > what the hardware configuration is, rather than requiring all
> > > OMAP3 boards to be set up exactly like the SDP...
> >
> > I think the initial version was written specific to SDP. Later
> > on it was changed to hsmmc.c and the platforms which required
> > simillar setup as SDP started using it.
>
> Right ...
>
>
> > Now, if support for a new board is required which is unlike SDP a
> > new file can still be added for that perticular board, right?
>
> Each board already gets a board-XYZ.c file. So the clean way
> to do this is to have that file set up its hsmmc config ... NOT
> to clone versions of hsmmc.c for each board!
Madhu, are you planning to abstract hsmmc.c further so various boards
can use it?
Tony
next prev parent reply other threads:[~2008-09-10 23:59 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-08-13 6:18 [PATCH]Add 3430 SDP board level support for HSMMC2 Madhusudhan Chikkature
2008-08-13 8:54 ` David Brownell
2008-08-13 9:27 ` Madhusudhan Chikkature
2008-09-08 19:26 ` David Brownell
2008-09-10 23:59 ` Tony Lindgren [this message]
2008-09-16 5:15 ` Madhusudhan Chikkature
2008-09-16 15:12 ` Tony Lindgren
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=20080910235913.GW21163@atomide.com \
--to=tony@atomide.com \
--cc=david-b@pacbell.net \
--cc=linux-omap@vger.kernel.org \
--cc=madhu.cr@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 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.