Re: 3ware driver in 2.4.x and 2.5.x not compatible with 6x00

[Bryan O'Sullivan]

--=-rzDn7ZGlTnuzxcuxIZSS
Content-Type: text/plain
Content-Transfer-Encoding: 7bit

On Wed, 2002-12-18 at 12:08, Alan Cox wrote:

> Please give the name of your 3ware contact so someone competent in 3ware
> so they can be 're-educated'
> 
> I use the new driver with an old card, it works.

I've talked with some clue-enabled people at 3ware about this.  The
newest driver is indeed broken on cards with very old firmware, and the
next revision of the driver will fix that problem.

There's a trivial fix for 2.4.20: just get rid of the error case around
line 1016 of 3w-xxxx.c.  This is basically the fix 3ware is planning to
include in the next driver release.

Patch attached.

	<b

--=-rzDn7ZGlTnuzxcuxIZSS
Content-Disposition: attachment; filename=3ware.patch
Content-Type: text/x-patch; name=3ware.patch; charset=ISO-8859-15
Content-Transfer-Encoding: 7bit

diff -u /home/bos/3w-xxxx.c.~1~ /home/bos/3w-xxxx.c
--- /home/bos/3w-xxxx.c.~1~	2002-12-18 17:09:20.000000000 -0800
+++ /home/bos/3w-xxxx.c	2002-12-18 17:09:20.000000000 -0800
@@ -1014,12 +1014,6 @@
 			error = tw_setfeature(tw_dev2, 2, 1, &c);
 			if (error) {
 				printk(KERN_WARNING "3w-xxxx: tw_setfeature(): Error setting features for card %d.\n", j);
-				scsi_unregister(host);
-				release_region((tw_dev->tw_pci_dev->resource[0].start), TW_IO_ADDRESS_RANGE);
-				tw_free_device_extension(tw_dev);
-				kfree(tw_dev);
-				numcards--;
-				continue;
 			}
 
 			/* Now setup the interrupt handler */

--=-rzDn7ZGlTnuzxcuxIZSS--