diff options
| author | Lyude Paul <lyude@redhat.com> | 2025-03-31 18:23:55 -0400 |
|---|---|---|
| committer | Lyude Paul <lyude@redhat.com> | 2025-04-02 17:20:01 -0400 |
| commit | 8abaa80b403e1fc40877eb9b339ae0ad07e8968f (patch) | |
| tree | 2820b04e2356bb66cdb537d470831f2d835788e3 | |
| parent | 4190aa3a76a6f840ace7d5e30719f85af0ed6302 (diff) | |
drm/mode_config: Make drm_mode_config.(max|min)_(width|height) unsigned
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
| -rw-r--r-- | include/drm/drm_mode_config.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/drm/drm_mode_config.h b/include/drm/drm_mode_config.h index 271765e2e9f2..4b8f0370b79b 100644 --- a/include/drm/drm_mode_config.h +++ b/include/drm/drm_mode_config.h @@ -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 */ |