Re: [Intel-gfx] [PATCH 02/15] drm/i915/glk: Introduce Geminilake platform definition

2016-11-11 Thread Ander Conselvan De Oliveira
On Thu, 2016-11-10 at 09:03 -0800, Rodrigo Vivi wrote: > Yep, it is probably better to merge Jani patch before while no platform > is using that flag, but one way or another feel free to use: Agreed. I'll send an updated version of the first two patches because of the ddb_size that is wrong, so I

Re: [Intel-gfx] [PATCH 02/15] drm/i915/glk: Introduce Geminilake platform definition

2016-11-10 Thread Matt Roper
On Thu, Nov 10, 2016 at 05:23:07PM +0200, Ander Conselvan de Oliveira wrote: > Geminilake is an Intel® Processor containing Intel® HD Graphics > following Broxton. > > Let's start by adding the platform definition. PCI IDs and plaform > specific code will follow. > > v2: Rebase (don't allow dev

Re: [Intel-gfx] [PATCH 02/15] drm/i915/glk: Introduce Geminilake platform definition

2016-11-10 Thread Rodrigo Vivi
Yep, it is probably better to merge Jani patch before while no platform is using that flag, but one way or another feel free to use: Reviewed-by: Rodrigo Vivi On Thu, Nov 10, 2016 at 06:40:29PM +0200, Jani Nikula wrote: > On Thu, 10 Nov 2016, Ander Conselvan de Oliveira

Re: [Intel-gfx] [PATCH 02/15] drm/i915/glk: Introduce Geminilake platform definition

2016-11-10 Thread Jani Nikula
On Thu, 10 Nov 2016, Ander Conselvan de Oliveira wrote: > Geminilake is an Intel® Processor containing Intel® HD Graphics > following Broxton. > > Let's start by adding the platform definition. PCI IDs and plaform > specific code will follow. > > v2: Rebase

[Intel-gfx] [PATCH 02/15] drm/i915/glk: Introduce Geminilake platform definition

2016-11-10 Thread Ander Conselvan de Oliveira
Geminilake is an Intel® Processor containing Intel® HD Graphics following Broxton. Let's start by adding the platform definition. PCI IDs and plaform specific code will follow. v2: Rebase (don't allow dev to be used with the new macro). Signed-off-by: Ander Conselvan de Oliveira