Message ID | 1319535558-20782-2-git-send-email-marek.vasut@gmail.com |
---|---|
State | Accepted |
Headers | show |
Dear Marek Vasut, In message <1319535558-20782-2-git-send-email-marek.vasut@gmail.com> you wrote: > usb.c: In function ‘usb_parse_config’: > usb.c:331:17: warning: variable ‘ch’ set but not used > [-Wunused-but-set-variable] > usb.c: In function ‘usb_hub_port_connect_change’: > usb.c:1123:29: warning: variable ‘portchange’ set but not used > [-Wunused-but-set-variable] > usb.c: In function ‘usb_hub_configure’: > usb.c:1183:25: warning: variable ‘hubsts’ set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Marek Vasut <marek.vasut@gmail.com> > Cc: Wolfgang Denk <wd@denx.de> > Cc: Simon Glass <sjg@chromium.org> > Cc: Mike Frysinger <vapier@gentoo.org> > --- > common/usb.c | 21 +++++++-------------- > 1 files changed, 7 insertions(+), 14 deletions(-) Applied, thanks. Best regards, Wolfgang Denk
diff --git a/common/usb.c b/common/usb.c index 2cd50db..bed5116 100644 --- a/common/usb.c +++ b/common/usb.c @@ -56,16 +56,16 @@ #endif #ifdef DEBUG -#define USB_DEBUG -#define USB_HUB_DEBUG -#endif - -#ifdef USB_DEBUG -#define USB_PRINTF(fmt, args...) printf(fmt , ##args) +#define USB_DEBUG 1 +#define USB_HUB_DEBUG 1 #else -#define USB_PRINTF(fmt, args...) +#define USB_DEBUG 0 +#define USB_HUB_DEBUG 0 #endif +#define USB_PRINTF(fmt, args...) debug_cond(USB_DEBUG, fmt, ##args) +#define USB_HUB_PRINTF(fmt, args...) debug_cond(USB_HUB_DEBUG, fmt, ##args) + #define USB_BUFSIZ 512 static struct usb_device usb_dev[USB_MAX_DEVICE]; @@ -968,13 +968,6 @@ void usb_scan_devices(void) * Probes device for being a hub and configurate it */ -#ifdef USB_HUB_DEBUG -#define USB_HUB_PRINTF(fmt, args...) printf(fmt , ##args) -#else -#define USB_HUB_PRINTF(fmt, args...) -#endif - - static struct usb_hub_device hub_dev[USB_MAX_HUB]; static int usb_hub_index;
usb.c: In function ‘usb_parse_config’: usb.c:331:17: warning: variable ‘ch’ set but not used [-Wunused-but-set-variable] usb.c: In function ‘usb_hub_port_connect_change’: usb.c:1123:29: warning: variable ‘portchange’ set but not used [-Wunused-but-set-variable] usb.c: In function ‘usb_hub_configure’: usb.c:1183:25: warning: variable ‘hubsts’ set but not used [-Wunused-but-set-variable] Signed-off-by: Marek Vasut <marek.vasut@gmail.com> Cc: Wolfgang Denk <wd@denx.de> Cc: Simon Glass <sjg@chromium.org> Cc: Mike Frysinger <vapier@gentoo.org> --- common/usb.c | 21 +++++++-------------- 1 files changed, 7 insertions(+), 14 deletions(-)