Commit d7e92f7f authored by Greg Kroah-Hartman's avatar Greg Kroah-Hartman

Watchdog: pcwd_usb: remove CONFIG_USB_DEBUG usage

CONFIG_USB_DEBUG is going away, and all of the other USB drivers no
longer rely on "debug" module parameters for debugging lines, so move
the pcwd_usb driver to use the dynamic debug infrastructure to be more
in line with the rest of the kernel.
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Wim Van Sebroeck <wim@iguana.be>
Reviewed-by: default avatarGuenter Roeck <linux@roeck-us.net>
parent 252d74f6
...@@ -44,23 +44,6 @@ ...@@ -44,23 +44,6 @@
#include <linux/hid.h> /* For HID_REQ_SET_REPORT & HID_DT_REPORT */ #include <linux/hid.h> /* For HID_REQ_SET_REPORT & HID_DT_REPORT */
#include <linux/uaccess.h> /* For copy_to_user/put_user/... */ #include <linux/uaccess.h> /* For copy_to_user/put_user/... */
#ifdef CONFIG_USB_DEBUG
static int debug = 1;
#else
static int debug;
#endif
/* Use our own dbg macro */
#undef dbg
#ifndef DEBUG
#define DEBUG
#endif
#define dbg(format, ...) \
do { \
if (debug) \
pr_debug(format "\n", ##__VA_ARGS__); \
} while (0)
/* Module and Version Information */ /* Module and Version Information */
#define DRIVER_VERSION "1.02" #define DRIVER_VERSION "1.02"
...@@ -73,10 +56,6 @@ MODULE_AUTHOR(DRIVER_AUTHOR); ...@@ -73,10 +56,6 @@ MODULE_AUTHOR(DRIVER_AUTHOR);
MODULE_DESCRIPTION(DRIVER_DESC); MODULE_DESCRIPTION(DRIVER_DESC);
MODULE_LICENSE(DRIVER_LICENSE); MODULE_LICENSE(DRIVER_LICENSE);
/* Module Parameters */
module_param(debug, int, 0);
MODULE_PARM_DESC(debug, "Debug enabled or not");
#define WATCHDOG_HEARTBEAT 0 /* default heartbeat = #define WATCHDOG_HEARTBEAT 0 /* default heartbeat =
delay-time from dip-switches */ delay-time from dip-switches */
static int heartbeat = WATCHDOG_HEARTBEAT; static int heartbeat = WATCHDOG_HEARTBEAT;
...@@ -193,6 +172,7 @@ static void usb_pcwd_intr_done(struct urb *urb) ...@@ -193,6 +172,7 @@ static void usb_pcwd_intr_done(struct urb *urb)
struct usb_pcwd_private *usb_pcwd = struct usb_pcwd_private *usb_pcwd =
(struct usb_pcwd_private *)urb->context; (struct usb_pcwd_private *)urb->context;
unsigned char *data = usb_pcwd->intr_buffer; unsigned char *data = usb_pcwd->intr_buffer;
struct device *dev = &usb_pcwd->interface->dev;
int retval; int retval;
switch (urb->status) { switch (urb->status) {
...@@ -202,17 +182,17 @@ static void usb_pcwd_intr_done(struct urb *urb) ...@@ -202,17 +182,17 @@ static void usb_pcwd_intr_done(struct urb *urb)
case -ENOENT: case -ENOENT:
case -ESHUTDOWN: case -ESHUTDOWN:
/* this urb is terminated, clean up */ /* this urb is terminated, clean up */
dbg("%s - urb shutting down with status: %d", __func__, dev_dbg(dev, "%s - urb shutting down with status: %d",
urb->status); __func__, urb->status);
return; return;
/* -EPIPE: should clear the halt */ /* -EPIPE: should clear the halt */
default: /* error */ default: /* error */
dbg("%s - nonzero urb status received: %d", __func__, dev_dbg(dev, "%s - nonzero urb status received: %d",
urb->status); __func__, urb->status);
goto resubmit; goto resubmit;
} }
dbg("received following data cmd=0x%02x msb=0x%02x lsb=0x%02x", dev_dbg(dev, "received following data cmd=0x%02x msb=0x%02x lsb=0x%02x",
data[0], data[1], data[2]); data[0], data[1], data[2]);
usb_pcwd->cmd_command = data[0]; usb_pcwd->cmd_command = data[0];
...@@ -251,7 +231,8 @@ static int usb_pcwd_send_command(struct usb_pcwd_private *usb_pcwd, ...@@ -251,7 +231,8 @@ static int usb_pcwd_send_command(struct usb_pcwd_private *usb_pcwd,
buf[2] = *lsb; /* Byte 2 = Data LSB */ buf[2] = *lsb; /* Byte 2 = Data LSB */
buf[3] = buf[4] = buf[5] = 0; /* All other bytes not used */ buf[3] = buf[4] = buf[5] = 0; /* All other bytes not used */
dbg("sending following data cmd=0x%02x msb=0x%02x lsb=0x%02x", dev_dbg(&usb_pcwd->interface->dev,
"sending following data cmd=0x%02x msb=0x%02x lsb=0x%02x",
buf[0], buf[1], buf[2]); buf[0], buf[1], buf[2]);
atomic_set(&usb_pcwd->cmd_received, 0); atomic_set(&usb_pcwd->cmd_received, 0);
...@@ -260,8 +241,9 @@ static int usb_pcwd_send_command(struct usb_pcwd_private *usb_pcwd, ...@@ -260,8 +241,9 @@ static int usb_pcwd_send_command(struct usb_pcwd_private *usb_pcwd,
HID_REQ_SET_REPORT, HID_DT_REPORT, HID_REQ_SET_REPORT, HID_DT_REPORT,
0x0200, usb_pcwd->interface_number, buf, 6, 0x0200, usb_pcwd->interface_number, buf, 6,
USB_COMMAND_TIMEOUT) != 6) { USB_COMMAND_TIMEOUT) != 6) {
dbg("usb_pcwd_send_command: error in usb_control_msg for " dev_dbg(&usb_pcwd->interface->dev,
"cmd 0x%x 0x%x 0x%x\n", cmd, *msb, *lsb); "usb_pcwd_send_command: error in usb_control_msg for cmd 0x%x 0x%x 0x%x\n",
cmd, *msb, *lsb);
} }
/* wait till the usb card processed the command, /* wait till the usb card processed the command,
* with a max. timeout of USB_COMMAND_TIMEOUT */ * with a max. timeout of USB_COMMAND_TIMEOUT */
......
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