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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to