summaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorDomagoj Trsan <domagoj.trsan@gmail.com>2014-09-20 14:40:38 +0200
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2014-09-23 23:41:25 -0700
commitd1d1a9d3a104531e25b7a33d60b627e93700327f (patch)
treef4ae1adf06506f0d6dc2e2470a8e5c1c90201b84 /drivers
parentc74920124750e5270c689d8c892a1c4263e5a547 (diff)
downloadop-kernel-dev-d1d1a9d3a104531e25b7a33d60b627e93700327f.zip
op-kernel-dev-d1d1a9d3a104531e25b7a33d60b627e93700327f.tar.gz
staging: line6: fix midibuf.c coding style issue
Fix the following checkpatch.pl warning: - else is not generally useful after a break or return Signed-off-by: Domagoj Trsan <domagoj.trsan@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/staging/line6/midibuf.c11
1 files changed, 8 insertions, 3 deletions
diff --git a/drivers/staging/line6/midibuf.c b/drivers/staging/line6/midibuf.c
index f0adb7b..1ff8569 100644
--- a/drivers/staging/line6/midibuf.c
+++ b/drivers/staging/line6/midibuf.c
@@ -15,11 +15,14 @@
static int midibuf_message_length(unsigned char code)
{
+ int message_length;
+
if (code < 0x80)
- return -1;
+ message_length = -1;
else if (code < 0xf0) {
static const int length[] = { 3, 3, 3, 3, 2, 2, 3 };
- return length[(code >> 4) - 8];
+
+ message_length = length[(code >> 4) - 8];
} else {
/*
Note that according to the MIDI specification 0xf2 is
@@ -29,8 +32,10 @@ static int midibuf_message_length(unsigned char code)
static const int length[] = { -1, 2, -1, 2, -1, -1, 1, 1, 1, 1,
1, 1, 1, -1, 1, 1
};
- return length[code & 0x0f];
+ message_length = length[code & 0x0f];
}
+
+ return message_length;
}
static int midibuf_is_empty(struct midi_buffer *this)
OpenPOWER on IntegriCloud