On Tuesday 17 April 2012, Roland Stigge wrote: > +static int match(struct device *dev, void *data) > +{ > + struct device_node *node = (struct device_node *)data; > + return dev->of_node == node ? 1 : 0; > +} One small correction: you should probably check if the device is actually bound to isp1301_driver: return (dev->of_node == node) && (dev->driver == &isp1301_driver.driver); Otherwise, the changes from last version look all good. 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