From: Ville Syrjälä <ville.syrj...@linux.intel.com>

Reorder the drm_encoder_init() vs. encoder->port
assignment so that we don't need the extra 'port'
variable.

Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com>
---
 drivers/gpu/drm/i915/display/intel_dvo.c | 14 ++++++--------
 1 file changed, 6 insertions(+), 8 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_dvo.c 
b/drivers/gpu/drm/i915/display/intel_dvo.c
index 9ce3b4ec6a4d..87ee913cf89b 100644
--- a/drivers/gpu/drm/i915/display/intel_dvo.c
+++ b/drivers/gpu/drm/i915/display/intel_dvo.c
@@ -453,7 +453,6 @@ void intel_dvo_init(struct drm_i915_private *dev_priv)
                bool dvoinit;
                enum pipe pipe;
                u32 dpll[I915_MAX_PIPES];
-               enum port port;
 
                /*
                 * Allow the I2C driver info to specify the GPIO to be used in
@@ -506,21 +505,20 @@ void intel_dvo_init(struct drm_i915_private *dev_priv)
                if (!dvoinit)
                        continue;
 
-               port = intel_dvo_port(dvo->dvo_reg);
-               drm_encoder_init(&dev_priv->drm, &intel_encoder->base,
-                                &intel_dvo_enc_funcs,
-                                intel_dvo_encoder_type(dvo),
-                                "DVO %c", port_name(port));
-
                intel_encoder->type = INTEL_OUTPUT_DVO;
                intel_encoder->power_domain = POWER_DOMAIN_PORT_OTHER;
-               intel_encoder->port = port;
+               intel_encoder->port = intel_dvo_port(dvo->dvo_reg);
                intel_encoder->pipe_mask = ~0;
 
                if (dvo->type != INTEL_DVO_CHIP_LVDS)
                        intel_encoder->cloneable = BIT(INTEL_OUTPUT_ANALOG) |
                                BIT(INTEL_OUTPUT_DVO);
 
+               drm_encoder_init(&dev_priv->drm, &intel_encoder->base,
+                                &intel_dvo_enc_funcs,
+                                intel_dvo_encoder_type(dvo),
+                                "DVO %c", port_name(intel_encoder->port));
+
                if (dvo->type == INTEL_DVO_CHIP_TMDS)
                        intel_connector->polled = DRM_CONNECTOR_POLL_CONNECT |
                                DRM_CONNECTOR_POLL_DISCONNECT;
-- 
2.37.4

Reply via email to