From: Anton Vorontsov <avorontsov@ru.mvista.com>
To: Pierre Ossman <drzeus-mmc@drzeus.cx>
Cc: David Brownell <dbrownell@users.sourceforge.net>,
Gary Jennejohn <garyj@denx.de>,
linux-kernel@vger.kernel.org, linuxppc-dev@ozlabs.org,
Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Subject: Re: [PATCH] [RFC v3] OF: OpenFirmware bindings for the mmc_spi driver
Date: Mon, 16 Jun 2008 17:23:30 +0400 [thread overview]
Message-ID: <20080616132330.GA7468@polina.dev.rtsoft.ru> (raw)
In-Reply-To: <20080614175723.209e93dc@mjolnir.drzeus.cx>
On Sat, Jun 14, 2008 at 05:57:23PM +0200, Pierre Ossman wrote:
> On Thu, 5 Jun 2008 20:16:24 +0400
> Anton Vorontsov <avorontsov@ru.mvista.com> wrote:
>
> > Here is v3. I'm out of ideas if you won't like it. :-)
> >
> > v3:
> > - Now these bindings are using bus notifiers chain, thus we adhere to the
> > spi bus.
> >
>
> Now this was nice and clean. I take it Grant doesn't like this version
> though. What's the downside of it?
To me the main downside of it is inability to make this modular. :-/
--
Anton Vorontsov
email: cbouatmailru@gmail.com
irc://irc.freenode.net/bd2
WARNING: multiple messages have this Message-ID (diff)
From: Anton Vorontsov <avorontsov@ru.mvista.com>
To: Pierre Ossman <drzeus-mmc@drzeus.cx>
Cc: David Brownell <dbrownell@users.sourceforge.net>,
Grant Likely <grant.likely@secretlab.ca>,
Jochen Friedrich <jochen@scram.de>,
Segher Boessenkool <segher@kernel.crashing.org>,
Gary Jennejohn <garyj@denx.de>,
Guennadi Liakhovetski <g.liakhovetski@gmx.de>,
linuxppc-dev@ozlabs.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] [RFC v3] OF: OpenFirmware bindings for the mmc_spi driver
Date: Mon, 16 Jun 2008 17:23:30 +0400 [thread overview]
Message-ID: <20080616132330.GA7468@polina.dev.rtsoft.ru> (raw)
In-Reply-To: <20080614175723.209e93dc@mjolnir.drzeus.cx>
On Sat, Jun 14, 2008 at 05:57:23PM +0200, Pierre Ossman wrote:
> On Thu, 5 Jun 2008 20:16:24 +0400
> Anton Vorontsov <avorontsov@ru.mvista.com> wrote:
>
> > Here is v3. I'm out of ideas if you won't like it. :-)
> >
> > v3:
> > - Now these bindings are using bus notifiers chain, thus we adhere to the
> > spi bus.
> >
>
> Now this was nice and clean. I take it Grant doesn't like this version
> though. What's the downside of it?
To me the main downside of it is inability to make this modular. :-/
--
Anton Vorontsov
email: cbouatmailru@gmail.com
irc://irc.freenode.net/bd2
next prev parent reply other threads:[~2008-06-16 13:23 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-06-05 16:16 [PATCH] [RFC v3] OF: OpenFirmware bindings for the mmc_spi driver Anton Vorontsov
2008-06-05 16:16 ` Anton Vorontsov
2008-06-05 16:45 ` Grant Likely
2008-06-05 16:45 ` Grant Likely
2008-06-05 17:27 ` Anton Vorontsov
2008-06-05 17:27 ` Anton Vorontsov
2008-06-05 17:36 ` Grant Likely
2008-06-05 17:36 ` Grant Likely
2008-06-05 18:00 ` Anton Vorontsov
2008-06-05 18:00 ` Anton Vorontsov
2008-06-05 18:18 ` Grant Likely
2008-06-05 18:18 ` Grant Likely
2008-06-05 18:31 ` Anton Vorontsov
2008-06-05 18:31 ` Anton Vorontsov
2008-06-05 18:42 ` Grant Likely
2008-06-05 18:42 ` Grant Likely
2008-06-14 15:57 ` Pierre Ossman
2008-06-16 13:23 ` Anton Vorontsov [this message]
2008-06-16 13:23 ` Anton Vorontsov
2008-06-16 14:13 ` Grant Likely
2008-06-16 14:13 ` Grant Likely
2008-07-03 3:26 ` Jon Smirl
2008-07-03 3:26 ` Jon Smirl
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=20080616132330.GA7468@polina.dev.rtsoft.ru \
--to=avorontsov@ru.mvista.com \
--cc=dbrownell@users.sourceforge.net \
--cc=drzeus-mmc@drzeus.cx \
--cc=g.liakhovetski@gmx.de \
--cc=garyj@denx.de \
--cc=linux-kernel@vger.kernel.org \
--cc=linuxppc-dev@ozlabs.org \
/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.