From: Chris Ball <cjb@laptop.org>
To: Olof Johansson <olof@lixom.net>
Cc: Wolfram Sang <w.sang@pengutronix.de>,
Mike Rapoport <mike@compulab.co.il>,
linux-mmc@vger.kernel.org, linux-tegra@vger.kernel.org,
Yvonne Yip <y@palm.com>
Subject: Re: [PATCH 2/2] mmc: add sdhci-tegra driver for Tegra SoCs
Date: Fri, 31 Dec 2010 04:11:26 +0000 [thread overview]
Message-ID: <20101231041124.GB19122@void.printf.net> (raw)
In-Reply-To: <AANLkTinpfKrLoZEr4RkiLJvNNHKavKCY4VBaJn+eGCv_@mail.gmail.com>
Hi Olof,
On Thu, Dec 23, 2010 at 10:27:29AM -0800, Olof Johansson wrote:
> But yeah, defaulting to RW makes more sense. v4 coming once I know if
> you want me to rip out the new quirk or not.
I'll take a v4 with the remaining quirk for now, and we'd be extremely
interested in seeing sdhci_ops->fixup work and quirk removal from you.
For what it's worth, Wolfram's not singling you out -- recently we've
been consistently rejecting patches that add new SDHCI quirks and
asking for rework. (See Takashi's JMicron 388 patch, for example.)
Thanks!
--
Chris Ball <cjb@laptop.org> <http://printf.net/>
One Laptop Per Child
next prev parent reply other threads:[~2010-12-31 4:11 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-12-23 9:27 [PATCH 0/3 v3] Add SDHCI driver for Tegra Olof Johansson
2010-12-23 9:27 ` [PATCH 1/2] sdhci: add quirk for max len ADMA descriptors Olof Johansson
2010-12-23 11:26 ` Wolfram Sang
2010-12-23 18:26 ` Olof Johansson
2011-01-04 14:14 ` Wolfram Sang
2010-12-23 9:33 ` [PATCH 2/2] mmc: add sdhci-tegra driver for Tegra SoCs Olof Johansson
2010-12-23 11:23 ` Wolfram Sang
2010-12-23 18:27 ` Olof Johansson
2010-12-31 4:11 ` Chris Ball [this message]
2011-01-02 0:28 ` Olof Johansson
2011-01-04 14:24 ` Wolfram Sang
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=20101231041124.GB19122@void.printf.net \
--to=cjb@laptop.org \
--cc=linux-mmc@vger.kernel.org \
--cc=linux-tegra@vger.kernel.org \
--cc=mike@compulab.co.il \
--cc=olof@lixom.net \
--cc=w.sang@pengutronix.de \
--cc=y@palm.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.