linux-mmc.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Amitkumar Karwar <akarwar@marvell.com>
To: Ulf Hansson <ulf.hansson@linaro.org>
Cc: "linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
	Nishant Sarmukadam <nishants@marvell.com>,
	linux-mmc <linux-mmc@vger.kernel.org>,
	Bing Zhao <bin9zha0@gmail.com>, Cathy Luo <cluo@marvell.com>,
	Xinming Hu <huxm@marvell.com>,
	Wei-Ning Huang <wnhuang@chromium.org>
Subject: RE: [PATCH] mmc: add API for data write using scatter gather DMA
Date: Thu, 5 May 2016 10:59:06 +0000	[thread overview]
Message-ID: <29cd9bbb3e3944f3bb1ac571b427bc4c@SC-EXCH04.marvell.com> (raw)
In-Reply-To: <CAPDyKFpDyBtGKxqhuo2NnioXfXcn8ZKB=EbyoaFkZ1LX34ZzkQ@mail.gmail.com>

Hi Ulf,

> From: linux-wireless-owner@vger.kernel.org [mailto:linux-wireless-
> owner@vger.kernel.org] On Behalf Of Ulf Hansson
> Sent: Friday, April 22, 2016 7:48 PM
> To: Amitkumar Karwar
> Cc: linux-wireless@vger.kernel.org; Nishant Sarmukadam; linux-mmc; Bing
> Zhao; Cathy Luo; Bing Zhao; Xinming Hu
> Subject: Re: [PATCH] mmc: add API for data write using scatter gather
> DMA
> 
> On 29 March 2016 at 14:57, Amitkumar Karwar <akarwar@marvell.com> wrote:
> > From: Bing Zhao <bzhao@marvell.com>
> >
> > This patch adds new API for SDIO scatter gather.
> >
> > Existing mmc_io_rw_extended() API expects caller to pass single
> > contiguous buffer. It will be split if it exceeds segment size, SG
> > table is prepared and used it for reading/writing the data.
> >
> > Our intention for defining new API here is to facilitate caller to
> > provide ready SG table(scattered buffer list). It will be useful for
> > the drivers which intend to handle SDIO SG internally.
> >
> > Signed-off-by: Bing Zhao <bzhao@marvell.com>
> > Signed-off-by: Xinming Hu <huxm@marvell.com>
> > Signed-off-by: Amitkumar Karwar <akarwar@marvell.com>
> > ---
> 
> So this has been posted and discussed earlier. In upcoming revisions,
> please attach that history so people know when reviewing.
> 
> Moreover, if you have been using this API to improve throughput, perhaps
> you can provide us with some numbers as well!?
> 

Thanks for your comments.
We are working on the updated version to resolve them. We will include the changelist history and share the throughput improvement numbers.

Regards,
Amitkumar

      reply	other threads:[~2016-05-05 11:02 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-29 12:57 [PATCH] mmc: add API for data write using scatter gather DMA Amitkumar Karwar
     [not found] ` <1459256261-22492-1-git-send-email-akarwar-eYqpPyKDWXRBDgjK7y7TUQ@public.gmane.org>
2016-04-22 14:17   ` Ulf Hansson
2016-05-05 10:59     ` Amitkumar Karwar [this message]

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=29cd9bbb3e3944f3bb1ac571b427bc4c@SC-EXCH04.marvell.com \
    --to=akarwar@marvell.com \
    --cc=bin9zha0@gmail.com \
    --cc=cluo@marvell.com \
    --cc=huxm@marvell.com \
    --cc=linux-mmc@vger.kernel.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=nishants@marvell.com \
    --cc=ulf.hansson@linaro.org \
    --cc=wnhuang@chromium.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).