Re: [PATCH RESEND v2 02/32] drm/vkms: Explicitly display plane type

José Expósito
2025-11-13T14:07:20Z

On Wed, Oct 29, 2025 at 03:36:39PM +0100, Louis Chauvet wrote:
> Debug information are mainly read by human, so use full name instead
> of values.

Reviewed-by: José Expósito <jose.exposito@redhat.com>

> Signed-off-by: Louis Chauvet <louis.chauvet@bootlin.com>
> ---
> drivers/gpu/drm/vkms/vkms_config.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/vkms/vkms_config.c b/drivers/gpu/drm/vkms/vkms_config.c
> index f8394a063ecf..cfdd5f32f234 100644
> --- a/drivers/gpu/drm/vkms/vkms_config.c
> +++ b/drivers/gpu/drm/vkms/vkms_config.c
> @@ -348,8 +348,8 @@ static int vkms_config_show(struct seq_file *m, void *data)
>
> vkms_config_for_each_plane(vkmsdev->config, plane_cfg) {
> seq_puts(m, "plane:\n");
> - seq_printf(m, "\ttype=%d\n",
> - vkms_config_plane_get_type(plane_cfg));
> + seq_printf(m, "\ttype=%s\n",
> + drm_get_plane_type_name(vkms_config_plane_get_type(plane_cfg)));
> }
>
> vkms_config_for_each_crtc(vkmsdev->config, crtc_cfg) {
>
> --
> 2.51.0
>