Skip to content

Commit

Permalink
[media] dvb_usb_v2: use dvb_usb_dbg_usb_control_msg()
Browse files Browse the repository at this point in the history
Convert drivers: au6610, ce6230, ec168, rtl28xxu for
dvb_usb_dbg_usb_control_msg() macro.

Signed-off-by: Antti Palosaari <crope@iki.fi>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
  • Loading branch information
Antti Palosaari authored and Mauro Carvalho Chehab committed Sep 15, 2012
1 parent 4ab7928 commit d89b936
Show file tree
Hide file tree
Showing 7 changed files with 13 additions and 39 deletions.
5 changes: 5 additions & 0 deletions drivers/media/usb/dvb-usb-v2/au6610.c
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,11 @@ static int au6610_usb_msg(struct dvb_usb_device *d, u8 operation, u8 addr,
ret = usb_control_msg(d->udev, usb_rcvctrlpipe(d->udev, 0), operation,
USB_TYPE_VENDOR|USB_DIR_IN, addr << 1, index,
usb_buf, 6, AU6610_USB_TIMEOUT);

dvb_usb_dbg_usb_control_msg(d->udev, operation,
(USB_TYPE_VENDOR|USB_DIR_IN), addr << 1, index,
usb_buf, 6);

if (ret < 0)
goto error;

Expand Down
4 changes: 2 additions & 2 deletions drivers/media/usb/dvb-usb-v2/ce6230.c
Original file line number Diff line number Diff line change
Expand Up @@ -74,8 +74,8 @@ static int ce6230_ctrl_msg(struct dvb_usb_device *d, struct usb_req *req)
ret = usb_control_msg(d->udev, pipe, request, requesttype, value, index,
buf, req->data_len, CE6230_USB_TIMEOUT);

ce6230_debug_dump(request, requesttype, value, index, buf,
req->data_len);
dvb_usb_dbg_usb_control_msg(d->udev, request, requesttype, value, index,
buf, req->data_len);

if (ret < 0)
pr_err("%s: usb_control_msg() failed=%d\n", KBUILD_MODNAME,
Expand Down
11 changes: 0 additions & 11 deletions drivers/media/usb/dvb-usb-v2/ce6230.h
Original file line number Diff line number Diff line change
Expand Up @@ -26,17 +26,6 @@
#include "zl10353.h"
#include "mxl5005s.h"

#define ce6230_debug_dump(r, t, v, i, b, l) { \
char *direction; \
if (t == (USB_TYPE_VENDOR | USB_DIR_OUT)) \
direction = ">>>"; \
else \
direction = "<<<"; \
pr_debug("%s: %02x %02x %02x %02x %02x %02x %02x %02x %s [%d bytes]\n", \
__func__, t, r, v & 0xff, v >> 8, i & 0xff, i >> 8, \
l & 0xff, l >> 8, direction, l); \
}

#define CE6230_USB_TIMEOUT 1000

struct usb_req {
Expand Down
4 changes: 2 additions & 2 deletions drivers/media/usb/dvb-usb-v2/ec168.c
Original file line number Diff line number Diff line change
Expand Up @@ -86,8 +86,8 @@ static int ec168_ctrl_msg(struct dvb_usb_device *d, struct ec168_req *req)
ret = usb_control_msg(d->udev, pipe, request, requesttype, req->value,
req->index, buf, req->size, EC168_USB_TIMEOUT);

ec168_debug_dump(request, requesttype, req->value, req->index, buf,
req->size);
dvb_usb_dbg_usb_control_msg(d->udev, request, requesttype, req->value,
req->index, buf, req->size);

if (ret < 0)
goto err_dealloc;
Expand Down
11 changes: 0 additions & 11 deletions drivers/media/usb/dvb-usb-v2/ec168.h
Original file line number Diff line number Diff line change
Expand Up @@ -24,17 +24,6 @@

#include "dvb_usb.h"

#define ec168_debug_dump(r, t, v, i, b, l) { \
char *direction; \
if (t == (USB_TYPE_VENDOR | USB_DIR_OUT)) \
direction = ">>>"; \
else \
direction = "<<<"; \
pr_debug("%s: %02x %02x %02x %02x %02x %02x %02x %02x %s\n", \
__func__, t, r, v & 0xff, v >> 8, i & 0xff, i >> 8, \
l & 0xff, l >> 8, direction); \
}

#define EC168_USB_TIMEOUT 1000

struct ec168_req {
Expand Down
6 changes: 4 additions & 2 deletions drivers/media/usb/dvb-usb-v2/rtl28xxu.c
Original file line number Diff line number Diff line change
Expand Up @@ -59,11 +59,13 @@ static int rtl28xxu_ctrl_msg(struct dvb_usb_device *d, struct rtl28xxu_req *req)

ret = usb_control_msg(d->udev, pipe, 0, requesttype, req->value,
req->index, buf, req->size, 1000);

dvb_usb_dbg_usb_control_msg(d->udev, 0, requesttype, req->value,
req->index, buf, req->size);

if (ret > 0)
ret = 0;

deb_dump(0, requesttype, req->value, req->index, buf, req->size);

/* read request, copy returned data to return buf */
if (!ret && requesttype == (USB_TYPE_VENDOR | USB_DIR_IN))
memcpy(req->data, buf, req->size);
Expand Down
11 changes: 0 additions & 11 deletions drivers/media/usb/dvb-usb-v2/rtl28xxu.h
Original file line number Diff line number Diff line change
Expand Up @@ -24,17 +24,6 @@

#include "dvb_usb.h"

#define deb_dump(r, t, v, i, b, l) { \
char *direction; \
if (t == (USB_TYPE_VENDOR | USB_DIR_OUT)) \
direction = ">>>"; \
else \
direction = "<<<"; \
dev_dbg(&d->udev->dev, "%s: %02x %02x %02x %02x %02x %02x %02x %02x " \
"%s [%d bytes]\n", __func__, t, r, v & 0xff, v >> 8, \
i & 0xff, i >> 8, l & 0xff, l >> 8, direction, l); \
}

/*
* USB commands
* (usb_control_msg() index parameter)
Expand Down

0 comments on commit d89b936

Please sign in to comment.