staging: media: lirc: remove unnecessary break after return
authorTapasweni Pathak <tapaswenipathak@gmail.com>
Sun, 21 Sep 2014 06:23:20 +0000 (11:53 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 23 Sep 2014 18:15:43 +0000 (11:15 -0700)
This patch fixes checkpatch.pl warning in lirc_serial.c file
WARNING : break is not useful after goto or return

Signed-off-by: Tapasweni Pathak <tapaswenipathak@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/media/lirc/lirc_serial.c

index 443954b6d3d817a6d3221bf01caf1f6669a3af6c..181b92bfe25deef04d4826bcc9cb3fefbbbda582 100644 (file)
@@ -948,7 +948,6 @@ static long lirc_ioctl(struct file *filep, unsigned int cmd, unsigned long arg)
 
        case LIRC_GET_LENGTH:
                return -ENOIOCTLCMD;
-               break;
 
        case LIRC_SET_SEND_DUTY_CYCLE:
                dprintk("SET_SEND_DUTY_CYCLE\n");
@@ -961,7 +960,6 @@ static long lirc_ioctl(struct file *filep, unsigned int cmd, unsigned long arg)
                if (value <= 0 || value > 100)
                        return -EINVAL;
                return init_timing_params(value, freq);
-               break;
 
        case LIRC_SET_SEND_CARRIER:
                dprintk("SET_SEND_CARRIER\n");
@@ -974,7 +972,6 @@ static long lirc_ioctl(struct file *filep, unsigned int cmd, unsigned long arg)
                if (value > 500000 || value < 20000)
                        return -EINVAL;
                return init_timing_params(duty_cycle, value);
-               break;
 
        default:
                return lirc_dev_fop_ioctl(filep, cmd, arg);