On Tue, Nov 13, 2012 at 04:07:17PM +0800, Lan Tianyu wrote: > + for (i = 1; i <= ports; i++) { > + if (usb_get_hub_port_connect_type( > + ehci_to_hcd(ehci)->self.root_hub, i) > + == USB_PORT_CONNECT_TYPE_HARD_WIRED) > + desc->u.hs.DeviceRemovable[i/8] |= 1 << (i%8); > + } > + > + I'm amazed that checkpatch.pl doesn't throw up all over this. Sneaky way of getting around it by messing with the line breaks. Please fix this pattern up, you also do it in other places in this series. greg k-h -- 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