lkml.org 
[lkml]   [2001]   [Jan]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Date
Subject[PATCH] i91uscsi.c misreads BIOS settings

Here's a patch for i91uscsi.c as delivered with the 2.4.0 kernel to correct the problem
of it reading the BIOS settings for double the SCSI id that it should be looking at.

This is a two liner to correct this problem but I've done some fairly major cosmetic
surgery on the rest of the driver - it currently uses two header files, i91uscsi.h and
ini9100u.h - and these contain duplicate definitions of the same structs but with
different contents.

This sounds to me like a recipe for problems so I've combined the two header files
and eliminated the duplication. I've also added /proc fs support. The resulting patch
is too big to post to a mailing list and a complete replacement of the source can be
found on http://www.os2warp.org/sysbench/i91uv104.tar.gz - this
adds ini9100_proc.c but i91uscsi.h can be rm'ed as no longer required. This
replacement has been tested on 2.2.13, 2.2.18 and 2.4.0 compiled into the
kernel and also on 2.4.0 as a module.

--- /backup/linuxold/drivers/scsi/i91uscsi.c Mon Aug 23 17:23:23 1999
+++ /backup/linux/drivers/scsi/i91uscsi.c Thu Jan 18 21:42:52 2001
@@ -590,7 +590,7 @@
int init_tulip(HCS * pCurHcb, SCB * scbp, int tul_num_scb, BYTE * pbBiosAdr, int seconds)
{
int i;
- WORD *pwFlags;
+ BYTE *pwFlags;
BYTE *pbHeads;
SCB *pTmpScb, *pPrevScb = NULL;

@@ -674,7 +674,7 @@
((pCurHcb->HCS_Config & HCC_AUTO_TERM) >> 4) | (TUL_RD(pCurHcb->HCS_Base, TUL_GCTRL1) & 0xFE));

for (i = 0,
- pwFlags = (WORD *) & (i91unvramp->NVM_SCSIInfo[0].NVM_Targ0Config),
+ pwFlags = & (i91unvramp->NVM_SCSIInfo[0].NVM_Targ0Config),
pbHeads = pbBiosAdr + 0x180;
i < pCurHcb->HCS_MaxTar;
i++, pwFlags++) {

Trevor Hemsley, Brighton, UK.
Trevor-Hemsley@dial.pipex.com

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 13:27    [W:0.048 / U:0.324 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site