[alsa-devel] [PATCH 5/5] ALSA/i915: Check power well API existense before calling

Wang Xingchao xingchao.wang at linux.intel.com
Mon May 13 09:37:28 CEST 2013


I915 module maybe loaded after snd_hda_intel, the power-well
API doesnot exist in such case. This patch intended to avoid
loading dependency between snd-hda-intel and i915 module.

Signed-off-by: Wang Xingchao <xingchao.wang at linux.intel.com>
---
 drivers/gpu/drm/i915/i915_dma.c  |    3 ++
 drivers/gpu/drm/i915/intel_drv.h |    2 ++
 drivers/gpu/drm/i915/intel_pm.c  |   12 +++++++
 include/drm/i915_powerwell.h     |    1 +
 sound/pci/hda/hda_i915.c         |   69 ++++++++++++++++++++++++--------------
 sound/pci/hda/hda_i915.h         |    5 +++
 sound/pci/hda/hda_intel.c        |    8 +++--
 7 files changed, 72 insertions(+), 28 deletions(-)

diff --git a/drivers/gpu/drm/i915/i915_dma.c b/drivers/gpu/drm/i915/i915_dma.c
index a1648eb..307ca4b 100644
--- a/drivers/gpu/drm/i915/i915_dma.c
+++ b/drivers/gpu/drm/i915/i915_dma.c
@@ -1671,6 +1671,9 @@ int i915_driver_load(struct drm_device *dev, unsigned long flags)
 	if (IS_GEN5(dev))
 		intel_gpu_ips_init(dev_priv);
 
+	if (IS_HASWELL(dev))
+		intel_gpu_audio_init();
+
 	return 0;
 
 out_gem_unload:
diff --git a/drivers/gpu/drm/i915/intel_drv.h b/drivers/gpu/drm/i915/intel_drv.h
index dfcf546..f159f12 100644
--- a/drivers/gpu/drm/i915/intel_drv.h
+++ b/drivers/gpu/drm/i915/intel_drv.h
@@ -758,6 +758,8 @@ extern void intel_update_fbc(struct drm_device *dev);
 /* IPS */
 extern void intel_gpu_ips_init(struct drm_i915_private *dev_priv);
 extern void intel_gpu_ips_teardown(void);
+/* Display audio */
+extern void intel_gpu_audio_init(void);
 
 extern bool intel_using_power_well(struct drm_device *dev);
 extern void intel_init_power_well(struct drm_device *dev);
diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
index 642c4b6..8c1df21 100644
--- a/drivers/gpu/drm/i915/intel_pm.c
+++ b/drivers/gpu/drm/i915/intel_pm.c
@@ -29,6 +29,7 @@
 #include "i915_drv.h"
 #include "intel_drv.h"
 #include "../../../platform/x86/intel_ips.h"
+#include "../../../../sound/pci/hda/hda_i915.h"
 #include <linux/module.h>
 
 #define FORCEWAKE_ACK_TIMEOUT_MS 2
@@ -4393,6 +4394,17 @@ struct i915_power_well_user {
 	struct list_head list;
 };
 
+void intel_gpu_audio_init(void)
+{
+	void (*link)(void);
+
+	link = symbol_get(audio_link_to_i915_driver);
+	if (link) {
+		link();
+		symbol_put(audio_link_to_i915_driver);
+	}
+}
+
 void i915_request_power_well(const char *name)
 {
 	struct i915_power_well_user *user;
diff --git a/include/drm/i915_powerwell.h b/include/drm/i915_powerwell.h
index 87e0a2e..de03dc8 100644
--- a/include/drm/i915_powerwell.h
+++ b/include/drm/i915_powerwell.h
@@ -33,6 +33,7 @@
 #ifndef _I915_POWERWELL_H_
 #define _I915_POWERWELL_H_
 
+/* For use by hda_i915 driver */
 extern void i915_request_power_well(const char *name);
 extern void i915_release_power_well(const char *name);
 
diff --git a/sound/pci/hda/hda_i915.c b/sound/pci/hda/hda_i915.c
index edf1a08..d11f255 100644
--- a/sound/pci/hda/hda_i915.c
+++ b/sound/pci/hda/hda_i915.c
@@ -22,54 +22,71 @@
 #include <drm/i915_powerwell.h>
 #include "hda_i915.h"
 
-const char *name = "i915";
-static void (*get_power)(const char *name);
-static void (*put_power)(const char *name);
+char *module_name = "i915";
+void (*get_power)(const char *);
+void (*put_power)(const char *);
+static bool hsw_i915_load;
+
+void audio_link_to_i915_driver(void)
+{
+	/* it's time to try getting the symbols again.*/
+	hsw_i915_load = true;
+}
+EXPORT_SYMBOL_GPL(audio_link_to_i915_driver);
 
 /* Power well has impact on Haswell controller and codecs */
 void hda_display_power(bool enable)
 {
-	snd_printk(KERN_INFO "HDA display power %s \n", enable ? "Enable" : "Disable");
-
-	if (!get_power || !put_power)
-		return;
+	if (!get_power || !put_power) {
+		if (hsw_i915_load) {
+			get_power = i915_request_power_well;
+			put_power = i915_release_power_well;
+		} else
+			return;
+	}
 
+	snd_printk(KERN_DEBUG "HDA display power %s \n",
+			enable ? "Enable" : "Disable");
 	if (enable)
 		get_power("hda");
 	else
 		put_power("hda");
 }
-EXPORT_SYMBOL(hda_display_power);
+EXPORT_SYMBOL_GPL(hda_display_power);
 
-static int __init hda_i915_init(void)
+/* In case i915 module loaded first, the APIs are there.
+ * Otherwise wait until i915 module notify us. */
+int hda_i915_init(void)
 {
-	struct module *i915;
-	mutex_lock(&module_mutex);
-	i915 = find_module(name);
-	mutex_unlock(&module_mutex);
+	struct module *i915;
 
-	if (!i915)
-		request_module_nowait(name);
+	mutex_lock(&module_mutex);
+	i915 = find_module(module_name);
+	mutex_unlock(&module_mutex);
 
-	if (!try_module_get(i915))
-		return -EFAULT;
+	if (!i915)
+		request_module_nowait(module_name);
 
 	get_power = symbol_get(i915_request_power_well);
+	if (!get_power)
+		goto out;
+
 	put_power = symbol_get(i915_release_power_well);
+	if (!put_power)
+		goto out;
 
-	module_put(i915);
+	snd_printk(KERN_DEBUG "HDA driver get symbol successfully from i915 module\n");
+out:
 	return 0;
 }
+EXPORT_SYMBOL_GPL(hda_i915_init);
 
-#if 0
-static void __exit hda_i915_exit(void)
+int hda_i915_exit(void)
 {
-	drm_pci_exit(&driver, &i915_pci_driver);
+	symbol_put(i915_request_power_well);
+	symbol_put(i915_release_power_well);
 }
+EXPORT_SYMBOL_GPL(hda_i915_exit);
 
-module_init(hda_i915_init);
-module_exit(hda_i915_exit);
-#endif
-module_init(hda_i915_init);
-MODULE_DESCRIPTION("HDA power well");
+MODULE_DESCRIPTION("HDA power well For Haswell");
 MODULE_LICENSE("GPL");
diff --git a/sound/pci/hda/hda_i915.h b/sound/pci/hda/hda_i915.h
index a7e5324..b0516ab 100644
--- a/sound/pci/hda/hda_i915.h
+++ b/sound/pci/hda/hda_i915.h
@@ -3,8 +3,13 @@
 
 #ifdef CONFIG_SND_HDA_I915
 void hda_display_power(bool enable);
+int hda_i915_init(void);
+int hda_i915_exit(void);
 #else
 void hda_display_power(bool enable) {}
+int hda_i915_init(void) {}
+int hda_i915_exit(void) {}
 #endif
 
+void audio_link_to_i915_driver(void);
 #endif
diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c
index 8bb6075..431027d 100644
--- a/sound/pci/hda/hda_intel.c
+++ b/sound/pci/hda/hda_intel.c
@@ -3684,8 +3684,10 @@ static int azx_probe(struct pci_dev *pci,
 		return -ENOENT;
 	}
 
-	if (IS_HSW(pci))
+	if (IS_HSW(pci)) {
+		hda_i915_init();
 		hda_display_power(true);
+	}
 
 	err = snd_card_create(index[dev], id[dev], THIS_MODULE, 0, &card);
 	if (err < 0) {
@@ -3822,8 +3824,10 @@ static void azx_remove(struct pci_dev *pci)
 	if (card)
 		snd_card_free(card);
 	pci_set_drvdata(pci, NULL);
-	if (IS_HSW(pci))
+	if (IS_HSW(pci)) {
 		hda_display_power(false);
+		hda_i915_exit();
+	}
 }
 
 /* PCI IDs */
-- 
1.7.9.5



More information about the Alsa-devel mailing list