Re: [Intel-gfx] [PATCH v2 3/8] drm/i915/gen11: Link nv12 Y and UV planes in the atomic state, v4.

2018-10-22 Thread Maarten Lankhorst
Op 19-10-18 om 19:14 schreef Ville Syrjälä: > On Fri, Oct 19, 2018 at 04:22:29PM +0200, Maarten Lankhorst wrote: >> Op 18-10-18 om 18:00 schreef Ville Syrjälä: >>> On Thu, Oct 18, 2018 at 01:51:29PM +0200, Maarten Lankhorst wrote: To make NV12 working on icl, we need to update 2 planes

Re: [Intel-gfx] [PATCH v2 3/8] drm/i915/gen11: Link nv12 Y and UV planes in the atomic state, v4.

2018-10-19 Thread Ville Syrjälä
On Fri, Oct 19, 2018 at 04:22:29PM +0200, Maarten Lankhorst wrote: > Op 18-10-18 om 18:00 schreef Ville Syrjälä: > > On Thu, Oct 18, 2018 at 01:51:29PM +0200, Maarten Lankhorst wrote: > >> To make NV12 working on icl, we need to update 2 planes simultaneously. > >> I've chosen to do this in the

Re: [Intel-gfx] [PATCH v2 3/8] drm/i915/gen11: Link nv12 Y and UV planes in the atomic state, v4.

2018-10-19 Thread Maarten Lankhorst
Op 18-10-18 om 18:00 schreef Ville Syrjälä: > On Thu, Oct 18, 2018 at 01:51:29PM +0200, Maarten Lankhorst wrote: >> To make NV12 working on icl, we need to update 2 planes simultaneously. >> I've chosen to do this in the CRTC step after plane validation is done, >> so we know what planes are

Re: [Intel-gfx] [PATCH v2 3/8] drm/i915/gen11: Link nv12 Y and UV planes in the atomic state, v4.

2018-10-18 Thread Ville Syrjälä
On Thu, Oct 18, 2018 at 01:51:29PM +0200, Maarten Lankhorst wrote: > To make NV12 working on icl, we need to update 2 planes simultaneously. > I've chosen to do this in the CRTC step after plane validation is done, > so we know what planes are (in)visible. The linked Y plane will get > updated in

[Intel-gfx] [PATCH v2 3/8] drm/i915/gen11: Link nv12 Y and UV planes in the atomic state, v4.

2018-10-18 Thread Maarten Lankhorst
To make NV12 working on icl, we need to update 2 planes simultaneously. I've chosen to do this in the CRTC step after plane validation is done, so we know what planes are (in)visible. The linked Y plane will get updated in intel_plane_update_planes_on_crtc(), by the call to update_slave, which