LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* [PATCH] drivers/net/cs89x0.c: compilation warning fix
@ 2008-02-17 17:23 Leonardo Potenza
2008-02-20 16:51 ` Jeff Garzik
0 siblings, 1 reply; 2+ messages in thread
From: Leonardo Potenza @ 2008-02-17 17:23 UTC (permalink / raw)
To: kernel-janitors; +Cc: linux-kernel
From: Leonardo Potenza <lpotenza@inwind.it>
Suppress the warning message about the 'netcard_portlist' defined but not used.
Signed-off-by: Leonardo Potenza <lpotenza@inwind.it>
---
When building cs89x0 as a module, the following warning message is generated:
drivers/net/cs89x0.c:198: warning: 'netcard_portlist' defined but not used
Since the netcard_portlist is not so big, I just marked the array as __used.
--- linux-2.6.orig/drivers/net/cs89x0.c
+++ linux-2.6/drivers/net/cs89x0.c
@@ -172,30 +172,30 @@ static char version[] __initdata =
them to system IRQ numbers. This mapping is card specific and is set to
the configuration of the Cirrus Eval board for this chip. */
#ifdef CONFIG_ARCH_CLPS7500
-static unsigned int netcard_portlist[] __initdata =
+static unsigned int netcard_portlist[] __used __initdata =
{ 0x80090303, 0x300, 0x320, 0x340, 0x360, 0x200, 0x220, 0x240, 0x260, 0x280, 0x2a0, 0x2c0, 0x2e0, 0};
static unsigned int cs8900_irq_map[] = {12,0,0,0};
#elif defined(CONFIG_SH_HICOSH4)
-static unsigned int netcard_portlist[] __initdata =
+static unsigned int netcard_portlist[] __used __initdata =
{ 0x0300, 0};
static unsigned int cs8900_irq_map[] = {1,0,0,0};
#elif defined(CONFIG_MACH_IXDP2351)
-static unsigned int netcard_portlist[] __initdata = {IXDP2351_VIRT_CS8900_BASE, 0};
+static unsigned int netcard_portlist[] __used __initdata = {IXDP2351_VIRT_CS8900_BASE, 0};
static unsigned int cs8900_irq_map[] = {IRQ_IXDP2351_CS8900, 0, 0, 0};
#include <asm/irq.h>
#elif defined(CONFIG_ARCH_IXDP2X01)
#include <asm/irq.h>
-static unsigned int netcard_portlist[] __initdata = {IXDP2X01_CS8900_VIRT_BASE, 0};
+static unsigned int netcard_portlist[] __used __initdata = {IXDP2X01_CS8900_VIRT_BASE, 0};
static unsigned int cs8900_irq_map[] = {IRQ_IXDP2X01_CS8900, 0, 0, 0};
#elif defined(CONFIG_ARCH_PNX010X)
#include <asm/irq.h>
#include <asm/arch/gpio.h>
#define CIRRUS_DEFAULT_BASE IO_ADDRESS(EXT_STATIC2_s0_BASE + 0x200000) /* = Physical address 0x48200000 */
#define CIRRUS_DEFAULT_IRQ VH_INTC_INT_NUM_CASCADED_INTERRUPT_1 /* Event inputs bank 1 - ID 35/bit 3 */
-static unsigned int netcard_portlist[] __initdata = {CIRRUS_DEFAULT_BASE, 0};
+static unsigned int netcard_portlist[] __used __initdata = {CIRRUS_DEFAULT_BASE, 0};
static unsigned int cs8900_irq_map[] = {CIRRUS_DEFAULT_IRQ, 0, 0, 0};
#else
-static unsigned int netcard_portlist[] __initdata =
+static unsigned int netcard_portlist[] __used __initdata =
{ 0x300, 0x320, 0x340, 0x360, 0x200, 0x220, 0x240, 0x260, 0x280, 0x2a0, 0x2c0, 0x2e0, 0};
static unsigned int cs8900_irq_map[] = {10,11,12,5};
#endif
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] drivers/net/cs89x0.c: compilation warning fix
2008-02-17 17:23 [PATCH] drivers/net/cs89x0.c: compilation warning fix Leonardo Potenza
@ 2008-02-20 16:51 ` Jeff Garzik
0 siblings, 0 replies; 2+ messages in thread
From: Jeff Garzik @ 2008-02-20 16:51 UTC (permalink / raw)
To: Leonardo Potenza; +Cc: kernel-janitors, linux-kernel
Leonardo Potenza wrote:
> From: Leonardo Potenza <lpotenza@inwind.it>
>
> Suppress the warning message about the 'netcard_portlist' defined but not used.
>
> Signed-off-by: Leonardo Potenza <lpotenza@inwind.it>
> ---
>
> When building cs89x0 as a module, the following warning message is generated:
> drivers/net/cs89x0.c:198: warning: 'netcard_portlist' defined but not used
>
> Since the netcard_portlist is not so big, I just marked the array as __used.
applied
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2008-02-20 16:51 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-02-17 17:23 [PATCH] drivers/net/cs89x0.c: compilation warning fix Leonardo Potenza
2008-02-20 16:51 ` Jeff Garzik
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).