OK, I think I misunderstood your explanation. Rather than judging based on English emails, I went back and looked at the old and new code.

Looking at the old code, there is a clear 1-to-1-to-1-to-1 correspondence between adapter, request_irq() call, scsi_host, and boardp. So what the old code is doing during probe is quite normal.

I thought you were saying there were multiple request_irq() calls for a single adapter.

The only thing that's abnormal is the interrupt handler checking multiple boards for events, which you rightly removed.

There is absolutely no need to add IRQF_SHARED to non-PCI devices or anything like that.

At this point, looking at your new code, I would only add
        "replace spin_lock_irqsave with spin_lock in interrupt handler"
and
        "delete no-ops DvcEnterCritical and DvcLeaveCritical"
to the todo list.

        Jeff



-
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to