lkml.org 
[lkml]   [2018]   [Dec]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH v2 4/7] drm: sti: Cleanup drm_display_mode print str
Le jeu. 20 déc. 2018 à 13:27, Shayenne Moura
<shayenneluzmoura@gmail.com> a écrit :
>
> This patch adjust the print string of drm_display_mode object
> to remove drm_mode_object dependency in sti files.
>
> Signed-off-by: Shayenne Moura <shayenneluzmoura@gmail.com>

Acked-by: Benjamin Gaignard <benjamin;gaignard@linaro.org>

>
> ---
> Changes in v2:
> - Use DRM_MODE_FMT/ARG macros (Daniel)
> - Make the commit message more clear
>
> drivers/gpu/drm/sti/sti_crtc.c | 16 ++++------------
> 1 file changed, 4 insertions(+), 12 deletions(-)
>
> diff --git a/drivers/gpu/drm/sti/sti_crtc.c b/drivers/gpu/drm/sti/sti_crtc.c
> index ed76e52eb213..ec9f87483e39 100644
> --- a/drivers/gpu/drm/sti/sti_crtc.c
> +++ b/drivers/gpu/drm/sti/sti_crtc.c
> @@ -53,18 +53,10 @@ sti_crtc_mode_set(struct drm_crtc *crtc, struct drm_display_mode *mode)
> struct clk *compo_clk, *pix_clk;
> int rate = mode->clock * 1000;
>
> - DRM_DEBUG_KMS("CRTC:%d (%s) mode:%d (%s)\n",
> - crtc->base.id, sti_mixer_to_str(mixer),
> - mode->base.id, mode->name);
> -
> - DRM_DEBUG_KMS("%d %d %d %d %d %d %d %d %d %d 0x%x 0x%x\n",
> - mode->vrefresh, mode->clock,
> - mode->hdisplay,
> - mode->hsync_start, mode->hsync_end,
> - mode->htotal,
> - mode->vdisplay,
> - mode->vsync_start, mode->vsync_end,
> - mode->vtotal, mode->type, mode->flags);
> + DRM_DEBUG_KMS("CRTC:%d (%s) mode: (%s)\n",
> + crtc->base.id, sti_mixer_to_str(mixer), mode->name);
> +
> + DRM_DEBUG_KMS(DRM_MODE_FMT "\n", DRM_MODE_ARG(mode));
>
> if (mixer->id == STI_MIXER_MAIN) {
> compo_clk = compo->clk_compo_main;
> --
> 2.17.1
>


--
Benjamin Gaignard

Graphic Study Group

Linaro.org │ Open source software for ARM SoCs

Follow Linaro: Facebook | Twitter | Blog

\
 
 \ /
  Last update: 2018-12-20 14:25    [W:0.097 / U:1.308 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site