Re: [Intel-gfx] [PATCH 2/9] drm/i915: Add per-pipe plane identifier

2016-11-18 Thread Paulo Zanoni
Em Sex, 2016-11-18 às 16:32 +0200, Ville Syrjälä escreveu: > On Fri, Nov 18, 2016 at 12:17:06PM -0200, Paulo Zanoni wrote: > > > > Em Qui, 2016-11-17 às 21:43 +0200, Ville Syrjälä escreveu: > > > > > > On Thu, Nov 17, 2016 at 05:09:38PM -0200, Paulo Zanoni wrote: > > > > > > > > > > > > Em Ter,

Re: [Intel-gfx] [PATCH 2/9] drm/i915: Add per-pipe plane identifier

2016-11-18 Thread Matt Roper
On Thu, Nov 17, 2016 at 05:09:38PM -0200, Paulo Zanoni wrote: > Em Ter, 2016-11-08 às 16:47 +0200, ville.syrj...@linux.intel.com > escreveu: > > From: Ville Syrjälä > > > > As I told people in [1] we really should not be confusing enum plane > > as a per-pipe plane identifier. Looks like that hap

Re: [Intel-gfx] [PATCH 2/9] drm/i915: Add per-pipe plane identifier

2016-11-18 Thread Ville Syrjälä
On Fri, Nov 18, 2016 at 12:17:06PM -0200, Paulo Zanoni wrote: > Em Qui, 2016-11-17 às 21:43 +0200, Ville Syrjälä escreveu: > > On Thu, Nov 17, 2016 at 05:09:38PM -0200, Paulo Zanoni wrote: > > > > > > Em Ter, 2016-11-08 às 16:47 +0200, ville.syrj...@linux.intel.com > > > escreveu: > > > > > > > >

Re: [Intel-gfx] [PATCH 2/9] drm/i915: Add per-pipe plane identifier

2016-11-18 Thread Paulo Zanoni
Em Qui, 2016-11-17 às 21:43 +0200, Ville Syrjälä escreveu: > On Thu, Nov 17, 2016 at 05:09:38PM -0200, Paulo Zanoni wrote: > > > > Em Ter, 2016-11-08 às 16:47 +0200, ville.syrj...@linux.intel.com > > escreveu: > > > > > > From: Ville Syrjälä > > > > > > As I told people in [1] we really should

Re: [Intel-gfx] [PATCH 2/9] drm/i915: Add per-pipe plane identifier

2016-11-17 Thread Ville Syrjälä
On Thu, Nov 17, 2016 at 05:09:38PM -0200, Paulo Zanoni wrote: > Em Ter, 2016-11-08 às 16:47 +0200, ville.syrj...@linux.intel.com > escreveu: > > From: Ville Syrjälä > > > > As I told people in [1] we really should not be confusing enum plane > > as a per-pipe plane identifier. Looks like that hap

Re: [Intel-gfx] [PATCH 2/9] drm/i915: Add per-pipe plane identifier

2016-11-17 Thread Paulo Zanoni
Em Ter, 2016-11-08 às 16:47 +0200, ville.syrj...@linux.intel.com escreveu: > From: Ville Syrjälä > > As I told people in [1] we really should not be confusing enum plane > as a per-pipe plane identifier. Looks like that happened nonetheless, > so > let's fix it up by splitting the two into two en

Re: [Intel-gfx] [PATCH 2/9] drm/i915: Add per-pipe plane identifier

2016-11-09 Thread Ville Syrjälä
On Tue, Nov 08, 2016 at 04:53:20PM -0800, Matt Roper wrote: > On Tue, Nov 08, 2016 at 04:47:12PM +0200, ville.syrj...@linux.intel.com wrote: > > From: Ville Syrjälä > > > > As I told people in [1] we really should not be confusing enum plane > > as a per-pipe plane identifier. Looks like that hap

Re: [Intel-gfx] [PATCH 2/9] drm/i915: Add per-pipe plane identifier

2016-11-08 Thread Matt Roper
On Tue, Nov 08, 2016 at 04:47:12PM +0200, ville.syrj...@linux.intel.com wrote: > From: Ville Syrjälä > > As I told people in [1] we really should not be confusing enum plane > as a per-pipe plane identifier. Looks like that happened nonetheless, so > let's fix it up by splitting the two into two

Re: [Intel-gfx] [PATCH 2/9] drm/i915: Add per-pipe plane identifier

2016-11-08 Thread Ville Syrjälä
On Tue, Nov 08, 2016 at 03:26:31PM +, Chris Wilson wrote: > On Tue, Nov 08, 2016 at 04:47:12PM +0200, ville.syrj...@linux.intel.com wrote: > > From: Ville Syrjälä > > > > As I told people in [1] we really should not be confusing enum plane > > as a per-pipe plane identifier. Looks like that h

Re: [Intel-gfx] [PATCH 2/9] drm/i915: Add per-pipe plane identifier

2016-11-08 Thread Chris Wilson
On Tue, Nov 08, 2016 at 04:47:12PM +0200, ville.syrj...@linux.intel.com wrote: > From: Ville Syrjälä > > As I told people in [1] we really should not be confusing enum plane > as a per-pipe plane identifier. Looks like that happened nonetheless, so > let's fix it up by splitting the two into two

[Intel-gfx] [PATCH 2/9] drm/i915: Add per-pipe plane identifier

2016-11-08 Thread ville . syrjala
From: Ville Syrjälä As I told people in [1] we really should not be confusing enum plane as a per-pipe plane identifier. Looks like that happened nonetheless, so let's fix it up by splitting the two into two enums. We'll also want something we just directly pass to various register offset macros