From: Shawn Guo <shawn.guo@freescale.com>
To: Per Forlin <per.forlin@linaro.org>
Cc: <linux-mmc@vger.kernel.org>,
<linux-arm-kernel@lists.infradead.org>,
<linux-kernel@vger.kernel.org>, <linaro-dev@lists.linaro.org>,
Chris Ball <cjb@laptop.org>
Subject: Re: [PATCH v2 00/12] mmc: use nonblock mmc requests to minimize latency
Date: Sat, 16 Apr 2011 23:48:41 +0800 [thread overview]
Message-ID: <20110416154841.GA17935@S2100-06.ap.freescale.net> (raw)
In-Reply-To: <1302116833-24540-1-git-send-email-per.forlin@linaro.org>
Hi Per,
On Wed, Apr 06, 2011 at 09:07:01PM +0200, Per Forlin wrote:
[...]
>
> Per Forlin (12):
> mmc: add none blocking mmc request function
> mmc: mmc_test: add debugfs file to list all tests
> mmc: mmc_test: add test for none blocking transfers
> mmc: add member in mmc queue struct to hold request data
> mmc: add a block request prepare function
> mmc: move error code in mmc_block_issue_rw_rq to a separate function.
> mmc: add a second mmc queue request member
> mmc: add handling for two parallel block requests in issue_rw_rq
> mmc: test: add random fault injection in core.c
> omap_hsmmc: use original sg_len for dma_unmap_sg
> omap_hsmmc: add support for pre_req and post_req
> mmci: implement pre_req() and post_req()
>
> drivers/mmc/card/block.c | 493 +++++++++++++++++++++++++++--------------
> drivers/mmc/card/mmc_test.c | 342 ++++++++++++++++++++++++++++-
> drivers/mmc/card/queue.c | 171 +++++++++------
> drivers/mmc/card/queue.h | 31 ++-
> drivers/mmc/core/core.c | 132 ++++++++++-
> drivers/mmc/core/debugfs.c | 5 +
> drivers/mmc/host/mmci.c | 146 +++++++++++-
> drivers/mmc/host/mmci.h | 8 +
> drivers/mmc/host/omap_hsmmc.c | 90 +++++++-
> include/linux/mmc/core.h | 9 +-
> include/linux/mmc/host.h | 13 +-
> lib/Kconfig.debug | 11 +
> 12 files changed, 1172 insertions(+), 279 deletions(-)
>
I'm playing the patch set and seeing the following warnings.
CC drivers/mmc/card/block.o
drivers/mmc/card/block.c: In function ‘mmc_blk_issue_rq’:
drivers/mmc/card/block.c:429:6: warning: ‘status’ may be used uninitialized in this function
CC drivers/mmc/core/core.o
drivers/mmc/core/core.c: In function ‘mmc_request_done’:
drivers/mmc/core/core.c:163:3: warning: passing argument 2 of ‘mmc_should_fail_request’ from incompatible pointer type
drivers/mmc/core/core.c:129:20: note: expected ‘struct mmc_data *’ but argument is of type ‘struct mmc_request *’
--
Regards,
Shawn
next prev parent reply other threads:[~2011-04-16 15:45 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-04-06 19:07 Per Forlin
2011-04-06 19:07 ` [PATCH v2 01/12] mmc: add none blocking mmc request function Per Forlin
2011-04-15 10:34 ` David Vrabel
2011-04-20 7:17 ` Per Forlin
2011-04-26 13:29 ` David Vrabel
2011-04-26 14:22 ` Per Forlin
2011-04-06 19:07 ` [PATCH v2 02/12] mmc: mmc_test: add debugfs file to list all tests Per Forlin
2011-04-06 19:07 ` [PATCH v2 03/12] mmc: mmc_test: add test for none blocking transfers Per Forlin
2011-04-17 7:09 ` Lin Tony-B19295
2011-04-20 7:30 ` Per Forlin
2011-04-17 15:46 ` Shawn Guo
2011-04-20 7:41 ` Per Forlin
2011-04-06 19:07 ` [PATCH v2 04/12] mmc: add member in mmc queue struct to hold request data Per Forlin
2011-04-06 19:07 ` [PATCH v2 05/12] mmc: add a block request prepare function Per Forlin
2011-04-06 19:07 ` [PATCH v2 06/12] mmc: move error code in mmc_block_issue_rw_rq to a separate function Per Forlin
2011-04-06 19:07 ` [PATCH v2 07/12] mmc: add a second mmc queue request member Per Forlin
2011-04-06 19:07 ` [PATCH v2 08/12] mmc: add handling for two parallel block requests in issue_rw_rq Per Forlin
2011-04-20 11:32 ` Per Forlin
2011-04-06 19:07 ` [PATCH v2 09/12] mmc: test: add random fault injection in core.c Per Forlin
2011-04-06 19:07 ` [PATCH v2 10/12] omap_hsmmc: use original sg_len for dma_unmap_sg Per Forlin
2011-04-06 19:07 ` [PATCH v2 11/12] omap_hsmmc: add support for pre_req and post_req Per Forlin
2011-04-06 19:07 ` [PATCH v2 12/12] mmci: implement pre_req() and post_req() Per Forlin
2011-04-08 16:49 ` [PATCH v2 00/12] mmc: use nonblock mmc requests to minimize latency Linus Walleij
2011-04-09 11:55 ` Jae hoon Chung
[not found] ` <BANLkTikwCveocjPdoMoLEVpNneKkq66s4g@mail.gmail.com>
2011-04-11 9:03 ` Per Forlin
2011-04-11 9:08 ` Per Forlin
2011-04-19 14:30 ` Jae hoon Chung
2011-04-16 15:48 ` Shawn Guo [this message]
2011-04-20 8:19 ` Per Forlin
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=20110416154841.GA17935@S2100-06.ap.freescale.net \
--to=shawn.guo@freescale.com \
--cc=cjb@laptop.org \
--cc=linaro-dev@lists.linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mmc@vger.kernel.org \
--cc=per.forlin@linaro.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
all inboxes | Powered by JetHome®