Re: [PATCH] And yet more PCI fixes for 2.5.70

[Greg KH <greg@kroah.com>]

On Wed, Jun 11, 2003 at 01:37:37PM +0100, Alan Cox wrote:
> On Mer, 2003-06-11 at 01:11, Greg KH wrote:
> >  			/* user supplied value */
> >  			system_bus_speed = idebus_parameter;
> > -		} else if (pci_present()) {
> > +		} else if (pci_find_device(PCI_ANY_ID, PCI_ANY_ID, NULL) != NULL) {
> 
> That is just gross. pci_present() is far more readable even if you make
> it an inline in pci.h that is pci_find_device(PCI_ANY_ID, PCI_ANY_ID,
> NULL)

Bartlomiej was actually the one who suggested this patch, I didn't do it
on my own :)

Anyway, there are only 2 places in the whole kernel that want a
pci_present() check, this place, and drivers/sbus/sbus.c.  sbus.c can
probably be changed to not need it at all, but I did it this way to be
safe.

So that leaves only this file.  Jeff Garzik and I talked about removing
pci_present() as it's not needed, and I think for this one case we can
live without it.  Do you want me to make the pci_present() macro earlier
in this file, so it's readable again?  I don't want to put it back into
pci.h.

thanks,

greg k-h