Hi Louis, On Wed Oct 29, 2025 at 3:36 PM CET, Louis Chauvet wrote: > Create and export an helper to display plane type using the > property string. This could be used to display debug > information in VKMS. > > Signed-off-by: Louis Chauvet <[email protected]> > --- > drivers/gpu/drm/drm_mode_config.c | 13 +++++++++++++ > include/drm/drm_mode_config.h | 3 +++ > 2 files changed, 16 insertions(+) > > diff --git a/drivers/gpu/drm/drm_mode_config.c > b/drivers/gpu/drm/drm_mode_config.c > index 25f376869b3a..1a1a3f43db4d 100644 > --- a/drivers/gpu/drm/drm_mode_config.c > +++ b/drivers/gpu/drm/drm_mode_config.c > @@ -226,6 +226,19 @@ static const struct drm_prop_enum_list > drm_plane_type_enum_list[] = { > { DRM_PLANE_TYPE_CURSOR, "Cursor" }, > }; > > +/** > + * drm_get_plane_type_name - return a string for plane name > + * @type: plane type to compute name of > + */ > +const char *drm_get_plane_type_name(enum drm_plane_type type) > +{ > + if (type < ARRAY_SIZE(drm_plane_type_enum_list)) > + return drm_plane_type_enum_list[type].name; > + else > + return "(unknown)"; > +}
AFAIK an enum can be signed, so you should check for >= 0 too for extra safety. Otherwise looks good. Luca -- Luca Ceresoli, Bootlin Embedded Linux and Kernel engineering https://bootlin.com
