[PATCH/RFC,v2,5/8] dsbr100: cleanup return value of start/stop handlers
Commit Message
The value returned from the device in radio->transfer_buffer[0]
isn't clearly defined, but is used as a return code. This value
is an unsigned 8-bit integer that is implicitly converted to an
int. Since this value can never be less than 0, return 0 instead.
This simplifies the verification of calling dsbr100_start and
dsbr100_stop.
Signed-off-by: David Ellingsworth <david@identd.dyndns.org>
---
drivers/media/radio/dsbr100.c | 12 ++++++------
1 files changed, 6 insertions(+), 6 deletions(-)
@@ -206,7 +206,7 @@ static int dsbr100_start(struct dsbr100_device *radio)
}
radio->status = STARTED;
- return (radio->transfer_buffer)[0];
+ return 0;
usb_control_msg_failed:
dev_err(&radio->usbdev->dev,
@@ -247,7 +247,7 @@ static int dsbr100_stop(struct dsbr100_device *radio)
}
radio->status = STOPPED;
- return (radio->transfer_buffer)[0];
+ return 0;
usb_control_msg_failed:
dev_err(&radio->usbdev->dev,
@@ -461,14 +461,14 @@ static int vidioc_s_ctrl(struct file *file, void *priv,
case V4L2_CID_AUDIO_MUTE:
if (ctrl->value) {
retval = dsbr100_stop(radio);
- if (retval < 0) {
+ if (retval) {
dev_warn(&radio->usbdev->dev,
"Radio did not respond properly\n");
return -EBUSY;
}
} else {
retval = dsbr100_start(radio);
- if (retval < 0) {
+ if (retval) {
dev_warn(&radio->usbdev->dev,
"Radio did not respond properly\n");
return -EBUSY;
@@ -542,7 +542,7 @@ static int usb_dsbr100_suspend(struct usb_interface *intf, pm_message_t message)
if (radio->status == STARTED) {
retval = dsbr100_stop(radio);
- if (retval < 0)
+ if (retval)
dev_warn(&intf->dev, "dsbr100_stop failed\n");
/* After dsbr100_stop() status set to STOPPED.
@@ -569,7 +569,7 @@ static int usb_dsbr100_resume(struct usb_interface *intf)
if (radio->status == STARTED) {
retval = dsbr100_start(radio);
- if (retval < 0)
+ if (retval)
dev_warn(&intf->dev, "dsbr100_start failed\n");
}