Re: [Intel-gfx] [PATCH 2/8] drm/i915/skl+: Remove data_rate from watermark struct.

2016-10-24 Thread Maarten Lankhorst
Op 20-10-16 om 19:18 schreef Paulo Zanoni: > Em Qua, 2016-10-19 às 15:13 -0700, Matt Roper escreveu: >> On Wed, Oct 12, 2016 at 03:28:15PM +0200, Maarten Lankhorst wrote: >>> It's only used in one function, and can be calculated without >>> caching it >>> in the global struct by using >>>

Re: [Intel-gfx] [PATCH 2/8] drm/i915/skl+: Remove data_rate from watermark struct.

2016-10-20 Thread Paulo Zanoni
Em Qui, 2016-10-20 às 15:18 -0200, Paulo Zanoni escreveu: > Em Qua, 2016-10-19 às 15:13 -0700, Matt Roper escreveu: > > > > On Wed, Oct 12, 2016 at 03:28:15PM +0200, Maarten Lankhorst wrote: > > > > > > > > > It's only used in one function, and can be calculated without > > > caching it > > >

Re: [Intel-gfx] [PATCH 2/8] drm/i915/skl+: Remove data_rate from watermark struct.

2016-10-20 Thread Paulo Zanoni
Em Qua, 2016-10-19 às 15:13 -0700, Matt Roper escreveu: > On Wed, Oct 12, 2016 at 03:28:15PM +0200, Maarten Lankhorst wrote: > > > > It's only used in one function, and can be calculated without > > caching it > > in the global struct by using > > drm_atomic_crtc_state_for_each_plane_state. > >

Re: [Intel-gfx] [PATCH 2/8] drm/i915/skl+: Remove data_rate from watermark struct.

2016-10-19 Thread Matt Roper
On Wed, Oct 12, 2016 at 03:28:15PM +0200, Maarten Lankhorst wrote: > It's only used in one function, and can be calculated without caching it > in the global struct by using drm_atomic_crtc_state_for_each_plane_state. > > Signed-off-by: Maarten Lankhorst > ---

[Intel-gfx] [PATCH 2/8] drm/i915/skl+: Remove data_rate from watermark struct.

2016-10-12 Thread Maarten Lankhorst
It's only used in one function, and can be calculated without caching it in the global struct by using drm_atomic_crtc_state_for_each_plane_state. Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/i915/intel_drv.h | 4 drivers/gpu/drm/i915/intel_pm.c