[Intel-gfx] [PATCH 1/2] drm/i915: use DRM_ERROR() instead of drm_err()

[Ville Syrjälä]

On Wed, Oct 02, 2019 at 05:54:04PM +0300, Jani Nikula wrote:
> Unify on current common usage to allow repurposing drm_err() later. Fix
> newlines while at it.
> 
> Signed-off-by: Jani Nikula <jani.nikula at intel.com>

Series is
Reviewed-by: Ville Syrjälä <ville.syrjala at linux.intel.com>

> ---
>  drivers/gpu/drm/i915/display/intel_display.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c
> index d99c59e97568..781f6ea7ccf9 100644
> --- a/drivers/gpu/drm/i915/display/intel_display.c
> +++ b/drivers/gpu/drm/i915/display/intel_display.c
> @@ -12562,10 +12562,10 @@ pipe_config_infoframe_mismatch(struct drm_i915_private *dev_priv,
>  		drm_dbg(DRM_UT_KMS, "found");
>  		hdmi_infoframe_log(KERN_DEBUG, dev_priv->drm.dev, b);
>  	} else {
> -		drm_err("mismatch in %s infoframe", name);
> -		drm_err("expected:");
> +		DRM_ERROR("mismatch in %s infoframe\n", name);
> +		DRM_ERROR("expected:\n");
>  		hdmi_infoframe_log(KERN_ERR, dev_priv->drm.dev, a);
> -		drm_err("found");
> +		DRM_ERROR("found:\n");
>  		hdmi_infoframe_log(KERN_ERR, dev_priv->drm.dev, b);
>  	}
>  }
> @@ -12583,7 +12583,7 @@ pipe_config_mismatch(bool fastset, const char *name, const char *format, ...)
>  	if (fastset)
>  		drm_dbg(DRM_UT_KMS, "fastset mismatch in %s %pV", name, &vaf);
>  	else
> -		drm_err("mismatch in %s %pV", name, &vaf);
> +		DRM_ERROR("mismatch in %s %pV\n", name, &vaf);
>  
>  	va_end(args);
>  }
> -- 
> 2.20.1
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Ville Syrjälä
Intel