[PATCH] drm: Don't force all planes to be added to the state due to zpos

2016-10-26 Thread Daniel Vetter
On Wed, Oct 26, 2016 at 04:09:00PM +0200, Benjamin Gaignard wrote: > I have just tested it on my board, no regression :-) > > Acked-by: Benjamin Gaignard > > 2016-10-25 22:53 GMT+02:00 Sean Paul : > > On Tue, Oct 25, 2016 at 10:43 AM, Ville Syrjälä > > wrote: > >> On Mon, Oct 10, 2016 at 03:1

[PATCH] drm: Don't force all planes to be added to the state due to zpos

2016-10-26 Thread Benjamin Gaignard
I have just tested it on my board, no regression :-) Acked-by: Benjamin Gaignard 2016-10-25 22:53 GMT+02:00 Sean Paul : > On Tue, Oct 25, 2016 at 10:43 AM, Ville Syrjälä > wrote: >> On Mon, Oct 10, 2016 at 03:19:47PM +0300, ville.syrjala at linux.intel.com >> wrote: >>> From: Ville Syrjälä

[PATCH] drm: Don't force all planes to be added to the state due to zpos

2016-10-25 Thread Ville Syrjälä
On Mon, Oct 10, 2016 at 03:19:47PM +0300, ville.syrjala at linux.intel.com wrote: > From: Ville Syrjälä > > We don't want all planes to be added to the state whenever a > plane with fixed zpos gets enabled/disabled. This is true > especially for eg. cursor planes on i915, as we want cursor > u

[PATCH] drm: Don't force all planes to be added to the state due to zpos

2016-10-25 Thread Sean Paul
On Tue, Oct 25, 2016 at 10:43 AM, Ville Syrjälä wrote: > On Mon, Oct 10, 2016 at 03:19:47PM +0300, ville.syrjala at linux.intel.com > wrote: >> From: Ville Syrjälä >> >> We don't want all planes to be added to the state whenever a >> plane with fixed zpos gets enabled/disabled. This is true

[PATCH] drm: Don't force all planes to be added to the state due to zpos

2016-10-10 Thread Ville Syrjälä
On Mon, Oct 10, 2016 at 03:14:22PM +0200, Daniel Vetter wrote: > On Mon, Oct 10, 2016 at 2:56 PM, Ville Syrjälä > wrote: > >> I think the > >> proper way is to keep track of a per-plane zpos changed (or compute > >> that ad-hoc, we have both states). And only grab more planes if a zpos > >> valu

[PATCH] drm: Don't force all planes to be added to the state due to zpos

2016-10-10 Thread Ville Syrjälä
On Mon, Oct 10, 2016 at 02:46:35PM +0200, Daniel Vetter wrote: > On Mon, Oct 10, 2016 at 2:19 PM, wrote: > > From: Ville Syrjälä > > > > We don't want all planes to be added to the state whenever a > > plane with fixed zpos gets enabled/disabled. This is true > > especially for eg. cursor plan

[PATCH] drm: Don't force all planes to be added to the state due to zpos

2016-10-10 Thread Daniel Vetter
On Mon, Oct 10, 2016 at 3:32 PM, Ville Syrjälä wrote: > On Mon, Oct 10, 2016 at 03:14:22PM +0200, Daniel Vetter wrote: >> On Mon, Oct 10, 2016 at 2:56 PM, Ville Syrjälä >> wrote: >> >> I think the >> >> proper way is to keep track of a per-plane zpos changed (or compute >> >> that ad-hoc, we

[PATCH] drm: Don't force all planes to be added to the state due to zpos

2016-10-10 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä We don't want all planes to be added to the state whenever a plane with fixed zpos gets enabled/disabled. This is true especially for eg. cursor planes on i915, as we want cursor updates to go through w/o throttling. Same holds for drivers that don't support zpos at all (i91

[PATCH] drm: Don't force all planes to be added to the state due to zpos

2016-10-10 Thread Daniel Vetter
On Mon, Oct 10, 2016 at 2:56 PM, Ville Syrjälä wrote: >> I think the >> proper way is to keep track of a per-plane zpos changed (or compute >> that ad-hoc, we have both states). And only grab more planes if a zpos >> value changed. > > Doesn't work with normalized zpos. The plane's actual zpos m

[PATCH] drm: Don't force all planes to be added to the state due to zpos

2016-10-10 Thread Daniel Vetter
On Mon, Oct 10, 2016 at 2:19 PM, wrote: > From: Ville Syrjälä > > We don't want all planes to be added to the state whenever a > plane with fixed zpos gets enabled/disabled. This is true > especially for eg. cursor planes on i915, as we want cursor > updates to go through w/o throttling. Same