Re: [Intel-gfx] [PATCH 2/3] drm/i915/gen9: Compute data rates for all planes on first commit

2016-06-20 Thread Daniel Vetter
On Fri, Jun 17, 2016 at 02:03:07PM -0700, Matt Roper wrote: > On Mon, Jun 13, 2016 at 04:50:31PM +0200, Daniel Vetter wrote: > > On Thu, Jun 09, 2016 at 03:14:54PM -0700, Matt Roper wrote: > > > When we sanitize our DDB and watermark info during the first atomic > > > commit, we need to calculate t

Re: [Intel-gfx] [PATCH 2/3] drm/i915/gen9: Compute data rates for all planes on first commit

2016-06-17 Thread Matt Roper
On Mon, Jun 13, 2016 at 04:50:31PM +0200, Daniel Vetter wrote: > On Thu, Jun 09, 2016 at 03:14:54PM -0700, Matt Roper wrote: > > When we sanitize our DDB and watermark info during the first atomic > > commit, we need to calculate the total data rate. Since we haven't > > explicitly added the plane

Re: [Intel-gfx] [PATCH 2/3] drm/i915/gen9: Compute data rates for all planes on first commit

2016-06-13 Thread Daniel Vetter
On Thu, Jun 09, 2016 at 03:14:54PM -0700, Matt Roper wrote: > When we sanitize our DDB and watermark info during the first atomic > commit, we need to calculate the total data rate. Since we haven't > explicitly added the planes for each CRTC to our atomic state, the total > data rate calculation

Re: [Intel-gfx] [PATCH 2/3] drm/i915/gen9: Compute data rates for all planes on first commit

2016-06-13 Thread Maarten Lankhorst
Op 10-06-16 om 00:14 schreef Matt Roper: > When we sanitize our DDB and watermark info during the first atomic > commit, we need to calculate the total data rate. Since we haven't > explicitly added the planes for each CRTC to our atomic state, the total > data rate calculation will try to use the

[Intel-gfx] [PATCH 2/3] drm/i915/gen9: Compute data rates for all planes on first commit

2016-06-09 Thread Matt Roper
When we sanitize our DDB and watermark info during the first atomic commit, we need to calculate the total data rate. Since we haven't explicitly added the planes for each CRTC to our atomic state, the total data rate calculation will try to use the cached values from a previous commit (which are