Commit f1051676 authored by H Hartley Sweeten's avatar H Hartley Sweeten Committed by Greg Kroah-Hartman

staging: comedi: cb_pcidas64: checkpatch.pl cleanup (break not useful)

Fix the checkpatch.pl warnings:

WARNING: break is not useful after a goto or return
Signed-off-by: default avatarH Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent ee99bcc7
...@@ -1874,7 +1874,6 @@ static int ai_config_master_clock_4020(struct comedi_device *dev, ...@@ -1874,7 +1874,6 @@ static int ai_config_master_clock_4020(struct comedi_device *dev,
break; break;
default: default:
return -EINVAL; return -EINVAL;
break;
} }
data[4] = divisor; data[4] = divisor;
...@@ -1890,10 +1889,8 @@ static int ai_config_master_clock(struct comedi_device *dev, unsigned int *data) ...@@ -1890,10 +1889,8 @@ static int ai_config_master_clock(struct comedi_device *dev, unsigned int *data)
switch (thisboard->layout) { switch (thisboard->layout) {
case LAYOUT_4020: case LAYOUT_4020:
return ai_config_master_clock_4020(dev, data); return ai_config_master_clock_4020(dev, data);
break;
default: default:
return -EINVAL; return -EINVAL;
break;
} }
return -EINVAL; return -EINVAL;
...@@ -1907,16 +1904,12 @@ static int ai_config_insn(struct comedi_device *dev, struct comedi_subdevice *s, ...@@ -1907,16 +1904,12 @@ static int ai_config_insn(struct comedi_device *dev, struct comedi_subdevice *s,
switch (id) { switch (id) {
case INSN_CONFIG_ALT_SOURCE: case INSN_CONFIG_ALT_SOURCE:
return ai_config_calibration_source(dev, data); return ai_config_calibration_source(dev, data);
break;
case INSN_CONFIG_BLOCK_SIZE: case INSN_CONFIG_BLOCK_SIZE:
return ai_config_block_size(dev, data); return ai_config_block_size(dev, data);
break;
case INSN_CONFIG_TIMER_1: case INSN_CONFIG_TIMER_1:
return ai_config_master_clock(dev, data); return ai_config_master_clock(dev, data);
break;
default: default:
return -EINVAL; return -EINVAL;
break;
} }
return -EINVAL; return -EINVAL;
} }
...@@ -2224,7 +2217,6 @@ static uint32_t ai_scan_counter_6xxx(struct comedi_device *dev, ...@@ -2224,7 +2217,6 @@ static uint32_t ai_scan_counter_6xxx(struct comedi_device *dev,
break; break;
default: default:
return 0; return 0;
break;
} }
return count - 3; return count - 3;
} }
...@@ -3568,7 +3560,6 @@ static int caldac_i2c_write(struct comedi_device *dev, ...@@ -3568,7 +3560,6 @@ static int caldac_i2c_write(struct comedi_device *dev,
default: default:
comedi_error(dev, "invalid caldac channel\n"); comedi_error(dev, "invalid caldac channel\n");
return -1; return -1;
break;
} }
serial_bytes[1] = NOT_CLEAR_REGISTERS | ((value >> 8) & 0xf); serial_bytes[1] = NOT_CLEAR_REGISTERS | ((value >> 8) & 0xf);
serial_bytes[2] = value & 0xff; serial_bytes[2] = value & 0xff;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment