ASoC: atmel: Don't squash error codes from atmel_ssc_set_audio()
authorMark Brown <broonie@kernel.org>
Fri, 25 Mar 2022 15:42:41 +0000 (15:42 +0000)
committerMark Brown <broonie@kernel.org>
Mon, 4 Apr 2022 13:24:20 +0000 (14:24 +0100)
The AT91SAM9G20-EK audio driver is replacing any error code returned by
atmel_ssc_set_audio() with -EINVAL which could be unhelpful for debugging.
Pass through the error code, and include it in the log message we print for
good measure.

Signed-off-by: Mark Brown <broonie@kernel.org>
Reviewed-by: Codrin Ciubotariu <codrin.ciubotariu@microchip.com>
Link: https://lore.kernel.org/r/20220325154241.1600757-4-broonie@kernel.org
sound/soc/atmel/sam9g20_wm8731.c

index d771843..0365b58 100644 (file)
@@ -127,8 +127,8 @@ static int at91sam9g20ek_audio_probe(struct platform_device *pdev)
 
        ret = atmel_ssc_set_audio(0);
        if (ret) {
-               dev_err(&pdev->dev, "ssc channel is not valid\n");
-               return -EINVAL;
+               dev_err(&pdev->dev, "ssc channel is not valid: %d\n", ret);
+               return ret;
        }
 
        card->dev = &pdev->dev;