please let us know your results after you have commented out the offending line. It should work. Jean Delvare wrote: > > > And the code with modifications: > > > > static int __init i2c_amd756_init(void) > > { > > struct pci_dev *dev; > > const struct pci_device_id *id; > > > > printk(KERN_INFO "i2c-amd756.o version %s (%s)\n", LM_VERSION, > > LM_DATE); > > > > return pci_module_init(&amd756_driver); > ^^^^^^ > > That return in the middle of the function might explain why you don't > see the change... ;) Strange if gcc did not tell you about that. > > > pci_for_each_dev(dev) { > > id = pci_match_device(amd756_ids, dev); > > if(id) > > if(amd756_probe(dev, id) >= 0) > > return 0; > > } > > return -ENODEV; > > > > > > } > > -- > Jean Delvare > http://www.ensicaen.ismra.fr/~delvare/