[0/2] drm: Make some resolution info unsigned

Message

Lyude Paul March 25, 2025, 9:27 p.m. UTC

During the review of some of my patches for KMS bindings in Rust, it was
pointed out we have some areas of DRM that are storing resolutions as
signed integers when it doesn't really make sense. Since there's no real
usecase for this and it's a bit more obvious when writing rust code then
it is in C, let's fix this.

Lyude Paul (2):
  drm/edid: Use unsigned int in drm_add_modes_noedid()
  drm/mode_config: Make drm_mode_config.(max|min)_(width|height) signed

 drivers/gpu/drm/drm_edid.c    | 2 +-
 include/drm/drm_edid.h        | 2 +-
 include/drm/drm_mode_config.h | 4 ++--
 3 files changed, 4 insertions(+), 4 deletions(-)


base-commit: 5da39dce1fa3c81dc6552a16a9f748ba2980d630

Comments

Maxime Ripard March 26, 2025, 9:07 a.m. UTC | #1

On Tue, 25 Mar 2025 17:27:03 -0400, Lyude Paul wrote:
> During the review of some of my patches for KMS bindings in Rust, it was
> pointed out we have some areas of DRM that are storing resolutions as
> signed integers when it doesn't really make sense. Since there's no real
> usecase for this and it's a bit more obvious when writing rust code then
> it is in C, let's fix this.
> 
> [ ... ]

Reviewed-by: Maxime Ripard <[e-mail]>

Thanks!
Maxime