Re: [PATCH 1/13]: PCI Err: pci.h header file changes

From: Greg KH
Date: Tue Jun 28 2005 - 19:36:42 EST


On Tue, Jun 28, 2005 at 06:58:17PM -0500, Linas Vepstas wrote:
> @@ -673,6 +704,7 @@ struct pci_driver {
> int (*enable_wake) (struct pci_dev *dev, pci_power_t state, int enable); /* Enable wake event */
> void (*shutdown) (struct pci_dev *dev);
>
> + struct pci_error_handlers err_handler;
> struct device_driver driver;
> struct pci_dynids dynids;
> };

Shouldn't that be a pointer and not the whole structure? Wouldn't that
make it easier to "reuse" error handlers?

thanks,

greg k-h
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/