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

drm/i915: Nuke pointless variable 



No point in assigning the function return value to a local
variable if we're just going to use it the one time.

Signed-off-by: default avatarVille Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20200630215601.28557-13-ville.syrjala@linux.intel.com


Reviewed-by: default avatarJosé Roberto de Souza <jose.souza@intel.com>
parent 6d3144eb

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

+1 −2
Original line number Diff line number Diff line
@@ -482,7 +482,6 @@ void intel_hpd_irq_handler(struct drm_i915_private *dev_priv, 
	 * only the one of them (DP) will have ->hpd_pulse().

	 */

	for_each_intel_encoder(&dev_priv->drm, encoder) {

		bool has_hpd_pulse = intel_encoder_has_hpd_pulse(encoder);

		enum port port = encoder->port;

		bool long_hpd;
@@ -490,7 +489,7 @@ void intel_hpd_irq_handler(struct drm_i915_private *dev_priv, 
		if (!(BIT(pin) & pin_mask))

			continue;



		if (!has_hpd_pulse)

		if (!intel_encoder_has_hpd_pulse(encoder))

			continue;



		long_hpd = long_mask & BIT(pin);