V4L/DVB (12612): si4713: simplify the code to remove a compiler warning.
authorHans Verkuil <hverkuil@xs4all.nl>
Mon, 31 Aug 2009 20:21:04 +0000 (17:21 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Sat, 12 Sep 2009 15:19:50 +0000 (12:19 -0300)
The compiler warned about an uninitialized stereo variable. By simplifying
the code it 1) improved readability and 2) fixed the compiler warning.

Acked-by: Eduardo Valentin <eduardo.valentin@nokia.com>
Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/radio/si4713-i2c.c

index 8cbbe48b01bdae57f412192d01c92f2db8927b5b..6a0028eb461f91ea767a609c56c09a9732ec2544 100644 (file)
@@ -1841,15 +1841,11 @@ static int si4713_s_modulator(struct v4l2_subdev *sd, struct v4l2_modulator *vm)
        u16 stereo, rds;
        u32 p;
 
-       if (!sdev) {
-               rval = -ENODEV;
-               goto exit;
-       }
+       if (!sdev)
+               return -ENODEV;
 
-       if (vm->index > 0) {
-               rval = -EINVAL;
-               goto exit;
-       }
+       if (vm->index > 0)
+               return -EINVAL;
 
        /* Set audio mode: mono or stereo */
        if (vm->txsubchans & V4L2_TUNER_SUB_STEREO)
@@ -1857,9 +1853,7 @@ static int si4713_s_modulator(struct v4l2_subdev *sd, struct v4l2_modulator *vm)
        else if (vm->txsubchans & V4L2_TUNER_SUB_MONO)
                stereo = 0;
        else
-               rval = -EINVAL;
-       if (rval < 0)
-               goto exit;
+               return -EINVAL;
 
        rds = !!(vm->txsubchans & V4L2_TUNER_SUB_RDS);
 
@@ -1885,7 +1879,6 @@ static int si4713_s_modulator(struct v4l2_subdev *sd, struct v4l2_modulator *vm)
 
 unlock:
        mutex_unlock(&sdev->mutex);
-exit:
        return rval;
 }