drm/i915/audio: Hook up component bindings even if displays are disabled
authorChris Wilson <chris@chris-wilson.co.uk>
Fri, 17 Aug 2018 10:02:41 +0000 (11:02 +0100)
committerChris Wilson <chris@chris-wilson.co.uk>
Mon, 20 Aug 2018 13:30:43 +0000 (14:30 +0100)
If the display has been disabled by modparam, we still want to connect
together the HW bits and bobs with the associated drivers so that we can
continue to manage their runtime power gating.

Fixes: 108109444ff6 ("drm/i915: Check num_pipes before initializing audio component")
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Imre Deak <imre.deak@intel.com>
Cc: Takashi Iwai <tiwai@suse.de>
Cc: Jani Nikula <jani.nikula@linux.intel.com>
Cc: Elaine Wang <elaine.wang@intel.com>
Reviewed-by: Imre Deak <imre.deak@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20180817100241.4628-1-chris@chris-wilson.co.uk
drivers/gpu/drm/i915/intel_audio.c

index bb94172ffc07402461bbfdecc9906387ef8bbfd1..f02cb211d3e766cfe26d99a7911745599eae536c 100644 (file)
@@ -960,9 +960,6 @@ void i915_audio_component_init(struct drm_i915_private *dev_priv)
 {
        int ret;
 
-       if (INTEL_INFO(dev_priv)->num_pipes == 0)
-               return;
-
        ret = component_add(dev_priv->drm.dev, &i915_audio_component_bind_ops);
        if (ret < 0) {
                DRM_ERROR("failed to add audio component (%d)\n", ret);