All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Bryan O'Sullivan" <bos@serpentine.com>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: Nathan Neulinger <nneul@umr.edu>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	uetrecht@umr.edu
Subject: Re: 3ware driver in 2.4.x and 2.5.x not compatible with 6x00 series cards
Date: 18 Dec 2002 17:11:38 -0800	[thread overview]
Message-ID: <1040260298.800.7.camel@camp4.serpentine.com> (raw)
In-Reply-To: <1040242085.24561.22.camel@irongate.swansea.linux.org.uk>

[-- Attachment #1: Type: text/plain, Size: 601 bytes --]

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

[-- Attachment #2: 3ware.patch --]
[-- Type: text/x-patch, Size: 610 bytes --]

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 */

  reply	other threads:[~2002-12-19  1:03 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-12-18 18:10 3ware driver in 2.4.x and 2.5.x not compatible with 6x00 series cards Nathan Neulinger
2002-12-18 18:26 ` Dave Jones
2002-12-18 20:08 ` Alan Cox
2002-12-19  1:11   ` Bryan O'Sullivan [this message]
  -- strict thread matches above, loose matches on Subject: below --
2002-12-18 18:29 Neulinger, Nathan
2002-12-18 18:45 Neulinger, Nathan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1040260298.800.7.camel@camp4.serpentine.com \
    --to=bos@serpentine.com \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=linux-kernel@vger.kernel.org \
    --cc=nneul@umr.edu \
    --cc=uetrecht@umr.edu \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.