From: Adrian Bunk <bunk@stusta.de>
To: drzeus-wbsd@drzeus.cx
Cc: wbsd-devel@list.drzeus.cx, linux-kernel@vger.kernel.org
Subject: [2.6 patch] drivers/mmc/wbsd.c: possible cleanups
Date: Tue, 19 Apr 2005 04:43:12 +0200 [thread overview]
Message-ID: <20050419024312.GW5489@stusta.de> (raw)
This patch contains the following possible cleanups:
- make some needlessly global code static
- remove the unneeded global function DBG_REG
Signed-off-by: Adrian Bunk <bunk@stusta.de>
---
drivers/mmc/wbsd.c | 30 ++++++++----------------------
drivers/mmc/wbsd.h | 7 -------
2 files changed, 8 insertions(+), 29 deletions(-)
--- linux-2.6.12-rc2-mm3-full/drivers/mmc/wbsd.h.old 2005-04-19 02:56:24.000000000 +0200
+++ linux-2.6.12-rc2-mm3-full/drivers/mmc/wbsd.h 2005-04-19 02:57:17.000000000 +0200
@@ -8,13 +8,6 @@
* published by the Free Software Foundation.
*/
-const int config_ports[] = { 0x2E, 0x4E };
-const int unlock_codes[] = { 0x83, 0x87 };
-
-const int valid_ids[] = {
- 0x7112,
- };
-
#define LOCK_CODE 0xAA
#define WBSD_CONF_SWRST 0x02
--- linux-2.6.12-rc2-mm3-full/drivers/mmc/wbsd.c.old 2005-04-19 02:55:39.000000000 +0200
+++ linux-2.6.12-rc2-mm3-full/drivers/mmc/wbsd.c 2005-04-19 02:57:06.000000000 +0200
@@ -52,32 +52,18 @@
#define DBGF(x...) do { } while (0)
#endif
+static const int config_ports[] = { 0x2E, 0x4E };
+static const int unlock_codes[] = { 0x83, 0x87 };
+
+static const int valid_ids[] = {
+ 0x7112,
+ };
+
+
static unsigned int io = 0x248;
static unsigned int irq = 6;
static int dma = 2;
-#ifdef CONFIG_MMC_DEBUG
-void DBG_REG(int reg, u8 value)
-{
- int i;
-
- printk(KERN_DEBUG "wbsd: Register %d: 0x%02X %3d '%c' ",
- reg, (int)value, (int)value, (value < 0x20)?'.':value);
-
- for (i = 7;i >= 0;i--)
- {
- if (value & (1 << i))
- printk("x");
- else
- printk(".");
- }
-
- printk("\n");
-}
-#else
-#define DBG_REG(r, v) do {} while (0)
-#endif
-
/*
* Basic functions
*/
next reply other threads:[~2005-04-19 2:44 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-04-19 2:43 Adrian Bunk [this message]
2005-04-19 7:19 ` [2.6 patch] drivers/mmc/wbsd.c: possible cleanups Russell King
-- strict thread matches above, loose matches on Subject: below --
2005-05-01 15:43 Adrian Bunk
2005-05-13 0:47 Adrian Bunk
2005-05-13 9:58 ` Russell King
2005-06-30 0:47 Adrian Bunk
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=20050419024312.GW5489@stusta.de \
--to=bunk@stusta.de \
--cc=drzeus-wbsd@drzeus.cx \
--cc=linux-kernel@vger.kernel.org \
--cc=wbsd-devel@list.drzeus.cx \
/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