diff options
author | Jerome Brunet <jbrunet@baylibre.com> | 2020-02-14 14:13:48 +0100 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2020-02-14 20:51:15 +0000 |
commit | 6e700f0672199f773ad645c2b7e886c1d2e2046e (patch) | |
tree | ebda03100f11f27b1e60c83fddc3bb639406926c /sound/soc/meson/aiu.c | |
parent | 269f00171273e47eebc915cc6ee8ceececa37a3a (diff) | |
download | lwn-6e700f0672199f773ad645c2b7e886c1d2e2046e.tar.gz lwn-6e700f0672199f773ad645c2b7e886c1d2e2046e.zip |
ASoC: meson: aiu: fix irq registration
The aiu stored the irq in an unsigned integer which may have discarded an
error returned by platform_get_irq_byname(). This is incorrect and should
have been a signed integer.
Also drop the irq error traces from the probe function as this is already
done by platform_get_irq_byname().
Fixes: 6ae9ca9ce986 ("ASoC: meson: aiu: add i2s and spdif support")
Reported-by: kbuild test robot <lkp@intel.com>
Signed-off-by: Jerome Brunet <jbrunet@baylibre.com>
Link: https://lore.kernel.org/r/20200214131350.337968-4-jbrunet@baylibre.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound/soc/meson/aiu.c')
-rw-r--r-- | sound/soc/meson/aiu.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/sound/soc/meson/aiu.c b/sound/soc/meson/aiu.c index de678a9d5cab..34b40b8b8299 100644 --- a/sound/soc/meson/aiu.c +++ b/sound/soc/meson/aiu.c @@ -314,16 +314,12 @@ static int aiu_probe(struct platform_device *pdev) } aiu->i2s.irq = platform_get_irq_byname(pdev, "i2s"); - if (aiu->i2s.irq < 0) { - dev_err(dev, "Can't get i2s irq\n"); + if (aiu->i2s.irq < 0) return aiu->i2s.irq; - } aiu->spdif.irq = platform_get_irq_byname(pdev, "spdif"); - if (aiu->spdif.irq < 0) { - dev_err(dev, "Can't get spdif irq\n"); + if (aiu->spdif.irq < 0) return aiu->spdif.irq; - } ret = aiu_clk_get(dev); if (ret) |