Re: [Intel-gfx] [PATCH] drm/i915: remove duplicated memsets in skl_allocate_pipe_ddb()

2016-11-10 Thread Jani Nikula
On Wed, 09 Nov 2016, Matt Roper wrote: > On Wed, Nov 09, 2016 at 11:27:37AM +0200, Jani Nikula wrote: >> On Wed, 09 Nov 2016, Matt Roper wrote: >> > On Tue, Nov 08, 2016 at 03:21:22PM -0200, Paulo Zanoni wrote: >> >> One of the memsets was added by 5a920b85f2c6 ("drm/i915/gen9: fix DDB >> >> part

Re: [Intel-gfx] [PATCH] drm/i915: remove duplicated memsets in skl_allocate_pipe_ddb()

2016-11-09 Thread Matt Roper
On Wed, Nov 09, 2016 at 11:27:37AM +0200, Jani Nikula wrote: > On Wed, 09 Nov 2016, Matt Roper wrote: > > On Tue, Nov 08, 2016 at 03:21:22PM -0200, Paulo Zanoni wrote: > >> One of the memsets was added by 5a920b85f2c6 ("drm/i915/gen9: fix DDB > >> partitioning for multi-screen cases"), and the oth

Re: [Intel-gfx] [PATCH] drm/i915: remove duplicated memsets in skl_allocate_pipe_ddb()

2016-11-09 Thread Jani Nikula
On Wed, 09 Nov 2016, Matt Roper wrote: > On Tue, Nov 08, 2016 at 03:21:22PM -0200, Paulo Zanoni wrote: >> One of the memsets was added by 5a920b85f2c6 ("drm/i915/gen9: fix DDB >> partitioning for multi-screen cases"), and the other was added by >> 01c72d6c17 ("drm/i915/gen9: fix DDB partitioning f

Re: [Intel-gfx] [PATCH] drm/i915: remove duplicated memsets in skl_allocate_pipe_ddb()

2016-11-08 Thread Matt Roper
On Tue, Nov 08, 2016 at 03:21:22PM -0200, Paulo Zanoni wrote: > One of the memsets was added by 5a920b85f2c6 ("drm/i915/gen9: fix DDB > partitioning for multi-screen cases"), and the other was added by > 01c72d6c17 ("drm/i915/gen9: fix DDB partitioning for multi-screen > cases"). I'm confused and I

[Intel-gfx] [PATCH] drm/i915: remove duplicated memsets in skl_allocate_pipe_ddb()

2016-11-08 Thread Paulo Zanoni
One of the memsets was added by 5a920b85f2c6 ("drm/i915/gen9: fix DDB partitioning for multi-screen cases"), and the other was added by 01c72d6c17 ("drm/i915/gen9: fix DDB partitioning for multi-screen cases"). I'm confused and I'll let the maintainers find out what went wrong here. Cc: Jani Nikul