diff options
| author | Matt Roper <matthew.d.roper@intel.com> | 2021-03-20 07:42:42 +0300 | 
|---|---|---|
| committer | Matt Roper <matthew.d.roper@intel.com> | 2021-03-24 02:41:11 +0300 | 
| commit | 005e95377249cb65133bf698926d0ab7876dddc3 (patch) | |
| tree | 27fe5658270ed29d805a52e9a47d54f5f8b30f13 /drivers/gpu/drm/i915/display/intel_atomic.c | |
| parent | 01eb15c9165e4169c8637703b025371586d27fbb (diff) | |
| download | linux-005e95377249cb65133bf698926d0ab7876dddc3.tar.xz | |
drm/i915/display: Eliminate most usage of INTEL_GEN()
Use Coccinelle to convert most of the usage of INTEL_GEN() and IS_GEN()
in the display code to use DISPLAY_VER() comparisons instead.  The
following semantic patch was used:
        @@ expression dev_priv, E; @@
        - INTEL_GEN(dev_priv) == E
        + IS_DISPLAY_VER(dev_priv, E)
        @@ expression dev_priv; @@
        - INTEL_GEN(dev_priv)
        + DISPLAY_VER(dev_priv)
        @@ expression dev_priv; expression E; @@
        - IS_GEN(dev_priv, E)
        + IS_DISPLAY_VER(dev_priv, E)
        @@
        expression dev_priv;
        expression from, until;
        @@
        - IS_GEN_RANGE(dev_priv, from, until)
        + IS_DISPLAY_RANGE(dev_priv, from, until)
There are still some display-related uses of INTEL_GEN() in intel_pm.c
(watermark code) and i915_irq.c.  Those will be updated separately.
v2:
 - Use new IS_DISPLAY_RANGE and IS_DISPLAY_VER helpers.  (Jani)
Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
Reviewed-by: Lucas De Marchi <lucas.demarchi@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20210320044245.3920043-4-matthew.d.roper@intel.com
Diffstat (limited to 'drivers/gpu/drm/i915/display/intel_atomic.c')
| -rw-r--r-- | drivers/gpu/drm/i915/display/intel_atomic.c | 6 | 
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/gpu/drm/i915/display/intel_atomic.c b/drivers/gpu/drm/i915/display/intel_atomic.c index 27f7d7109ca3..2b928795755e 100644 --- a/drivers/gpu/drm/i915/display/intel_atomic.c +++ b/drivers/gpu/drm/i915/display/intel_atomic.c @@ -332,7 +332,7 @@ static void intel_atomic_setup_scaler(struct intel_crtc_scaler_state *scaler_sta  	    plane_state->hw.fb->format->is_yuv &&  	    plane_state->hw.fb->format->num_planes > 1) {  		struct intel_plane *plane = to_intel_plane(plane_state->uapi.plane); -		if (IS_GEN(dev_priv, 9) && +		if (IS_DISPLAY_VER(dev_priv, 9) &&  		    !IS_GEMINILAKE(dev_priv)) {  			mode = SKL_PS_SCALER_MODE_NV12;  		} else if (icl_is_hdr_plane(dev_priv, plane->id)) { @@ -351,7 +351,7 @@ static void intel_atomic_setup_scaler(struct intel_crtc_scaler_state *scaler_sta  			if (linked)  				mode |= PS_PLANE_Y_SEL(linked->id);  		} -	} else if (INTEL_GEN(dev_priv) > 9 || IS_GEMINILAKE(dev_priv)) { +	} else if (DISPLAY_VER(dev_priv) > 9 || IS_GEMINILAKE(dev_priv)) {  		mode = PS_SCALER_MODE_NORMAL;  	} else if (num_scalers_need == 1 && intel_crtc->num_scalers > 1) {  		/* @@ -460,7 +460,7 @@ int intel_atomic_setup_scalers(struct drm_i915_private *dev_priv,  				 * isn't necessary to change between HQ and dyn mode  				 * on those platforms.  				 */ -				if (INTEL_GEN(dev_priv) >= 10 || IS_GEMINILAKE(dev_priv)) +				if (DISPLAY_VER(dev_priv) >= 10 || IS_GEMINILAKE(dev_priv))  					continue;  				plane = drm_plane_from_index(&dev_priv->drm, i);  | 
