Hello.

On 04/13/2014 06:06 PM, vichy wrote:

Allways run your patches through checkpatch.pl so someone doesn't point
out the formatting issues in it when you send it to the mailing list :)

Well, I didn't want to complain about the example patch but now I'll just have to... :-)

Here it is

diff --git a/drivers/usb/core/hub.c b/drivers/usb/core/hub.c
index 090469e..2192776 100644
--- a/drivers/usb/core/hub.c
+++ b/drivers/usb/core/hub.c
@@ -4316,6 +4316,9 @@ fail:
         if (retval) {
                 hub_port_disable(hub, port1, 0);
                 update_devnum(udev, devnum);    /* for disconnect processing */
+               if(udev->speed == USB_SPEED_SUPER)

If you'd really run the patch thru checkpatch.pl, it would have complained about missing space after *if*. ;-)

+                       hub_port_reset(hub, port1, udev,
+                               HUB_BH_RESET_TIME, true);

WBR, Sergei

--
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