[igt-dev] [i-g-t 1/4] lib/igt_fb: Use small case for modifier name

B, Jeevan jeevan.b at intel.com
Fri Oct 6 08:18:44 UTC 2023


> -----Original Message-----
> From: igt-dev <igt-dev-bounces at lists.freedesktop.org> On Behalf Of
> Bhanuprakash Modem
> Sent: Friday, October 6, 2023 1:28 PM
> To: igt-dev at lists.freedesktop.org
> Subject: [igt-dev] [i-g-t 1/4] lib/igt_fb: Use small case for modifier name
> 
> As few tests are using modifier name as subtest, let's keep it as a small case. Also,
> don't mix '_' & '-'.
> 
> Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem at intel.com>
LGTM
Reviewed-by: Jeevan B <jeevan.b at intel.com> 

> ---
>  lib/igt_fb.c | 22 +++++++++++-----------
>  1 file changed, 11 insertions(+), 11 deletions(-)
> 
> diff --git a/lib/igt_fb.c b/lib/igt_fb.c index 54a66eb6a..50b49102b 100644
> --- a/lib/igt_fb.c
> +++ b/lib/igt_fb.c
> @@ -4919,32 +4919,32 @@ const char *igt_fb_modifier_name(uint64_t
> modifier)
>  	case DRM_FORMAT_MOD_LINEAR:
>  		return "linear";
>  	case I915_FORMAT_MOD_X_TILED:
> -		return "X";
> +		return "x";
>  	case I915_FORMAT_MOD_Y_TILED:
> -		return "Y";
> +		return "y";
>  	case I915_FORMAT_MOD_Yf_TILED:
> -		return "Yf";
> +		return "yf";
>  	case I915_FORMAT_MOD_Y_TILED_CCS:
> -		return "Y-CCS";
> +		return "y-ccs";
>  	case I915_FORMAT_MOD_Yf_TILED_CCS:
> -		return "Yf-CCS";
> +		return "yf-ccs";
>  	case I915_FORMAT_MOD_Y_TILED_GEN12_RC_CCS:
> -		return "Y-RC_CCS";
> +		return "y-rc-ccs";
>  	case I915_FORMAT_MOD_Y_TILED_GEN12_RC_CCS_CC:
> -		return "Y-RC_CCS-CC";
> +		return "y-rc-ccs-cc";
>  	case I915_FORMAT_MOD_Y_TILED_GEN12_MC_CCS:
> -		return "Y-MC_CCS";
> +		return "y-mc-ccs";
>  	case I915_FORMAT_MOD_4_TILED:
>  		return "4";
>  	case I915_FORMAT_MOD_4_TILED_MTL_RC_CCS:
>  	case I915_FORMAT_MOD_4_TILED_DG2_RC_CCS:
> -		return "4-RC_CCS";
> +		return "4-rc-ccs";
>  	case I915_FORMAT_MOD_4_TILED_MTL_MC_CCS:
>  	case I915_FORMAT_MOD_4_TILED_DG2_MC_CCS:
> -		return "4-MC_CCS";
> +		return "4-mc-ccs";
>  	case I915_FORMAT_MOD_4_TILED_MTL_RC_CCS_CC:
>  	case I915_FORMAT_MOD_4_TILED_DG2_RC_CCS_CC:
> -		return "4-RC_CCS-CC";
> +		return "4-rc-ccs-cc";
>  	default:
>  		return "?";
>  	}
> --
> 2.40.0



More information about the igt-dev mailing list