Re: [Intel-gfx] [PATCH 04/14] drm/i915: Clean up the .get_cdclk() assignment if ladder

2017-01-25 Thread Ville Syrjälä
On Tue, Jan 24, 2017 at 02:29:00PM +0200, David Weinehall wrote: > On Fri, Jan 20, 2017 at 08:21:55PM +0200, ville.syrj...@linux.intel.com wrote: > > From: Ville Syrjälä > > > > Let's clean up the mess we have in the if ladder that assigns the > > .get_cdclk()

Re: [Intel-gfx] [PATCH 04/14] drm/i915: Clean up the .get_cdclk() assignment if ladder

2017-01-24 Thread David Weinehall
On Fri, Jan 20, 2017 at 08:21:55PM +0200, ville.syrj...@linux.intel.com wrote: > From: Ville Syrjälä > > Let's clean up the mess we have in the if ladder that assigns the > .get_cdclk() hooks. The grouping of the platforms by the function > results in a thing

[Intel-gfx] [PATCH 04/14] drm/i915: Clean up the .get_cdclk() assignment if ladder

2017-01-20 Thread ville . syrjala
From: Ville Syrjälä Let's clean up the mess we have in the if ladder that assigns the .get_cdclk() hooks. The grouping of the platforms by the function results in a thing that's not really legible, so let's do it the other way around and order the if ladder by

Re: [Intel-gfx] [PATCH 04/14] drm/i915: Clean up the .get_cdclk() assignment if ladder

2016-12-20 Thread Ander Conselvan De Oliveira
On Mon, 2016-12-19 at 19:28 +0200, ville.syrj...@linux.intel.com wrote: > From: Ville Syrjälä > > Let's clean up the mess we have in the if ladder that assigns the > .get_cdclk() hooks. The grouping of the platforms by the function > results in a thing that's not

Re: [Intel-gfx] [PATCH 04/14] drm/i915: Clean up the .get_cdclk() assignment if ladder

2016-12-20 Thread Ander Conselvan De Oliveira
On Tue, 2016-12-20 at 12:08 +0200, Jani Nikula wrote: > On Tue, 20 Dec 2016, Ander Conselvan De Oliveira wrote: > > > > On Mon, 2016-12-19 at 19:35 +0200, Jani Nikula wrote: > > > > > > On Mon, 19 Dec 2016, ville.syrj...@linux.intel.com wrote: > > > > > > > > > > > >

Re: [Intel-gfx] [PATCH 04/14] drm/i915: Clean up the .get_cdclk() assignment if ladder

2016-12-20 Thread Jani Nikula
On Tue, 20 Dec 2016, Ander Conselvan De Oliveira wrote: > On Mon, 2016-12-19 at 19:35 +0200, Jani Nikula wrote: >> On Mon, 19 Dec 2016, ville.syrj...@linux.intel.com wrote: >> > >> > From: Ville Syrjälä >> > >> > Let's clean up the mess we

Re: [Intel-gfx] [PATCH 04/14] drm/i915: Clean up the .get_cdclk() assignment if ladder

2016-12-20 Thread Ander Conselvan De Oliveira
On Mon, 2016-12-19 at 19:35 +0200, Jani Nikula wrote: > On Mon, 19 Dec 2016, ville.syrj...@linux.intel.com wrote: > > > > From: Ville Syrjälä > > > > Let's clean up the mess we have in the if ladder that assigns the > > .get_cdclk() hooks. The grouping of the

Re: [Intel-gfx] [PATCH 04/14] drm/i915: Clean up the .get_cdclk() assignment if ladder

2016-12-19 Thread Jani Nikula
On Mon, 19 Dec 2016, ville.syrj...@linux.intel.com wrote: > From: Ville Syrjälä > > Let's clean up the mess we have in the if ladder that assigns the > .get_cdclk() hooks. The grouping of the platforms by the function > results in a thing that's not really legible,

[Intel-gfx] [PATCH 04/14] drm/i915: Clean up the .get_cdclk() assignment if ladder

2016-12-19 Thread ville . syrjala
From: Ville Syrjälä Let's clean up the mess we have in the if ladder that assigns the .get_cdclk() hooks. The grouping of the platforms by the function results in a thing that's not really legible, so let's do it the other way around and order the if ladder by

[Intel-gfx] [PATCH 04/14] drm/i915: Clean up the .get_cdclk() assignment if ladder

2016-12-19 Thread ville . syrjala
From: Ville Syrjälä Let's clean up the mess we have in the if ladder that assigns the .get_cdclk() hooks. The grouping of the platforms by the function results in a thing that's not really legible, so let's do it the other way around and order the if ladder by