Commit 843152b4 authored by Chris Wilson's avatar Chris Wilson

drm/i915: Move connector registration to driver registration

Defer connector registration from during construction to the driver
registration phase. This is important for ordering the action correctly,
e.g. not using debugfs before it is ready.
Signed-off-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
Link: http://patchwork.freedesktop.org/patch/msgid/1466773227-7994-4-git-send-email-chris@chris-wilson.co.uk
parent 1ebaa0b9
...@@ -1398,6 +1398,7 @@ static void i915_driver_register(struct drm_i915_private *dev_priv) ...@@ -1398,6 +1398,7 @@ static void i915_driver_register(struct drm_i915_private *dev_priv)
I915_WRITE(vgtif_reg(display_ready), VGT_DRV_DISPLAY_READY); I915_WRITE(vgtif_reg(display_ready), VGT_DRV_DISPLAY_READY);
i915_setup_sysfs(dev); i915_setup_sysfs(dev);
intel_modeset_register(dev_priv);
if (INTEL_INFO(dev_priv)->num_pipes) { if (INTEL_INFO(dev_priv)->num_pipes) {
/* Must be done after probing outputs */ /* Must be done after probing outputs */
...@@ -1430,6 +1431,7 @@ static void i915_driver_unregister(struct drm_i915_private *dev_priv) ...@@ -1430,6 +1431,7 @@ static void i915_driver_unregister(struct drm_i915_private *dev_priv)
intel_gpu_ips_teardown(); intel_gpu_ips_teardown();
acpi_video_unregister(); acpi_video_unregister();
intel_opregion_unregister(dev_priv); intel_opregion_unregister(dev_priv);
intel_modeset_unregister(dev_priv);
i915_teardown_sysfs(dev_priv->dev); i915_teardown_sysfs(dev_priv->dev);
i915_gem_shrinker_cleanup(dev_priv); i915_gem_shrinker_cleanup(dev_priv);
} }
......
...@@ -3734,6 +3734,8 @@ extern void intel_modeset_init_hw(struct drm_device *dev); ...@@ -3734,6 +3734,8 @@ extern void intel_modeset_init_hw(struct drm_device *dev);
extern void intel_modeset_init(struct drm_device *dev); extern void intel_modeset_init(struct drm_device *dev);
extern void intel_modeset_gem_init(struct drm_device *dev); extern void intel_modeset_gem_init(struct drm_device *dev);
extern void intel_modeset_cleanup(struct drm_device *dev); extern void intel_modeset_cleanup(struct drm_device *dev);
extern void intel_modeset_register(struct drm_i915_private *dev_priv);
extern void intel_modeset_unregister(struct drm_i915_private *dev_priv);
extern int intel_connector_register(struct drm_connector *); extern int intel_connector_register(struct drm_connector *);
extern void intel_connector_unregister(struct drm_connector *); extern void intel_connector_unregister(struct drm_connector *);
extern int intel_modeset_vga_set_state(struct drm_device *dev, bool state); extern int intel_modeset_vga_set_state(struct drm_device *dev, bool state);
......
...@@ -880,8 +880,6 @@ void intel_crt_init(struct drm_device *dev) ...@@ -880,8 +880,6 @@ void intel_crt_init(struct drm_device *dev)
drm_connector_helper_add(connector, &intel_crt_connector_helper_funcs); drm_connector_helper_add(connector, &intel_crt_connector_helper_funcs);
drm_connector_register(connector);
if (!I915_HAS_HOTPLUG(dev)) if (!I915_HAS_HOTPLUG(dev))
intel_connector->polled = DRM_CONNECTOR_POLL_CONNECT; intel_connector->polled = DRM_CONNECTOR_POLL_CONNECT;
......
...@@ -15551,6 +15551,16 @@ void intel_modeset_init_hw(struct drm_device *dev) ...@@ -15551,6 +15551,16 @@ void intel_modeset_init_hw(struct drm_device *dev)
intel_enable_gt_powersave(dev_priv); intel_enable_gt_powersave(dev_priv);
} }
void intel_modeset_register(struct drm_i915_private *dev_priv)
{
drm_connector_register_all(dev_priv->dev);
}
void intel_modeset_unregister(struct drm_i915_private *dev_priv)
{
drm_connector_unregister_all(dev_priv->dev);
}
/* /*
* Calculate what we think the watermarks should be for the state we've read * Calculate what we think the watermarks should be for the state we've read
* out of the hardware and then immediately program those watermarks so that * out of the hardware and then immediately program those watermarks so that
...@@ -16365,8 +16375,6 @@ void intel_modeset_cleanup(struct drm_device *dev) ...@@ -16365,8 +16375,6 @@ void intel_modeset_cleanup(struct drm_device *dev)
/* flush any delayed tasks or pending work */ /* flush any delayed tasks or pending work */
flush_scheduled_work(); flush_scheduled_work();
drm_connector_unregister_all(dev);
drm_mode_config_cleanup(dev); drm_mode_config_cleanup(dev);
intel_cleanup_overlay(dev_priv); intel_cleanup_overlay(dev_priv);
......
...@@ -5583,7 +5583,6 @@ intel_dp_init_connector(struct intel_digital_port *intel_dig_port, ...@@ -5583,7 +5583,6 @@ intel_dp_init_connector(struct intel_digital_port *intel_dig_port,
edp_panel_vdd_work); edp_panel_vdd_work);
intel_connector_attach_encoder(intel_connector, intel_encoder); intel_connector_attach_encoder(intel_connector, intel_encoder);
drm_connector_register(connector);
if (HAS_DDI(dev)) if (HAS_DDI(dev))
intel_connector->get_hw_state = intel_ddi_connector_get_hw_state; intel_connector->get_hw_state = intel_ddi_connector_get_hw_state;
...@@ -5639,7 +5638,6 @@ intel_dp_init_connector(struct intel_digital_port *intel_dig_port, ...@@ -5639,7 +5638,6 @@ intel_dp_init_connector(struct intel_digital_port *intel_dig_port,
return true; return true;
fail: fail:
drm_connector_unregister(connector);
drm_connector_cleanup(connector); drm_connector_cleanup(connector);
return false; return false;
......
...@@ -1592,8 +1592,6 @@ void intel_dsi_init(struct drm_device *dev) ...@@ -1592,8 +1592,6 @@ void intel_dsi_init(struct drm_device *dev)
intel_dsi_add_properties(intel_connector); intel_dsi_add_properties(intel_connector);
drm_connector_register(connector);
return; return;
err: err:
......
...@@ -551,7 +551,6 @@ void intel_dvo_init(struct drm_device *dev) ...@@ -551,7 +551,6 @@ void intel_dvo_init(struct drm_device *dev)
intel_dvo->panel_wants_dither = true; intel_dvo->panel_wants_dither = true;
} }
drm_connector_register(connector);
return; return;
} }
......
...@@ -1915,7 +1915,6 @@ void intel_hdmi_init_connector(struct intel_digital_port *intel_dig_port, ...@@ -1915,7 +1915,6 @@ void intel_hdmi_init_connector(struct intel_digital_port *intel_dig_port,
intel_hdmi_add_properties(intel_hdmi, connector); intel_hdmi_add_properties(intel_hdmi, connector);
intel_connector_attach_encoder(intel_connector, intel_encoder); intel_connector_attach_encoder(intel_connector, intel_encoder);
drm_connector_register(connector);
intel_hdmi->attached_connector = intel_connector; intel_hdmi->attached_connector = intel_connector;
/* For G4X desktop chip, PEG_BAND_GAP_DATA 3:0 must first be written /* For G4X desktop chip, PEG_BAND_GAP_DATA 3:0 must first be written
......
...@@ -1134,7 +1134,6 @@ void intel_lvds_init(struct drm_device *dev) ...@@ -1134,7 +1134,6 @@ void intel_lvds_init(struct drm_device *dev)
DRM_DEBUG_KMS("lid notifier registration failed\n"); DRM_DEBUG_KMS("lid notifier registration failed\n");
lvds_connector->lid_notifier.notifier_call = NULL; lvds_connector->lid_notifier.notifier_call = NULL;
} }
drm_connector_register(connector);
return; return;
......
...@@ -2396,16 +2396,8 @@ intel_sdvo_connector_init(struct intel_sdvo_connector *connector, ...@@ -2396,16 +2396,8 @@ intel_sdvo_connector_init(struct intel_sdvo_connector *connector,
connector->base.get_hw_state = intel_sdvo_connector_get_hw_state; connector->base.get_hw_state = intel_sdvo_connector_get_hw_state;
intel_connector_attach_encoder(&connector->base, &encoder->base); intel_connector_attach_encoder(&connector->base, &encoder->base);
ret = drm_connector_register(drm_connector);
if (ret < 0)
goto err1;
return 0; return 0;
err1:
drm_connector_cleanup(drm_connector);
return ret;
} }
static void static void
...@@ -2532,7 +2524,6 @@ intel_sdvo_tv_init(struct intel_sdvo *intel_sdvo, int type) ...@@ -2532,7 +2524,6 @@ intel_sdvo_tv_init(struct intel_sdvo *intel_sdvo, int type)
return true; return true;
err: err:
drm_connector_unregister(connector);
intel_sdvo_destroy(connector); intel_sdvo_destroy(connector);
return false; return false;
} }
...@@ -2611,7 +2602,6 @@ intel_sdvo_lvds_init(struct intel_sdvo *intel_sdvo, int device) ...@@ -2611,7 +2602,6 @@ intel_sdvo_lvds_init(struct intel_sdvo *intel_sdvo, int device)
return true; return true;
err: err:
drm_connector_unregister(connector);
intel_sdvo_destroy(connector); intel_sdvo_destroy(connector);
return false; return false;
} }
......
...@@ -1642,5 +1642,4 @@ intel_tv_init(struct drm_device *dev) ...@@ -1642,5 +1642,4 @@ intel_tv_init(struct drm_device *dev)
drm_object_attach_property(&connector->base, drm_object_attach_property(&connector->base,
dev->mode_config.tv_bottom_margin_property, dev->mode_config.tv_bottom_margin_property,
intel_tv->margin[TV_MARGIN_BOTTOM]); intel_tv->margin[TV_MARGIN_BOTTOM]);
drm_connector_register(connector);
} }
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment