| From 11d6d834ae9e3521231d520b8dde663b8cc39646 Mon Sep 17 00:00:00 2001 |
| From: Daniel Vetter <daniel.vetter@ffwll.ch> |
| Date: Sun, 21 Jul 2013 21:37:06 +0200 |
| Subject: drm/i915/lvds: use the native encoder ->mode_set callback |
| |
| Does nothing, so trivial conversion. But update the outdated comment |
| while at it. |
| |
| Reviewed-by: Rodrigo Vivi <rodrigo.vivi@gmail.com> |
| Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch> |
| (cherry picked from commit 66df24d926fe0686034bb8d47a0f586310602178) |
| Signed-off-by: Darren Hart <dvhart@linux.intel.com> |
| --- |
| drivers/gpu/drm/i915/intel_lvds.c | 16 +++++----------- |
| 1 file changed, 5 insertions(+), 11 deletions(-) |
| |
| --- a/drivers/gpu/drm/i915/intel_lvds.c |
| +++ b/drivers/gpu/drm/i915/intel_lvds.c |
| @@ -319,14 +319,12 @@ static bool intel_lvds_compute_config(st |
| return true; |
| } |
| |
| -static void intel_lvds_mode_set(struct drm_encoder *encoder, |
| - struct drm_display_mode *mode, |
| - struct drm_display_mode *adjusted_mode) |
| +static void intel_lvds_mode_set(struct intel_encoder *encoder) |
| { |
| /* |
| - * The LVDS pin pair will already have been turned on in the |
| - * intel_crtc_mode_set since it has a large impact on the DPLL |
| - * settings. |
| + * We don't do anything here, the LVDS port is fully set up in the pre |
| + * enable hook - the ordering constraints for enabling the lvds port vs. |
| + * enabling the display pll are too strict. |
| */ |
| } |
| |
| @@ -507,10 +505,6 @@ static int intel_lvds_set_property(struc |
| return 0; |
| } |
| |
| -static const struct drm_encoder_helper_funcs intel_lvds_helper_funcs = { |
| - .mode_set = intel_lvds_mode_set, |
| -}; |
| - |
| static const struct drm_connector_helper_funcs intel_lvds_connector_helper_funcs = { |
| .get_modes = intel_lvds_get_modes, |
| .mode_valid = intel_lvds_mode_valid, |
| @@ -1014,6 +1008,7 @@ void intel_lvds_init(struct drm_device * |
| intel_encoder->enable = intel_enable_lvds; |
| intel_encoder->pre_enable = intel_pre_enable_lvds; |
| intel_encoder->compute_config = intel_lvds_compute_config; |
| + intel_encoder->mode_set = intel_lvds_mode_set; |
| intel_encoder->disable = intel_disable_lvds; |
| intel_encoder->get_hw_state = intel_lvds_get_hw_state; |
| intel_encoder->get_config = intel_lvds_get_config; |
| @@ -1030,7 +1025,6 @@ void intel_lvds_init(struct drm_device * |
| else |
| intel_encoder->crtc_mask = (1 << 1); |
| |
| - drm_encoder_helper_add(encoder, &intel_lvds_helper_funcs); |
| drm_connector_helper_add(connector, &intel_lvds_connector_helper_funcs); |
| connector->display_info.subpixel_order = SubPixelHorizontalRGB; |
| connector->interlace_allowed = false; |