Commit b8f1ba99 authored by Dan Carpenter's avatar Dan Carpenter Committed by Greg Kroah-Hartman

usb: hub: make wait_for_connected() take an int instead of a pointer to int

The wait_for_connected() function doesn't modify "*port1" and there is
no need to pass a pointer.  Just pass the int itself.
Signed-off-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Link: https://lore.kernel.org/r/20211210142028.GB18906@kiliSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent f59f93cd
...@@ -3572,7 +3572,7 @@ static int finish_port_resume(struct usb_device *udev) ...@@ -3572,7 +3572,7 @@ static int finish_port_resume(struct usb_device *udev)
* This routine should only be called when persist is enabled. * This routine should only be called when persist is enabled.
*/ */
static int wait_for_connected(struct usb_device *udev, static int wait_for_connected(struct usb_device *udev,
struct usb_hub *hub, int *port1, struct usb_hub *hub, int port1,
u16 *portchange, u16 *portstatus) u16 *portchange, u16 *portstatus)
{ {
int status = 0, delay_ms = 0; int status = 0, delay_ms = 0;
...@@ -3586,7 +3586,7 @@ static int wait_for_connected(struct usb_device *udev, ...@@ -3586,7 +3586,7 @@ static int wait_for_connected(struct usb_device *udev,
} }
msleep(20); msleep(20);
delay_ms += 20; delay_ms += 20;
status = hub_port_status(hub, *port1, portstatus, portchange); status = hub_port_status(hub, port1, portstatus, portchange);
} }
dev_dbg(&udev->dev, "Waited %dms for CONNECT\n", delay_ms); dev_dbg(&udev->dev, "Waited %dms for CONNECT\n", delay_ms);
return status; return status;
...@@ -3692,7 +3692,7 @@ int usb_port_resume(struct usb_device *udev, pm_message_t msg) ...@@ -3692,7 +3692,7 @@ int usb_port_resume(struct usb_device *udev, pm_message_t msg)
} }
if (udev->persist_enabled) if (udev->persist_enabled)
status = wait_for_connected(udev, hub, &port1, &portchange, status = wait_for_connected(udev, hub, port1, &portchange,
&portstatus); &portstatus);
status = check_port_resume_type(udev, status = check_port_resume_type(udev,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment