Messages in this thread | | | Subject | Re: AMD 8131 and MSI quirk | From | Roland Dreier <> | Date | Tue, 14 Feb 2006 09:17:41 -0800 |
| |
> 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.
Hmm:
> +typedef unsigned short __bitwise pci_bus_flags_t;
and:
> - unsigned short pad2; > + pci_bus_flags_t bus_flags; /* Inherited by child busses */
This does make pci_bus_flags_t a short -- it just lets sparse catch misuses of the enum values.
It's debatable whether it's worth the source obfuscation to let sparse check this, since it seems rather unlikely that someone will screw it up. But I don't see how it makes any difference in the generated code; certainly there's no bloat (beyond the extra source code)
- R. - 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/
| |