Re: [Intel-gfx] [PATCH 1/5] drm/i915: Separate cherryview from valleyview

2015-12-10 Thread Jani Nikula
On Wed, 09 Dec 2015, Ville Syrjälä wrote: > On Wed, Dec 09, 2015 at 12:29:35PM -0800, Wayne Boyer wrote: >> The cherryview device shares many characteristics with the valleyview >> device. When support was added to the driver for cherryview, the >> corresponding device info structure included .is

Re: [Intel-gfx] [PATCH 1/5] drm/i915: Separate cherryview from valleyview

2015-12-09 Thread Ville Syrjälä
On Wed, Dec 09, 2015 at 12:29:35PM -0800, Wayne Boyer wrote: > The cherryview device shares many characteristics with the valleyview > device. When support was added to the driver for cherryview, the > corresponding device info structure included .is_valleyview = 1. > This is not correct and leads

[Intel-gfx] [PATCH 1/5] drm/i915: Separate cherryview from valleyview

2015-12-09 Thread Wayne Boyer
The cherryview device shares many characteristics with the valleyview device. When support was added to the driver for cherryview, the corresponding device info structure included .is_valleyview = 1. This is not correct and leads to some confusion. This patch changes .is_valleyview to .is_cherryv

Re: [Intel-gfx] [PATCH 1/5] drm/i915: Separate cherryview from valleyview

2015-12-09 Thread Boyer, Wayne
Thanks Ville. I'll send one more update with some of the changes. I'll leave the others for a different patch. Details below. Wayne On 12/9/15, 9:10 AM, "Ville Syrjälä" wrote: >On Tue, Dec 08, 2015 at 11:46:53AM -0800, Wayne Boyer wrote: >> The cherryview device shares many characteristics

Re: [Intel-gfx] [PATCH 1/5] drm/i915: Separate cherryview from valleyview

2015-12-09 Thread Ville Syrjälä
On Tue, Dec 08, 2015 at 11:46:53AM -0800, Wayne Boyer wrote: > The cherryview device shares many characteristics with the valleyview > device. When support was added to the driver for cherryview, the > corresponding device info structure included .is_valleyview = 1. > This is not correct and leads

[Intel-gfx] [PATCH 1/5] drm/i915: Separate cherryview from valleyview

2015-12-08 Thread Wayne Boyer
The cherryview device shares many characteristics with the valleyview device. When support was added to the driver for cherryview, the corresponding device info structure included .is_valleyview = 1. This is not correct and leads to some confusion. This patch changes .is_valleyview to .is_cherryv

Re: [Intel-gfx] [PATCH 1/5] drm/i915: Separate cherryview from valleyview

2015-12-08 Thread Ville Syrjälä
On Tue, Dec 08, 2015 at 01:44:10PM +0200, Jani Nikula wrote: > On Mon, 07 Dec 2015, Wayne Boyer wrote: > > The cherryview device shares many characteristics with the valleyview > > device. When support was added to the driver for cherryview, the > > corresponding device info structure included .i

Re: [Intel-gfx] [PATCH 1/5] drm/i915: Separate cherryview from valleyview

2015-12-08 Thread Jani Nikula
On Mon, 07 Dec 2015, Wayne Boyer wrote: > The cherryview device shares many characteristics with the valleyview > device. When support was added to the driver for cherryview, the > corresponding device info structure included .is_valleyview = 1. > This is not correct and leads to some confusion.

[Intel-gfx] [PATCH 1/5] drm/i915: Separate cherryview from valleyview

2015-12-07 Thread Wayne Boyer
The cherryview device shares many characteristics with the valleyview device. When support was added to the driver for cherryview, the corresponding device info structure included .is_valleyview = 1. This is not correct and leads to some confusion. This patch changes .is_valleyview to .is_cherryv