USB: isp116x: remove dependency on CONFIG_USB_DEBUG

Move all debugging messages in the driver to use the dynamic debug
subsystem, and not rely on CONFIG_USB_DEBUG to turn them on or off.

This lets debugging be enabled without having to rebuild the driver, an
important thing for users that can not do it.

It also removes the pointless IRQ_TEST() macro, as that was totally
useless and obviously never used.

Cc: Olav Kongas <ok@artecdesign.ee>
Acked-by: Felipe Balbi <balbi@ti.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Greg Kroah-Hartman 2013-06-28 11:33:02 -07:00
parent 641c86cd1c
commit 1d896ceae4

View File

@ -325,11 +325,7 @@ struct isp116x_ep {
/*-------------------------------------------------------------------------*/ /*-------------------------------------------------------------------------*/
#ifdef DEBUG #define DBG(stuff...) pr_debug("116x: " stuff)
#define DBG(stuff...) printk(KERN_DEBUG "116x: " stuff)
#else
#define DBG(stuff...) do{}while(0)
#endif
#ifdef VERBOSE #ifdef VERBOSE
# define VDBG DBG # define VDBG DBG
@ -358,15 +354,8 @@ struct isp116x_ep {
#define isp116x_check_platform_delay(h) 0 #define isp116x_check_platform_delay(h) 0
#endif #endif
#if defined(DEBUG)
#define IRQ_TEST() BUG_ON(!irqs_disabled())
#else
#define IRQ_TEST() do{}while(0)
#endif
static inline void isp116x_write_addr(struct isp116x *isp116x, unsigned reg) static inline void isp116x_write_addr(struct isp116x *isp116x, unsigned reg)
{ {
IRQ_TEST();
writew(reg & 0xff, isp116x->addr_reg); writew(reg & 0xff, isp116x->addr_reg);
isp116x_delay(isp116x, 300); isp116x_delay(isp116x, 300);
} }