[PATCH] drm/amd/display: Use seq_puts() in 'amdgpu_current_colorspace_show()' & 'edp_ilr_show()'

[Rodrigo Siqueira Jordao]

On 6/22/23 02:58, Srinivasan Shanmugam wrote:
> Replace seq_printf with seq_puts when there is no argument list.
> 
> Fix the checkpatch warning:
> WARNING: Prefer seq_puts to seq_printf
> 
> Cc: Wayne Lin <Wayne.Lin at amd.com>
> Cc: Harry Wentland <harry.wentland at amd.com>
> Cc: Rodrigo Siqueira <Rodrigo.Siqueira at amd.com>
> Cc: Aurabindo Pillai <aurabindo.pillai at amd.com>
> Signed-off-by: Srinivasan Shanmugam <srinivasan.shanmugam at amd.com>
> ---
>   .../drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c  | 14 +++++++-------
>   1 file changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c
> index d63ee636483b..7c21e21bcc51 100644
> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c
> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c
> @@ -1075,24 +1075,24 @@ static int amdgpu_current_colorspace_show(struct seq_file *m, void *data)
>   
>   	switch (dm_crtc_state->stream->output_color_space) {
>   	case COLOR_SPACE_SRGB:
> -		seq_printf(m, "sRGB");
> +		seq_puts(m, "sRGB");
>   		break;
>   	case COLOR_SPACE_YCBCR601:
>   	case COLOR_SPACE_YCBCR601_LIMITED:
> -		seq_printf(m, "BT601_YCC");
> +		seq_puts(m, "BT601_YCC");
>   		break;
>   	case COLOR_SPACE_YCBCR709:
>   	case COLOR_SPACE_YCBCR709_LIMITED:
> -		seq_printf(m, "BT709_YCC");
> +		seq_puts(m, "BT709_YCC");
>   		break;
>   	case COLOR_SPACE_ADOBERGB:
> -		seq_printf(m, "opRGB");
> +		seq_puts(m, "opRGB");
>   		break;
>   	case COLOR_SPACE_2020_RGB_FULLRANGE:
> -		seq_printf(m, "BT2020_RGB");
> +		seq_puts(m, "BT2020_RGB");
>   		break;
>   	case COLOR_SPACE_2020_YCBCR:
> -		seq_printf(m, "BT2020_YCC");
> +		seq_puts(m, "BT2020_YCC");
>   		break;
>   	default:
>   		goto unlock;
> @@ -3022,7 +3022,7 @@ static int edp_ilr_show(struct seq_file *m, void *unused)
>   			seq_printf(m, "[%d] %d kHz\n", entry/2, link_rate_in_khz);
>   		}
>   	} else {
> -		seq_printf(m, "ILR is not supported by this eDP panel.\n");
> +		seq_puts(m, "ILR is not supported by this eDP panel.\n");
>   	}
>   
>   	return 0;

Reviewed-by: Rodrigo Siqueira <Rodrigo.Siqueira at amd.com>