drm/i915/sdvo: Pass the requesed mode to intel_sdvo_create_preferred_input_timing()
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Fri, 11 Mar 2022 17:24:14 +0000 (19:24 +0200)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Mon, 14 Mar 2022 22:12:54 +0000 (00:12 +0200)
We want to stop using connector->panel.fixed_mode directtly.
In order to look it up in the future we'll need to have the
requested mode around, so pass that in fully (instead of just
passing bits of it).

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20220311172428.14685-3-ville.syrjala@linux.intel.com
Reviewed-by: Jani Nikula <jani.nikula@intel.com>
drivers/gpu/drm/i915/display/intel_sdvo.c

index 76e1188..129f661 100644 (file)
@@ -783,24 +783,22 @@ static bool intel_sdvo_get_input_timing(struct intel_sdvo *intel_sdvo,
 static bool
 intel_sdvo_create_preferred_input_timing(struct intel_sdvo *intel_sdvo,
                                         struct intel_sdvo_connector *intel_sdvo_connector,
-                                        u16 clock,
-                                        u16 width,
-                                        u16 height)
+                                        const struct drm_display_mode *mode)
 {
        struct intel_sdvo_preferred_input_timing_args args;
 
        memset(&args, 0, sizeof(args));
-       args.clock = clock;
-       args.width = width;
-       args.height = height;
+       args.clock = mode->clock / 10;
+       args.width = mode->hdisplay;
+       args.height = mode->vdisplay;
        args.interlace = 0;
 
        if (IS_LVDS(intel_sdvo_connector)) {
                const struct drm_display_mode *fixed_mode =
                        intel_sdvo_connector->base.panel.fixed_mode;
 
-               if (fixed_mode->hdisplay != width ||
-                   fixed_mode->vdisplay != height)
+               if (fixed_mode->hdisplay != args.width ||
+                   fixed_mode->vdisplay != args.height)
                        args.scaled = 1;
        }
 
@@ -1236,9 +1234,7 @@ intel_sdvo_get_preferred_input_mode(struct intel_sdvo *intel_sdvo,
 
        if (!intel_sdvo_create_preferred_input_timing(intel_sdvo,
                                                      intel_sdvo_connector,
-                                                     mode->clock / 10,
-                                                     mode->hdisplay,
-                                                     mode->vdisplay))
+                                                     mode))
                return false;
 
        if (!intel_sdvo_get_preferred_input_timing(intel_sdvo,