On Mon, Dec 12, 2011 at 10:02:52AM +0100, Oliver Neukum wrote: > Am Montag, 12. Dezember 2011, 10:06:04 schrieb Henrik Rydberg: > > @@ -895,6 +857,10 @@ static int bcm5974_probe(struct usb_interface *iface, > > /* save our data pointer in this interface device */ > > usb_set_intfdata(iface, dev); > > > > + error = bcm5974_start_traffic(dev); > > + if (error) > > + goto err_free_buffer; > > + > > return 0; > > > > err_free_buffer: > > -- > > Why do you want to start traffic before open() in probe() ? > It doesn't make sense to ask a device to deliver data nobody > wants. Seems I was a bit trigger happy here, will send another one. Thanks, Henrik -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
