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

staging: comedi: cb_pcidas64: checkpatch.pl cleanup (else after return)

Fix these checkpatch.pl warnings:

WARNING: else is not generally useful after a break or return

Also, remove an unnecessary trailing ';' at the end of a function.
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 34880ec7
...@@ -636,7 +636,7 @@ static inline unsigned int ai_dma_ring_count(const struct pcidas64_board *board) ...@@ -636,7 +636,7 @@ static inline unsigned int ai_dma_ring_count(const struct pcidas64_board *board)
{ {
if (board->layout == LAYOUT_4020) if (board->layout == LAYOUT_4020)
return MAX_AI_DMA_RING_COUNT; return MAX_AI_DMA_RING_COUNT;
else
return MIN_AI_DMA_RING_COUNT; return MIN_AI_DMA_RING_COUNT;
} }
...@@ -1045,9 +1045,9 @@ static inline unsigned short se_diff_bit_6xxx(struct comedi_device *dev, ...@@ -1045,9 +1045,9 @@ static inline unsigned short se_diff_bit_6xxx(struct comedi_device *dev,
if ((thisboard->layout == LAYOUT_64XX && !use_differential) || if ((thisboard->layout == LAYOUT_64XX && !use_differential) ||
(thisboard->layout == LAYOUT_60XX && use_differential)) (thisboard->layout == LAYOUT_60XX && use_differential))
return ADC_SE_DIFF_BIT; return ADC_SE_DIFF_BIT;
else
return 0; return 0;
}; }
struct ext_clock_info { struct ext_clock_info {
/* master clock divisor to use for scans with external master clock */ /* master clock divisor to use for scans with external master clock */
...@@ -2162,7 +2162,7 @@ static int use_hw_sample_counter(struct comedi_cmd *cmd) ...@@ -2162,7 +2162,7 @@ static int use_hw_sample_counter(struct comedi_cmd *cmd)
if (cmd->stop_src == TRIG_COUNT && cmd->stop_arg <= max_counter_value) if (cmd->stop_src == TRIG_COUNT && cmd->stop_arg <= max_counter_value)
return 1; return 1;
else
return 0; return 0;
} }
...@@ -3384,9 +3384,8 @@ static int dio_callback(int dir, int port, int data, unsigned long arg) ...@@ -3384,9 +3384,8 @@ static int dio_callback(int dir, int port, int data, unsigned long arg)
if (dir) { if (dir) {
writeb(data, iobase + port); writeb(data, iobase + port);
return 0; return 0;
} else {
return readb(iobase + port);
} }
return readb(iobase + port);
} }
static int dio_callback_4020(int dir, int port, int data, unsigned long arg) static int dio_callback_4020(int dir, int port, int data, unsigned long arg)
...@@ -3395,9 +3394,8 @@ static int dio_callback_4020(int dir, int port, int data, unsigned long arg) ...@@ -3395,9 +3394,8 @@ static int dio_callback_4020(int dir, int port, int data, unsigned long arg)
if (dir) { if (dir) {
writew(data, iobase + 2 * port); writew(data, iobase + 2 * port);
return 0; return 0;
} else {
return readw(iobase + 2 * port);
} }
return readw(iobase + 2 * port);
} }
static int di_rbits(struct comedi_device *dev, struct comedi_subdevice *s, static int di_rbits(struct comedi_device *dev, struct comedi_subdevice *s,
......
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