[prev in list] [next in list] [prev in thread] [next in thread]
List: linux-ia64
Subject: [PATCH] fix sn_serial warning
From: Jesse Barnes <jbarnes () sgi ! com>
Date: 2004-03-29 18:58:47
Message-ID: 200403291058.47096.jbarnes () sgi ! com
[Download RAW message or body]
In my fixes to quiet the sn_serial driver, I made sn_debug_printf only get
used when the DEBUG define was turned on in the driver but forgot to #ifdef
out the function definition, causing an 'defined but not used' warning. This
patch fixes that.
Jesse
["sn-serial-warning-fixes.patch" (text/plain)]
===== drivers/char/sn_serial.c 1.10 vs edited =====
--- 1.10/drivers/char/sn_serial.c Mon Mar 22 02:29:18 2004
+++ edited/drivers/char/sn_serial.c Mon Mar 29 10:48:14 2004
@@ -82,10 +82,10 @@
static unsigned long sn_interrupt_timeout;
extern u64 master_node_bedrock_address;
-static int sn_debug_printf(const char *fmt, ...);
#undef DEBUG
#ifdef DEBUG
+static int sn_debug_printf(const char *fmt, ...);
#define DPRINTF(x...) sn_debug_printf(x)
#else
#define DPRINTF(x...) do { } while (0)
@@ -247,6 +247,7 @@
sn_func->sal_puts(s, count);
}
+#ifdef DEBUG
/* this is as "close to the metal" as we can get, used when the driver
* itself may be broken */
static int
@@ -262,6 +263,7 @@
va_end(args);
return printed_len;
}
+#endif /* DEBUG */
/*
* Interrupt handling routines.
-
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic