Re: [Intel-gfx] [PATCH v2] drm/i915: Simplify CCS and UV plane alignment handling

2021-04-24 Thread Imre Deak
On Fri, Apr 23, 2021 at 08:04:35PM +0300, Ville Syrjälä wrote: > On Wed, Apr 21, 2021 at 08:32:20PM +0300, Imre Deak wrote: > > We can handle the surface alignment of CCS and UV color planes for all > > modifiers at one place, so do this. An AUX color plane can be a CCS or a > > UV plane, use only

Re: [Intel-gfx] [PATCH v2] drm/i915: Simplify CCS and UV plane alignment handling

2021-04-23 Thread Ville Syrjälä
On Wed, Apr 21, 2021 at 08:32:20PM +0300, Imre Deak wrote: > We can handle the surface alignment of CCS and UV color planes for all > modifiers at one place, so do this. An AUX color plane can be a CCS or a > UV plane, use only the more specific query functions and remove > is_aux_plane() becoming

Re: [Intel-gfx] [PATCH v2] drm/i915: Simplify CCS and UV plane alignment handling

2021-04-22 Thread Juha-Pekka Heikkila
look ok to me. Reviewed-by: Juha-Pekka Heikkila On 21.4.2021 20.32, Imre Deak wrote: We can handle the surface alignment of CCS and UV color planes for all modifiers at one place, so do this. An AUX color plane can be a CCS or a UV plane, use only the more specific query functions and remove i

[Intel-gfx] [PATCH v2] drm/i915: Simplify CCS and UV plane alignment handling

2021-04-21 Thread Imre Deak
We can handle the surface alignment of CCS and UV color planes for all modifiers at one place, so do this. An AUX color plane can be a CCS or a UV plane, use only the more specific query functions and remove is_aux_plane() becoming redundant. While at it add a TODO for linear UV color plane alignm