On Wed, 2016-09-28 at 16:51 +0300, ville.syrjala at linux.intel.com wrote:
> From: Ville Syrjälä <ville.syrjala at linux.intel.com>
> 
> Reduce the eyesore with a local variable.
> 
> Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>

Reviewed-by: Ander Conselvan de Oliveira <conselvan2 at gmail.com>

> ---
>  drivers/gpu/drm/i915/intel_display.c | 14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/intel_display.c
> b/drivers/gpu/drm/i915/intel_display.c
> index 8e464e089794..34ca03e621ba 100644
> --- a/drivers/gpu/drm/i915/intel_display.c
> +++ b/drivers/gpu/drm/i915/intel_display.c
> @@ -12657,22 +12657,22 @@ static void
>  connected_sink_compute_bpp(struct intel_connector *connector,
>                         struct intel_crtc_state *pipe_config)
>  {
> +     const struct drm_display_info *info = &connector->base.display_info;
>      int bpp = pipe_config->pipe_bpp;
>  
>      DRM_DEBUG_KMS("[CONNECTOR:%d:%s] checking for sink bpp constrains\n",
> -             connector->base.base.id,
> -             connector->base.name);
> +                   connector->base.base.id,
> +                   connector->base.name);
>  
>      /* Don't use an invalid EDID bpc value */
> -     if (connector->base.display_info.bpc &&
> -         connector->base.display_info.bpc * 3 < bpp) {
> +     if (info->bpc != 0 && info->bpc * 3 < bpp) {
>              DRM_DEBUG_KMS("clamping display bpp (was %d) to EDID reported
> max of %d\n",
> -                           bpp, connector->base.display_info.bpc*3);
> -             pipe_config->pipe_bpp = connector->base.display_info.bpc*3;
> +                           bpp, info->bpc * 3);
> +             pipe_config->pipe_bpp = info->bpc * 3;
>      }
>  
>      /* Clamp bpp to 8 on screens without EDID 1.4 */
> -     if (connector->base.display_info.bpc == 0 && bpp > 24) {
> +     if (info->bpc == 0 && bpp > 24) {
>              DRM_DEBUG_KMS("clamping display bpp (was %d) to default limit
> of 24\n",
>                            bpp);
>              pipe_config->pipe_bpp = 24;

Reply via email to