diff options
| author | Sean Williams | 2013-09-08 00:51:34 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman | 2013-09-17 16:47:40 +0200 |
| commit | cb30fc50feb70762d7d82dab9f832ede3de1d067 (patch) | |
| tree | d312a3727590d14692f30e9a6998b026063c2646 | |
| parent | staging: comedi: ni_labpc: use comedi_range_is_unipolar() (diff) | |
| download | kernel-qcow2-linux-cb30fc50feb70762d7d82dab9f832ede3de1d067.tar.gz kernel-qcow2-linux-cb30fc50feb70762d7d82dab9f832ede3de1d067.tar.xz kernel-qcow2-linux-cb30fc50feb70762d7d82dab9f832ede3de1d067.zip | |
Staging: comedi: ni_at_a2150: Remove a few superfluous braces
Please don't flame me :) I'm getting my feet wet with kernel contribution.
One example I saw in a video by GKH suggested cleaning up coding style as a good first commit.
Signed-off-by: Sean Williams <unixed@gmail.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | drivers/staging/comedi/drivers/ni_at_a2150.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/drivers/staging/comedi/drivers/ni_at_a2150.c b/drivers/staging/comedi/drivers/ni_at_a2150.c index 2512ce8dfcaa..c77d34306119 100644 --- a/drivers/staging/comedi/drivers/ni_at_a2150.c +++ b/drivers/staging/comedi/drivers/ni_at_a2150.c @@ -684,13 +684,12 @@ static int a2150_set_chanlist(struct comedi_device *dev, devpriv->config_bits |= CHANNEL_BITS(0x4 | start_channel); break; case 2: - if (start_channel == 0) { + if (start_channel == 0) devpriv->config_bits |= CHANNEL_BITS(0x2); - } else if (start_channel == 2) { + else if (start_channel == 2) devpriv->config_bits |= CHANNEL_BITS(0x3); - } else { + else return -1; - } break; case 4: devpriv->config_bits |= CHANNEL_BITS(0x1); |