Eric, The lpci_set_subsystem() in amd8111_ac97.c is exactly the same as the default pci_dev_set_subsystem(), why do you define it instead of using the default one? BTW, why can't the set_subsystem() method be set at runtime ? Ollie