Commit 92271551 authored by Hoang Tran's avatar Hoang Tran Committed by Greg Kroah-Hartman

staging: goldfish: fix coding style.

Using an else following a break or return can unnecessarily
indent code blocks.

This patch fixes coding style reported by checkpatch.pl, a part
of eudyptula challenge.
Signed-off-by: default avatarHoang Tran <hoangtran.gwr@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 9c340ac9
...@@ -203,10 +203,10 @@ static int goldfish_audio_open(struct inode *ip, struct file *fp) ...@@ -203,10 +203,10 @@ static int goldfish_audio_open(struct inode *ip, struct file *fp)
AUDIO_INT_WRITE_BUFFER_2_EMPTY); AUDIO_INT_WRITE_BUFFER_2_EMPTY);
AUDIO_WRITE(audio_data, AUDIO_INT_ENABLE, AUDIO_INT_MASK); AUDIO_WRITE(audio_data, AUDIO_INT_ENABLE, AUDIO_INT_MASK);
return 0; return 0;
} else {
atomic_dec(&open_count);
return -EBUSY;
} }
atomic_dec(&open_count);
return -EBUSY;
} }
static int goldfish_audio_release(struct inode *ip, struct file *fp) static int goldfish_audio_release(struct inode *ip, struct file *fp)
...@@ -223,8 +223,8 @@ static long goldfish_audio_ioctl(struct file *fp, unsigned int cmd, ...@@ -223,8 +223,8 @@ static long goldfish_audio_ioctl(struct file *fp, unsigned int cmd,
/* temporary workaround, until we switch to the ALSA API */ /* temporary workaround, until we switch to the ALSA API */
if (cmd == 315) if (cmd == 315)
return -1; return -1;
else
return 0; return 0;
} }
static irqreturn_t goldfish_audio_interrupt(int irq, void *dev_id) static irqreturn_t goldfish_audio_interrupt(int irq, void *dev_id)
......
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