diff options
author | Charles Keepax <ckeepax@opensource.wolfsonmicro.com> | 2016-05-04 14:59:11 +0100 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2016-05-09 17:35:50 +0200 |
commit | 875f6fffa2e09b48fa07ecbf2e28dd2425b5ce01 (patch) | |
tree | 8e6b082b705f1d528b0ca40fa17450a136a10023 | |
parent | 1d03f2bd56f3a45123a7572fb536c063068cfb83 (diff) | |
download | linux-875f6fffa2e09b48fa07ecbf2e28dd2425b5ce01.tar.bz2 |
ALSA: compress: Replace complex if statement with switch
A switch statement looks a bit cleaner than an if statement
spread over 3 lines, as such update this to a switch.
Signed-off-by: Charles Keepax <ckeepax@opensource.wolfsonmicro.com>
Acked-by: Vinod Koul <vinod.koul@intel.com>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
-rw-r--r-- | sound/core/compress_offload.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/sound/core/compress_offload.c b/sound/core/compress_offload.c index f56f4e358b76..9b3334be9df2 100644 --- a/sound/core/compress_offload.c +++ b/sound/core/compress_offload.c @@ -288,9 +288,12 @@ static ssize_t snd_compr_write(struct file *f, const char __user *buf, stream = &data->stream; mutex_lock(&stream->device->lock); /* write is allowed when stream is running or has been steup */ - if (stream->runtime->state != SNDRV_PCM_STATE_SETUP && - stream->runtime->state != SNDRV_PCM_STATE_PREPARED && - stream->runtime->state != SNDRV_PCM_STATE_RUNNING) { + switch (stream->runtime->state) { + case SNDRV_PCM_STATE_SETUP: + case SNDRV_PCM_STATE_PREPARED: + case SNDRV_PCM_STATE_RUNNING: + break; + default: mutex_unlock(&stream->device->lock); return -EBADFD; } |