]> git.baikalelectronics.ru Git - kernel.git/commit
usb: core: phy: fix return value of usb_phy_roothub_exit()
authorMartin Blumenstingl <martin.blumenstingl@googlemail.com>
Wed, 18 Apr 2018 19:39:46 +0000 (21:39 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 22 Apr 2018 13:01:30 +0000 (15:01 +0200)
commitba5363e9424bb7d0a6ea329d8523cc5d6bf949e2
treeba83c043a54009d56fc22a347ddc6ec5ffeeeeff
parentff35cd78e3ba376fcbc9123a5a31573fb81146d5
usb: core: phy: fix return value of usb_phy_roothub_exit()

usb_phy_roothub_exit() should return the error code from the phy_exit()
call if exiting the PHY failed.
However, since a wrong variable is used usb_phy_roothub_exit() currently
always returns 0, even if one of the phy_exit calls returned an error.
Clang also reports this bug:
kernel/drivers/usb/core/phy.c:114:8: warning: explicitly assigning value of
variable of type 'int' to itself [-Wself-assign] error, forbidden
warning: phy.c:114

Fix this by assigning the error code from phy_exit() to the "ret"
variable to propagate the error correctly.

Fixes: f619caa5f10889 ("usb: core: add a wrapper for the USB PHYs on the HCD")
Signed-off-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com>
Signed-off-by: Rishabh Bhatnagar <rishabhb@codeaurora.org>
Tested-by: Keerthy <j-keerthy@ti.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/core/phy.c