Messages in this thread Patch in this message | | | Date | Tue, 30 Jan 2001 06:58:57 -0800 (PST) | From | rakesh rakesh <> | Subject | Re: [PATCH] drivers/scsi/BusLogic.c: No resource probing before pci_enable_device (241p11) |
| |
hi List, What I can make out from the below statement is that its better to first enable the device and then allocate the resources for that.Since from the hot pluggable point of view if the device is not enabled ,then it does not mean anything to allocate the resources for it.It would be simply wasting the resources which serves no good purpose. If my understanding is incorrect or if there are few more important issues attached from the Hot-pluggable point of view,then please let me know.
Thanks
--- Rasmus Andersen <rasmus@jaquet.dk> wrote: > Hi. > > The following patch makes drivers/scsi/BusLogic.c > wait with probing > pdev->irq and pdev->resource[] until we call > pci_enable_device. This > is recommended due to hot-plug considerations > (according to Jeff Garzik). > > It applies against ac12 and 241p11. > > Comments? > > > --- linux-ac12-clean/drivers/scsi/BusLogic.c Sat Jan > 27 21:16:24 2001 > +++ linux-ac12/drivers/scsi/BusLogic.c Sat Jan 27 > 21:32:33 2001 > @@ -770,15 +770,19 @@ > BusLogic_ModifyIOAddressRequest_T > ModifyIOAddressRequest; > unsigned char Bus = PCI_Device->bus->number; > unsigned char Device = PCI_Device->devfn >> > 3; > - unsigned int IRQ_Channel = PCI_Device->irq; > - unsigned long BaseAddress0 = > pci_resource_start(PCI_Device, 0); > - unsigned long BaseAddress1 = > pci_resource_start(PCI_Device, 1); > - BusLogic_IO_Address_T IO_Address = > BaseAddress0; > - BusLogic_PCI_Address_T PCI_Address = > BaseAddress1; > + unsigned int IRQ_Channel; > + unsigned long BaseAddress0; > + unsigned long BaseAddress1; > + BusLogic_IO_Address_T IO_Address; > + BusLogic_PCI_Address_T PCI_Address; > > if (pci_enable_device(PCI_Device)) > continue; > > + IRQ_Channel = PCI_Device->irq; > + IO_Address = BaseAddress0 = > pci_resource_start(PCI_Device, 0); > + PCI_Address = BaseAddress1 = > pci_resource_start(PCI_Device, 1); > + > if (pci_resource_flags(PCI_Device, 0) & > IORESOURCE_MEM) > { > BusLogic_Error("BusLogic: Base Address0 0x%X not > I/O for " > > -- > Regards, > Rasmus(rasmus@jaquet.dk) > > "The obvious mathematical breakthrough would be > development of an easy way > to factor large prime numbers." > -- Bill Gates, The Road Ahead, Viking Penguin > (1995) > - > To unsubscribe from this list: send the line > "unsubscribe linux-scsi" in > the body of a message to majordomo@vger.kernel.org
--- Rasmus Andersen <rasmus@jaquet.dk> wrote:
__________________________________________________ Get personalized email addresses from Yahoo! Mail - only $35 a year! http://personal.mail.yahoo.com/ - 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/
| |