Re: [Intel-gfx] [PATCH 2/2] drm/i915: Simplify scaler init during CRTC HW readout

2017-07-21 Thread Imre Deak
On Fri, Jul 21, 2017 at 07:36:22PM +0530, Mahesh Kumar wrote: > Hi, > > > On Friday 21 July 2017 06:56 PM, Imre Deak wrote: > > On Fri, Jul 21, 2017 at 06:44:24PM +0530, Mahesh Kumar wrote: > > > Hi, > > > > > > > > > On Thursday 20 July 2017 04:20 AM, Imre Deak wrote: > > > > The crtc state st

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Simplify scaler init during CRTC HW readout

2017-07-21 Thread Mahesh Kumar
Hi, On Friday 21 July 2017 06:56 PM, Imre Deak wrote: On Fri, Jul 21, 2017 at 06:44:24PM +0530, Mahesh Kumar wrote: Hi, On Thursday 20 July 2017 04:20 AM, Imre Deak wrote: The crtc state starts out being bzero'd, so no need to clear scaler_users. Also intel_crtc_init_scalers() knows already

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Simplify scaler init during CRTC HW readout

2017-07-21 Thread Imre Deak
On Fri, Jul 21, 2017 at 06:44:24PM +0530, Mahesh Kumar wrote: > Hi, > > > On Thursday 20 July 2017 04:20 AM, Imre Deak wrote: > > The crtc state starts out being bzero'd, so no need to clear > > scaler_users. Also intel_crtc_init_scalers() knows already which > > platforms have scalers, so no nee

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Simplify scaler init during CRTC HW readout

2017-07-21 Thread Mahesh Kumar
Hi, On Thursday 20 July 2017 04:20 AM, Imre Deak wrote: The crtc state starts out being bzero'd, so no need to clear scaler_users. Also intel_crtc_init_scalers() knows already which platforms have scalers, so no need for the platform check here. Similarly intel_crtc_init_scalers() will init sca

[Intel-gfx] [PATCH 2/2] drm/i915: Simplify scaler init during CRTC HW readout

2017-07-19 Thread Imre Deak
The crtc state starts out being bzero'd, so no need to clear scaler_users. Also intel_crtc_init_scalers() knows already which platforms have scalers, so no need for the platform check here. Similarly intel_crtc_init_scalers() will init scaler_id as required, so no need to do it here separately. Cc