Re: [Intel-gfx] [PATCH v2 13/13] drm/i915: Pass the plane to icl_program_input_csc_coeff()

2018-11-26 Thread Matt Roper
On Wed, Nov 14, 2018 at 11:07:29PM +0200, Ville Syrjala wrote: > From: Ville Syrjälä > > On icl+ the plane state that gets passed to update_slave() is not > the plane state of the plane we're programming. With NV12 the > plane state would be coming from the master (UV) plane whereas > the plane

Re: [Intel-gfx] [PATCH v2 13/13] drm/i915: Pass the plane to icl_program_input_csc_coeff()

2018-11-15 Thread Maarten Lankhorst
Op 14-11-18 om 22:07 schreef Ville Syrjala: > From: Ville Syrjälä > > On icl+ the plane state that gets passed to update_slave() is not > the plane state of the plane we're programming. With NV12 the > plane state would be coming from the master (UV) plane whereas > the plane we're programming is

Re: [Intel-gfx] [PATCH v2 13/13] drm/i915: Pass the plane to icl_program_input_csc_coeff()

2018-11-15 Thread Shankar, Uma
>-Original Message- >From: Ville Syrjala [mailto:ville.syrj...@linux.intel.com] >Sent: Thursday, November 15, 2018 2:37 AM >To: intel-gfx@lists.freedesktop.org >Cc: Shankar, Uma ; Maarten Lankhorst > >Subject: [PATCH v2 13/13] drm/i915: Pass the plane to >icl_program_input_csc_coeff() >

[Intel-gfx] [PATCH v2 13/13] drm/i915: Pass the plane to icl_program_input_csc_coeff()

2018-11-14 Thread Ville Syrjala
From: Ville Syrjälä On icl+ the plane state that gets passed to update_slave() is not the plane state of the plane we're programming. With NV12 the plane state would be coming from the master (UV) plane whereas the plane we're programming is the slave (Y) plane. For that reason we need to