Re: [Intel-gfx] [PATCH 09/12] drm: Shrink drm_display_mode timings

2020-02-24 Thread Ville Syrjälä
On Fri, Feb 21, 2020 at 06:27:22PM +0100, Sam Ravnborg wrote: > Hi Ville. > > On Wed, Feb 19, 2020 at 10:35:41PM +0200, Ville Syrjala wrote: > > From: Ville Syrjälä > > > > Store the timings (apart from the clock) as u16. The uapi mode > > struct already uses u16 for everything so using

Re: [Intel-gfx] [PATCH 09/12] drm: Shrink drm_display_mode timings

2020-02-21 Thread Sam Ravnborg
Hi Ville. On Wed, Feb 19, 2020 at 10:35:41PM +0200, Ville Syrjala wrote: > From: Ville Syrjälä > > Store the timings (apart from the clock) as u16. The uapi mode > struct already uses u16 for everything so using something bigger > internally doesn't really help us. > > Signed-off-by: Ville

Re: [Intel-gfx] [PATCH 09/12] drm: Shrink drm_display_mode timings

2020-02-21 Thread Sam Ravnborg
Hi Ville. On Wed, Feb 19, 2020 at 10:35:41PM +0200, Ville Syrjala wrote: > From: Ville Syrjälä > > Store the timings (apart from the clock) as u16. The uapi mode > struct already uses u16 for everything so using something bigger > internally doesn't really help us. > > Signed-off-by: Ville

Re: [Intel-gfx] [PATCH 09/12] drm: Shrink drm_display_mode timings

2020-02-20 Thread Ville Syrjälä
On Thu, Feb 20, 2020 at 07:19:08PM +0100, Daniel Vetter wrote: > On Wed, Feb 19, 2020 at 10:35:41PM +0200, Ville Syrjala wrote: > > From: Ville Syrjälä > > > > Store the timings (apart from the clock) as u16. The uapi mode > > struct already uses u16 for everything so using something bigger > >

Re: [Intel-gfx] [PATCH 09/12] drm: Shrink drm_display_mode timings

2020-02-20 Thread Daniel Vetter
On Wed, Feb 19, 2020 at 10:35:41PM +0200, Ville Syrjala wrote: > From: Ville Syrjälä > > Store the timings (apart from the clock) as u16. The uapi mode > struct already uses u16 for everything so using something bigger > internally doesn't really help us. > > Signed-off-by: Ville Syrjälä

[Intel-gfx] [PATCH 09/12] drm: Shrink drm_display_mode timings

2020-02-19 Thread Ville Syrjala
From: Ville Syrjälä Store the timings (apart from the clock) as u16. The uapi mode struct already uses u16 for everything so using something bigger internally doesn't really help us. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/drm_modes.c | 7 -- include/drm/drm_modes.h | 46