drm/i915: Create generic intel_panel for LVDS and eDP
authorJani Nikula <jani.nikula@intel.com>
Fri, 19 Oct 2012 11:51:49 +0000 (14:51 +0300)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Mon, 22 Oct 2012 16:31:20 +0000 (18:31 +0200)
Create a generic struct intel_panel for sharing a data structure and code
between eDP and LVDS panels. Add the new struct to intel_connector so that
later on we can have generic EDID and mode reading functions with EDID
caching that transparently fallback to fixed mode when EDID is not
available.

Add intel_panel as a dummy first, and move data (such as the mentioned
fixed mode) to it in later patches.

Based on earlier work by Chris Wilson <chris@chris-wilson.co.uk>

CC: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Reviewed-by: Jesse Barnes <jbarnes@virtuousgeek.org>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
[danvet: Fixup tiny conflict in intel_dp_destroy.]
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/intel_dp.c
drivers/gpu/drm/i915/intel_drv.h
drivers/gpu/drm/i915/intel_lvds.c
drivers/gpu/drm/i915/intel_panel.c

index be47efff90f079ecdd2730d069c072230ab86667..83a0bc0b7e93d3cc698c484b51f287b19deb4480 100644 (file)
@@ -2497,9 +2497,12 @@ intel_dp_destroy(struct drm_connector *connector)
 {
        struct drm_device *dev = connector->dev;
        struct intel_dp *intel_dp = intel_attached_dp(connector);
+       struct intel_connector *intel_connector = to_intel_connector(connector);
 
-       if (is_edp(intel_dp))
+       if (is_edp(intel_dp)) {
                intel_panel_destroy_backlight(dev);
+               intel_panel_fini(&intel_connector->panel);
+       }
 
        drm_sysfs_connector_remove(connector);
        drm_connector_cleanup(connector);
@@ -2828,8 +2831,10 @@ intel_dp_init(struct drm_device *dev, int output_reg, enum port port)
 
        intel_encoder->hot_plug = intel_dp_hot_plug;
 
-       if (is_edp(intel_dp))
+       if (is_edp(intel_dp)) {
+               intel_panel_init(&intel_connector->panel);
                intel_panel_setup_backlight(connector);
+       }
 
        intel_dp_add_properties(intel_dp, connector);
 
index fa388fdbfe08938f25c94fb9071f0ba42c9309ee..b37bf28d830af9f5dfa6dcc3b358b51abcec0b7b 100644 (file)
@@ -163,6 +163,9 @@ struct intel_encoder {
        int crtc_mask;
 };
 
+struct intel_panel {
+};
+
 struct intel_connector {
        struct drm_connector base;
        /*
@@ -179,6 +182,9 @@ struct intel_connector {
        /* Reads out the current hw, returning true if the connector is enabled
         * and active (i.e. dpms ON state). */
        bool (*get_hw_state)(struct intel_connector *);
+
+       /* Panel info for eDP and LVDS */
+       struct intel_panel panel;
 };
 
 struct intel_crtc {
@@ -436,6 +442,9 @@ extern void intel_flush_display_plane(struct drm_i915_private *dev_priv,
                                      enum plane plane);
 
 /* intel_panel.c */
+extern int intel_panel_init(struct intel_panel *panel);
+extern void intel_panel_fini(struct intel_panel *panel);
+
 extern void intel_fixed_panel_mode(struct drm_display_mode *fixed_mode,
                                   struct drm_display_mode *adjusted_mode);
 extern void intel_pch_panel_fitting(struct drm_device *dev,
index 6435bf3ae56c6e27aa7a1b002b2b39841a6caa08..6b31e2556de0bb0cfcb20ed6d6c04ad19ec1bfde 100644 (file)
@@ -554,6 +554,7 @@ static void intel_lvds_destroy(struct drm_connector *connector)
                acpi_lid_notifier_unregister(&lvds_connector->lid_notifier);
 
        intel_panel_destroy_backlight(connector->dev);
+       intel_panel_fini(&lvds_connector->base.panel);
 
        drm_sysfs_connector_remove(connector);
        drm_connector_cleanup(connector);
@@ -1106,6 +1107,7 @@ out:
        }
        drm_sysfs_connector_add(connector);
 
+       intel_panel_init(&intel_connector->panel);
        intel_panel_setup_backlight(connector);
 
        return true;
index d9752a3bf1f8e64d0d7abf72a4fb73d8d56d48ca..4c64ebc3f743cb0ca504f0fada450e07ad7df1ce 100644 (file)
@@ -464,3 +464,12 @@ void intel_panel_destroy_backlight(struct drm_device *dev)
        return;
 }
 #endif
+
+int intel_panel_init(struct intel_panel *panel)
+{
+       return 0;
+}
+
+void intel_panel_fini(struct intel_panel *panel)
+{
+}