Thank you for the help to improve the patch! - Henry

On 07/04/2018 05:21 PM, Chris Wilson wrote:
Quoting hang.y...@linux.intel.com (2018-07-04 04:07:45)
From: Hang Yuan <hang.y...@linux.intel.com>

This helps initramfs builder and other tools to know the full dependencies
of i915 and have gvt module loaded with i915.

v2: add condition and change to pre-dependency (Chris)
v3: move declaration to gvt.c. (Chris)

Signed-off-by: Hang Yuan <hang.y...@linux.intel.com>
---
  drivers/gpu/drm/i915/gvt/gvt.c | 7 +++++++
  1 file changed, 7 insertions(+)

diff --git a/drivers/gpu/drm/i915/gvt/gvt.c b/drivers/gpu/drm/i915/gvt/gvt.c
index 4e65266..00f487e9 100644
--- a/drivers/gpu/drm/i915/gvt/gvt.c
+++ b/drivers/gpu/drm/i915/gvt/gvt.c
@@ -468,3 +468,10 @@ int intel_gvt_init_device(struct drm_i915_private 
*dev_priv)
         kfree(gvt);
         return ret;
  }
+
+#if IS_ENABLED(CONFIG_DRM_I915_GVT_KVMGT)
+MODULE_SOFTDEP("pre: kvmgt");
+#elif IS_ENABLED(CONFIG_DRM_I915_GVT_XENGT)
+MODULE_SOFTDEP("pre: xengt");
+#endif

This looks like the best one can do under the constraints of the macro.
It matches my limited knowledge of how this works, so
Reviewed-by: Chris Wilson <ch...@chris-wilson.co.uk>
-Chris

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to