From: Sean Williams Date: Sat, 7 Sep 2013 22:51:34 +0000 (-0700) Subject: Staging: comedi: ni_at_a2150: Remove a few superfluous braces X-Git-Tag: v3.13-rc1~170^2~781 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=cb30fc50feb70762d7d82dab9f832ede3de1d067;p=kernel%2Fkernel-generic.git 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 Reviewed-by: Ian Abbott Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/comedi/drivers/ni_at_a2150.c b/drivers/staging/comedi/drivers/ni_at_a2150.c index 2512ce8..c77d343 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);