qiurui
/
rv1126-uboot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

UPSTREAM: usb: Correct use of debug() 

With clang this gives a warning because hubsts appears to be used before
it is set, even if ultimately it is not used. Simplify the code to avoid
this problem.

Change-Id: If765f7b0c909de125e163875856e43ddf9b6c5d5
Signed-off-by: Simon Glass <sjg@chromium.org>
Signed-off-by: Kever Yang <kever.yang@rock-chips.com>
(cherry picked from commit 0ad0458c76107b29325b8c38804e0407bed42a79)
This commit is contained in:
Simon Glass 2017-12-04 13:48:22 -07:00 committed by Kever Yang
parent 00ac81b92d
commit 143f37beff
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
common/usb_hub.c
View File

@ -625,7 +625,7 @@ static int usb_hub_configure(struct usb_device *dev)
short hubCharacteristics; short hubCharacteristics;
struct usb_hub_descriptor *descriptor; struct usb_hub_descriptor *descriptor;
struct usb_hub_device *hub; struct usb_hub_device *hub;
__maybe_unused struct usb_hub_status *hubsts; struct usb_hub_status *hubsts;
int ret; int ret;
hub = usb_get_hub_device(dev); hub = usb_get_hub_device(dev);
@ -779,9 +779,7 @@ static int usb_hub_configure(struct usb_device *dev)
return ret; return ret;
} }
#ifdef DEBUG
hubsts = (struct usb_hub_status *)buffer; hubsts = (struct usb_hub_status *)buffer;
#endif
debug("get_hub_status returned status %X, change %X\n", debug("get_hub_status returned status %X, change %X\n",
le16_to_cpu(hubsts->wHubStatus), le16_to_cpu(hubsts->wHubStatus),