On Wed, 19 Feb 2020 at 20:36, Ville Syrjala <[email protected]> wrote: > > From: Ville Syrjälä <[email protected]> > > 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ä <[email protected]> > --- > 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 <[email protected]> -Emil _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
