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 <[email protected]>

Series is
Reviewed-by: Ville Syrjälä <[email protected]>

> ---
>  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
> [email protected]
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Ville Syrjälä
Intel
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to