Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>
---
 drivers/gpu/drm/i915/i915_drv.h   |    3 ---
 drivers/gpu/drm/i915/intel_lvds.c |   33 +++++++++++++++++----------------
 2 files changed, 17 insertions(+), 19 deletions(-)

diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
index b214709..4c96dfc 100644
--- a/drivers/gpu/drm/i915/i915_drv.h
+++ b/drivers/gpu/drm/i915/i915_drv.h
@@ -364,8 +364,6 @@ typedef struct drm_i915_private {
        } edp;
        bool no_aux_handshake;
 
-       struct notifier_block lid_notifier;
-
        int crt_ddc_pin;
        struct drm_i915_fence_reg fence_regs[16]; /* assume 965 */
        int fence_reg_start; /* 4 if userland hasn't ioctl'd us yet */
@@ -682,7 +680,6 @@ typedef struct drm_i915_private {
        u16 orig_clock;
        int child_dev_num;
        struct child_device_config *child_dev;
-       struct drm_connector *int_lvds_connector;
 
        bool mchbar_need_disable;
 
diff --git a/drivers/gpu/drm/i915/intel_lvds.c 
b/drivers/gpu/drm/i915/intel_lvds.c
index d82c8da..e0ada0c 100644
--- a/drivers/gpu/drm/i915/intel_lvds.c
+++ b/drivers/gpu/drm/i915/intel_lvds.c
@@ -56,6 +56,8 @@ struct intel_lvds_encoder {
 
 struct intel_lvds_connector {
        struct intel_connector base;
+
+       struct notifier_block lid_notifier;
 };
 
 static struct intel_lvds_encoder *to_lvds_encoder(struct drm_encoder *encoder)
@@ -543,18 +545,18 @@ static const struct dmi_system_id 
intel_no_modeset_on_lid[] = {
 static int intel_lid_notify(struct notifier_block *nb, unsigned long val,
                            void *unused)
 {
-       struct drm_i915_private *dev_priv =
-               container_of(nb, struct drm_i915_private, lid_notifier);
-       struct drm_device *dev = dev_priv->dev;
-       struct drm_connector *connector = dev_priv->int_lvds_connector;
+       struct intel_lvds_connector *lvds_connector =
+               container_of(nb, struct intel_lvds_connector, lid_notifier);
+       struct drm_device *dev = lvds_connector->base.base.dev;
+       struct drm_i915_private *dev_priv = dev->dev_private;
 
        /*
         * check and update the status of LVDS connector after receiving
         * the LID nofication event.
         */
-       if (connector)
-               connector->status = connector->funcs->detect(connector,
-                                                            false);
+       lvds_connector->base.base.status =
+               
lvds_connector->base.base.funcs->detect(&lvds_connector->base.base,
+                                                       false);
 
        /* Don't force modeset on machines where it causes a GPU lockup */
        if (dmi_check_system(intel_no_modeset_on_lid))
@@ -585,11 +587,12 @@ static int intel_lid_notify(struct notifier_block *nb, 
unsigned long val,
  */
 static void intel_lvds_connector_destroy(struct drm_connector *connector)
 {
-       struct drm_device *dev = connector->dev;
-       struct drm_i915_private *dev_priv = dev->dev_private;
+       struct intel_lvds_connector *lvds_connector =
+               to_lvds_connector(connector);
+
+       if (lvds_connector->lid_notifier.notifier_call)
+               acpi_lid_notifier_unregister(&lvds_connector->lid_notifier);
 
-       if (dev_priv->lid_notifier.notifier_call)
-               acpi_lid_notifier_unregister(&dev_priv->lid_notifier);
        drm_sysfs_connector_remove(connector);
        drm_connector_cleanup(connector);
        kfree(connector);
@@ -1061,13 +1064,11 @@ out:
                pwm |= PWM_PCH_ENABLE;
                I915_WRITE(BLC_PWM_PCH_CTL1, pwm);
        }
-       dev_priv->lid_notifier.notifier_call = intel_lid_notify;
-       if (acpi_lid_notifier_register(&dev_priv->lid_notifier)) {
+       lvds_connector->lid_notifier.notifier_call = intel_lid_notify;
+       if (acpi_lid_notifier_register(&lvds_connector->lid_notifier)) {
                DRM_DEBUG_KMS("lid notifier registration failed\n");
-               dev_priv->lid_notifier.notifier_call = NULL;
+               lvds_connector->lid_notifier.notifier_call = NULL;
        }
-       /* keep the LVDS connector */
-       dev_priv->int_lvds_connector = connector;
        drm_sysfs_connector_add(connector);
        return true;
 
-- 
1.7.4.1

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

Reply via email to