Commit 9e783375 authored by Rodrigo Vivi's avatar Rodrigo Vivi

drm/i915: Prefer IS_GEN<n> check with bitmask.

Whenever possible we should stick with IS_GEN<n> checks.

Bitmaks has been introduced on commit ae7617f0 ("drm/i915:
Allow optimized platform checks") for efficiency.

Let's stick with it whenever possible.

This patch was generated with coccinelle:

spatch -sp_file is_gen.cocci *{c,h} --in-place

is_gen.cocci:
@gen2@ expression e; @@
-INTEL_GEN(e) == 2
+IS_GEN2(e)
@gen3@ expression e; @@
-INTEL_GEN(e) == 3
+IS_GEN3(e)
@gen4@ expression e; @@
-INTEL_GEN(e) == 4
+IS_GEN4(e)
@gen5@ expression e; @@
-INTEL_GEN(e) == 5
+IS_GEN5(e)
@gen6@ expression e; @@
-INTEL_GEN(e) == 6
+IS_GEN6(e)
@gen7@ expression e; @@
-INTEL_GEN(e) == 7
+IS_GEN7(e)
@gen8@ expression e; @@
-INTEL_GEN(e) == 8
+IS_GEN8(e)
@gen9@ expression e; @@
-INTEL_GEN(e) == 9
+IS_GEN9(e)
@gen10@ expression e; @@
-INTEL_GEN(e) == 10
+IS_GEN10(e)
@gen11@ expression e; @@
-INTEL_GEN(e) == 11
+IS_GEN11(e)

Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Signed-off-by: default avatarRodrigo Vivi <rodrigo.vivi@intel.com>
Reviewed-by: default avatarVille Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20181026195143.20353-1-rodrigo.vivi@intel.com
parent 3ab0a6ed
...@@ -1330,7 +1330,7 @@ intel_get_dram_info(struct drm_i915_private *dev_priv) ...@@ -1330,7 +1330,7 @@ intel_get_dram_info(struct drm_i915_private *dev_priv)
/* Need to calculate bandwidth only for Gen9 */ /* Need to calculate bandwidth only for Gen9 */
if (IS_BROXTON(dev_priv)) if (IS_BROXTON(dev_priv))
ret = bxt_get_dram_info(dev_priv); ret = bxt_get_dram_info(dev_priv);
else if (INTEL_GEN(dev_priv) == 9) else if (IS_GEN9(dev_priv))
ret = skl_get_dram_info(dev_priv); ret = skl_get_dram_info(dev_priv);
else else
ret = skl_dram_get_channels_info(dev_priv); ret = skl_dram_get_channels_info(dev_priv);
......
...@@ -232,7 +232,7 @@ static void intel_atomic_setup_scaler(struct intel_crtc_scaler_state *scaler_sta ...@@ -232,7 +232,7 @@ static void intel_atomic_setup_scaler(struct intel_crtc_scaler_state *scaler_sta
if (plane_state && plane_state->base.fb && if (plane_state && plane_state->base.fb &&
plane_state->base.fb->format->is_yuv && plane_state->base.fb->format->is_yuv &&
plane_state->base.fb->format->num_planes > 1) { plane_state->base.fb->format->num_planes > 1) {
if (INTEL_GEN(dev_priv) == 9 && if (IS_GEN9(dev_priv) &&
!IS_GEMINILAKE(dev_priv)) { !IS_GEMINILAKE(dev_priv)) {
mode = SKL_PS_SCALER_MODE_NV12; mode = SKL_PS_SCALER_MODE_NV12;
} else if (icl_is_hdr_plane(to_intel_plane(plane_state->base.plane))) { } else if (icl_is_hdr_plane(to_intel_plane(plane_state->base.plane))) {
......
...@@ -744,7 +744,7 @@ void intel_device_info_runtime_init(struct intel_device_info *info) ...@@ -744,7 +744,7 @@ void intel_device_info_runtime_init(struct intel_device_info *info)
if (INTEL_GEN(dev_priv) >= 10) { if (INTEL_GEN(dev_priv) >= 10) {
for_each_pipe(dev_priv, pipe) for_each_pipe(dev_priv, pipe)
info->num_scalers[pipe] = 2; info->num_scalers[pipe] = 2;
} else if (INTEL_GEN(dev_priv) == 9) { } else if (IS_GEN9(dev_priv)) {
info->num_scalers[PIPE_A] = 2; info->num_scalers[PIPE_A] = 2;
info->num_scalers[PIPE_B] = 2; info->num_scalers[PIPE_B] = 2;
info->num_scalers[PIPE_C] = 1; info->num_scalers[PIPE_C] = 1;
...@@ -847,9 +847,9 @@ void intel_device_info_runtime_init(struct intel_device_info *info) ...@@ -847,9 +847,9 @@ void intel_device_info_runtime_init(struct intel_device_info *info)
cherryview_sseu_info_init(dev_priv); cherryview_sseu_info_init(dev_priv);
else if (IS_BROADWELL(dev_priv)) else if (IS_BROADWELL(dev_priv))
broadwell_sseu_info_init(dev_priv); broadwell_sseu_info_init(dev_priv);
else if (INTEL_GEN(dev_priv) == 9) else if (IS_GEN9(dev_priv))
gen9_sseu_info_init(dev_priv); gen9_sseu_info_init(dev_priv);
else if (INTEL_GEN(dev_priv) == 10) else if (IS_GEN10(dev_priv))
gen10_sseu_info_init(dev_priv); gen10_sseu_info_init(dev_priv);
else if (INTEL_GEN(dev_priv) >= 11) else if (INTEL_GEN(dev_priv) >= 11)
gen11_sseu_info_init(dev_priv); gen11_sseu_info_init(dev_priv);
......
...@@ -5238,7 +5238,7 @@ static bool needs_nv12_wa(struct drm_i915_private *dev_priv, ...@@ -5238,7 +5238,7 @@ static bool needs_nv12_wa(struct drm_i915_private *dev_priv,
if (IS_SKYLAKE(dev_priv) || IS_BROXTON(dev_priv)) if (IS_SKYLAKE(dev_priv) || IS_BROXTON(dev_priv))
return false; return false;
if ((INTEL_GEN(dev_priv) == 9 && !IS_GEMINILAKE(dev_priv)) || if ((IS_GEN9(dev_priv) && !IS_GEMINILAKE(dev_priv)) ||
IS_CANNONLAKE(dev_priv)) IS_CANNONLAKE(dev_priv))
return true; return true;
......
...@@ -455,7 +455,7 @@ intel_dp_set_source_rates(struct intel_dp *intel_dp) ...@@ -455,7 +455,7 @@ intel_dp_set_source_rates(struct intel_dp *intel_dp)
if (INTEL_GEN(dev_priv) >= 10) { if (INTEL_GEN(dev_priv) >= 10) {
source_rates = cnl_rates; source_rates = cnl_rates;
size = ARRAY_SIZE(cnl_rates); size = ARRAY_SIZE(cnl_rates);
if (INTEL_GEN(dev_priv) == 10) if (IS_GEN10(dev_priv))
max_rate = cnl_max_source_rate(intel_dp); max_rate = cnl_max_source_rate(intel_dp);
else else
max_rate = icl_max_source_rate(intel_dp); max_rate = icl_max_source_rate(intel_dp);
......
...@@ -812,7 +812,7 @@ u32 intel_calculate_mcr_s_ss_select(struct drm_i915_private *dev_priv) ...@@ -812,7 +812,7 @@ u32 intel_calculate_mcr_s_ss_select(struct drm_i915_private *dev_priv)
u32 slice = fls(sseu->slice_mask); u32 slice = fls(sseu->slice_mask);
u32 subslice = fls(sseu->subslice_mask[slice]); u32 subslice = fls(sseu->subslice_mask[slice]);
if (INTEL_GEN(dev_priv) == 10) if (IS_GEN10(dev_priv))
mcr_s_ss_select = GEN8_MCR_SLICE(slice) | mcr_s_ss_select = GEN8_MCR_SLICE(slice) |
GEN8_MCR_SUBSLICE(subslice); GEN8_MCR_SUBSLICE(subslice);
else if (INTEL_GEN(dev_priv) >= 11) else if (INTEL_GEN(dev_priv) >= 11)
......
...@@ -84,7 +84,7 @@ static int intel_fbc_calculate_cfb_size(struct drm_i915_private *dev_priv, ...@@ -84,7 +84,7 @@ static int intel_fbc_calculate_cfb_size(struct drm_i915_private *dev_priv,
int lines; int lines;
intel_fbc_get_plane_source_size(cache, NULL, &lines); intel_fbc_get_plane_source_size(cache, NULL, &lines);
if (INTEL_GEN(dev_priv) == 7) if (IS_GEN7(dev_priv))
lines = min(lines, 2048); lines = min(lines, 2048);
else if (INTEL_GEN(dev_priv) >= 8) else if (INTEL_GEN(dev_priv) >= 8)
lines = min(lines, 2560); lines = min(lines, 2560);
......
...@@ -4741,13 +4741,13 @@ static int skl_compute_plane_wm(const struct drm_i915_private *dev_priv, ...@@ -4741,13 +4741,13 @@ static int skl_compute_plane_wm(const struct drm_i915_private *dev_priv,
selected_result = method2; selected_result = method2;
} else if (ddb_allocation >= } else if (ddb_allocation >=
fixed16_to_u32_round_up(wp->plane_blocks_per_line)) { fixed16_to_u32_round_up(wp->plane_blocks_per_line)) {
if (INTEL_GEN(dev_priv) == 9 && if (IS_GEN9(dev_priv) &&
!IS_GEMINILAKE(dev_priv)) !IS_GEMINILAKE(dev_priv))
selected_result = min_fixed16(method1, method2); selected_result = min_fixed16(method1, method2);
else else
selected_result = method2; selected_result = method2;
} else if (latency >= wp->linetime_us) { } else if (latency >= wp->linetime_us) {
if (INTEL_GEN(dev_priv) == 9 && if (IS_GEN9(dev_priv) &&
!IS_GEMINILAKE(dev_priv)) !IS_GEMINILAKE(dev_priv))
selected_result = min_fixed16(method1, method2); selected_result = min_fixed16(method1, method2);
else else
......
...@@ -574,7 +574,7 @@ static void intel_psr_enable_source(struct intel_dp *intel_dp, ...@@ -574,7 +574,7 @@ static void intel_psr_enable_source(struct intel_dp *intel_dp,
if (dev_priv->psr.psr2_enabled) { if (dev_priv->psr.psr2_enabled) {
u32 chicken = I915_READ(CHICKEN_TRANS(cpu_transcoder)); u32 chicken = I915_READ(CHICKEN_TRANS(cpu_transcoder));
if (INTEL_GEN(dev_priv) == 9 && !IS_GEMINILAKE(dev_priv)) if (IS_GEN9(dev_priv) && !IS_GEMINILAKE(dev_priv))
chicken |= (PSR2_VSC_ENABLE_PROG_HEADER chicken |= (PSR2_VSC_ENABLE_PROG_HEADER
| PSR2_ADD_VERTICAL_LINE_COUNT); | PSR2_ADD_VERTICAL_LINE_COUNT);
......
...@@ -93,11 +93,11 @@ hangcheck_action_to_str(const enum intel_engine_hangcheck_action a) ...@@ -93,11 +93,11 @@ hangcheck_action_to_str(const enum intel_engine_hangcheck_action a)
#define I915_MAX_SUBSLICES 8 #define I915_MAX_SUBSLICES 8
#define instdone_slice_mask(dev_priv__) \ #define instdone_slice_mask(dev_priv__) \
(INTEL_GEN(dev_priv__) == 7 ? \ (IS_GEN7(dev_priv__) ? \
1 : INTEL_INFO(dev_priv__)->sseu.slice_mask) 1 : INTEL_INFO(dev_priv__)->sseu.slice_mask)
#define instdone_subslice_mask(dev_priv__) \ #define instdone_subslice_mask(dev_priv__) \
(INTEL_GEN(dev_priv__) == 7 ? \ (IS_GEN7(dev_priv__) ? \
1 : INTEL_INFO(dev_priv__)->sseu.subslice_mask[0]) 1 : INTEL_INFO(dev_priv__)->sseu.subslice_mask[0])
#define for_each_instdone_slice_subslice(dev_priv__, slice__, subslice__) \ #define for_each_instdone_slice_subslice(dev_priv__, slice__, subslice__) \
......
...@@ -1869,7 +1869,7 @@ static bool skl_plane_has_planar(struct drm_i915_private *dev_priv, ...@@ -1869,7 +1869,7 @@ static bool skl_plane_has_planar(struct drm_i915_private *dev_priv,
if (IS_SKYLAKE(dev_priv) || IS_BROXTON(dev_priv)) if (IS_SKYLAKE(dev_priv) || IS_BROXTON(dev_priv))
return false; return false;
if (INTEL_GEN(dev_priv) == 9 && !IS_GEMINILAKE(dev_priv) && pipe == PIPE_C) if (IS_GEN9(dev_priv) && !IS_GEMINILAKE(dev_priv) && pipe == PIPE_C)
return false; return false;
if (plane_id != PLANE_PRIMARY && plane_id != PLANE_SPRITE0) if (plane_id != PLANE_PRIMARY && plane_id != PLANE_SPRITE0)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment