On Mon, Aug 03, 2015 at 01:08:26PM -0400, Kris Borer wrote: > else > + switch (ctl->ioctl_code) { This else statement isn't indented. It looks weird and it will cause static checker warnings (hopefully). Maybe leave that one out and send the rest. Do complicated one in a different patch. regards, dan carpenter -- 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