Re: [PATCH v5 08/10] drm/arm: malidp: Use crtc->mode_valid() callback

2017-05-31 Thread Liviu Dudau
On Wed, May 31, 2017 at 12:56:59PM +0200, Daniel Vetter wrote: > On Wed, May 31, 2017 at 12:48 PM, Liviu Dudau wrote: > > On Wed, May 31, 2017 at 10:20:04AM +0200, Daniel Vetter wrote: > >> On Tue, May 30, 2017 at 10:37:29AM +0100, Liviu Dudau wrote: > >> > On Tue, May 30,

Re: [PATCH v5 08/10] drm/arm: malidp: Use crtc->mode_valid() callback

2017-05-31 Thread Daniel Vetter
On Wed, May 31, 2017 at 12:48 PM, Liviu Dudau wrote: > On Wed, May 31, 2017 at 10:20:04AM +0200, Daniel Vetter wrote: >> On Tue, May 30, 2017 at 10:37:29AM +0100, Liviu Dudau wrote: >> > On Tue, May 30, 2017 at 09:29:44AM +0200, Neil Armstrong wrote: >> > > On 05/25/2017

Re: [PATCH v5 08/10] drm/arm: malidp: Use crtc->mode_valid() callback

2017-05-31 Thread Liviu Dudau
On Wed, May 31, 2017 at 10:20:04AM +0200, Daniel Vetter wrote: > On Tue, May 30, 2017 at 10:37:29AM +0100, Liviu Dudau wrote: > > On Tue, May 30, 2017 at 09:29:44AM +0200, Neil Armstrong wrote: > > > On 05/25/2017 04:19 PM, Jose Abreu wrote: > > > > Now that we have a callback to check if crtc

Re: [PATCH v5 08/10] drm/arm: malidp: Use crtc->mode_valid() callback

2017-05-31 Thread Daniel Vetter
On Tue, May 30, 2017 at 10:37:29AM +0100, Liviu Dudau wrote: > On Tue, May 30, 2017 at 09:29:44AM +0200, Neil Armstrong wrote: > > On 05/25/2017 04:19 PM, Jose Abreu wrote: > > > Now that we have a callback to check if crtc supports a given mode > > > we can use it in malidp so that we restrict

Re: [PATCH v5 08/10] drm/arm: malidp: Use crtc->mode_valid() callback

2017-05-30 Thread Liviu Dudau
On Tue, May 30, 2017 at 09:29:44AM +0200, Neil Armstrong wrote: > On 05/25/2017 04:19 PM, Jose Abreu wrote: > > Now that we have a callback to check if crtc supports a given mode > > we can use it in malidp so that we restrict the number of probbed > > modes to the ones we can actually display. >

Re: [PATCH v5 08/10] drm/arm: malidp: Use crtc->mode_valid() callback

2017-05-30 Thread Neil Armstrong
On 05/25/2017 04:19 PM, Jose Abreu wrote: > Now that we have a callback to check if crtc supports a given mode > we can use it in malidp so that we restrict the number of probbed > modes to the ones we can actually display. > > Also, remove the mode_fixup() callback as this is no longer needed >

[PATCH v5 08/10] drm/arm: malidp: Use crtc->mode_valid() callback

2017-05-25 Thread Jose Abreu
Now that we have a callback to check if crtc supports a given mode we can use it in malidp so that we restrict the number of probbed modes to the ones we can actually display. Also, remove the mode_fixup() callback as this is no longer needed because mode_valid() will be called before. NOTE: Not