[PATCH 2/2] drm/nouveau: Add HD-audio component notifier support

Ilia Mirkin imirkin at alum.mit.edu
Mon Jul 22 15:00:15 UTC 2019


On Mon, Jul 22, 2019 at 10:38 AM Takashi Iwai <tiwai at suse.de> wrote:
> +static void
> +nv50_audio_component_init(struct nouveau_drm *drm)
> +{
> +       if (!component_add(drm->dev->dev, &nv50_audio_component_bind_ops))
> +               drm->audio.component_registered = true;
> +}

Pardon my ignorance on this topic ... but are there any ill effects
from always doing this? For example, the board in question might not
have ELD support at all (the original G80, covered by dispnv50), or it
might not have any HDMI/DP ports, or it may have them but just nothing
is plugged in.

Could you confirm that this is all OK?

Thanks,

  -ilia


More information about the dri-devel mailing list