Re: [Intel-gfx] [CI 08/12] drm/i915: Perform manual conversions for plane uapi/hw split, v2.

2019-10-31 Thread Maarten Lankhorst
Op 31-10-2019 om 11:30 schreef Ville Syrjälä: > On Thu, Oct 31, 2019 at 10:15:41AM +0100, Maarten Lankhorst wrote: >> Op 30-10-2019 om 17:19 schreef Ville Syrjälä: >>> On Wed, Oct 30, 2019 at 03:26:53PM +0100, Maarten Lankhorst wrote: get_crtc_from_states() is called before plane_state is

Re: [Intel-gfx] [CI 08/12] drm/i915: Perform manual conversions for plane uapi/hw split, v2.

2019-10-31 Thread Ville Syrjälä
On Thu, Oct 31, 2019 at 10:15:41AM +0100, Maarten Lankhorst wrote: > Op 30-10-2019 om 17:19 schreef Ville Syrjälä: > > On Wed, Oct 30, 2019 at 03:26:53PM +0100, Maarten Lankhorst wrote: > >> get_crtc_from_states() is called before plane_state is copied to uapi, > >> so use the uapi state there. >

Re: [Intel-gfx] [CI 08/12] drm/i915: Perform manual conversions for plane uapi/hw split, v2.

2019-10-31 Thread Maarten Lankhorst
Op 30-10-2019 om 17:19 schreef Ville Syrjälä: > On Wed, Oct 30, 2019 at 03:26:53PM +0100, Maarten Lankhorst wrote: >> get_crtc_from_states() is called before plane_state is copied to uapi, >> so use the uapi state there. >> >> intel_legacy_cursor_update() could probably get away with looking at >>

Re: [Intel-gfx] [CI 08/12] drm/i915: Perform manual conversions for plane uapi/hw split, v2.

2019-10-30 Thread Ville Syrjälä
On Wed, Oct 30, 2019 at 03:26:53PM +0100, Maarten Lankhorst wrote: > get_crtc_from_states() is called before plane_state is copied to uapi, > so use the uapi state there. > > intel_legacy_cursor_update() could probably get away with looking at > the hw state, but for clarity always look at the

[Intel-gfx] [CI 08/12] drm/i915: Perform manual conversions for plane uapi/hw split, v2.

2019-10-30 Thread Maarten Lankhorst
get_crtc_from_states() is called before plane_state is copied to uapi, so use the uapi state there. intel_legacy_cursor_update() could probably get away with looking at the hw state, but for clarity always look at the uapi state. Changes since v1: - Convert entirety of intel_legacy_cursor_update