* [patch] 2.4.17 drivers/scsi/NCR5380.c
@ 2001-12-28 4:19 Keith Owens
2001-12-28 16:26 ` Alan Cox
0 siblings, 1 reply; 2+ messages in thread
From: Keith Owens @ 2001-12-28 4:19 UTC (permalink / raw)
To: linux-kernel
Several drivers #include "NCR5380.c". If two or more of those drivers
are built into vmlinux then you get duplicate global symbols.
Functions in NCR5380.c must be static. There are a couple of other
suspect functions but they are conditioned by #ifdef so I left them
alone.
Index: 17.9/drivers/scsi/NCR5380.c
--- 17.9/drivers/scsi/NCR5380.c Sat, 08 Dec 2001 10:12:02 +1100 kaos (linux-2.4/U/b/0_NCR5380.c 1.4 644)
+++ 17.9(w)/drivers/scsi/NCR5380.c Fri, 28 Dec 2001 15:15:16 +1100 kaos (linux-2.4/U/b/0_NCR5380.c 1.4 644)
@@ -612,7 +612,7 @@ static int NCR5380_set_timer(struct Scsi
* Locks: disables irqs, takes and frees io_request_lock
*/
-void NCR5380_timer_fn(unsigned long unused)
+static void NCR5380_timer_fn(unsigned long unused)
{
struct Scsi_Host *instance;
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [patch] 2.4.17 drivers/scsi/NCR5380.c
2001-12-28 4:19 [patch] 2.4.17 drivers/scsi/NCR5380.c Keith Owens
@ 2001-12-28 16:26 ` Alan Cox
0 siblings, 0 replies; 2+ messages in thread
From: Alan Cox @ 2001-12-28 16:26 UTC (permalink / raw)
To: Keith Owens; +Cc: linux-kernel
> Functions in NCR5380.c must be static. There are a couple of other
> suspect functions but they are conditioned by #ifdef so I left them
> alone.
Im not totally confident the ifdef ones are correct either. The timer one
is definitely correct. Marcelo I guess I'm the nearest thing to NCR5380
maintainer - please apply this one
> Index: 17.9/drivers/scsi/NCR5380.c
> --- 17.9/drivers/scsi/NCR5380.c Sat, 08 Dec 2001 10:12:02 +1100 kaos (linux-2.4/U/b/0_NCR5380.c 1.4 644)
> +++ 17.9(w)/drivers/scsi/NCR5380.c Fri, 28 Dec 2001 15:15:16 +1100 kaos (linux-2.4/U/b/0_NCR5380.c 1.4 644)
> @@ -612,7 +612,7 @@ static int NCR5380_set_timer(struct Scsi
> * Locks: disables irqs, takes and frees io_request_lock
> */
>
> -void NCR5380_timer_fn(unsigned long unused)
> +static void NCR5380_timer_fn(unsigned long unused)
> {
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2001-12-28 16:16 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2001-12-28 4:19 [patch] 2.4.17 drivers/scsi/NCR5380.c Keith Owens
2001-12-28 16:26 ` Alan Cox
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®