lkml.org 
[lkml]   [2001]   [Jan]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
Subject[PATCH] drivers/scsi/dmx3191d.c: small cleanup patch (241p11)
(Forgot l-k. Please cc dafastidio@libero.it on replies to this mail.)

----- Forwarded message from Rasmus Andersen <rasmus@jaquet.dk> -----

Hi.

The following patch makes drivers/scsi/dmx3191d.c call
pci_enable_device before probing resources and replaces
a check_region+request_region with request_region.

It applies against ac12 and 241p11.

Comments?



--- linux-ac12-clean/drivers/scsi/dmx3191d.c Sun Nov 12 04:01:11 2000
+++ linux-ac12/drivers/scsi/dmx3191d.c Sat Jan 27 21:27:44 2001
@@ -68,18 +68,16 @@
while ((pdev = pci_find_device(PCI_VENDOR_ID_DOMEX,
PCI_DEVICE_ID_DOMEX_DMX3191D, pdev))) {

- unsigned long port = pci_resource_start (pdev, 0);
-
if (pci_enable_device(pdev))
continue;

- if (check_region(port, DMX3191D_REGION)) {
+ unsigned long port = pci_resource_start (pdev, 0);
+
+ if (!request_region(port, DMX3191D_REGION, DMX3191D_DRIVER_NAME)) {
dmx3191d_printk("region 0x%lx-0x%lx already reserved\n",
port, port + DMX3191D_REGION);
continue;
}
-
- request_region(port, DMX3191D_REGION, DMX3191D_DRIVER_NAME);

instance = scsi_register(tmpl, sizeof(struct NCR5380_hostdata));
if(instance == NULL)
--
Regards,
Rasmus(rasmus@jaquet.dk)

"There are also enough rocks on Earth to kill the world's population several
times over."
-- Lt. General Daniel Graham, DIA, explaining why it's necessary to
have more than enough nukes

----- End forwarded message -----
--
Rasmus(rasmus@jaquet.dk)

Even if you're on the right track, you'll get run over if you just sit there.
-- Will Rogers
-
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:25    [W:0.037 / U:0.316 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site