summaryrefslogtreecommitdiffstats
path: root/sound
diff options
context:
space:
mode:
authorSodhi, VunnyX <vunnyx.sodhi@intel.com>2016-10-28 16:59:41 +0530
committerMark Brown <broonie@kernel.org>2016-10-28 17:29:09 +0100
commit7a5857c3c282c12a8bd0cfd2dd4a17a9252c2b4d (patch)
tree9fdbc6cbca57a287f29fe93a59a321f06964492a /sound
parent6d13f62d931ba638e54ba56f3a7dd3080ffb485a (diff)
downloadop-kernel-dev-7a5857c3c282c12a8bd0cfd2dd4a17a9252c2b4d.zip
op-kernel-dev-7a5857c3c282c12a8bd0cfd2dd4a17a9252c2b4d.tar.gz
ASoC: Intel: Skylake: Fix to turn off hdmi power on probe failure
HDMI codec is required to be powered up before controller initialization for successful enumeration of codec. If the probe fails it needs to be powered off to balance the power state of HDMI codec. This fix balances the reference count in the error path before turning off the codec. Reported-by: Takashi Sakamoto <o-takashi@sakamocchi.jp> Signed-off-by: Sodhi, VunnyX <vunnyx.sodhi@intel.com> Signed-off-by: Subhransu S. Prusty <subhransu.s.prusty@intel.com> Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound')
-rw-r--r--sound/soc/intel/skylake/skl.c5
1 files changed, 4 insertions, 1 deletions
diff --git a/sound/soc/intel/skylake/skl.c b/sound/soc/intel/skylake/skl.c
index 7b7a380..3fc30cb 100644
--- a/sound/soc/intel/skylake/skl.c
+++ b/sound/soc/intel/skylake/skl.c
@@ -674,7 +674,7 @@ static int skl_probe(struct pci_dev *pci,
if (skl->nhlt == NULL) {
err = -ENODEV;
- goto out_free;
+ goto out_display_power_off;
}
skl_nhlt_update_topology_bin(skl);
@@ -746,6 +746,9 @@ out_mach_free:
skl_machine_device_unregister(skl);
out_nhlt_free:
skl_nhlt_free(skl->nhlt);
+out_display_power_off:
+ if (IS_ENABLED(CONFIG_SND_SOC_HDAC_HDMI))
+ snd_hdac_display_power(bus, false);
out_free:
skl->init_failed = 1;
skl_free(ebus);
OpenPOWER on IntegriCloud