| Message ID | 20200219203544.31013-7-ville.syrjala@linux.intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | drm: Put drm_display_mode on diet | expand |
On Wed, 19 Feb 2020 at 20:36, Ville Syrjala <ville.syrjala@linux.intel.com> wrote: > > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > > Instead of supporting ~2000km wide displayes let's limit ourselves > to ~65m. That seems plenty big enough to me. > > Even with EDID_QUIRK_DETAILED_IN_CM EDIDs seem to be limited to > 10*0xfff which fits into the 16 bits. > > Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> > --- > include/drm/drm_modes.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/include/drm/drm_modes.h b/include/drm/drm_modes.h > index 52e8ca613e4b..2bb2b1a8592a 100644 > --- a/include/drm/drm_modes.h > +++ b/include/drm/drm_modes.h > @@ -330,7 +330,7 @@ struct drm_display_mode { > * Addressable size of the output in mm, projectors should set this to > * 0. > */ > - int width_mm; > + u16 width_mm; > > /** > * @height_mm: > @@ -338,7 +338,7 @@ struct drm_display_mode { > * Addressable size of the output in mm, projectors should set this to > * 0. > */ > - int height_mm; > + u16 height_mm; > Fwiw we could do the same for struct drm_display_info, although we should be carefull since that info sets passed to userspace. Regardless, this patch is: Reviewed-by: Emil Velikov <emil.velikov@collabora.com> -Emil
diff --git a/include/drm/drm_modes.h b/include/drm/drm_modes.h index 52e8ca613e4b..2bb2b1a8592a 100644 --- a/include/drm/drm_modes.h +++ b/include/drm/drm_modes.h @@ -330,7 +330,7 @@ struct drm_display_mode { * Addressable size of the output in mm, projectors should set this to * 0. */ - int width_mm; + u16 width_mm; /** * @height_mm: @@ -338,7 +338,7 @@ struct drm_display_mode { * Addressable size of the output in mm, projectors should set this to * 0. */ - int height_mm; + u16 height_mm; /** * @crtc_clock: