]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
drm/vc4: hdmi: Avoid log spam for audio start failure
authorDom Cobley <popcornmix@gmail.com>
Fri, 21 Jun 2024 15:20:28 +0000 (16:20 +0100)
committerDave Stevenson <dave.stevenson@raspberrypi.com>
Mon, 9 Sep 2024 12:02:52 +0000 (13:02 +0100)
We regularly get dmesg error reports of:
[   18.184066] hdmi-audio-codec hdmi-audio-codec.3.auto: ASoC: error at snd_soc_dai_startup on i2s-hifi: -19
[   18.184098]  MAI: soc_pcm_open() failed (-19)

These are generated for any disconnected hdmi interface when pulseaudio
attempts to open the associated ALSA device (numerous times). Each open
generates a kernel error message, generating general log spam.

The error messages all come from _soc_pcm_ret in sound/soc/soc-pcm.c#L39
which suggests returning ENOTSUPP, rather that ENODEV will be quiet.
And indeed it is.

Signed-off-by: Dom Cobley <popcornmix@gmail.com>
Reviewed-by: Maxime Ripard <mripard@kernel.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20240621152055.4180873-5-dave.stevenson@raspberrypi.com
Signed-off-by: Dave Stevenson <dave.stevenson@raspberrypi.com>
drivers/gpu/drm/vc4/vc4_hdmi.c

index 6611ab7c26a63cafe6adf0d691d5df2a7ce61b81..43f4e150d72678ffbcb4813a87b3517983409437 100644 (file)
@@ -1920,7 +1920,7 @@ static int vc4_hdmi_audio_startup(struct device *dev, void *data)
        }
 
        if (!vc4_hdmi_audio_can_stream(vc4_hdmi)) {
-               ret = -ENODEV;
+               ret = -ENOTSUPP;
                goto out_dev_exit;
        }