Re: [PATCH v2] ASoC: soc-core: Create device_link to ensure correct suspend order
From: Marek Szyprowski
Date: Mon Jun 22 2026 - 03:08:05 EST
On 20.06.2026 17:57, Richard Fitzgerald wrote:
> On 17/6/26 15:10, Marek Szyprowski wrote:
>> On 11.06.2026 13:08, Richard Fitzgerald wrote:
>>> In snd_soc_bind_card() create a device_link from card to all components
>>> to ensure correct order of system_suspend. The card is the consumer and
>>> the components are the supplier, so that the card will system_suspend
>>> before any of the components.
>>>
>
> <SNIP>
>
>> This patch landed recently in linux-next as commit 0f54ce994b23 ("ASoC:
>> soc-core: Create device_link to ensure correct suspend order"). In my
>> tests I found that it breaks probing of VC4 DRM subsystem on Raspberry Pi
>> 3 and 4 boards due to an issue with hdmi-audio-codec:
>>
>> # dmesg | grep vc4
>> vc4-drm gpu: bound fe400000.hvs (ops vc4_hvs_ops [vc4])
>> vc4_hdmi fef00700.hdmi: Failed to create device link to hdmi-audio-codec.1.auto
>> vc4_hdmi fef00700.hdmi: error -EINVAL: Could not register sound card
>> vc4-drm gpu: failed to bind fef00700.hdmi (ops vc4_hdmi_ops [vc4]): -22
>> vc4-drm gpu: adev bind failed: -22
>> vc4-drm gpu: probe with driver vc4-drm failed with error -22
>>
> Marek,
>
> Can you try the patch below?
>
> If this works for you, and passes our tests, and nobody has any
> objections, I can send it as a proper patch submission.
The patch was completely malformed (broken lines, tabs vs. spaces), I had to apply
it manually line by line, but it fixed the issue I've observed:
# dmesg | grep vc4
vc4-drm gpu: bound fe400000.hvs (ops vc4_hvs_ops [vc4])
vc4_hdmi fef00700.hdmi: Could not create device link to hdmi-audio-codec.1.auto
input: vc4-hdmi-0 HDMI Jack as /devices/platform/soc/fef00700.hdmi/sound/card1/input0
vc4-drm gpu: bound fef00700.hdmi (ops vc4_hdmi_ops [vc4])
vc4_hdmi fef05700.hdmi: Could not create device link to hdmi-audio-codec.3.auto
input: vc4-hdmi-1 HDMI Jack as /devices/platform/soc/fef05700.hdmi/sound/card2/input1
vc4-drm gpu: bound fef05700.hdmi (ops vc4_hdmi_ops [vc4])
vc4-drm gpu: bound fe004000.txp (ops vc4_txp_ops [vc4])
vc4-drm gpu: bound fe206000.pixelvalve (ops vc4_crtc_ops [vc4])
vc4-drm gpu: bound fe207000.pixelvalve (ops vc4_crtc_ops [vc4])
vc4-drm gpu: bound fe20a000.pixelvalve (ops vc4_crtc_ops [vc4])
vc4-drm gpu: bound fe216000.pixelvalve (ops vc4_crtc_ops [vc4])
[drm] Initialized vc4 0.0.0 for gpu on minor 1
Feel free to add:
Tested-by: Marek Szyprowski <m.szyprowski@xxxxxxxxxxx>
> ---
>
> From 08de5810ef813c9945d01a2ce23cd9dd71cf3d99 Mon Sep 17 00:00:00 2001
> From: Richard Fitzgerald <rf@xxxxxxxxxxxxxxxxxxxxx>
> Date: Sat, 20 Jun 2026 14:33:18 +0100
> Subject: [PATCH] ASoC: soc-core: Don't fail if device_link could not be
> created
>
> Signed-off-by: Richard Fitzgerald <rf@xxxxxxxxxxxxxxxxxxxxx>
> ---
> include/sound/soc-component.h | 4 ++++
> sound/soc/soc-core.c | 32 ++++++++++++--------------------
> 2 files changed, 16 insertions(+), 20 deletions(-)
>
> diff --git a/include/sound/soc-component.h b/include/sound/soc-component.h
> index 11bc9527653f..aa423865dbe7 100644
> --- a/include/sound/soc-component.h
> +++ b/include/sound/soc-component.h
> @@ -10,6 +10,8 @@
>
> #include <sound/soc.h>
>
> +struct device_link;
> +
> /*
> * Component probe and remove ordering levels for components with runtime
> * dependencies.
> @@ -216,6 +218,8 @@ struct snd_soc_component {
> struct list_head card_aux_list; /* for auxiliary bound components */
> struct list_head card_list;
>
> + struct device_link *card_device_link;
> +
> const struct snd_soc_component_driver *driver;
>
> struct list_head dai_list;
> diff --git a/sound/soc/soc-core.c b/sound/soc/soc-core.c
> index 86b6c752a56b..8c24049d44d3 100644
> --- a/sound/soc/soc-core.c
> +++ b/sound/soc/soc-core.c
> @@ -1980,19 +1980,15 @@ static void soc_cleanup_card_resources(struct snd_soc_card *card)
> }
> }
>
> -static void snd_soc_remove_device_links(struct snd_soc_card *card,
> - struct snd_soc_component *stop_at)
> +static void snd_soc_remove_device_links(struct snd_soc_card *card)
> {
> struct snd_soc_component *component;
>
> for_each_card_components(card, component) {
> - if (card->dev == component->dev)
> - continue;
> -
> - device_link_remove(card->dev, component->dev);
> -
> - if (component == stop_at)
> - return;
> + if (component->card_device_link) {
> + device_link_del(component->card_device_link);
> + component->card_device_link = NULL;
> + }
> }
> }
>
> @@ -2001,7 +1997,7 @@ static void snd_soc_unbind_card(struct snd_soc_card *card)
> if (snd_soc_card_is_instantiated(card)) {
> card->instantiated = false;
>
> - snd_soc_remove_device_links(card, NULL);
> + snd_soc_remove_device_links(card);
>
> soc_cleanup_card_resources(card);
> }
> @@ -2012,7 +2008,6 @@ static int snd_soc_bind_card(struct snd_soc_card *card)
> struct snd_soc_pcm_runtime *rtd;
> struct snd_soc_component *component;
> struct snd_soc_dapm_context *dapm = snd_soc_card_to_dapm(card);
> - struct snd_soc_component *last_devlinked_component = NULL;
> int ret;
>
> snd_soc_card_mutex_lock_root(card);
> @@ -2146,14 +2141,13 @@ static int snd_soc_bind_card(struct snd_soc_card *card)
> if (card->dev == component->dev)
> continue;
>
> - if (!device_link_add(card->dev, component->dev, DL_FLAG_STATELESS)) {
> - dev_warn(card->dev, "Failed to create device link to %s\n",
> + component->card_device_link = device_link_add(card->dev,
> + component->dev,
> + DL_FLAG_STATELESS);
> + if (!component->card_device_link) {
> + dev_warn(card->dev, "Could not create device link to %s\n",
> dev_name(component->dev));
> - ret = -EINVAL;
> - goto probe_end;
> }
> -
> - last_devlinked_component = component;
> }
>
> ret = snd_soc_card_late_probe(card);
> @@ -2185,9 +2179,7 @@ static int snd_soc_bind_card(struct snd_soc_card *card)
>
> probe_end:
> if (ret < 0) {
> - if (last_devlinked_component)
> - snd_soc_remove_device_links(card, last_devlinked_component);
> -
> + snd_soc_remove_device_links(card);
> soc_cleanup_card_resources(card);
> }
>
> --
> 2.47.3
>
Best regards
--
Marek Szyprowski, PhD
Samsung R&D Institute Poland