drm/mode_config: Make drm_mode_config.(max|min)_(width|height) unsigned
authorLyude Paul <lyude@redhat.com>
Mon, 31 Mar 2025 22:23:55 +0000 (18:23 -0400)
committerLyude Paul <lyude@redhat.com>
Wed, 2 Apr 2025 21:20:01 +0000 (17:20 -0400)
It doesn't make much sense to allow devices to specify their min/max
resolution as signed integers, and in Rust with CONFIG_RUST_OVERFLOW_CHECKS
enabled this provides us actual over/underflow checks. Similarly, it
doesn't really make much sense for us to allow devices to specify their
minimum/maximum resolution as signed.

Signed-off-by: Lyude Paul <lyude@redhat.com>
Reviewed-by: Maxime Ripard <mripard@kernel.org>
Reviewed-by: Thomas Zimmermann <tzimmermann@suse.de>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Thomas Zimmermann <tzimmermann@suse.de>
Link: https://lore.kernel.org/r/20250331222556.454334-3-lyude@redhat.com
include/drm/drm_mode_config.h

index 271765e2e9f2da62aaf0d258828ef4196e14822e..4b8f0370b79bf573a03cffaf5bcb776010b88f31 100644 (file)
@@ -532,8 +532,8 @@ struct drm_mode_config {
         */
        struct list_head privobj_list;
 
-       int min_width, min_height;
-       int max_width, max_height;
+       unsigned int min_width, min_height;
+       unsigned int max_width, max_height;
        const struct drm_mode_config_funcs *funcs;
 
        /* output poll support */