From: micky <micky_ching@realsil.com.cn>
To: Lee Jones <lee.jones@linaro.org>, Ulf Hansson <ulf.hansson@linaro.org>
Cc: Samuel Ortiz <sameo@linux.intel.com>,
Chris Ball <chris@printf.net>, <devel@linuxdriverproject.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
linux-mmc <linux-mmc@vger.kernel.org>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Dan Carpenter <dan.carpenter@oracle.com>,
Roger <rogerable@realtek.com>, Wei WANG <wei_wang@realsil.com.cn>
Subject: Re: [PATCH 1/2] mfd: rtsx: add dma transfer function
Date: Wed, 2 Jul 2014 17:14:08 +0800 [thread overview]
Message-ID: <53B3CD60.80302@realsil.com.cn> (raw)
In-Reply-To: <20140618080056.GM21030@lee--X1>
On 06/18/2014 04:00 PM, Lee Jones wrote:
> On Mon, 16 Jun 2014, Ulf Hansson wrote:
>
>> On 16 June 2014 14:20, Lee Jones <lee.jones@linaro.org> wrote:
>>>> From: Micky Ching <micky_ching@realsil.com.cn>
>>>>
>>>> rtsx driver using a single function for transfer data, dma map/unmap are
>>>> placed in one fix function. We need map/unmap dma in different place(for
>>>> mmc async driver), so add three function for dma map, dma transfer and
>>>> dma unmap.
>>>>
>>>> Signed-off-by: Micky Ching <micky_ching@realsil.com.cn>
>>>> ---
>>>> drivers/mfd/rtsx_pcr.c | 76 ++++++++++++++++++++++++++----------------
>>>> include/linux/mfd/rtsx_pci.h | 6 ++++
>>>> 2 files changed, 54 insertions(+), 28 deletions(-)
>>> I don't see any glaring issues with this patch. Does it rely on the
>>> first patch, or vise versa, or can it just be applied?
>> The mmc part in patch2 relies on this one, but please go ahead and
>> apply the mfd patch if you see it good.
>>
>> I can later provide my ack for the mmc parts, in patch2, when it's a
>> reviewed properly and thus you can take it through your tree.
> There's no rush. Once you're happy with the MMC patch, Micky can
> submit them both again with my Ack on the MFD part and I'll take them
> as a set.
>
Hi Lee,
Can you pick this patch directly? need resend it now?
Thanks.
Best Regards.
micky.
next prev parent reply other threads:[~2014-07-02 9:09 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-06-06 7:05 [PATCH 0/2] mmc: rtsx: add support for async request micky_ching
2014-06-06 7:05 ` [PATCH 1/2] mfd: rtsx: add dma transfer function micky_ching
2014-06-16 12:20 ` Lee Jones
2014-06-16 14:24 ` Ulf Hansson
2014-06-18 8:00 ` Lee Jones
2014-07-02 9:14 ` micky [this message]
2014-07-02 12:15 ` Lee Jones
2014-06-17 1:08 ` micky
2014-06-06 7:05 ` [PATCH 2/2] mmc: rtsx: add support for async request micky_ching
2014-06-16 8:42 ` Ulf Hansson
2014-06-16 8:51 ` Arend van Spriel
2014-06-16 9:09 ` micky
2014-06-16 12:40 ` Ulf Hansson
2014-06-17 1:04 ` micky
2014-06-17 7:45 ` Ulf Hansson
2014-06-18 1:17 ` micky
2014-06-18 7:39 ` Ulf Hansson
2014-06-18 10:08 ` micky
2014-06-18 11:03 ` Ulf Hansson
2014-06-19 1:57 ` micky
2014-06-23 9:26 ` micky
2014-07-02 8:53 ` Ulf Hansson
2014-07-18 7:28 ` [PATCH 0/2] " Lee Jones
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=53B3CD60.80302@realsil.com.cn \
--to=micky_ching@realsil.com.cn \
--cc=chris@printf.net \
--cc=dan.carpenter@oracle.com \
--cc=devel@linuxdriverproject.org \
--cc=gregkh@linuxfoundation.org \
--cc=lee.jones@linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mmc@vger.kernel.org \
--cc=rogerable@realtek.com \
--cc=sameo@linux.intel.com \
--cc=ulf.hansson@linaro.org \
--cc=wei_wang@realsil.com.cn \
/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
Powered by JetHome