Re: [PATCH] ALSA: hda/hdmi: run eld notify in delay work

Takashi Iwai
2022-09-27T14:11:43Z

On Tue, 27 Sep 2022 15:58:07 +0200,
Brent Lu wrote:
>
> During resolution change, display driver would disable HDMI audio then
> enable it in a short time. There is possibility that eld notify for
> HDMI audio enable is called when previous runtime suspend is still
> running. In this case, the elf nofity just returns and not updating the
> status of corresponding HDMI pin/port. Here we move the eld nofity to
> a delay work so we don't lose it.
>
> Signed-off-by: Brent Lu <brent.lu@intel.com>

We have already a dedicated per-pin work for the delayed ELD check.
Can we reuse it instead of inventing yet another work?
More work needs more cares, and better to avoid unless really needed
(e.g. you forgot cleanup at suspend/removal in this patch).

thanks,

Takashi