Commit e5436206 authored by Ville Syrjälä's avatar Ville Syrjälä
Browse files

drm/i915: Drop redundant aspec ratio prop value initialization 



HDMI_PICTURE_ASPECT_NONE is zero and the connector state is kzalloc()'d
so no need to initialize conn_state->picture_aspect_ratio with it.

Cc: Ilia Mirkin <imirkin@alum.mit.edu>
Signed-off-by: default avatarVille Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20190620142639.17518-6-ville.syrjala@linux.intel.com


Reviewed-by: default avatarUma Shankar <uma.shankar@intel.com>
parent c504f4df

drivers/gpu/drm/i915/display/intel_hdmi.c

+0 −1
Original line number Diff line number Diff line
@@ -2876,7 +2876,6 @@ intel_hdmi_add_properties(struct intel_hdmi *intel_hdmi, struct drm_connector *c 
		intel_attach_colorspace_property(connector);



	drm_connector_attach_content_type_property(connector);

	connector->state->picture_aspect_ratio = HDMI_PICTURE_ASPECT_NONE;



	if (INTEL_GEN(dev_priv) >= 10 || IS_GEMINILAKE(dev_priv))

		drm_object_attach_property(&connector->base,

drivers/gpu/drm/i915/display/intel_sdvo.c

+0 −1
Original line number Diff line number Diff line
@@ -2654,7 +2654,6 @@ intel_sdvo_add_hdmi_properties(struct intel_sdvo *intel_sdvo, 
		intel_attach_broadcast_rgb_property(&connector->base.base);

	}

	intel_attach_aspect_ratio_property(&connector->base.base);

	connector->base.base.state->picture_aspect_ratio = HDMI_PICTURE_ASPECT_NONE;

}



static struct intel_sdvo_connector *intel_sdvo_connector_alloc(void)

CRA Git | Maintained and supported by SUSTech CRA and CCSE