On Wed, 2006-04-19 at 13:20 -0600, Moore, Eric wrote: > +static void > +mptspi_renegotiate(struct scsi_device *sdev) > +{ > + struct _MPT_SCSI_HOST *hd = (struct _MPT_SCSI_HOST > *)sdev->host->hostdata; > + > + if (sdev->channel == 1 || > + !(hd->ioc->raid_data.isRaid & (1 << sdev->id))) The equivalent of this check is *already* in mptspi_dv_device(), so you don't need to check it again. Simply replacing mptspi_renegotiate with mptspi_dv_device() should work fine. James - : send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html