[Intel-gfx] [PATCH 2/5] drm/i915/dp: Cache EDID for a detection cycle

Jani Nikula jani.nikula at linux.intel.com
Wed Sep 3 10:23:09 CEST 2014


On Tue, 02 Sep 2014, Chris Wilson <chris at chris-wilson.co.uk> wrote:
> @@ -3819,27 +3819,55 @@ intel_dp_get_edid(struct drm_connector *connector, struct i2c_adapter *adapter)
>  			return NULL;
>  
>  		return drm_edid_duplicate(intel_connector->edid);
> -	}
> +	} else
> +		return drm_get_edid(&intel_connector->base,
> +				    &intel_dp->aux.ddc);

Nitpick, I'd like to see braces on all branches if one branch requires
them. Also CodingStyle.

BR,
Jani.

-- 
Jani Nikula, Intel Open Source Technology Center



More information about the Intel-gfx mailing list