V4L/DVB (10062): dsbr100: change return values in 3 functions
authorAlexey Klimov <klimov.linux@gmail.com>
Sun, 28 Dec 2008 01:40:57 +0000 (22:40 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Tue, 30 Dec 2008 11:40:12 +0000 (09:40 -0200)
Patch replace return -1; with return retval; in 3 functions.
And we also change if-checks for this returned values.

Signed-off-by: Alexey Klimov <klimov.linux@gmail.com>
Signed-off-by: Douglas Schilling Landgraf <dougsland@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/radio/dsbr100.c

index 5e0d651ffdd2dcd8ae57e61fab8c84c706c30b37..5474a22c1b2235ef627163c73ba27fe519d1b471 100644 (file)
@@ -236,7 +236,7 @@ usb_control_msg_failed:
        dev_err(&radio->usbdev->dev,
                "%s - usb_control_msg returned %i, request %i\n",
                        __func__, retval, request);
-       return -1;
+       return retval;
 
 }
 
@@ -279,7 +279,7 @@ usb_control_msg_failed:
        dev_err(&radio->usbdev->dev,
                "%s - usb_control_msg returned %i, request %i\n",
                        __func__, retval, request);
-       return -1;
+       return retval;
 
 }
 
@@ -336,7 +336,7 @@ usb_control_msg_failed:
        dev_err(&radio->usbdev->dev,
                "%s - usb_control_msg returned %i, request %i\n",
                        __func__, retval, request);
-       return -1;
+       return retval;
 }
 
 /* return the device status.  This is, in effect, just whether it
@@ -452,7 +452,7 @@ static int vidioc_s_frequency(struct file *file, void *priv,
 
        radio->curfreq = f->frequency;
        retval = dsbr100_setfreq(radio, radio->curfreq);
-       if (retval == -1)
+       if (retval < 0)
                dev_warn(&radio->usbdev->dev, "Set frequency failed\n");
        return 0;
 }
@@ -516,14 +516,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 == -1) {
+                       if (retval < 0) {
                                dev_warn(&radio->usbdev->dev,
                                         "Radio did not respond properly\n");
                                return -EBUSY;
                        }
                } else {
                        retval = dsbr100_start(radio);
-                       if (retval == -1) {
+                       if (retval < 0) {
                                dev_warn(&radio->usbdev->dev,
                                         "Radio did not respond properly\n");
                                return -EBUSY;
@@ -585,7 +585,7 @@ static int usb_dsbr100_open(struct inode *inode, struct file *file)
        }
 
        retval = dsbr100_setfreq(radio, radio->curfreq);
-       if (retval == -1)
+       if (retval < 0)
                dev_warn(&radio->usbdev->dev,
                        "set frequency failed\n");
 
@@ -604,7 +604,7 @@ static int usb_dsbr100_close(struct inode *inode, struct file *file)
        radio->users = 0;
        if (!radio->removed) {
                retval = dsbr100_stop(radio);
-               if (retval == -1) {
+               if (retval < 0) {
                        dev_warn(&radio->usbdev->dev,
                                "dsbr100_stop failed\n");
                }
@@ -620,7 +620,7 @@ static int usb_dsbr100_suspend(struct usb_interface *intf, pm_message_t message)
        int retval;
 
        retval = dsbr100_stop(radio);
-       if (retval == -1)
+       if (retval < 0)
                dev_warn(&intf->dev, "dsbr100_stop failed\n");
 
        dev_info(&intf->dev, "going into suspend..\n");
@@ -635,7 +635,7 @@ static int usb_dsbr100_resume(struct usb_interface *intf)
        int retval;
 
        retval = dsbr100_start(radio);
-       if (retval == -1)
+       if (retval < 0)
                dev_warn(&intf->dev, "dsbr100_start failed\n");
 
        dev_info(&intf->dev, "coming out of suspend..\n");