diff options
author | Maxime Ripard <maxime.ripard@free-electrons.com> | 2017-11-08 10:47:08 -0500 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2017-11-08 13:25:48 -0500 |
commit | 560bfe774f058e97596f30ff71cffdac52b72914 (patch) | |
tree | 593b5525b0fff3aa0a47a4c62d655e8fcf4ae1df | |
parent | 2bd6bf03f4c1c59381d62c61d03f6cc3fe71f66e (diff) |
ASoC: sun8i-codec: Invert Master / Slave condition
The current code had the condition backward when checking if the codec
should be running in slave or master mode.
Fix it, and make the comment a bit more readable.
Fixes: 36c684936fae ("ASoC: Add sun8i digital audio codec")
Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>
Reviewed-by: Chen-Yu Tsai <wens@csie.org>
Signed-off-by: Mark Brown <broonie@kernel.org>
Cc: <stable@vger.kernel.org>
-rw-r--r-- | sound/soc/sunxi/sun8i-codec.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/sound/soc/sunxi/sun8i-codec.c b/sound/soc/sunxi/sun8i-codec.c index abfb710df7cb..038107baf414 100644 --- a/sound/soc/sunxi/sun8i-codec.c +++ b/sound/soc/sunxi/sun8i-codec.c | |||
@@ -170,11 +170,11 @@ static int sun8i_set_fmt(struct snd_soc_dai *dai, unsigned int fmt) | |||
170 | 170 | ||
171 | /* clock masters */ | 171 | /* clock masters */ |
172 | switch (fmt & SND_SOC_DAIFMT_MASTER_MASK) { | 172 | switch (fmt & SND_SOC_DAIFMT_MASTER_MASK) { |
173 | case SND_SOC_DAIFMT_CBS_CFS: /* DAI Slave */ | 173 | case SND_SOC_DAIFMT_CBS_CFS: /* Codec slave, DAI master */ |
174 | value = 0x0; /* Codec Master */ | 174 | value = 0x1; |
175 | break; | 175 | break; |
176 | case SND_SOC_DAIFMT_CBM_CFM: /* DAI Master */ | 176 | case SND_SOC_DAIFMT_CBM_CFM: /* Codec Master, DAI slave */ |
177 | value = 0x1; /* Codec Slave */ | 177 | value = 0x0; |
178 | break; | 178 | break; |
179 | default: | 179 | default: |
180 | return -EINVAL; | 180 | return -EINVAL; |