[PATCH 4/4] Revert "drm/i915: Move out non-modeset calls from modeset init and cleanup"
José Roberto de Souza
jose.souza at intel.com
Fri Jul 20 00:17:25 UTC 2018
This reverts commit 49b3ec21b5892f342da8a521bbc8445fc7482161.
---
drivers/gpu/drm/i915/i915_drv.c | 56 ++++++++++------------------
drivers/gpu/drm/i915/i915_drv.h | 1 -
drivers/gpu/drm/i915/intel_display.c | 16 +++++---
3 files changed, 31 insertions(+), 42 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c
index f7cb5d1d5482..ba44023ac41c 100644
--- a/drivers/gpu/drm/i915/i915_drv.c
+++ b/drivers/gpu/drm/i915/i915_drv.c
@@ -665,15 +665,25 @@ static int i915_load_modeset_init(struct drm_device *dev)
/* must happen before intel_power_domains_init_hw() on VLV/CHV */
intel_update_rawclk(dev_priv);
+ intel_power_domains_init_hw(dev_priv, false);
+
intel_csr_ucode_init(dev_priv);
+ ret = intel_irq_install(dev_priv);
+ if (ret)
+ goto cleanup_csr;
+
intel_setup_gmbus(dev_priv);
/* Important: The output setup functions called by modeset_init need
* working irqs for e.g. gmbus and dp aux transfers. */
ret = intel_modeset_init(dev);
if (ret)
- goto cleanup_gmbus;
+ goto cleanup_irq;
+
+ ret = i915_gem_init(dev_priv);
+ if (ret)
+ goto cleanup_modeset;
intel_setup_overlay(dev_priv);
@@ -682,17 +692,23 @@ static int i915_load_modeset_init(struct drm_device *dev)
ret = intel_fbdev_init(dev);
if (ret)
- goto cleanup_modeset;
+ goto cleanup_gem;
/* Only enable hotplug handling once the fbdev is fully set up. */
intel_hpd_init(dev_priv);
return 0;
+cleanup_gem:
+ if (i915_gem_suspend(dev_priv))
+ DRM_ERROR("failed to idle hardware; continuing to unload!\n");
+ i915_gem_fini(dev_priv);
cleanup_modeset:
intel_modeset_cleanup(dev);
-cleanup_gmbus:
+cleanup_irq:
+ drm_irq_uninstall(dev);
intel_teardown_gmbus(dev_priv);
+cleanup_csr:
intel_csr_ucode_fini(dev_priv);
intel_power_domains_fini(dev_priv);
vga_switcheroo_unregister_client(pdev);
@@ -1378,22 +1394,9 @@ int i915_driver_load(struct pci_dev *pdev, const struct pci_device_id *ent)
goto out_cleanup_hw;
}
- ret = intel_irq_install(dev_priv);
- if (ret)
- goto out_cleanup_hw;
-
- /* i915_gem_init() call chain will call
- * intel_display_power_put(i915, POWER_DOMAIN_GT_IRQ);
- */
- intel_power_domains_init_hw(dev_priv, false);
-
- ret = i915_gem_init(dev_priv);
- if (ret)
- goto cleanup_irq;
-
ret = i915_load_modeset_init(&dev_priv->drm);
if (ret < 0)
- goto cleanup_gem;
+ goto out_cleanup_hw;
i915_driver_register(dev_priv);
@@ -1407,12 +1410,6 @@ int i915_driver_load(struct pci_dev *pdev, const struct pci_device_id *ent)
return 0;
-cleanup_gem:
- if (i915_gem_suspend(dev_priv))
- DRM_ERROR("failed to idle hardware; continuing to unload!\n");
- i915_gem_fini(dev_priv);
-cleanup_irq:
- drm_irq_uninstall(&dev_priv->drm);
out_cleanup_hw:
i915_driver_cleanup_hw(dev_priv);
out_cleanup_mmio:
@@ -1447,21 +1444,8 @@ void i915_driver_unload(struct drm_device *dev)
intel_gvt_cleanup(dev_priv);
- intel_modeset_cleanup_prepare(dev);
-
- intel_disable_gt_powersave(dev_priv);
-
- /*
- * Interrupts and polling as the first thing to avoid creating havoc.
- * Too much stuff here (turning of connectors, ...) would
- * experience fancy races otherwise.
- */
- intel_irq_uninstall(dev_priv);
-
intel_modeset_cleanup(dev);
- intel_cleanup_gt_powersave(dev_priv);
-
intel_bios_cleanup(dev_priv);
vga_switcheroo_unregister_client(pdev);
diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
index d50fd9825b10..0f49f9988dfa 100644
--- a/drivers/gpu/drm/i915/i915_drv.h
+++ b/drivers/gpu/drm/i915/i915_drv.h
@@ -3411,7 +3411,6 @@ mkwrite_device_info(struct drm_i915_private *dev_priv)
/* modesetting */
extern void intel_modeset_init_hw(struct drm_device *dev);
extern int intel_modeset_init(struct drm_device *dev);
-void intel_modeset_cleanup_prepare(struct drm_device *dev);
extern void intel_modeset_cleanup(struct drm_device *dev);
extern int intel_connector_register(struct drm_connector *);
extern void intel_connector_unregister(struct drm_connector *);
diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
index f490450a3ba5..87e4cfbfd096 100644
--- a/drivers/gpu/drm/i915/intel_display.c
+++ b/drivers/gpu/drm/i915/intel_display.c
@@ -15905,7 +15905,7 @@ static void intel_hpd_poll_fini(struct drm_device *dev)
drm_connector_list_iter_end(&conn_iter);
}
-void intel_modeset_cleanup_prepare(struct drm_device *dev)
+void intel_modeset_cleanup(struct drm_device *dev)
{
struct drm_i915_private *dev_priv = to_i915(dev);
@@ -15913,11 +15913,15 @@ void intel_modeset_cleanup_prepare(struct drm_device *dev)
flush_work(&dev_priv->atomic_helper.free_work);
WARN_ON(!llist_empty(&dev_priv->atomic_helper.free_list));
-}
-void intel_modeset_cleanup(struct drm_device *dev)
-{
- struct drm_i915_private *dev_priv = to_i915(dev);
+ intel_disable_gt_powersave(dev_priv);
+
+ /*
+ * Interrupts and polling as the first thing to avoid creating havoc.
+ * Too much stuff here (turning of connectors, ...) would
+ * experience fancy races otherwise.
+ */
+ intel_irq_uninstall(dev_priv);
/*
* Due to the hpd irq storm handling the hotplug work can re-arm the
@@ -15939,6 +15943,8 @@ void intel_modeset_cleanup(struct drm_device *dev)
intel_cleanup_overlay(dev_priv);
+ intel_cleanup_gt_powersave(dev_priv);
+
intel_teardown_gmbus(dev_priv);
destroy_workqueue(dev_priv->modeset_wq);
--
2.18.0
More information about the Intel-gfx-trybot
mailing list