All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Vivi, Rodrigo" <rodrigo.vivi@intel.com>
To: "daniel@ffwll.ch" <daniel@ffwll.ch>,
	"jani.nikula@linux.intel.com" <jani.nikula@linux.intel.com>
Cc: "intel-gfx@lists.freedesktop.org" <intel-gfx@lists.freedesktop.org>
Subject: Re: [PATCH] drm/i915: Separate cherryview from	valleyview
Date: Fri, 4 Dec 2015 16:15:27 +0000	[thread overview]
Message-ID: <1449245746.1387.208.camel@intel.com> (raw)
In-Reply-To: <20151204150554.GT10243@phenom.ffwll.local>

On Fri, 2015-12-04 at 16:05 +0100, Daniel Vetter wrote:
> On Wed, Dec 02, 2015 at 02:30:28PM +0200, Jani Nikula wrote:
> > On Wed, 02 Dec 2015, Ville Syrjälä <ville.syrjala@linux.intel.com> 
> > wrote:
> > > On Tue, Dec 01, 2015 at 05:11:40PM -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 to some confusion.
> > > > 
> > > > This patch changes .is_valleyview to .is_cherryview in the 
> > > > cherryview
> > > > device info structure and defines the HAS_GEN7_LP_FEATURES 
> > > > macro.
> > > > Then where appropriate, instances of IS_VALLEYVIEW are replaced 
> > > > with
> > > > HAS_GEN7_LP_FEATURES to test for either a valleyview or a 
> > > > cherryview
> > > > device.
> > > 
> > > I don't like the name of the macro. Most of the shared bits are 
> > > display
> > > related, so we could have something like HAS_VLV_DISPLAY. For the 
> > > rest,
> > > I think we could just test IS_VLV||IS_CHV explicitly. Unless 
> > > someone
> > > can come up with a better name that covers everything?
> > 
> > Definitely NAK on HAS_GEN7_LP_FEATURES.
> > 
> > HAS_VLV_DISPLAY would be a subset of HAS_GMCH_DISPLAY, which I 
> > guess
> > wouldn't be that bad... unless someone starts using that for a 
> > VLV||CHV
> > shorthand in non-display code.
> > 
> > I think I might just go for the verbose (IS_VALLEYVIEW || 
> > IS_CHERRYVIEW)
> > all around. Especially since we've been brainwashed with the old 
> > vlv is
> > both vlv and chv code.
> 
> HAS_GMCH_DISPLAY is what I've generally used, since usually you have 
> a
> INTEL_INFO()->gen >= 5 test already somewhere. If we want to make 
> this
> more explicit the proper name for vlv is BAYTRAIL, and for truely byt
> specific stuff we've named things byt_. So what about Defining an
> IS_BAYTRAIL instead for the cases where it's not vlv || chv.

Baytrail is the platform name with the Valleyview graphics. Than we
would have Cherry Trail and/or Braswell for Cherryview graphics and
Apollo Lake for Broxton. So I would vote to stay with Valleyview,
Cherryview and Broxton only.

> 
> And what's the benefit of all this churn?

Organize and prepare the code for future platforms. 
Avoid more confusion like we had on IS_SKYLAKE x IS_KABYLAKE.
Make things more easy and clear if we decide to add .is_atom_lp on
these platforms definition.

> -Daniel
> 
> > 
> > BR,
> > Jani.
> > 
> > > 
> > > > 
> > > > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> > > > Signed-off-by: Wayne Boyer <wayne.boyer@intel.com>
> > > > ---
> > > >  drivers/gpu/drm/i915/i915_debugfs.c     | 14 ++++-----
> > > >  drivers/gpu/drm/i915/i915_dma.c         |  8 ++---
> > > >  drivers/gpu/drm/i915/i915_drv.c         | 10 +++---
> > > >  drivers/gpu/drm/i915/i915_drv.h         | 19 ++++++++----
> > > >  drivers/gpu/drm/i915/i915_gem.c         |  4 +--
> > > >  drivers/gpu/drm/i915/i915_gem_context.c |  2 +-
> > > >  drivers/gpu/drm/i915/i915_gem_gtt.c     |  2 +-
> > > >  drivers/gpu/drm/i915/i915_gpu_error.c   |  6 ++--
> > > >  drivers/gpu/drm/i915/i915_irq.c         |  8 ++---
> > > >  drivers/gpu/drm/i915/i915_suspend.c     |  4 +--
> > > >  drivers/gpu/drm/i915/i915_sysfs.c       | 10 +++---
> > > >  drivers/gpu/drm/i915/intel_audio.c      |  6 ++--
> > > >  drivers/gpu/drm/i915/intel_crt.c        |  4 +--
> > > >  drivers/gpu/drm/i915/intel_display.c    | 54 ++++++++++++++++-
> > > > ----------------
> > > >  drivers/gpu/drm/i915/intel_dp.c         | 38 +++++++++++------
> > > > ------
> > > >  drivers/gpu/drm/i915/intel_dsi.c        | 13 ++++----
> > > >  drivers/gpu/drm/i915/intel_dsi_pll.c    |  6 ++--
> > > >  drivers/gpu/drm/i915/intel_hdmi.c       |  4 +--
> > > >  drivers/gpu/drm/i915/intel_hotplug.c    |  2 +-
> > > >  drivers/gpu/drm/i915/intel_i2c.c        |  2 +-
> > > >  drivers/gpu/drm/i915/intel_panel.c      |  2 +-
> > > >  drivers/gpu/drm/i915/intel_pm.c         |  8 ++---
> > > >  drivers/gpu/drm/i915/intel_psr.c        |  4 +--
> > > >  drivers/gpu/drm/i915/intel_sprite.c     |  4 +--
> > > >  drivers/gpu/drm/i915/intel_uncore.c     |  6 ++--
> > > >  25 files changed, 123 insertions(+), 117 deletions(-)
> > > > 
> > > > diff --git a/drivers/gpu/drm/i915/i915_debugfs.c 
> > > > b/drivers/gpu/drm/i915/i915_debugfs.c
> > > > index bfd57fb..a2d50da 100644
> > > > --- a/drivers/gpu/drm/i915/i915_debugfs.c
> > > > +++ b/drivers/gpu/drm/i915/i915_debugfs.c
> > > > @@ -862,7 +862,7 @@ static int i915_interrupt_info(struct 
> > > > seq_file *m, void *data)
> > > >  			   I915_READ(GEN8_PCU_IIR));
> > > >  		seq_printf(m, "PCU interrupt enable:\t%08x\n",
> > > >  			   I915_READ(GEN8_PCU_IER));
> > > > -	} else if (IS_VALLEYVIEW(dev)) {
> > > > +	} else if (HAS_GEN7_LP_FEATURES(dev)) {
> > > 
> > > We alredy test for CHV earlier, so this should remain as is.
> > > 
> > > >  		seq_printf(m, "Display IER:\t%08x\n",
> > > >  			   I915_READ(VLV_IER));
> > > >  		seq_printf(m, "Display IIR:\t%08x\n",
> > > > @@ -1284,7 +1284,7 @@ static int i915_frequency_info(struct 
> > > > seq_file *m, void *unused)
> > > >  		seq_printf(m,
> > > >  			   "efficient (RPe) frequency: %d 
> > > > MHz\n",
> > > >  			   intel_gpu_freq(dev_priv, dev_priv
> > > > ->rps.efficient_freq));
> > > > -	} else if (IS_VALLEYVIEW(dev)) {
> > > > +	} else if (HAS_GEN7_LP_FEATURES(dev)) {
> > > 
> > > And for this you would also need to change the earlier condition 
> > > in the
> > > previous 'else if' branch. But I think it would be clearner to 
> > > hoist
> > > the VLV/CHV code to sit before the gen6+ code, and then the gen6+
> > > condition can be simplified significantly.
> > > 
> > > >  		u32 freq_sts;
> > > >  
> > > >  		mutex_lock(&dev_priv->rps.hw_lock);
> > > > @@ -1602,7 +1602,7 @@ static int i915_drpc_info(struct seq_file 
> > > > *m, void *unused)
> > > >  	struct drm_info_node *node = m->private;
> > > >  	struct drm_device *dev = node->minor->dev;
> > > >  
> > > > -	if (IS_VALLEYVIEW(dev))
> > > > +	if (HAS_GEN7_LP_FEATURES(dev))
> > > >  		return vlv_drpc_info(m);
> > > >  	else if (INTEL_INFO(dev)->gen >= 6)
> > > >  		return gen6_drpc_info(m);
> > > > @@ -1743,7 +1743,7 @@ static int i915_sr_status(struct seq_file 
> > > > *m, void *unused)
> > > >  		sr_enabled = I915_READ(INSTPM) & 
> > > > INSTPM_SELF_EN;
> > > >  	else if (IS_PINEVIEW(dev))
> > > >  		sr_enabled = I915_READ(DSPFW3) & 
> > > > PINEVIEW_SELF_REFRESH_EN;
> > > > -	else if (IS_VALLEYVIEW(dev))
> > > > +	else if (HAS_GEN7_LP_FEATURES(dev))
> > > >  		sr_enabled = I915_READ(FW_BLC_SELF_VLV) & 
> > > > FW_CSPWRDWNEN;
> > > >  
> > > >  	intel_runtime_pm_put(dev_priv);
> > > > @@ -3983,7 +3983,7 @@ static int pipe_crc_set_source(struct 
> > > > drm_device *dev, enum pipe pipe,
> > > >  		ret = i8xx_pipe_crc_ctl_reg(&source, &val);
> > > >  	else if (INTEL_INFO(dev)->gen < 5)
> > > >  		ret = i9xx_pipe_crc_ctl_reg(dev, pipe, 
> > > > &source, &val);
> > > > -	else if (IS_VALLEYVIEW(dev))
> > > > +	else if (HAS_GEN7_LP_FEATURES(dev))
> > > >  		ret = vlv_pipe_crc_ctl_reg(dev, pipe, &source, 
> > > > &val);
> > > >  	else if (IS_GEN5(dev) || IS_GEN6(dev))
> > > >  		ret = ilk_pipe_crc_ctl_reg(&source, &val);
> > > > @@ -4052,7 +4052,7 @@ static int pipe_crc_set_source(struct 
> > > > drm_device *dev, enum pipe pipe,
> > > >  
> > > >  		if (IS_G4X(dev))
> > > >  			g4x_undo_pipe_scramble_reset(dev, 
> > > > pipe);
> > > > -		else if (IS_VALLEYVIEW(dev))
> > > > +		else if (HAS_GEN7_LP_FEATURES(dev))
> > > >  			vlv_undo_pipe_scramble_reset(dev, 
> > > > pipe);
> > > >  		else if (IS_HASWELL(dev) && pipe == PIPE_A)
> > > >  			hsw_trans_edp_pipe_A_crc_wa(dev, 
> > > > false);
> > > > @@ -4442,7 +4442,7 @@ static void wm_latency_show(struct 
> > > > seq_file *m, const uint16_t wm[8])
> > > >  		 * - WM1+ latency values in 0.5us units
> > > >  		 * - latencies are in us on gen9/vlv/chv
> > > >  		 */
> > > > -		if (INTEL_INFO(dev)->gen >= 9 || 
> > > > IS_VALLEYVIEW(dev))
> > > > +		if (INTEL_INFO(dev)->gen >= 9 || 
> > > > HAS_GEN7_LP_FEATURES(dev))
> > > >  			latency *= 10;
> > > >  		else if (level > 0)
> > > >  			latency *= 5;
> > > > diff --git a/drivers/gpu/drm/i915/i915_dma.c 
> > > > b/drivers/gpu/drm/i915/i915_dma.c
> > > > index a81c766..e24aa65 100644
> > > > --- a/drivers/gpu/drm/i915/i915_dma.c
> > > > +++ b/drivers/gpu/drm/i915/i915_dma.c
> > > > @@ -256,7 +256,7 @@ intel_setup_mchbar(struct drm_device *dev)
> > > >  	u32 temp;
> > > >  	bool enabled;
> > > >  
> > > > -	if (IS_VALLEYVIEW(dev))
> > > > +	if (HAS_GEN7_LP_FEATURES(dev))
> > > >  		return;
> > > >  
> > > >  	dev_priv->mchbar_need_disable = false;
> > > > @@ -779,7 +779,7 @@ static void 
> > > > intel_device_info_runtime_init(struct drm_device *dev)
> > > >  		info->num_sprites[PIPE_A] = 2;
> > > >  		info->num_sprites[PIPE_B] = 2;
> > > >  		info->num_sprites[PIPE_C] = 1;
> > > > -	} else if (IS_VALLEYVIEW(dev))
> > > > +	} else if (HAS_GEN7_LP_FEATURES(dev))
> > > >  		for_each_pipe(dev_priv, pipe)
> > > >  			info->num_sprites[pipe] = 2;
> > > >  	else
> > > > @@ -791,7 +791,7 @@ static void 
> > > > intel_device_info_runtime_init(struct drm_device *dev)
> > > >  		info->num_pipes = 0;
> > > >  	} else if (info->num_pipes > 0 &&
> > > >  		   (INTEL_INFO(dev)->gen == 7 || 
> > > > INTEL_INFO(dev)->gen == 8) &&
> > > > -		   !IS_VALLEYVIEW(dev)) {
> > > > +		   !HAS_GEN7_LP_FEATURES(dev)) {
> > > 
> > > That should probably check for 'HAS_PCH_SPLIT && (GEN7 || GEN8)', 
> > > since
> > > it's reading SFUSE_STRAP which lives in the PCH.
> > > 
> > > >  		u32 fuse_strap = I915_READ(FUSE_STRAP);
> > > >  		u32 sfuse_strap = I915_READ(SFUSE_STRAP);
> > > >  
> > > > @@ -836,7 +836,7 @@ static void 
> > > > intel_device_info_runtime_init(struct drm_device *dev)
> > > >  
> > > >  static void intel_init_dpio(struct drm_i915_private *dev_priv)
> > > >  {
> > > > -	if (!IS_VALLEYVIEW(dev_priv))
> > > > +	if (!HAS_GEN7_LP_FEATURES(dev_priv))
> > > >  		return;
> > > >  
> > > >  	/*
> > > > diff --git a/drivers/gpu/drm/i915/i915_drv.c 
> > > > b/drivers/gpu/drm/i915/i915_drv.c
> > > > index 5f8b0ff..ac54b68 100644
> > > > --- a/drivers/gpu/drm/i915/i915_drv.c
> > > > +++ b/drivers/gpu/drm/i915/i915_drv.c
> > > > @@ -311,7 +311,7 @@ static const struct intel_device_info 
> > > > intel_cherryview_info = {
> > > >  	.gen = 8, .num_pipes = 3,
> > > >  	.need_gfx_hws = 1, .has_hotplug = 1,
> > > >  	.ring_mask = RENDER_RING | BSD_RING | BLT_RING | 
> > > > VEBOX_RING,
> > > > -	.is_valleyview = 1,
> > > > +	.is_cherryview = 1,
> > > >  	.display_mmio_offset = VLV_DISPLAY_BASE,
> > > >  	GEN_CHV_PIPEOFFSETS,
> > > >  	CURSOR_OFFSETS,
> > > > @@ -800,7 +800,7 @@ static int i915_drm_resume_early(struct 
> > > > drm_device *dev)
> > > >  
> > > >  	pci_set_master(dev->pdev);
> > > >  
> > > > -	if (IS_VALLEYVIEW(dev_priv))
> > > > +	if (HAS_GEN7_LP_FEATURES(dev_priv))
> > > >  		ret = vlv_resume_prepare(dev_priv, false);
> > > >  	if (ret)
> > > >  		DRM_ERROR("Resume prepare failed: %d, 
> > > > continuing anyway\n",
> > > > @@ -1530,7 +1530,7 @@ static int intel_runtime_resume(struct 
> > > > device *device)
> > > >  		ret = bxt_resume_prepare(dev_priv);
> > > >  	else if (IS_HASWELL(dev_priv) || 
> > > > IS_BROADWELL(dev_priv))
> > > >  		hsw_disable_pc8(dev_priv);
> > > > -	else if (IS_VALLEYVIEW(dev_priv))
> > > > +	else if (HAS_GEN7_LP_FEATURES(dev_priv))
> > > >  		ret = vlv_resume_prepare(dev_priv, true);
> > > >  
> > > >  	/*
> > > > @@ -1547,7 +1547,7 @@ static int intel_runtime_resume(struct 
> > > > device *device)
> > > >  	 * power well, so hpd is reinitialized from there. For
> > > >  	 * everyone else do it here.
> > > >  	 */
> > > > -	if (!IS_VALLEYVIEW(dev_priv))
> > > > +	if (!HAS_GEN7_LP_FEATURES(dev_priv))
> > > >  		intel_hpd_init(dev_priv);
> > > >  
> > > >  	intel_enable_gt_powersave(dev);
> > > > @@ -1572,7 +1572,7 @@ static int intel_suspend_complete(struct 
> > > > drm_i915_private *dev_priv)
> > > >  		ret = bxt_suspend_complete(dev_priv);
> > > >  	else if (IS_HASWELL(dev_priv) || 
> > > > IS_BROADWELL(dev_priv))
> > > >  		ret = hsw_suspend_complete(dev_priv);
> > > > -	else if (IS_VALLEYVIEW(dev_priv))
> > > > +	else if (HAS_GEN7_LP_FEATURES(dev_priv))
> > > >  		ret = vlv_suspend_complete(dev_priv);
> > > >  	else
> > > >  		ret = 0;
> > > > diff --git a/drivers/gpu/drm/i915/i915_drv.h 
> > > > b/drivers/gpu/drm/i915/i915_drv.h
> > > > index bc865e23..acb5e8b 100644
> > > > --- a/drivers/gpu/drm/i915/i915_drv.h
> > > > +++ b/drivers/gpu/drm/i915/i915_drv.h
> > > > @@ -763,6 +763,7 @@ struct intel_csr {
> > > >  	func(is_crestline) sep \
> > > >  	func(is_ivybridge) sep \
> > > >  	func(is_valleyview) sep \
> > > > +	func(is_cherryview) sep \
> > > >  	func(is_haswell) sep \
> > > >  	func(is_skylake) sep \
> > > >  	func(is_broxton) sep \
> > > > @@ -2456,9 +2457,9 @@ struct drm_i915_cmd_table {
> > > >  				 INTEL_DEVID(dev) == 0x0152 || 
> > > > \
> > > >  				 INTEL_DEVID(dev) == 0x015a)
> > > >  #define IS_VALLEYVIEW(dev)	(INTEL_INFO(dev)
> > > > ->is_valleyview)
> > > > -#define IS_CHERRYVIEW(dev)	(INTEL_INFO(dev)
> > > > ->is_valleyview && IS_GEN8(dev))
> > > > +#define IS_CHERRYVIEW(dev)	(INTEL_INFO(dev)
> > > > ->is_cherryview)
> > > >  #define IS_HASWELL(dev)	(INTEL_INFO(dev)->is_haswell)
> > > > -#define IS_BROADWELL(dev)	(!INTEL_INFO(dev)
> > > > ->is_valleyview && IS_GEN8(dev))
> > > > +#define IS_BROADWELL(dev)	(!INTEL_INFO(dev)
> > > > ->is_cherryview && IS_GEN8(dev))
> > > >  #define IS_SKYLAKE(dev)	(INTEL_INFO(dev)->is_skylake)
> > > >  #define IS_BROXTON(dev)		(INTEL_INFO(dev)
> > > > ->is_broxton)
> > > >  #define IS_KABYLAKE(dev)	(INTEL_INFO(dev)->is_kabylake)
> > > > @@ -2585,7 +2586,8 @@ struct drm_i915_cmd_table {
> > > >  				 IS_SKYLAKE(dev) || 
> > > > IS_KABYLAKE(dev))
> > > >  #define HAS_RUNTIME_PM(dev)	(IS_GEN6(dev) || 
> > > > IS_HASWELL(dev) || \
> > > >  				 IS_BROADWELL(dev) || 
> > > > IS_VALLEYVIEW(dev) || \
> > > > -				 IS_SKYLAKE(dev) || 
> > > > IS_KABYLAKE(dev))
> > > > +				 IS_CHERRYVIEW(dev) || 
> > > > IS_SKYLAKE(dev) || \
> > > > +				 IS_KABYLAKE(dev))
> > > >  #define HAS_RC6(dev)		(INTEL_INFO(dev)->gen >= 
> > > > 6)
> > > >  #define HAS_RC6p(dev)		(INTEL_INFO(dev)->gen == 
> > > > 6 || IS_IVYBRIDGE(dev))
> > > >  
> > > > @@ -2598,7 +2600,11 @@ struct drm_i915_cmd_table {
> > > >  				    INTEL_INFO(dev)->gen >= 8)
> > > >  
> > > >  #define HAS_CORE_RING_FREQ(dev)	(INTEL_INFO(dev)->gen 
> > > > >= 6 && \
> > > > -				 !IS_VALLEYVIEW(dev) && 
> > > > !IS_BROXTON(dev))
> > > > +				 !IS_VALLEYVIEW(dev) && 
> > > > !IS_CHERRYVIEW(dev) && \
> > > > +				 !IS_BROXTON(dev))
> > > > +
> > > > +#define HAS_GEN7_LP_FEATURES(dev) (INTEL_INFO(dev)
> > > > ->is_valleyview || \
> > > > +				   INTEL_INFO(dev)
> > > > ->is_cherryview)
> > > >  
> > > >  #define INTEL_PCH_DEVICE_ID_MASK		0xff00
> > > >  #define INTEL_PCH_IBX_DEVICE_ID_TYPE		0x3b00
> > > > @@ -2620,7 +2626,8 @@ struct drm_i915_cmd_table {
> > > >  #define HAS_PCH_NOP(dev) (INTEL_PCH_TYPE(dev) == PCH_NOP)
> > > >  #define HAS_PCH_SPLIT(dev) (INTEL_PCH_TYPE(dev) != PCH_NONE)
> > > >  
> > > > -#define HAS_GMCH_DISPLAY(dev) (INTEL_INFO(dev)->gen < 5 || 
> > > > IS_VALLEYVIEW(dev))
> > > > +#define HAS_GMCH_DISPLAY(dev) (INTEL_INFO(dev)->gen < 5 || \
> > > > +			       HAS_GEN7_LP_FEATURES(dev))
> > > >  
> > > >  /* DPF == dynamic parity feature */
> > > >  #define HAS_L3_DPF(dev) (IS_IVYBRIDGE(dev) || IS_HASWELL(dev))
> > > > @@ -3511,7 +3518,7 @@ __raw_write(64, q)
> > > >  
> > > >  static inline i915_reg_t i915_vgacntrl_reg(struct drm_device 
> > > > *dev)
> > > >  {
> > > > -	if (IS_VALLEYVIEW(dev))
> > > > +	if (HAS_GEN7_LP_FEATURES(dev))
> > > >  		return VLV_VGACNTRL;
> > > >  	else if (INTEL_INFO(dev)->gen >= 5)
> > > >  		return CPU_VGACNTRL;
> > > > diff --git a/drivers/gpu/drm/i915/i915_gem.c 
> > > > b/drivers/gpu/drm/i915/i915_gem.c
> > > > index a6997a8..15be9ea 100644
> > > > --- a/drivers/gpu/drm/i915/i915_gem.c
> > > > +++ b/drivers/gpu/drm/i915/i915_gem.c
> > > > @@ -4838,7 +4838,7 @@ int i915_gem_init(struct drm_device *dev)
> > > >  
> > > >  	mutex_lock(&dev->struct_mutex);
> > > >  
> > > > -	if (IS_VALLEYVIEW(dev)) {
> > > > +	if (HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		/* VLVA0 (potential hack), BIOS isn't actually 
> > > > waking us */
> > > >  		I915_WRITE(VLV_GTLC_WAKE_CTRL, 
> > > > VLV_GTLC_ALLOWWAKEREQ);
> > > >  		if (wait_for((I915_READ(VLV_GTLC_PW_STATUS) &
> > > 
> > > We still have that code? Should probably just nuke it.
> > > 
> > > > @@ -4963,7 +4963,7 @@ i915_gem_load(struct drm_device *dev)
> > > >  
> > > >  	dev_priv->relative_constants_mode = 
> > > > I915_EXEC_CONSTANTS_REL_GENERAL;
> > > >  
> > > > -	if (INTEL_INFO(dev)->gen >= 7 && !IS_VALLEYVIEW(dev))
> > > > +	if (INTEL_INFO(dev)->gen >= 7 && 
> > > > !HAS_GEN7_LP_FEATURES(dev))
> > > >  		dev_priv->num_fence_regs = 32;
> > > >  	else if (INTEL_INFO(dev)->gen >= 4 || IS_I945G(dev) || 
> > > > IS_I945GM(dev) || IS_G33(dev))
> > > >  		dev_priv->num_fence_regs = 16;
> > > > diff --git a/drivers/gpu/drm/i915/i915_gem_context.c 
> > > > b/drivers/gpu/drm/i915/i915_gem_context.c
> > > > index 43761c5..476fee0 100644
> > > > --- a/drivers/gpu/drm/i915/i915_gem_context.c
> > > > +++ b/drivers/gpu/drm/i915/i915_gem_context.c
> > > > @@ -190,7 +190,7 @@ i915_gem_alloc_context_obj(struct 
> > > > drm_device *dev, size_t size)
> > > >  	 * would make the object snooped which might have a
> > > >  	 * negative performance impact.
> > > >  	 */
> > > > -	if (INTEL_INFO(dev)->gen >= 7 && !IS_VALLEYVIEW(dev)) 
> > > > {
> > > > +	if (INTEL_INFO(dev)->gen >= 7 && 
> > > > !HAS_GEN7_LP_FEATURES(dev)) {
> > > 
> > > I think we should just make this IS_IVYBRIDGE since later 
> > > platforms
> > > don't have any L3 control bits in the PTE anyway.
> > > 
> > > >  		ret = i915_gem_object_set_cache_level(obj, 
> > > > I915_CACHE_L3_LLC);
> > > >  		/* Failure shouldn't ever happen this early */
> > > >  		if (WARN_ON(ret)) {
> > > > diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c 
> > > > b/drivers/gpu/drm/i915/i915_gem_gtt.c
> > > > index 1f7e6b9..ee80e26 100644
> > > > --- a/drivers/gpu/drm/i915/i915_gem_gtt.c
> > > > +++ b/drivers/gpu/drm/i915/i915_gem_gtt.c
> > > > @@ -3125,7 +3125,7 @@ int i915_gem_gtt_init(struct drm_device 
> > > > *dev)
> > > >  			gtt->base.pte_encode = 
> > > > iris_pte_encode;
> > > >  		else if (IS_HASWELL(dev))
> > > >  			gtt->base.pte_encode = hsw_pte_encode;
> > > > -		else if (IS_VALLEYVIEW(dev))
> > > > +		else if (HAS_GEN7_LP_FEATURES(dev))
> > > 
> > > No need to change.
> > > 
> > > >  			gtt->base.pte_encode = byt_pte_encode;
> > > >  		else if (INTEL_INFO(dev)->gen >= 7)
> > > >  			gtt->base.pte_encode = ivb_pte_encode;
> > > > diff --git a/drivers/gpu/drm/i915/i915_gpu_error.c 
> > > > b/drivers/gpu/drm/i915/i915_gpu_error.c
> > > > index 06ca408..e92be5f 100644
> > > > --- a/drivers/gpu/drm/i915/i915_gpu_error.c
> > > > +++ b/drivers/gpu/drm/i915/i915_gpu_error.c
> > > > @@ -383,7 +383,7 @@ int i915_error_state_to_str(struct 
> > > > drm_i915_error_state_buf *m,
> > > >  		for (i = 0; i < 4; i++)
> > > >  			err_printf(m, "GTIER gt %d: 0x%08x\n", 
> > > > i,
> > > >  				   error->gtier[i]);
> > > > -	} else if (HAS_PCH_SPLIT(dev) || IS_VALLEYVIEW(dev))
> > > > +	} else if (HAS_PCH_SPLIT(dev) || 
> > > > HAS_GEN7_LP_FEATURES(dev))
> > > 
> > > No need to change. CHV takes the gen>=8 branch earlier.
> > > 
> > > >  		err_printf(m, "GTIER: 0x%08x\n", error
> > > > ->gtier[0]);
> > > >  	err_printf(m, "PGTBL_ER: 0x%08x\n", error->pgtbl_er);
> > > >  	err_printf(m, "FORCEWAKE: 0x%08x\n", error
> > > > ->forcewake);
> > > > @@ -1208,7 +1208,7 @@ static void i915_capture_reg_state(struct 
> > > > drm_i915_private *dev_priv,
> > > >  	 */
> > > >  
> > > >  	/* 1: Registers specific to a single generation */
> > > > -	if (IS_VALLEYVIEW(dev)) {
> > > > +	if (HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		error->gtier[0] = I915_READ(GTIER);
> > > >  		error->ier = I915_READ(VLV_IER);
> > > >  		error->forcewake = 
> > > > I915_READ_FW(FORCEWAKE_VLV);
> > > 
> > > We want to capture the forcewake on CHV, but not the GTIER. Error
> > > capture on VLV/CHV is pretty decenrly broken currently. I have a 
> > > branch
> > > somewhere that tries to fix it up, I suppose I should dig it up 
> > > and post
> > > it.
> > > 
> > > > @@ -1257,7 +1257,7 @@ static void i915_capture_reg_state(struct 
> > > > drm_i915_private *dev_priv,
> > > >  		error->gtier[0] = I915_READ(GTIER);
> > > >  	} else if (IS_GEN2(dev)) {
> > > >  		error->ier = I915_READ16(IER);
> > > > -	} else if (!IS_VALLEYVIEW(dev)) {
> > > > +	} else if (!HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		error->ier = I915_READ(IER);
> > > >  	}
> > > >  	error->eir = I915_READ(EIR);
> > > > diff --git a/drivers/gpu/drm/i915/i915_irq.c 
> > > > b/drivers/gpu/drm/i915/i915_irq.c
> > > > index e88d692..9201e25 100644
> > > > --- a/drivers/gpu/drm/i915/i915_irq.c
> > > > +++ b/drivers/gpu/drm/i915/i915_irq.c
> > > > @@ -581,7 +581,7 @@ i915_enable_pipestat(struct 
> > > > drm_i915_private *dev_priv, enum pipe pipe,
> > > >  {
> > > >  	u32 enable_mask;
> > > >  
> > > > -	if (IS_VALLEYVIEW(dev_priv->dev))
> > > > +	if (HAS_GEN7_LP_FEATURES(dev_priv->dev))
> > > >  		enable_mask = 
> > > > vlv_get_pipestat_enable_mask(dev_priv->dev,
> > > >  							  
> > > >  status_mask);
> > > >  	else
> > > > @@ -595,7 +595,7 @@ i915_disable_pipestat(struct 
> > > > drm_i915_private *dev_priv, enum pipe pipe,
> > > >  {
> > > >  	u32 enable_mask;
> > > >  
> > > > -	if (IS_VALLEYVIEW(dev_priv->dev))
> > > > +	if (HAS_GEN7_LP_FEATURES(dev_priv->dev))
> > > >  		enable_mask = 
> > > > vlv_get_pipestat_enable_mask(dev_priv->dev,
> > > >  							  
> > > >  status_mask);
> > > >  	else
> > > > @@ -1723,7 +1723,7 @@ static void i9xx_hpd_irq_handler(struct 
> > > > drm_device *dev)
> > > >  	 */
> > > >  	POSTING_READ(PORT_HOTPLUG_STAT);
> > > >  
> > > > -	if (IS_G4X(dev) || IS_VALLEYVIEW(dev)) {
> > > > +	if (IS_G4X(dev) || HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		u32 hotplug_trigger = hotplug_status & 
> > > > HOTPLUG_INT_STATUS_G4X;
> > > >  
> > > >  		if (hotplug_trigger) {
> > > > @@ -4412,7 +4412,7 @@ void intel_irq_init(struct 
> > > > drm_i915_private *dev_priv)
> > > >  	INIT_WORK(&dev_priv->l3_parity.error_work, 
> > > > ivybridge_parity_work);
> > > >  
> > > >  	/* Let's track the enabled rps events */
> > > > -	if (IS_VALLEYVIEW(dev_priv) && 
> > > > !IS_CHERRYVIEW(dev_priv))
> > > > +	if (IS_VALLEYVIEW(dev_priv))
> > > >  		/* WaGsvRC0ResidencyMethod:vlv */
> > > >  		dev_priv->pm_rps_events = 
> > > > GEN6_PM_RP_DOWN_EI_EXPIRED | GEN6_PM_RP_UP_EI_EXPIRED;
> > > >  	else
> > > > diff --git a/drivers/gpu/drm/i915/i915_suspend.c 
> > > > b/drivers/gpu/drm/i915/i915_suspend.c
> > > > index 2d91821..95921fc 100644
> > > > --- a/drivers/gpu/drm/i915/i915_suspend.c
> > > > +++ b/drivers/gpu/drm/i915/i915_suspend.c
> > > > @@ -49,7 +49,7 @@ static void i915_save_display(struct 
> > > > drm_device *dev)
> > > >  		dev_priv->regfile.savePP_ON_DELAYS = 
> > > > I915_READ(PCH_PP_ON_DELAYS);
> > > >  		dev_priv->regfile.savePP_OFF_DELAYS = 
> > > > I915_READ(PCH_PP_OFF_DELAYS);
> > > >  		dev_priv->regfile.savePP_DIVISOR = 
> > > > I915_READ(PCH_PP_DIVISOR);
> > > > -	} else if (!IS_VALLEYVIEW(dev)) {
> > > > +	} else if (!HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		dev_priv->regfile.savePP_CONTROL = 
> > > > I915_READ(PP_CONTROL);
> > > >  		dev_priv->regfile.savePP_ON_DELAYS = 
> > > > I915_READ(PP_ON_DELAYS);
> > > >  		dev_priv->regfile.savePP_OFF_DELAYS = 
> > > > I915_READ(PP_OFF_DELAYS);
> > > > @@ -84,7 +84,7 @@ static void i915_restore_display(struct 
> > > > drm_device *dev)
> > > >  		I915_WRITE(PCH_PP_OFF_DELAYS, dev_priv
> > > > ->regfile.savePP_OFF_DELAYS);
> > > >  		I915_WRITE(PCH_PP_DIVISOR, dev_priv
> > > > ->regfile.savePP_DIVISOR);
> > > >  		I915_WRITE(PCH_PP_CONTROL, dev_priv
> > > > ->regfile.savePP_CONTROL);
> > > > -	} else if (!IS_VALLEYVIEW(dev)) {
> > > > +	} else if (!HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		I915_WRITE(PP_ON_DELAYS, dev_priv
> > > > ->regfile.savePP_ON_DELAYS);
> > > >  		I915_WRITE(PP_OFF_DELAYS, dev_priv
> > > > ->regfile.savePP_OFF_DELAYS);
> > > >  		I915_WRITE(PP_DIVISOR, dev_priv
> > > > ->regfile.savePP_DIVISOR);
> > > > diff --git a/drivers/gpu/drm/i915/i915_sysfs.c 
> > > > b/drivers/gpu/drm/i915/i915_sysfs.c
> > > > index f929c61..dd506e3 100644
> > > > --- a/drivers/gpu/drm/i915/i915_sysfs.c
> > > > +++ b/drivers/gpu/drm/i915/i915_sysfs.c
> > > > @@ -49,7 +49,7 @@ static u32 calc_residency(struct drm_device 
> > > > *dev,
> > > >  	intel_runtime_pm_get(dev_priv);
> > > >  
> > > >  	/* On VLV and CHV, residency time is in CZ units 
> > > > rather than 1.28us */
> > > > -	if (IS_VALLEYVIEW(dev)) {
> > > > +	if (HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		units = 1;
> > > >  		div = dev_priv->czclk_freq;
> > > >  
> > > > @@ -284,7 +284,7 @@ static ssize_t gt_act_freq_mhz_show(struct 
> > > > device *kdev,
> > > >  	intel_runtime_pm_get(dev_priv);
> > > >  
> > > >  	mutex_lock(&dev_priv->rps.hw_lock);
> > > > -	if (IS_VALLEYVIEW(dev_priv->dev)) {
> > > > +	if (HAS_GEN7_LP_FEATURES(dev_priv->dev)) {
> > > >  		u32 freq;
> > > >  		freq = vlv_punit_read(dev_priv, 
> > > > PUNIT_REG_GPU_FREQ_STS);
> > > >  		ret = intel_gpu_freq(dev_priv, (freq >> 8) & 
> > > > 0xff);
> > > > @@ -598,7 +598,7 @@ void i915_setup_sysfs(struct drm_device 
> > > > *dev)
> > > >  		if (ret)
> > > >  			DRM_ERROR("RC6p residency sysfs setup 
> > > > failed\n");
> > > >  	}
> > > > -	if (IS_VALLEYVIEW(dev)) {
> > > > +	if (HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		ret = sysfs_merge_group(&dev->primary->kdev
> > > > ->kobj,
> > > >  					&media_rc6_attr_group)
> > > > ;
> > > >  		if (ret)
> > > > @@ -619,7 +619,7 @@ void i915_setup_sysfs(struct drm_device 
> > > > *dev)
> > > >  	}
> > > >  
> > > >  	ret = 0;
> > > > -	if (IS_VALLEYVIEW(dev))
> > > > +	if (HAS_GEN7_LP_FEATURES(dev))
> > > >  		ret = sysfs_create_files(&dev->primary->kdev
> > > > ->kobj, vlv_attrs);
> > > >  	else if (INTEL_INFO(dev)->gen >= 6)
> > > >  		ret = sysfs_create_files(&dev->primary->kdev
> > > > ->kobj, gen6_attrs);
> > > > @@ -635,7 +635,7 @@ void i915_setup_sysfs(struct drm_device 
> > > > *dev)
> > > >  void i915_teardown_sysfs(struct drm_device *dev)
> > > >  {
> > > >  	sysfs_remove_bin_file(&dev->primary->kdev->kobj, 
> > > > &error_state_attr);
> > > > -	if (IS_VALLEYVIEW(dev))
> > > > +	if (HAS_GEN7_LP_FEATURES(dev))
> > > >  		sysfs_remove_files(&dev->primary->kdev->kobj, 
> > > > vlv_attrs);
> > > >  	else
> > > >  		sysfs_remove_files(&dev->primary->kdev->kobj, 
> > > > gen6_attrs);
> > > > diff --git a/drivers/gpu/drm/i915/intel_audio.c 
> > > > b/drivers/gpu/drm/i915/intel_audio.c
> > > > index 9aa83e7..6734eae 100644
> > > > --- a/drivers/gpu/drm/i915/intel_audio.c
> > > > +++ b/drivers/gpu/drm/i915/intel_audio.c
> > > > @@ -375,7 +375,7 @@ static void ilk_audio_codec_disable(struct 
> > > > intel_encoder *encoder)
> > > >  	if (HAS_PCH_IBX(dev_priv->dev)) {
> > > >  		aud_config = IBX_AUD_CFG(pipe);
> > > >  		aud_cntrl_st2 = IBX_AUD_CNTL_ST2;
> > > > -	} else if (IS_VALLEYVIEW(dev_priv)) {
> > > > +	} else if (HAS_GEN7_LP_FEATURES(dev_priv)) {
> > > >  		aud_config = VLV_AUD_CFG(pipe);
> > > >  		aud_cntrl_st2 = VLV_AUD_CNTL_ST2;
> > > >  	} else {
> > > > @@ -435,7 +435,7 @@ static void ilk_audio_codec_enable(struct 
> > > > drm_connector *connector,
> > > >  		aud_config = IBX_AUD_CFG(pipe);
> > > >  		aud_cntl_st = IBX_AUD_CNTL_ST(pipe);
> > > >  		aud_cntrl_st2 = IBX_AUD_CNTL_ST2;
> > > > -	} else if (IS_VALLEYVIEW(connector->dev)) {
> > > > +	} else if (HAS_GEN7_LP_FEATURES(connector->dev)) {
> > > >  		hdmiw_hdmiedid = VLV_HDMIW_HDMIEDID(pipe);
> > > >  		aud_config = VLV_AUD_CFG(pipe);
> > > >  		aud_cntl_st = VLV_AUD_CNTL_ST(pipe);
> > > > @@ -559,7 +559,7 @@ void intel_init_audio(struct drm_device 
> > > > *dev)
> > > >  	if (IS_G4X(dev)) {
> > > >  		dev_priv->display.audio_codec_enable = 
> > > > g4x_audio_codec_enable;
> > > >  		dev_priv->display.audio_codec_disable = 
> > > > g4x_audio_codec_disable;
> > > > -	} else if (IS_VALLEYVIEW(dev)) {
> > > > +	} else if (HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		dev_priv->display.audio_codec_enable = 
> > > > ilk_audio_codec_enable;
> > > >  		dev_priv->display.audio_codec_disable = 
> > > > ilk_audio_codec_disable;
> > > >  	} else if (IS_HASWELL(dev) || INTEL_INFO(dev)->gen >= 
> > > > 8) {
> > > > diff --git a/drivers/gpu/drm/i915/intel_crt.c 
> > > > b/drivers/gpu/drm/i915/intel_crt.c
> > > > index 12008af..94d1ab2 100644
> > > > --- a/drivers/gpu/drm/i915/intel_crt.c
> > > > +++ b/drivers/gpu/drm/i915/intel_crt.c
> > > > @@ -362,7 +362,7 @@ static bool intel_crt_detect_hotplug(struct 
> > > > drm_connector *connector)
> > > >  	if (HAS_PCH_SPLIT(dev))
> > > >  		return 
> > > > intel_ironlake_crt_detect_hotplug(connector);
> > > >  
> > > > -	if (IS_VALLEYVIEW(dev))
> > > > +	if (HAS_GEN7_LP_FEATURES(dev))
> > > >  		return 
> > > > valleyview_crt_detect_hotplug(connector);
> > > 
> > > There's no crt on CHV, so doesn't need changing.
> > > 
> > > >  
> > > >  	/*
> > > > @@ -786,7 +786,7 @@ void intel_crt_init(struct drm_device *dev)
> > > >  
> > > >  	if (HAS_PCH_SPLIT(dev))
> > > >  		adpa_reg = PCH_ADPA;
> > > > -	else if (IS_VALLEYVIEW(dev))
> > > > +	else if (HAS_GEN7_LP_FEATURES(dev))
> > > >  		adpa_reg = VLV_ADPA;
> > > >  	else
> > > >  		adpa_reg = ADPA;
> > > > diff --git a/drivers/gpu/drm/i915/intel_display.c 
> > > > b/drivers/gpu/drm/i915/intel_display.c
> > > > index aac17011..8e51b83 100644
> > > > --- a/drivers/gpu/drm/i915/intel_display.c
> > > > +++ b/drivers/gpu/drm/i915/intel_display.c
> > > > @@ -185,7 +185,7 @@ int intel_hrawclk(struct drm_device *dev)
> > > >  	uint32_t clkcfg;
> > > >  
> > > >  	/* There is no CLKCFG reg in Valleyview. VLV hrawclk 
> > > > is 200 MHz */
> > > > -	if (IS_VALLEYVIEW(dev))
> > > > +	if (HAS_GEN7_LP_FEATURES(dev))
> > > >  		return 200;
> > > >  
> > > >  	clkcfg = I915_READ(CLKCFG);
> > > > @@ -213,7 +213,7 @@ int intel_hrawclk(struct drm_device *dev)
> > > >  
> > > >  static void intel_update_czclk(struct drm_i915_private 
> > > > *dev_priv)
> > > >  {
> > > > -	if (!IS_VALLEYVIEW(dev_priv))
> > > > +	if (!HAS_GEN7_LP_FEATURES(dev_priv))
> > > >  		return;
> > > >  
> > > >  	dev_priv->czclk_freq = 
> > > > vlv_get_cck_clock_hpll(dev_priv, "czclk",
> > > > @@ -714,11 +714,11 @@ static bool intel_PLL_is_valid(struct 
> > > > drm_device *dev,
> > > >  	if (clock->m1  < limit->m1.min  || limit->m1.max  < 
> > > > clock->m1)
> > > >  		INTELPllInvalid("m1 out of range\n");
> > > >  
> > > > -	if (!IS_PINEVIEW(dev) && !IS_VALLEYVIEW(dev) && 
> > > > !IS_BROXTON(dev))
> > > > +	if (!IS_PINEVIEW(dev) && !HAS_GEN7_LP_FEATURES(dev) && 
> > > > !IS_BROXTON(dev))
> > > >  		if (clock->m1 <= clock->m2)
> > > >  			INTELPllInvalid("m1 <= m2\n");
> > > >  
> > > > -	if (!IS_VALLEYVIEW(dev) && !IS_BROXTON(dev)) {
> > > > +	if (!HAS_GEN7_LP_FEATURES(dev) && !IS_BROXTON(dev)) {
> > > >  		if (clock->p < limit->p.min || limit->p.max < 
> > > > clock->p)
> > > >  			INTELPllInvalid("p out of range\n");
> > > >  		if (clock->m < limit->m.min || limit->m.max < 
> > > > clock->m)
> > > > @@ -1303,7 +1303,7 @@ void assert_panel_unlocked(struct 
> > > > drm_i915_private *dev_priv,
> > > >  		    I915_READ(PCH_LVDS) & LVDS_PIPEB_SELECT)
> > > >  			panel_pipe = PIPE_B;
> > > >  		/* XXX: else fix for eDP */
> > > > -	} else if (IS_VALLEYVIEW(dev)) {
> > > > +	} else if (HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		/* presumably write lock depends on pipe, not 
> > > > port select */
> > > >  		pp_reg = VLV_PIPE_PP_CONTROL(pipe);
> > > >  		panel_pipe = pipe;
> > > > @@ -1421,7 +1421,7 @@ static void 
> > > > assert_sprites_disabled(struct drm_i915_private *dev_priv,
> > > >  			     "plane %d assertion failure, 
> > > > should be off on pipe %c but is still active\n",
> > > >  			     sprite, pipe_name(pipe));
> > > >  		}
> > > > -	} else if (IS_VALLEYVIEW(dev)) {
> > > > +	} else if (HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		for_each_sprite(dev_priv, pipe, sprite) {
> > > >  			u32 val = I915_READ(SPCNTR(pipe, 
> > > > sprite));
> > > >  			I915_STATE_WARN(val & SP_ENABLE,
> > > > @@ -1605,7 +1605,7 @@ static void vlv_enable_pll(struct 
> > > > intel_crtc *crtc,
> > > >  	assert_pipe_disabled(dev_priv, crtc->pipe);
> > > >  
> > > >  	/* No really, not for ILK+ */
> > > > -	BUG_ON(!IS_VALLEYVIEW(dev_priv->dev));
> > > > +	BUG_ON(!HAS_GEN7_LP_FEATURES(dev_priv->dev));
> > > 
> > > This is VLV specific, so no need to change. Could just kill these
> > > BUG_ONs though...
> > > 
> > > >  
> > > >  	/* PLL is protected by panel, make sure we can write 
> > > > it */
> > > >  	if (IS_MOBILE(dev_priv->dev))
> > > > @@ -2317,7 +2317,7 @@ static unsigned int 
> > > > intel_linear_alignment(struct drm_i915_private *dev_priv)
> > > >  	if (INTEL_INFO(dev_priv)->gen >= 9)
> > > >  		return 256 * 1024;
> > > >  	else if (IS_BROADWATER(dev_priv) || 
> > > > IS_CRESTLINE(dev_priv) ||
> > > > -		 IS_VALLEYVIEW(dev_priv))
> > > > +		 HAS_GEN7_LP_FEATURES(dev_priv))
> > > >  		return 128 * 1024;
> > > >  	else if (INTEL_INFO(dev_priv)->gen >= 4)
> > > >  		return 4 * 1024;
> > > > @@ -5448,7 +5448,7 @@ static void intel_update_cdclk(struct 
> > > > drm_device *dev)
> > > >  	 * BSpec erroneously claims we should aim for 4MHz, 
> > > > but
> > > >  	 * in fact 1MHz is the correct frequency.
> > > >  	 */
> > > > -	if (IS_VALLEYVIEW(dev)) {
> > > > +	if (HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		/*
> > > >  		 * Program the gmbus_freq based on the cdclk 
> > > > frequency.
> > > >  		 * BSpec erroneously claims we should aim for 
> > > > 4MHz, but
> > > > @@ -7169,7 +7169,7 @@ static int i9xx_get_refclk(const struct 
> > > > intel_crtc_state *crtc_state,
> > > >  
> > > >  	WARN_ON(!crtc_state->base.state);
> > > >  
> > > > -	if (IS_VALLEYVIEW(dev) || IS_BROXTON(dev)) {
> > > > +	if (HAS_GEN7_LP_FEATURES(dev) || IS_BROXTON(dev)) {
> > > >  		refclk = 100000;
> > > >  	} else if (intel_pipe_will_have_type(crtc_state, 
> > > > INTEL_OUTPUT_LVDS) &&
> > > >  	    intel_panel_use_ssc(dev_priv) && num_connectors < 
> > > > 2) {
> > > > @@ -7868,7 +7868,7 @@ static void i9xx_set_pipeconf(struct 
> > > > intel_crtc *intel_crtc)
> > > >  		pipeconf |= PIPECONF_DOUBLE_WIDE;
> > > >  
> > > >  	/* only g4x and later have fancy bpc/dither controls 
> > > > */
> > > > -	if (IS_G4X(dev) || IS_VALLEYVIEW(dev)) {
> > > > +	if (IS_G4X(dev) || HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		/* Bspec claims that we can't use dithering 
> > > > for 30bpp pipes. */
> > > >  		if (intel_crtc->config->dither && intel_crtc
> > > > ->config->pipe_bpp != 30)
> > > >  			pipeconf |= PIPECONF_DITHER_EN |
> > > > @@ -7908,7 +7908,7 @@ static void i9xx_set_pipeconf(struct 
> > > > intel_crtc *intel_crtc)
> > > >  	} else
> > > >  		pipeconf |= PIPECONF_PROGRESSIVE;
> > > >  
> > > > -	if (IS_VALLEYVIEW(dev) && intel_crtc->config
> > > > ->limited_color_range)
> > > > +	if (HAS_GEN7_LP_FEATURES(dev) && intel_crtc->config
> > > > ->limited_color_range)
> > > >  		pipeconf |= PIPECONF_COLOR_RANGE_SELECT;
> > > >  
> > > >  	I915_WRITE(PIPECONF(intel_crtc->pipe), pipeconf);
> > > > @@ -8155,7 +8155,7 @@ static bool i9xx_get_pipe_config(struct 
> > > > intel_crtc *crtc,
> > > >  	if (!(tmp & PIPECONF_ENABLE))
> > > >  		return false;
> > > >  
> > > > -	if (IS_G4X(dev) || IS_VALLEYVIEW(dev)) {
> > > > +	if (IS_G4X(dev) || HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		switch (tmp & PIPECONF_BPC_MASK) {
> > > >  		case PIPECONF_6BPC:
> > > >  			pipe_config->pipe_bpp = 18;
> > > > @@ -8171,7 +8171,7 @@ static bool i9xx_get_pipe_config(struct 
> > > > intel_crtc *crtc,
> > > >  		}
> > > >  	}
> > > >  
> > > > -	if (IS_VALLEYVIEW(dev) && (tmp & 
> > > > PIPECONF_COLOR_RANGE_SELECT))
> > > > +	if (HAS_GEN7_LP_FEATURES(dev) && (tmp & 
> > > > PIPECONF_COLOR_RANGE_SELECT))
> > > >  		pipe_config->limited_color_range = true;
> > > >  
> > > >  	if (INTEL_INFO(dev)->gen < 4)
> > > > @@ -8199,7 +8199,7 @@ static bool i9xx_get_pipe_config(struct 
> > > > intel_crtc *crtc,
> > > >  		pipe_config->pixel_multiplier = 1;
> > > >  	}
> > > >  	pipe_config->dpll_hw_state.dpll = I915_READ(DPLL(crtc
> > > > ->pipe));
> > > > -	if (!IS_VALLEYVIEW(dev)) {
> > > > +	if (!HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		/*
> > > >  		 * DPLL_DVO_2X_MODE must be enabled for both 
> > > > DPLLs
> > > >  		 * on 830. Filter it out here so that we don't
> > > > @@ -11516,7 +11516,7 @@ static int intel_crtc_page_flip(struct 
> > > > drm_crtc *crtc,
> > > >  	if (INTEL_INFO(dev)->gen >= 5 || IS_G4X(dev))
> > > >  		work->flip_count = 
> > > > I915_READ(PIPE_FLIPCOUNT_G4X(pipe)) + 1;
> > > >  
> > > > -	if (IS_VALLEYVIEW(dev)) {
> > > > +	if (HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		ring = &dev_priv->ring[BCS];
> > > >  		if (obj->tiling_mode != intel_fb_obj(work
> > > > ->old_fb)->tiling_mode)
> > > >  			/* vlv: DISPLAY_FLIP fails to change 
> > > > tiling */
> > > > @@ -11997,7 +11997,7 @@ compute_baseline_pipe_bpp(struct 
> > > > intel_crtc *crtc,
> > > >  	struct drm_connector_state *connector_state;
> > > >  	int bpp, i;
> > > >  
> > > > -	if ((IS_G4X(dev) || IS_VALLEYVIEW(dev)))
> > > > +	if ((IS_G4X(dev) || HAS_GEN7_LP_FEATURES(dev)))
> > > >  		bpp = 10*3;
> > > >  	else if (INTEL_INFO(dev)->gen >= 5)
> > > >  		bpp = 12*3;
> > > > @@ -12607,7 +12607,7 @@ intel_pipe_config_compare(struct 
> > > > drm_device *dev,
> > > >  	PIPE_CONF_CHECK_I(pixel_multiplier);
> > > >  	PIPE_CONF_CHECK_I(has_hdmi_sink);
> > > >  	if ((INTEL_INFO(dev)->gen < 8 && !IS_HASWELL(dev)) ||
> > > > -	    IS_VALLEYVIEW(dev))
> > > > +	    HAS_GEN7_LP_FEATURES(dev))
> > > >  		PIPE_CONF_CHECK_I(limited_color_range);
> > > >  	PIPE_CONF_CHECK_I(has_infoframe);
> > > >  
> > > > @@ -14256,7 +14256,7 @@ static bool intel_crt_present(struct 
> > > > drm_device *dev)
> > > >  	if (IS_CHERRYVIEW(dev))
> > > >  		return false;
> > > >  
> > > > -	if (IS_VALLEYVIEW(dev) && !dev_priv
> > > > ->vbt.int_crt_support)
> > > > +	if (HAS_GEN7_LP_FEATURES(dev) && !dev_priv
> > > > ->vbt.int_crt_support)
> > > >  		return false;
> > > 
> > > Again, no crt on CHV, so doesn't need changing.
> > > 
> > > >  
> > > >  	return true;
> > > > @@ -14341,7 +14341,7 @@ static void intel_setup_outputs(struct 
> > > > drm_device *dev)
> > > >  
> > > >  		if (I915_READ(PCH_DP_D) & DP_DETECTED)
> > > >  			intel_dp_init(dev, PCH_DP_D, PORT_D);
> > > > -	} else if (IS_VALLEYVIEW(dev)) {
> > > > +	} else if (HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		/*
> > > >  		 * The DP_DETECTED bit is the latched state of 
> > > > the DDC
> > > >  		 * SDA pin at boot. However since eDP doesn't 
> > > > require DDC
> > > > @@ -14490,7 +14490,7 @@ u32 intel_fb_pitch_limit(struct 
> > > > drm_device *dev, uint64_t fb_modifier,
> > > >  		 *  pixels and 32K bytes."
> > > >  		 */
> > > >  		 return 
> > > > min(8192*drm_format_plane_cpp(pixel_format, 0), 32768);
> > > > -	} else if (gen >= 5 && !IS_VALLEYVIEW(dev)) {
> > > > +	} else if (gen >= 5 && !HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		return 32*1024;
> > > >  	} else if (gen >= 4) {
> > > >  		if (fb_modifier == I915_FORMAT_MOD_X_TILED)
> > > > @@ -14594,7 +14594,7 @@ static int 
> > > > intel_framebuffer_init(struct drm_device *dev,
> > > >  		}
> > > >  		break;
> > > >  	case DRM_FORMAT_ABGR8888:
> > > > -		if (!IS_VALLEYVIEW(dev) && INTEL_INFO(dev)
> > > > ->gen < 9) {
> > > > +		if (!HAS_GEN7_LP_FEATURES(dev) && 
> > > > INTEL_INFO(dev)->gen < 9) {
> > > >  			DRM_DEBUG("unsupported pixel format: 
> > > > %s\n",
> > > >  				  drm_get_format_name(mode_cmd
> > > > ->pixel_format));
> > > >  			return -EINVAL;
> > > > @@ -14610,7 +14610,7 @@ static int 
> > > > intel_framebuffer_init(struct drm_device *dev,
> > > >  		}
> > > >  		break;
> > > >  	case DRM_FORMAT_ABGR2101010:
> > > > -		if (!IS_VALLEYVIEW(dev)) {
> > > > +		if (!HAS_GEN7_LP_FEATURES(dev)) {
> > > >  			DRM_DEBUG("unsupported pixel format: 
> > > > %s\n",
> > > >  				  drm_get_format_name(mode_cmd
> > > > ->pixel_format));
> > > >  			return -EINVAL;
> > > > @@ -14738,7 +14738,7 @@ static void intel_init_display(struct 
> > > > drm_device *dev)
> > > >  		dev_priv->display.crtc_disable = 
> > > > ironlake_crtc_disable;
> > > >  		dev_priv->display.update_primary_plane =
> > > >  			ironlake_update_primary_plane;
> > > > -	} else if (IS_VALLEYVIEW(dev)) {
> > > > +	} else if (HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		dev_priv->display.get_pipe_config = 
> > > > i9xx_get_pipe_config;
> > > >  		dev_priv->display.get_initial_plane_config =
> > > >  			i9xx_get_initial_plane_config;
> > > > @@ -14771,7 +14771,7 @@ static void intel_init_display(struct 
> > > > drm_device *dev)
> > > >  	else if (IS_HASWELL(dev))
> > > >  		dev_priv->display.get_display_clock_speed =
> > > >  			haswell_get_display_clock_speed;
> > > > -	else if (IS_VALLEYVIEW(dev))
> > > > +	else if (HAS_GEN7_LP_FEATURES(dev))
> > > >  		dev_priv->display.get_display_clock_speed =
> > > >  			valleyview_get_display_clock_speed;
> > > >  	else if (IS_GEN5(dev))
> > > > @@ -14832,7 +14832,7 @@ static void intel_init_display(struct 
> > > > drm_device *dev)
> > > >  			dev_priv->display.modeset_calc_cdclk =
> > > >  				broadwell_modeset_calc_cdclk;
> > > >  		}
> > > > -	} else if (IS_VALLEYVIEW(dev)) {
> > > > +	} else if (HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		dev_priv->display.modeset_commit_cdclk =
> > > >  			valleyview_modeset_commit_cdclk;
> > > >  		dev_priv->display.modeset_calc_cdclk =
> > > > @@ -15606,7 +15606,7 @@ intel_modeset_setup_hw_state(struct 
> > > > drm_device *dev)
> > > >  		pll->on = false;
> > > >  	}
> > > >  
> > > > -	if (IS_VALLEYVIEW(dev))
> > > > +	if (HAS_GEN7_LP_FEATURES(dev))
> > > >  		vlv_wm_get_hw_state(dev);
> > > >  	else if (IS_GEN9(dev))
> > > >  		skl_wm_get_hw_state(dev);
> > > > diff --git a/drivers/gpu/drm/i915/intel_dp.c 
> > > > b/drivers/gpu/drm/i915/intel_dp.c
> > > > index f2bfca0..ac84f83 100644
> > > > --- a/drivers/gpu/drm/i915/intel_dp.c
> > > > +++ b/drivers/gpu/drm/i915/intel_dp.c
> > > > @@ -517,7 +517,7 @@ void vlv_power_sequencer_reset(struct 
> > > > drm_i915_private *dev_priv)
> > > >  	struct drm_device *dev = dev_priv->dev;
> > > >  	struct intel_encoder *encoder;
> > > >  
> > > > -	if (WARN_ON(!IS_VALLEYVIEW(dev)))
> > > > +	if (WARN_ON(!HAS_GEN7_LP_FEATURES(dev)))
> > > >  		return;
> > > >  
> > > >  	/*
> > > > @@ -582,7 +582,7 @@ static int edp_notify_handler(struct 
> > > > notifier_block *this, unsigned long code,
> > > >  
> > > >  	pps_lock(intel_dp);
> > > >  
> > > > -	if (IS_VALLEYVIEW(dev)) {
> > > > +	if (HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		enum pipe pipe = 
> > > > vlv_power_sequencer_pipe(intel_dp);
> > > >  		i915_reg_t pp_ctrl_reg, pp_div_reg;
> > > >  		u32 pp_div;
> > > > @@ -610,7 +610,7 @@ static bool edp_have_panel_power(struct 
> > > > intel_dp *intel_dp)
> > > >  
> > > >  	lockdep_assert_held(&dev_priv->pps_mutex);
> > > >  
> > > > -	if (IS_VALLEYVIEW(dev) &&
> > > > +	if (HAS_GEN7_LP_FEATURES(dev) &&
> > > >  	    intel_dp->pps_pipe == INVALID_PIPE)
> > > >  		return false;
> > > >  
> > > > @@ -624,7 +624,7 @@ static bool edp_have_panel_vdd(struct 
> > > > intel_dp *intel_dp)
> > > >  
> > > >  	lockdep_assert_held(&dev_priv->pps_mutex);
> > > >  
> > > > -	if (IS_VALLEYVIEW(dev) &&
> > > > +	if (HAS_GEN7_LP_FEATURES(dev) &&
> > > >  	    intel_dp->pps_pipe == INVALID_PIPE)
> > > >  		return false;
> > > >  
> > > > @@ -1722,7 +1722,7 @@ static void intel_dp_prepare(struct 
> > > > intel_encoder *encoder)
> > > >  			trans_dp &= ~TRANS_DP_ENH_FRAMING;
> > > >  		I915_WRITE(TRANS_DP_CTL(crtc->pipe), 
> > > > trans_dp);
> > > >  	} else {
> > > > -		if (!HAS_PCH_SPLIT(dev) && !IS_VALLEYVIEW(dev) 
> > > > &&
> > > > +		if (!HAS_PCH_SPLIT(dev) && 
> > > > !HAS_GEN7_LP_FEATURES(dev) &&
> > > >  		    crtc->config->limited_color_range)
> > > >  			intel_dp->DP |= DP_COLOR_RANGE_16_235;
> > > >  
> > > > @@ -2417,7 +2417,7 @@ static void intel_dp_get_config(struct 
> > > > intel_encoder *encoder,
> > > >  
> > > >  	pipe_config->base.adjusted_mode.flags |= flags;
> > > >  
> > > > -	if (!HAS_PCH_SPLIT(dev) && !IS_VALLEYVIEW(dev) &&
> > > > +	if (!HAS_PCH_SPLIT(dev) && !HAS_GEN7_LP_FEATURES(dev) 
> > > > &&
> > > >  	    tmp & DP_COLOR_RANGE_16_235)
> > > >  		pipe_config->limited_color_range = true;
> > > >  
> > > > @@ -2694,7 +2694,7 @@ static void intel_enable_dp(struct 
> > > > intel_encoder *encoder)
> > > >  
> > > >  	pps_lock(intel_dp);
> > > >  
> > > > -	if (IS_VALLEYVIEW(dev))
> > > > +	if (HAS_GEN7_LP_FEATURES(dev))
> > > >  		vlv_init_panel_power_sequencer(intel_dp);
> > > >  
> > > >  	/*
> > > > @@ -2728,7 +2728,7 @@ static void intel_enable_dp(struct 
> > > > intel_encoder *encoder)
> > > >  
> > > >  	pps_unlock(intel_dp);
> > > >  
> > > > -	if (IS_VALLEYVIEW(dev)) {
> > > > +	if (HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		unsigned int lane_mask = 0x0;
> > > >  
> > > >  		if (IS_CHERRYVIEW(dev))
> > > > @@ -3218,7 +3218,7 @@ intel_dp_voltage_max(struct intel_dp 
> > > > *intel_dp)
> > > >  		if (dev_priv->edp_low_vswing && port == 
> > > > PORT_A)
> > > >  			return DP_TRAIN_VOLTAGE_SWING_LEVEL_3;
> > > >  		return DP_TRAIN_VOLTAGE_SWING_LEVEL_2;
> > > > -	} else if (IS_VALLEYVIEW(dev))
> > > > +	} else if (HAS_GEN7_LP_FEATURES(dev))
> > > >  		return DP_TRAIN_VOLTAGE_SWING_LEVEL_3;
> > > >  	else if (IS_GEN7(dev) && port == PORT_A)
> > > >  		return DP_TRAIN_VOLTAGE_SWING_LEVEL_2;
> > > > @@ -3259,7 +3259,7 @@ intel_dp_pre_emphasis_max(struct intel_dp 
> > > > *intel_dp, uint8_t voltage_swing)
> > > >  		default:
> > > >  			return DP_TRAIN_PRE_EMPH_LEVEL_0;
> > > >  		}
> > > > -	} else if (IS_VALLEYVIEW(dev)) {
> > > > +	} else if (HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		switch (voltage_swing & 
> > > > DP_TRAIN_VOLTAGE_SWING_MASK) {
> > > >  		case DP_TRAIN_VOLTAGE_SWING_LEVEL_0:
> > > >  			return DP_TRAIN_PRE_EMPH_LEVEL_3;
> > > > @@ -4539,7 +4539,7 @@ bool intel_digital_port_connected(struct 
> > > > drm_i915_private *dev_priv,
> > > >  		return cpt_digital_port_connected(dev_priv, 
> > > > port);
> > > >  	else if (IS_BROXTON(dev_priv))
> > > >  		return bxt_digital_port_connected(dev_priv, 
> > > > port);
> > > > -	else if (IS_VALLEYVIEW(dev_priv))
> > > > +	else if (HAS_GEN7_LP_FEATURES(dev_priv))
> > > >  		return vlv_digital_port_connected(dev_priv, 
> > > > port);
> > > >  	else
> > > >  		return g4x_digital_port_connected(dev_priv, 
> > > > port);
> > > > @@ -4933,7 +4933,7 @@ static void intel_dp_encoder_reset(struct 
> > > > drm_encoder *encoder)
> > > >  	 * Read out the current power sequencer assignment,
> > > >  	 * in case the BIOS did something with it.
> > > >  	 */
> > > > -	if (IS_VALLEYVIEW(encoder->dev))
> > > > +	if (HAS_GEN7_LP_FEATURES(encoder->dev))
> > > >  		vlv_initial_power_sequencer_setup(intel_dp);
> > > >  
> > > >  	intel_edp_panel_vdd_sanitize(intel_dp);
> > > > @@ -5293,7 +5293,7 @@ 
> > > > intel_dp_init_panel_power_sequencer_registers(struct drm_device 
> > > > *dev,
> > > >  
> > > >  	/* Haswell doesn't have any port selection bits for 
> > > > the panel
> > > >  	 * power sequencer any more. */
> > > > -	if (IS_VALLEYVIEW(dev)) {
> > > > +	if (HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		port_sel = PANEL_PORT_SELECT_VLV(port);
> > > >  	} else if (HAS_PCH_IBX(dev) || HAS_PCH_CPT(dev)) {
> > > >  		if (port == PORT_A)
> > > > @@ -5405,12 +5405,12 @@ static void 
> > > > intel_dp_set_drrs_state(struct drm_device *dev, int 
> > > > refresh_rate)
> > > >  
> > > >  		val = I915_READ(reg);
> > > >  		if (index > DRRS_HIGH_RR) {
> > > > -			if (IS_VALLEYVIEW(dev))
> > > > +			if (HAS_GEN7_LP_FEATURES(dev))
> > > >  				val |= 
> > > > PIPECONF_EDP_RR_MODE_SWITCH_VLV;
> > > >  			else
> > > >  				val |= 
> > > > PIPECONF_EDP_RR_MODE_SWITCH;
> > > >  		} else {
> > > > -			if (IS_VALLEYVIEW(dev))
> > > > +			if (HAS_GEN7_LP_FEATURES(dev))
> > > >  				val &= 
> > > > ~PIPECONF_EDP_RR_MODE_SWITCH_VLV;
> > > >  			else
> > > >  				val &= 
> > > > ~PIPECONF_EDP_RR_MODE_SWITCH;
> > > > @@ -5777,7 +5777,7 @@ static bool 
> > > > intel_edp_init_connector(struct intel_dp *intel_dp,
> > > >  	}
> > > >  	mutex_unlock(&dev->mode_config.mutex);
> > > >  
> > > > -	if (IS_VALLEYVIEW(dev)) {
> > > > +	if (HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		intel_dp->edp_notifier.notifier_call = 
> > > > edp_notify_handler;
> > > >  		register_reboot_notifier(&intel_dp
> > > > ->edp_notifier);
> > > >  
> > > > @@ -5825,7 +5825,7 @@ intel_dp_init_connector(struct 
> > > > intel_digital_port *intel_dig_port,
> > > >  	/* intel_dp vfuncs */
> > > >  	if (INTEL_INFO(dev)->gen >= 9)
> > > >  		intel_dp->get_aux_clock_divider = 
> > > > skl_get_aux_clock_divider;
> > > > -	else if (IS_VALLEYVIEW(dev))
> > > > +	else if (HAS_GEN7_LP_FEATURES(dev))
> > > >  		intel_dp->get_aux_clock_divider = 
> > > > vlv_get_aux_clock_divider;
> > > >  	else if (IS_HASWELL(dev) || IS_BROADWELL(dev))
> > > >  		intel_dp->get_aux_clock_divider = 
> > > > hsw_get_aux_clock_divider;
> > > > @@ -5860,7 +5860,7 @@ intel_dp_init_connector(struct 
> > > > intel_digital_port *intel_dig_port,
> > > >  		intel_encoder->type = INTEL_OUTPUT_EDP;
> > > >  
> > > >  	/* eDP only on port B and/or C on vlv/chv */
> > > > -	if (WARN_ON(IS_VALLEYVIEW(dev) && is_edp(intel_dp) &&
> > > > +	if (WARN_ON(HAS_GEN7_LP_FEATURES(dev) && 
> > > > is_edp(intel_dp) &&
> > > >  		    port != PORT_B && port != PORT_C))
> > > >  		return false;
> > > >  
> > > > @@ -5912,7 +5912,7 @@ intel_dp_init_connector(struct 
> > > > intel_digital_port *intel_dig_port,
> > > >  	if (is_edp(intel_dp)) {
> > > >  		pps_lock(intel_dp);
> > > >  		intel_dp_init_panel_power_timestamps(intel_dp)
> > > > ;
> > > > -		if (IS_VALLEYVIEW(dev))
> > > > +		if (HAS_GEN7_LP_FEATURES(dev))
> > > >  			vlv_initial_power_sequencer_setup(inte
> > > > l_dp);
> > > >  		else
> > > >  			intel_dp_init_panel_power_sequencer(de
> > > > v, intel_dp);
> > > > diff --git a/drivers/gpu/drm/i915/intel_dsi.c 
> > > > b/drivers/gpu/drm/i915/intel_dsi.c
> > > > index 230957f..9570761 100644
> > > > --- a/drivers/gpu/drm/i915/intel_dsi.c
> > > > +++ b/drivers/gpu/drm/i915/intel_dsi.c
> > > > @@ -369,7 +369,7 @@ static void intel_dsi_device_ready(struct 
> > > > intel_encoder *encoder)
> > > >  {
> > > >  	struct drm_device *dev = encoder->base.dev;
> > > >  
> > > > -	if (IS_VALLEYVIEW(dev))
> > > > +	if (HAS_GEN7_LP_FEATURES(dev))
> > > >  		vlv_dsi_device_ready(encoder);
> > > >  	else if (IS_BROXTON(dev))
> > > >  		bxt_dsi_device_ready(encoder);
> > > > @@ -487,7 +487,7 @@ static void intel_dsi_pre_enable(struct 
> > > > intel_encoder *encoder)
> > > >  
> > > >  	msleep(intel_dsi->panel_on_delay);
> > > >  
> > > > -	if (IS_VALLEYVIEW(dev)) {
> > > > +	if (HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		/*
> > > >  		 * Disable DPOunit clock gating, can stall 
> > > > pipe
> > > >  		 * and we need DPLL REFA always enabled
> > > > @@ -684,8 +684,7 @@ static bool intel_dsi_get_hw_state(struct 
> > > > intel_encoder *encoder,
> > > >  		 * Enable bit does not get set. To check 
> > > > whether DSI Port C
> > > >  		 * was enabled in BIOS, check the Pipe B 
> > > > enable bit
> > > >  		 */
> > > > -		if (IS_VALLEYVIEW(dev) && !IS_CHERRYVIEW(dev) 
> > > > &&
> > > > -		    (port == PORT_C))
> > > > +		if (IS_VALLEYVIEW(dev) && (port == PORT_C))
> > > >  			dpi_enabled = 
> > > > I915_READ(PIPECONF(PIPE_B)) &
> > > >  							PIPECO
> > > > NF_ENABLE;
> > > >  
> > > > @@ -716,7 +715,7 @@ static void intel_dsi_get_config(struct 
> > > > intel_encoder *encoder,
> > > >  
> > > >  	if (IS_BROXTON(encoder->base.dev))
> > > >  		pclk = bxt_get_dsi_pclk(encoder, pipe_config
> > > > ->pipe_bpp);
> > > > -	else if (IS_VALLEYVIEW(encoder->base.dev))
> > > > +	else if (HAS_GEN7_LP_FEATURES(encoder->base.dev))
> > > >  		pclk = vlv_get_dsi_pclk(encoder, pipe_config
> > > > ->pipe_bpp);
> > > >  
> > > >  	if (!pclk)
> > > > @@ -869,7 +868,7 @@ static void intel_dsi_prepare(struct 
> > > > intel_encoder *intel_encoder)
> > > >  	}
> > > >  
> > > >  	for_each_dsi_port(port, intel_dsi->ports) {
> > > > -		if (IS_VALLEYVIEW(dev)) {
> > > > +		if (HAS_GEN7_LP_FEATURES(dev)) {
> > > >  			/*
> > > >  			 * escape clock divider, 20MHz, shared 
> > > > for A and C.
> > > >  			 * device ready must be off when doing 
> > > > this! txclkesc?
> > > > @@ -1129,7 +1128,7 @@ void intel_dsi_init(struct drm_device 
> > > > *dev)
> > > >  	if (!dev_priv->vbt.has_mipi)
> > > >  		return;
> > > >  
> > > > -	if (IS_VALLEYVIEW(dev)) {
> > > > +	if (HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		dev_priv->mipi_mmio_base = VLV_MIPI_BASE;
> > > >  	} else {
> > > >  		DRM_ERROR("Unsupported Mipi device to reg 
> > > > base");
> > > > diff --git a/drivers/gpu/drm/i915/intel_dsi_pll.c 
> > > > b/drivers/gpu/drm/i915/intel_dsi_pll.c
> > > > index cb3cf39..dce9f93 100644
> > > > --- a/drivers/gpu/drm/i915/intel_dsi_pll.c
> > > > +++ b/drivers/gpu/drm/i915/intel_dsi_pll.c
> > > > @@ -561,7 +561,7 @@ void intel_enable_dsi_pll(struct 
> > > > intel_encoder *encoder)
> > > >  {
> > > >  	struct drm_device *dev = encoder->base.dev;
> > > >  
> > > > -	if (IS_VALLEYVIEW(dev))
> > > > +	if (HAS_GEN7_LP_FEATURES(dev))
> > > >  		vlv_enable_dsi_pll(encoder);
> > > >  	else if (IS_BROXTON(dev))
> > > >  		bxt_enable_dsi_pll(encoder);
> > > > @@ -571,7 +571,7 @@ void intel_disable_dsi_pll(struct 
> > > > intel_encoder *encoder)
> > > >  {
> > > >  	struct drm_device *dev = encoder->base.dev;
> > > >  
> > > > -	if (IS_VALLEYVIEW(dev))
> > > > +	if (HAS_GEN7_LP_FEATURES(dev))
> > > >  		vlv_disable_dsi_pll(encoder);
> > > >  	else if (IS_BROXTON(dev))
> > > >  		bxt_disable_dsi_pll(encoder);
> > > > @@ -599,6 +599,6 @@ void intel_dsi_reset_clocks(struct 
> > > > intel_encoder *encoder, enum port port)
> > > >  
> > > >  	if (IS_BROXTON(dev))
> > > >  		bxt_dsi_reset_clocks(encoder, port);
> > > > -	else if (IS_VALLEYVIEW(dev))
> > > > +	else if (HAS_GEN7_LP_FEATURES(dev))
> > > >  		vlv_dsi_reset_clocks(encoder, port);
> > > >  }
> > > > diff --git a/drivers/gpu/drm/i915/intel_hdmi.c 
> > > > b/drivers/gpu/drm/i915/intel_hdmi.c
> > > > index c3978ba..80c4d8e 100644
> > > > --- a/drivers/gpu/drm/i915/intel_hdmi.c
> > > > +++ b/drivers/gpu/drm/i915/intel_hdmi.c
> > > > @@ -638,7 +638,7 @@ static bool 
> > > > intel_hdmi_set_gcp_infoframe(struct drm_encoder *encoder)
> > > >  
> > > >  	if (HAS_DDI(dev_priv))
> > > >  		reg = HSW_TVIDEO_DIP_GCP(crtc->config
> > > > ->cpu_transcoder);
> > > > -	else if (IS_VALLEYVIEW(dev_priv))
> > > > +	else if (HAS_GEN7_LP_FEATURES(dev_priv))
> > > >  		reg = VLV_TVIDEO_DIP_GCP(crtc->pipe);
> > > >  	else if (HAS_PCH_SPLIT(dev_priv->dev))
> > > >  		reg = TVIDEO_DIP_GCP(crtc->pipe);
> > > > @@ -2100,7 +2100,7 @@ void intel_hdmi_init_connector(struct 
> > > > intel_digital_port *intel_dig_port,
> > > >  		BUG();
> > > >  	}
> > > >  
> > > > -	if (IS_VALLEYVIEW(dev)) {
> > > > +	if (HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		intel_hdmi->write_infoframe = 
> > > > vlv_write_infoframe;
> > > >  		intel_hdmi->set_infoframes = 
> > > > vlv_set_infoframes;
> > > >  		intel_hdmi->infoframe_enabled = 
> > > > vlv_infoframe_enabled;
> > > > diff --git a/drivers/gpu/drm/i915/intel_hotplug.c 
> > > > b/drivers/gpu/drm/i915/intel_hotplug.c
> > > > index b177857..e7ddf34 100644
> > > > --- a/drivers/gpu/drm/i915/intel_hotplug.c
> > > > +++ b/drivers/gpu/drm/i915/intel_hotplug.c
> > > > @@ -407,7 +407,7 @@ void intel_hpd_irq_handler(struct 
> > > > drm_device *dev,
> > > >  			 * hotplug bits itself. So only WARN 
> > > > about unexpected
> > > >  			 * interrupts on saner platforms.
> > > >  			 */
> > > > -			WARN_ONCE(INTEL_INFO(dev)->gen >= 5 && 
> > > > !IS_VALLEYVIEW(dev),
> > > > +			WARN_ONCE(INTEL_INFO(dev)->gen >= 5 && 
> > > > !HAS_GEN7_LP_FEATURES(dev),
> > > >  				  "Received HPD interrupt on 
> > > > pin %d although disabled\n", i);
> > > 
> > > Can replace with !HAS_GMCH_DISPLAY
> > > 
> > > >  			continue;
> > > >  		}
> > > > diff --git a/drivers/gpu/drm/i915/intel_i2c.c 
> > > > b/drivers/gpu/drm/i915/intel_i2c.c
> > > > index 1110c83..3e9e0e5 100644
> > > > --- a/drivers/gpu/drm/i915/intel_i2c.c
> > > > +++ b/drivers/gpu/drm/i915/intel_i2c.c
> > > > @@ -629,7 +629,7 @@ int intel_setup_gmbus(struct drm_device 
> > > > *dev)
> > > >  	if (HAS_PCH_NOP(dev))
> > > >  		return 0;
> > > >  
> > > > -	if (IS_VALLEYVIEW(dev))
> > > > +	if (HAS_GEN7_LP_FEATURES(dev))
> > > >  		dev_priv->gpio_mmio_base = VLV_DISPLAY_BASE;
> > > >  	else if (!HAS_GMCH_DISPLAY(dev_priv))
> > > >  		dev_priv->gpio_mmio_base =
> > > > diff --git a/drivers/gpu/drm/i915/intel_panel.c 
> > > > b/drivers/gpu/drm/i915/intel_panel.c
> > > > index a24df35..3b7e447 100644
> > > > --- a/drivers/gpu/drm/i915/intel_panel.c
> > > > +++ b/drivers/gpu/drm/i915/intel_panel.c
> > > > @@ -1767,7 +1767,7 @@ intel_panel_init_backlight_funcs(struct 
> > > > intel_panel *panel)
> > > >  		panel->backlight.set = pch_set_backlight;
> > > >  		panel->backlight.get = pch_get_backlight;
> > > >  		panel->backlight.hz_to_pwm = pch_hz_to_pwm;
> > > > -	} else if (IS_VALLEYVIEW(dev)) {
> > > > +	} else if (HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		if (dev_priv->vbt.has_mipi) {
> > > >  			panel->backlight.setup = 
> > > > pwm_setup_backlight;
> > > >  			panel->backlight.enable = 
> > > > pwm_enable_backlight;
> > > > diff --git a/drivers/gpu/drm/i915/intel_pm.c 
> > > > b/drivers/gpu/drm/i915/intel_pm.c
> > > > index 96f45d7..741c888 100644
> > > > --- a/drivers/gpu/drm/i915/intel_pm.c
> > > > +++ b/drivers/gpu/drm/i915/intel_pm.c
> > > > @@ -283,7 +283,7 @@ void intel_set_memory_cxsr(struct 
> > > > drm_i915_private *dev_priv, bool enable)
> > > >  	struct drm_device *dev = dev_priv->dev;
> > > >  	u32 val;
> > > >  
> > > > -	if (IS_VALLEYVIEW(dev)) {
> > > > +	if (HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		I915_WRITE(FW_BLC_SELF_VLV, enable ? 
> > > > FW_CSPWRDWNEN : 0);
> > > >  		POSTING_READ(FW_BLC_SELF_VLV);
> > > >  		dev_priv->wm.vlv.cxsr = enable;
> > > > @@ -4397,7 +4397,7 @@ void gen6_rps_idle(struct 
> > > > drm_i915_private *dev_priv)
> > > >  
> > > >  	mutex_lock(&dev_priv->rps.hw_lock);
> > > >  	if (dev_priv->rps.enabled) {
> > > > -		if (IS_VALLEYVIEW(dev))
> > > > +		if (HAS_GEN7_LP_FEATURES(dev))
> > > >  			vlv_set_rps_idle(dev_priv);
> > > >  		else
> > > >  			gen6_set_rps(dev_priv->dev, dev_priv
> > > > ->rps.idle_freq);
> > > > @@ -4450,7 +4450,7 @@ void gen6_rps_boost(struct 
> > > > drm_i915_private *dev_priv,
> > > >  
> > > >  void intel_set_rps(struct drm_device *dev, u8 val)
> > > >  {
> > > > -	if (IS_VALLEYVIEW(dev))
> > > > +	if (HAS_GEN7_LP_FEATURES(dev))
> > > >  		valleyview_set_rps(dev, val);
> > > >  	else
> > > >  		gen6_set_rps(dev, val);
> > > > @@ -4494,7 +4494,7 @@ static void valleyview_disable_rps(struct 
> > > > drm_device *dev)
> > > >  
> > > >  static void intel_print_rc6_info(struct drm_device *dev, u32 
> > > > mode)
> > > >  {
> > > > -	if (IS_VALLEYVIEW(dev)) {
> > > > +	if (HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		if (mode & (GEN7_RC_CTL_TO_MODE | 
> > > > GEN6_RC_CTL_EI_MODE(1)))
> > > >  			mode = GEN6_RC_CTL_RC6_ENABLE;
> > > >  		else
> > > > diff --git a/drivers/gpu/drm/i915/intel_psr.c 
> > > > b/drivers/gpu/drm/i915/intel_psr.c
> > > > index b6609e6..c4e8ad2 100644
> > > > --- a/drivers/gpu/drm/i915/intel_psr.c
> > > > +++ b/drivers/gpu/drm/i915/intel_psr.c
> > > > @@ -332,7 +332,7 @@ static bool 
> > > > intel_psr_match_conditions(struct intel_dp *intel_dp)
> > > >  		return false;
> > > >  	}
> > > >  
> > > > -	if (!IS_VALLEYVIEW(dev) && ((dev_priv
> > > > ->vbt.psr.full_link) ||
> > > > +	if (!HAS_GEN7_LP_FEATURES(dev) && ((dev_priv
> > > > ->vbt.psr.full_link) ||
> > > >  				    (dig_port->port != 
> > > > PORT_A))) {
> > > >  		DRM_DEBUG_KMS("PSR condition failed: Link 
> > > > Standby requested/needed but not supported on this 
> > > > platform\n");
> > > >  		return false;
> > > > @@ -650,7 +650,7 @@ void intel_psr_single_frame_update(struct 
> > > > drm_device *dev,
> > > >  	 * Single frame update is already supported on BDW+ 
> > > > but it requires
> > > >  	 * many W/A and it isn't really needed.
> > > >  	 */
> > > > -	if (!IS_VALLEYVIEW(dev))
> > > > +	if (!HAS_GEN7_LP_FEATURES(dev))
> > > >  		return;
> > > >  
> > > >  	mutex_lock(&dev_priv->psr.lock);
> > > > diff --git a/drivers/gpu/drm/i915/intel_sprite.c 
> > > > b/drivers/gpu/drm/i915/intel_sprite.c
> > > > index 2b96f33..2e7043b 100644
> > > > --- a/drivers/gpu/drm/i915/intel_sprite.c
> > > > +++ b/drivers/gpu/drm/i915/intel_sprite.c
> > > > @@ -951,7 +951,7 @@ int intel_sprite_set_colorkey(struct 
> > > > drm_device *dev, void *data,
> > > >  	if ((set->flags & (I915_SET_COLORKEY_DESTINATION | 
> > > > I915_SET_COLORKEY_SOURCE)) == (I915_SET_COLORKEY_DESTINATION | 
> > > > I915_SET_COLORKEY_SOURCE))
> > > >  		return -EINVAL;
> > > >  
> > > > -	if (IS_VALLEYVIEW(dev) &&
> > > > +	if (HAS_GEN7_LP_FEATURES(dev) &&
> > > >  	    set->flags & I915_SET_COLORKEY_DESTINATION)
> > > >  		return -EINVAL;
> > > >  
> > > > @@ -1086,7 +1086,7 @@ intel_plane_init(struct drm_device *dev, 
> > > > enum pipe pipe, int plane)
> > > >  			intel_plane->max_downscale = 1;
> > > >  		}
> > > >  
> > > > -		if (IS_VALLEYVIEW(dev)) {
> > > > +		if (HAS_GEN7_LP_FEATURES(dev)) {
> > > >  			intel_plane->update_plane = 
> > > > vlv_update_plane;
> > > >  			intel_plane->disable_plane = 
> > > > vlv_disable_plane;
> > > >  
> > > > diff --git a/drivers/gpu/drm/i915/intel_uncore.c 
> > > > b/drivers/gpu/drm/i915/intel_uncore.c
> > > > index c2358ba..ebbbe00 100644
> > > > --- a/drivers/gpu/drm/i915/intel_uncore.c
> > > > +++ b/drivers/gpu/drm/i915/intel_uncore.c
> > > > @@ -211,7 +211,7 @@ static int __gen6_gt_wait_for_fifo(struct 
> > > > drm_i915_private *dev_priv)
> > > >  
> > > >  	/* On VLV, FIFO will be shared by both SW and HW.
> > > >  	 * So, we need to read the FREE_ENTRIES everytime */
> > > > -	if (IS_VALLEYVIEW(dev_priv->dev))
> > > > +	if (HAS_GEN7_LP_FEATURES(dev_priv->dev))
> > > >  		dev_priv->uncore.fifo_count = 
> > > > fifo_free_entries(dev_priv);
> > > 
> > > This code isn't used on CHV, so can stay as is.
> > > 
> > > >  
> > > >  	if (dev_priv->uncore.fifo_count < 
> > > > GT_FIFO_NUM_RESERVED_ENTRIES) {
> > > > @@ -1115,7 +1115,7 @@ static void fw_domain_init(struct 
> > > > drm_i915_private *dev_priv,
> > > >  		d->val_clear = 
> > > > _MASKED_BIT_DISABLE(FORCEWAKE_KERNEL);
> > > >  	}
> > > >  
> > > > -	if (IS_VALLEYVIEW(dev_priv))
> > > > +	if (HAS_GEN7_LP_FEATURES(dev_priv))
> > > >  		d->reg_post = FORCEWAKE_ACK_VLV;
> > > >  	else if (IS_GEN6(dev_priv) || IS_GEN7(dev_priv) || 
> > > > IS_GEN8(dev_priv))
> > > >  		d->reg_post = ECOBUS;
> > > > @@ -1148,7 +1148,7 @@ static void 
> > > > intel_uncore_fw_domains_init(struct drm_device *dev)
> > > >  			       FORCEWAKE_ACK_BLITTER_GEN9);
> > > >  		fw_domain_init(dev_priv, FW_DOMAIN_ID_MEDIA,
> > > >  			       FORCEWAKE_MEDIA_GEN9, 
> > > > FORCEWAKE_ACK_MEDIA_GEN9);
> > > > -	} else if (IS_VALLEYVIEW(dev)) {
> > > > +	} else if (HAS_GEN7_LP_FEATURES(dev)) {
> > > >  		dev_priv->uncore.funcs.force_wake_get = 
> > > > fw_domains_get;
> > > >  		if (!IS_CHERRYVIEW(dev))
> > > >  			dev_priv->uncore.funcs.force_wake_put 
> > > > =
> > > > -- 
> > > > 2.6.3
> > > > 
> > > > _______________________________________________
> > > > Intel-gfx mailing list
> > > > Intel-gfx@lists.freedesktop.org
> > > > http://lists.freedesktop.org/mailman/listinfo/intel-gfx
> > 
> > -- 
> > Jani Nikula, Intel Open Source Technology Center
> > _______________________________________________
> > Intel-gfx mailing list
> > Intel-gfx@lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/intel-gfx
> 
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

  parent reply	other threads:[~2015-12-04 16:15 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-12-02  1:11 [PATCH] drm/i915: Separate cherryview from valleyview Wayne Boyer
2015-12-02  9:02 ` Ville Syrjälä
2015-12-02 12:30   ` Jani Nikula
2015-12-04 15:05     ` Daniel Vetter
2015-12-04 15:18       ` Ville Syrjälä
2015-12-04 16:15       ` Vivi, Rodrigo [this message]
2015-12-04 16:51         ` Daniel Vetter
2015-12-04 17:04           ` Ville Syrjälä
2015-12-04 17:14             ` Vivi, Rodrigo
2015-12-04 17:24               ` Ville Syrjälä
2015-12-05 19:52                 ` Boyer, Wayne

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1449245746.1387.208.camel@intel.com \
    --to=rodrigo.vivi@intel.com \
    --cc=daniel@ffwll.ch \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=jani.nikula@linux.intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.