aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMatthias Kaehlcke <mka@chromium.org>2018-01-19 18:36:50 -0500
committerMark Brown <broonie@kernel.org>2018-01-22 07:20:46 -0500
commit35b84bf0614a7da98bfbbac70ed3d01c3b5b6c58 (patch)
tree82ae84e6ebf99726291a5eff2b36fa0d6a2adab7
parent7fb59e940f6225beed0b24cd09e9fad9aebb7565 (diff)
ASoC: dmic: Fix check of return value from read of 'num-channels'
Commit 7fb59e940f62 ("ASoC: codecs: dmic: Make number of channels configurable") introduces an optional property to the device tree to specify the number of DMIC channels. dmic_codec_probe() uses of_property_read_u32() to read the DT value, and expects a return value of -ENOENT when the property does not exist. This expectation is incorrect, the actual value returned in this case is -EINVAL (see of_find_property_value_of_size(), which is called under the hood). Check for -EINVAL instead. Fixes: 7fb59e940f62 ("ASoC: codecs: dmic: Make number of channels configurable") Signed-off-by: Matthias Kaehlcke <mka@chromium.org> Signed-off-by: Mark Brown <broonie@kernel.org>
-rw-r--r--sound/soc/codecs/dmic.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/soc/codecs/dmic.c b/sound/soc/codecs/dmic.c
index c88f974ebe3e..cf83c423394d 100644
--- a/sound/soc/codecs/dmic.c
+++ b/sound/soc/codecs/dmic.c
@@ -113,7 +113,7 @@ static int dmic_dev_probe(struct platform_device *pdev)
113 113
114 if (pdev->dev.of_node) { 114 if (pdev->dev.of_node) {
115 err = of_property_read_u32(pdev->dev.of_node, "num-channels", &chans); 115 err = of_property_read_u32(pdev->dev.of_node, "num-channels", &chans);
116 if (err && (err != -ENOENT)) 116 if (err && (err != -EINVAL))
117 return err; 117 return err;
118 118
119 if (!err) { 119 if (!err) {