Ah, one more common style fixup that you should fixup:
> +static void sisusb_kill_all_busy(struct sisusb_usb_data *sisusb)
> +{
> + int i;
> +
> + if (sisusb_all_free(sisusb)) {
> + return;
> + }
For if and for loops that only have 1 statement, don't add the { } to
them, it's not needed.
thanks,
greg k-h
-------------------------------------------------------
This SF.Net email is sponsored by: IntelliVIEW -- Interactive Reporting
Tool for open source databases. Create drag-&-drop reports. Save time
by over 75%! Publish reports on the web. Export to DOC, XLS, RTF, etc.
Download a FREE copy at http://www.intelliview.com/go/osdn_nl
_______________________________________________
linux-usb-devel@lists.sourceforge.net
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
