From: Vikram Narayanan <vikram186@gmail.com>
To: linux-mtd@lists.infradead.org
Cc: Iwo Mergler <Iwo.Mergler@netcommwireless.com.au>,
Brian Norris <computersforpeace@gmail.com>,
Adrian Hunter <ext-adrian.hunter@nokia.com>,
Artem Bityutskiy <dedekind1@gmail.com>
Subject: [RFC PATCH 0/8] mtd_tests: Remove duplicate code and cleanups
Date: Tue, 02 Oct 2012 19:56:13 +0530 [thread overview]
Message-ID: <506AF985.30104@gmail.com> (raw)
Cc: Artem Bityutskiy <dedekind1@gmail.com>
Cc: Brian Norris <computersforpeace@gmail.com>
Cc: Adrian Hunter <ext-adrian.hunter@nokia.com>
Cc: Iwo Mergler <Iwo.Mergler@netcommwireless.com.au>
This RFC patchset moves some common code in all the tests to a common header
file. Also, all the printk,PRINT_PREF are replaced with pr_info style prints.
This way duplicates are eliminated and the code looks clean.
I deliberately missed out the mtd_oobtest as Brian Norris has already sent a
patch for that.
All the patches are compile tested and checkpatch clean(No checkpatch warnings
are introduced by the code that is added in this patch).
Vikram Narayanan(8):
mtd_tests: Refactor duplicate code
mtd_pagetest: Replace printk with pr_{info,crit}
mtd_torturetest: Replace printk with pr_{info,crit}
mtd_subpagetest: Replace printk with pr_info
mtd_stresstest: Replace printk with pr_{info,crit}
mtd_speedtest: Replace printk with pr_{info,crit}
mtd_readtest: Replace printk with pr_info
mtd_nandbiterrs: Replace msg macro with pr_info
drivers/mtd/tests/mtd_nandbiterrs.c | 97 +++++++-----------
drivers/mtd/tests/mtd_oobtest.c | 31 +-----
drivers/mtd/tests/mtd_pagetest.c | 191 +++++++++++++++--------------------
drivers/mtd/tests/mtd_readtest.c | 42 ++++----
drivers/mtd/tests/mtd_speedtest.c | 140 ++++++++------------------
drivers/mtd/tests/mtd_stresstest.c | 71 ++++---------
drivers/mtd/tests/mtd_subpagetest.c | 147 +++++++++++----------------
drivers/mtd/tests/mtd_tests.h | 60 +++++++++++
drivers/mtd/tests/mtd_torturetest.c | 104 +++++++------------
9 files changed, 370 insertions(+), 513 deletions(-)
next reply other threads:[~2012-10-02 14:26 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-10-02 14:26 Vikram Narayanan [this message]
2012-10-02 14:28 ` [PATCH 1/8] mtd_tests: Refactor duplicate code Vikram Narayanan
2012-10-05 10:17 ` Artem Bityutskiy
2012-10-02 14:28 ` [PATCH 2/8] mtd_pagetest: Replace printk with pr_{info,crit} Vikram Narayanan
2012-10-02 14:29 ` [PATCH 3/8] mtd_torturetest: " Vikram Narayanan
2012-10-02 14:30 ` [PATCH 4/8] mtd_subpagetest: Replace printk with pr_info Vikram Narayanan
2012-10-02 14:31 ` [PATCH 5/8] mtd_stresstest: Replace printk with pr_{info,crit} Vikram Narayanan
2012-10-02 14:32 ` [PATCH 6/8] mtd_speedtest: " Vikram Narayanan
2012-10-02 14:33 ` [PATCH 7/8] mtd_readtest: Replace printk with pr_info Vikram Narayanan
2012-10-02 14:33 ` [PATCH 8/8] mtd_nandbiterrs: Replace msg macro " Vikram Narayanan
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=506AF985.30104@gmail.com \
--to=vikram186@gmail.com \
--cc=Iwo.Mergler@netcommwireless.com.au \
--cc=computersforpeace@gmail.com \
--cc=dedekind1@gmail.com \
--cc=ext-adrian.hunter@nokia.com \
--cc=linux-mtd@lists.infradead.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