On Wednesday 08 August 2012, Stephen Warren wrote: > I think that's all just: > > module_platform_driver(vt8500_gpio_driver); > > (except that _init uses platform_driver_probe() rather than > platform_driver_register(), which seems unusual. I guess that explains > the of_find_matching_node() above too.) Ah, I totally missed both of these. Using platform_driver_register is definitely preferred over platform_driver_probe in cases like this, so using module_platform_driver is the right simplification. Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html