lkml.org 
[lkml]   [2004]   [Oct]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
SubjectRe: 2.6.9-rc3-mm3: `risc_code_addr01' multiple definition
From
Date
On Thu, 2004-10-07 at 11:13 -0700, Andrew Vasquez wrote:
> On Thu, 2004-10-07 at 12:50 -0500, James Bottomley wrote:
> > On Thu, 2004-10-07 at 12:29, Andrew Vasquez wrote:
> > > Hmm, seems the additional 1040 support in qla1280.c is causing name
> > > clashes with the firmware image in qlogicfc_asm.c. Try out the attached
> > > patch (not tested) which provides the 1040 firmware image unique
> > > variable names.
> > >
> > > Looks like there would be some name clashes in qlogicfc and qlogicisp.
> >
> > Is there any reason for these firmware image pointers not to be static?
> > At least for these drivers which are single files.
> >
>
> That certainly seems like a reasonable option for qlogicfc and
> qlogicisp. Attached is a small patch which will limit the scope of
> qlogicfc_asm variables.
>

Sorry, patch attached.

--
av
--- linux-2.6.9-rc3-mm3/drivers/scsi/qlogicfc_asm.c~ 2004-10-07 11:09:44.971796784 -0700
+++ linux-2.6.9-rc3-mm3/drivers/scsi/qlogicfc_asm.c 2004-10-07 11:10:08.103280264 -0700
@@ -19,10 +19,10 @@
* Firmware Version 1.19.16 (10:36 Nov 02, 2000)
*/

-unsigned short risc_code_addr01 = 0x1000 ;
+static unsigned short risc_code_addr01 = 0x1000 ;

-unsigned short risc_code_length2100 = 0x9260;
-unsigned short risc_code2100[] = {
+static unsigned short risc_code_length2100 = 0x9260;
+static unsigned short risc_code2100[] = {
0x0078, 0x102d, 0x0000, 0x9260, 0x0000, 0x0001, 0x0013, 0x0010,
0x0017, 0x2043, 0x4f50, 0x5952, 0x4947, 0x4854, 0x2031, 0x3939,
0x3920, 0x514c, 0x4f47, 0x4943, 0x2043, 0x4f52, 0x504f, 0x5241,
@@ -4729,8 +4729,8 @@ unsigned short risc_code2100[] = {
* Firmware Version 2.01.27 (11:07 Dec 18, 2000)
*/

-unsigned short risc_code_length2200 = 0x9cbf;
-unsigned short risc_code2200[] = {
+static unsigned short risc_code_length2200 = 0x9cbf;
+static unsigned short risc_code2200[] = {
0x0470, 0x0000, 0x0000, 0x9cbf, 0x0000, 0x0002, 0x0001, 0x001b,
0x0017, 0x2043, 0x4f50, 0x5952, 0x4947, 0x4854, 0x2031, 0x3939,
0x3920, 0x514c, 0x4f47, 0x4943, 0x2043, 0x4f52, 0x504f, 0x5241,
\
 
 \ /
  Last update: 2005-03-22 14:06    [W:0.051 / U:0.384 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site