[i-g-t V2] lib/igt_kms: Format cleanup in DRM mode printing

B, Jeevan jeevan.b at intel.com
Tue Jun 25 16:36:50 UTC 2024


LGTM

Reviewed-by: Jeevan B <jeevan.b at intel.com>

> -----Original Message-----
> From: Modem, Bhanuprakash <bhanuprakash.modem at intel.com>
> Sent: Tuesday, June 25, 2024 9:29 AM
> To: igt-dev at lists.freedesktop.org
> Cc: Modem, Bhanuprakash <bhanuprakash.modem at intel.com>; B, Jeevan
> <jeevan.b at intel.com>
> Subject: [i-g-t V2] lib/igt_kms: Format cleanup in DRM mode printing
> 
> While printing the DRM mode, align the style with the Kernel logs, so that it
> would be easy to compare & debug.
> 
> Example:
>  Old:- 2880x1800 60 2880 2928 2960 3040 1800 1803 1809 1906 0xa 0x48
> 347710
>  New:- 2880x1800: 60 347710 2880 2928 2960 3040 1800 1803 1809 1906
> 0x48 0xa
> 
> V2:
> - Minor cleanup (Jeevan)
> 
> Cc: Jeevan B <jeevan.b at intel.com>
> Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem at intel.com>
> ---
>  lib/igt_kms.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/lib/igt_kms.c b/lib/igt_kms.c index 2e6f44e3c..174bbe6c4 100644
> --- a/lib/igt_kms.c
> +++ b/lib/igt_kms.c
> @@ -1144,13 +1144,13 @@ void kmstest_dump_mode(drmModeModeInfo
> *mode)
>  	const char *stereo = mode_stereo_name(mode);
>  	const char *aspect = mode_picture_aspect_name(mode);
> 
> -	igt_info("  %s %d %d %d %d %d %d %d %d %d 0x%x 0x%x
> %d%s%s%s%s%s%s\n",
> -		 mode->name, mode->vrefresh,
> +	igt_info("  %s: %d %d %d %d %d %d %d %d %d %d 0x%x 0x%x
> %s%s%s%s%s%s\n",
> +		 mode->name, 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->flags, mode->type, mode->clock,
> +		 mode->type, mode->flags,
>  		 stereo ? " (3D:" : "",
>  		 stereo ? stereo : "", stereo ? ")" : "",
>  		 aspect ? " (PAR:" : "",
> --
> 2.43.2



More information about the igt-dev mailing list