Here is an updated patch addressing most of Uwe's and Peter's comments. Ripping out the ehci/uhci_init() code doesn't seem to have done any harm, and I got rid of a bunch of unused junk in i3100_smbus.h
I left the *_set_subsystem() arguments unsigned, as that's how the function is declared in include/device/pci.h.
Signed-off-by: Ed Swierk eswierk@arastra.com
--Ed