Messages in this thread | | | Date | Tue, 14 Feb 2006 09:56:01 -0700 | From | Matthew Wilcox <> | Subject | Re: AMD 8131 and MSI quirk |
| |
On Tue, Feb 14, 2006 at 06:52:22PM +0200, Michael S. Tsirkin wrote: > The following should do this IMO. Roland, could you test this patch please?
Going a bit overboard on the type safety. Please, leave bus_flags as an unsigned short so as not to bloat the pci_bus structure unnecessarily.
> +typedef unsigned short __bitwise pci_bus_flags_t; > +enum pci_bus_flags { > + PCI_BUS_FLAGS_NO_MSI = (pci_bus_flags_t) 1, > +}; > + > /* > * The pci_dev structure is used to describe PCI devices. > */ > @@ -203,7 +208,7 @@ struct pci_bus { > char name[48]; > > unsigned short bridge_ctl; /* manage NO_ISA/FBB/et al behaviors */ > - unsigned short pad2; > + pci_bus_flags_t bus_flags; /* Inherited by child busses */ > struct device *bridge; - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |