* [Intel-gfx] [PATCH 01/10] drm/i915: simplify prefixes on device_info
2019-12-20 17:27 [Intel-gfx] [PATCH 00/10] Prefer acronym for prefixes Lucas De Marchi
@ 2019-12-20 17:27 ` Lucas De Marchi
2019-12-20 17:45 ` Ville Syrjälä
2019-12-20 17:27 ` [Intel-gfx] [PATCH 02/10] drm/i915: prefer 3-letter acronym for pineview Lucas De Marchi
` (11 subsequent siblings)
12 siblings, 1 reply; 17+ messages in thread
From: Lucas De Marchi @ 2019-12-20 17:27 UTC (permalink / raw)
To: intel-gfx
Drop the intel prefix since all these structs are static and prefer
using the 3-letter prefix for each platform.
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
---
drivers/gpu/drm/i915/i915_pci.c | 230 ++++++++++++++++----------------
1 file changed, 115 insertions(+), 115 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_pci.c b/drivers/gpu/drm/i915/i915_pci.c
index 9571611b4b16..4cd25a83e6f1 100644
--- a/drivers/gpu/drm/i915/i915_pci.c
+++ b/drivers/gpu/drm/i915/i915_pci.c
@@ -193,23 +193,23 @@
GEN_DEFAULT_PAGE_SIZES, \
GEN_DEFAULT_REGIONS
-static const struct intel_device_info intel_i830_info = {
+static const struct intel_device_info i830_info = {
I830_FEATURES,
PLATFORM(INTEL_I830),
};
-static const struct intel_device_info intel_i845g_info = {
+static const struct intel_device_info i845g_info = {
I845_FEATURES,
PLATFORM(INTEL_I845G),
};
-static const struct intel_device_info intel_i85x_info = {
+static const struct intel_device_info i85x_info = {
I830_FEATURES,
PLATFORM(INTEL_I85X),
.display.has_fbc = 1,
};
-static const struct intel_device_info intel_i865g_info = {
+static const struct intel_device_info i865g_info = {
I845_FEATURES,
PLATFORM(INTEL_I865G),
};
@@ -228,7 +228,7 @@ static const struct intel_device_info intel_i865g_info = {
GEN_DEFAULT_PAGE_SIZES, \
GEN_DEFAULT_REGIONS
-static const struct intel_device_info intel_i915g_info = {
+static const struct intel_device_info i915g_info = {
GEN3_FEATURES,
PLATFORM(INTEL_I915G),
.has_coherent_ggtt = false,
@@ -239,7 +239,7 @@ static const struct intel_device_info intel_i915g_info = {
.unfenced_needs_alignment = 1,
};
-static const struct intel_device_info intel_i915gm_info = {
+static const struct intel_device_info i915gm_info = {
GEN3_FEATURES,
PLATFORM(INTEL_I915GM),
.is_mobile = 1,
@@ -252,7 +252,7 @@ static const struct intel_device_info intel_i915gm_info = {
.unfenced_needs_alignment = 1,
};
-static const struct intel_device_info intel_i945g_info = {
+static const struct intel_device_info i945g_info = {
GEN3_FEATURES,
PLATFORM(INTEL_I945G),
.display.has_hotplug = 1,
@@ -263,7 +263,7 @@ static const struct intel_device_info intel_i945g_info = {
.unfenced_needs_alignment = 1,
};
-static const struct intel_device_info intel_i945gm_info = {
+static const struct intel_device_info i945gm_info = {
GEN3_FEATURES,
PLATFORM(INTEL_I945GM),
.is_mobile = 1,
@@ -277,21 +277,21 @@ static const struct intel_device_info intel_i945gm_info = {
.unfenced_needs_alignment = 1,
};
-static const struct intel_device_info intel_g33_info = {
+static const struct intel_device_info g33_info = {
GEN3_FEATURES,
PLATFORM(INTEL_G33),
.display.has_hotplug = 1,
.display.has_overlay = 1,
};
-static const struct intel_device_info intel_pineview_g_info = {
+static const struct intel_device_info pnv_g_info = {
GEN3_FEATURES,
PLATFORM(INTEL_PINEVIEW),
.display.has_hotplug = 1,
.display.has_overlay = 1,
};
-static const struct intel_device_info intel_pineview_m_info = {
+static const struct intel_device_info pnv_m_info = {
GEN3_FEATURES,
PLATFORM(INTEL_PINEVIEW),
.is_mobile = 1,
@@ -314,7 +314,7 @@ static const struct intel_device_info intel_pineview_m_info = {
GEN_DEFAULT_PAGE_SIZES, \
GEN_DEFAULT_REGIONS
-static const struct intel_device_info intel_i965g_info = {
+static const struct intel_device_info i965g_info = {
GEN4_FEATURES,
PLATFORM(INTEL_I965G),
.display.has_overlay = 1,
@@ -322,7 +322,7 @@ static const struct intel_device_info intel_i965g_info = {
.has_snoop = false,
};
-static const struct intel_device_info intel_i965gm_info = {
+static const struct intel_device_info i965gm_info = {
GEN4_FEATURES,
PLATFORM(INTEL_I965GM),
.is_mobile = 1,
@@ -333,14 +333,14 @@ static const struct intel_device_info intel_i965gm_info = {
.has_snoop = false,
};
-static const struct intel_device_info intel_g45_info = {
+static const struct intel_device_info g45_info = {
GEN4_FEATURES,
PLATFORM(INTEL_G45),
.engine_mask = BIT(RCS0) | BIT(VCS0),
.gpu_reset_clobbers_display = false,
};
-static const struct intel_device_info intel_gm45_info = {
+static const struct intel_device_info gm45_info = {
GEN4_FEATURES,
PLATFORM(INTEL_GM45),
.is_mobile = 1,
@@ -365,12 +365,12 @@ static const struct intel_device_info intel_gm45_info = {
GEN_DEFAULT_PAGE_SIZES, \
GEN_DEFAULT_REGIONS
-static const struct intel_device_info intel_ironlake_d_info = {
+static const struct intel_device_info ilk_d_info = {
GEN5_FEATURES,
PLATFORM(INTEL_IRONLAKE),
};
-static const struct intel_device_info intel_ironlake_m_info = {
+static const struct intel_device_info ilk_m_info = {
GEN5_FEATURES,
PLATFORM(INTEL_IRONLAKE),
.is_mobile = 1,
@@ -400,12 +400,12 @@ static const struct intel_device_info intel_ironlake_m_info = {
GEN6_FEATURES, \
PLATFORM(INTEL_SANDYBRIDGE)
-static const struct intel_device_info intel_sandybridge_d_gt1_info = {
+static const struct intel_device_info snb_d_gt1_info = {
SNB_D_PLATFORM,
.gt = 1,
};
-static const struct intel_device_info intel_sandybridge_d_gt2_info = {
+static const struct intel_device_info snb_d_gt2_info = {
SNB_D_PLATFORM,
.gt = 2,
};
@@ -416,12 +416,12 @@ static const struct intel_device_info intel_sandybridge_d_gt2_info = {
.is_mobile = 1
-static const struct intel_device_info intel_sandybridge_m_gt1_info = {
+static const struct intel_device_info snb_m_gt1_info = {
SNB_M_PLATFORM,
.gt = 1,
};
-static const struct intel_device_info intel_sandybridge_m_gt2_info = {
+static const struct intel_device_info snb_m_gt2_info = {
SNB_M_PLATFORM,
.gt = 2,
};
@@ -450,12 +450,12 @@ static const struct intel_device_info intel_sandybridge_m_gt2_info = {
PLATFORM(INTEL_IVYBRIDGE), \
.has_l3_dpf = 1
-static const struct intel_device_info intel_ivybridge_d_gt1_info = {
+static const struct intel_device_info ivb_d_gt1_info = {
IVB_D_PLATFORM,
.gt = 1,
};
-static const struct intel_device_info intel_ivybridge_d_gt2_info = {
+static const struct intel_device_info ivb_d_gt2_info = {
IVB_D_PLATFORM,
.gt = 2,
};
@@ -466,17 +466,17 @@ static const struct intel_device_info intel_ivybridge_d_gt2_info = {
.is_mobile = 1, \
.has_l3_dpf = 1
-static const struct intel_device_info intel_ivybridge_m_gt1_info = {
+static const struct intel_device_info ivb_m_gt1_info = {
IVB_M_PLATFORM,
.gt = 1,
};
-static const struct intel_device_info intel_ivybridge_m_gt2_info = {
+static const struct intel_device_info ivb_m_gt2_info = {
IVB_M_PLATFORM,
.gt = 2,
};
-static const struct intel_device_info intel_ivybridge_q_info = {
+static const struct intel_device_info ivb_q_info = {
GEN7_FEATURES,
PLATFORM(INTEL_IVYBRIDGE),
.gt = 2,
@@ -484,7 +484,7 @@ static const struct intel_device_info intel_ivybridge_q_info = {
.has_l3_dpf = 1,
};
-static const struct intel_device_info intel_valleyview_info = {
+static const struct intel_device_info vlv_info = {
PLATFORM(INTEL_VALLEYVIEW),
GEN(7),
.is_lp = 1,
@@ -523,17 +523,17 @@ static const struct intel_device_info intel_valleyview_info = {
PLATFORM(INTEL_HASWELL), \
.has_l3_dpf = 1
-static const struct intel_device_info intel_haswell_gt1_info = {
+static const struct intel_device_info hsw_gt1_info = {
HSW_PLATFORM,
.gt = 1,
};
-static const struct intel_device_info intel_haswell_gt2_info = {
+static const struct intel_device_info hsw_gt2_info = {
HSW_PLATFORM,
.gt = 2,
};
-static const struct intel_device_info intel_haswell_gt3_info = {
+static const struct intel_device_info hsw_gt3_info = {
HSW_PLATFORM,
.gt = 3,
};
@@ -551,17 +551,17 @@ static const struct intel_device_info intel_haswell_gt3_info = {
GEN8_FEATURES, \
PLATFORM(INTEL_BROADWELL)
-static const struct intel_device_info intel_broadwell_gt1_info = {
+static const struct intel_device_info bdw_gt1_info = {
BDW_PLATFORM,
.gt = 1,
};
-static const struct intel_device_info intel_broadwell_gt2_info = {
+static const struct intel_device_info bdw_gt2_info = {
BDW_PLATFORM,
.gt = 2,
};
-static const struct intel_device_info intel_broadwell_rsvd_info = {
+static const struct intel_device_info bdw_rsvd_info = {
BDW_PLATFORM,
.gt = 3,
/* According to the device ID those devices are GT3, they were
@@ -569,14 +569,14 @@ static const struct intel_device_info intel_broadwell_rsvd_info = {
*/
};
-static const struct intel_device_info intel_broadwell_gt3_info = {
+static const struct intel_device_info bdw_gt3_info = {
BDW_PLATFORM,
.gt = 3,
.engine_mask =
BIT(RCS0) | BIT(VCS0) | BIT(BCS0) | BIT(VECS0) | BIT(VCS1),
};
-static const struct intel_device_info intel_cherryview_info = {
+static const struct intel_device_info chv_info = {
PLATFORM(INTEL_CHERRYVIEW),
GEN(8),
.pipe_mask = BIT(PIPE_A) | BIT(PIPE_B) | BIT(PIPE_C),
@@ -621,12 +621,12 @@ static const struct intel_device_info intel_cherryview_info = {
GEN9_FEATURES, \
PLATFORM(INTEL_SKYLAKE)
-static const struct intel_device_info intel_skylake_gt1_info = {
+static const struct intel_device_info skl_gt1_info = {
SKL_PLATFORM,
.gt = 1,
};
-static const struct intel_device_info intel_skylake_gt2_info = {
+static const struct intel_device_info skl_gt2_info = {
SKL_PLATFORM,
.gt = 2,
};
@@ -637,12 +637,12 @@ static const struct intel_device_info intel_skylake_gt2_info = {
BIT(RCS0) | BIT(VCS0) | BIT(BCS0) | BIT(VECS0) | BIT(VCS1)
-static const struct intel_device_info intel_skylake_gt3_info = {
+static const struct intel_device_info skl_gt3_info = {
SKL_GT3_PLUS_PLATFORM,
.gt = 3,
};
-static const struct intel_device_info intel_skylake_gt4_info = {
+static const struct intel_device_info skl_gt4_info = {
SKL_GT3_PLUS_PLATFORM,
.gt = 4,
};
@@ -679,13 +679,13 @@ static const struct intel_device_info intel_skylake_gt4_info = {
GEN9_DEFAULT_PAGE_SIZES, \
GEN_DEFAULT_REGIONS
-static const struct intel_device_info intel_broxton_info = {
+static const struct intel_device_info bxt_info = {
GEN9_LP_FEATURES,
PLATFORM(INTEL_BROXTON),
.ddb_size = 512,
};
-static const struct intel_device_info intel_geminilake_info = {
+static const struct intel_device_info glk_info = {
GEN9_LP_FEATURES,
PLATFORM(INTEL_GEMINILAKE),
.ddb_size = 1024,
@@ -696,17 +696,17 @@ static const struct intel_device_info intel_geminilake_info = {
GEN9_FEATURES, \
PLATFORM(INTEL_KABYLAKE)
-static const struct intel_device_info intel_kabylake_gt1_info = {
+static const struct intel_device_info kbl_gt1_info = {
KBL_PLATFORM,
.gt = 1,
};
-static const struct intel_device_info intel_kabylake_gt2_info = {
+static const struct intel_device_info kbl_gt2_info = {
KBL_PLATFORM,
.gt = 2,
};
-static const struct intel_device_info intel_kabylake_gt3_info = {
+static const struct intel_device_info kbl_gt3_info = {
KBL_PLATFORM,
.gt = 3,
.engine_mask =
@@ -717,17 +717,17 @@ static const struct intel_device_info intel_kabylake_gt3_info = {
GEN9_FEATURES, \
PLATFORM(INTEL_COFFEELAKE)
-static const struct intel_device_info intel_coffeelake_gt1_info = {
+static const struct intel_device_info cfl_gt1_info = {
CFL_PLATFORM,
.gt = 1,
};
-static const struct intel_device_info intel_coffeelake_gt2_info = {
+static const struct intel_device_info cfl_gt2_info = {
CFL_PLATFORM,
.gt = 2,
};
-static const struct intel_device_info intel_coffeelake_gt3_info = {
+static const struct intel_device_info cfl_gt3_info = {
CFL_PLATFORM,
.gt = 3,
.engine_mask =
@@ -742,7 +742,7 @@ static const struct intel_device_info intel_coffeelake_gt3_info = {
.has_coherent_ggtt = false, \
GLK_COLORS
-static const struct intel_device_info intel_cannonlake_info = {
+static const struct intel_device_info cnl_info = {
GEN10_FEATURES,
PLATFORM(INTEL_CANNONLAKE),
.gt = 2,
@@ -777,14 +777,14 @@ static const struct intel_device_info intel_cannonlake_info = {
.has_logical_ring_elsq = 1, \
.color = { .degamma_lut_size = 33, .gamma_lut_size = 262145 }
-static const struct intel_device_info intel_icelake_11_info = {
+static const struct intel_device_info icl_11_info = {
GEN11_FEATURES,
PLATFORM(INTEL_ICELAKE),
.engine_mask =
BIT(RCS0) | BIT(BCS0) | BIT(VECS0) | BIT(VCS0) | BIT(VCS2),
};
-static const struct intel_device_info intel_elkhartlake_info = {
+static const struct intel_device_info ehl_info = {
GEN11_FEATURES,
PLATFORM(INTEL_ELKHARTLAKE),
.require_force_probe = 1,
@@ -815,7 +815,7 @@ static const struct intel_device_info intel_elkhartlake_info = {
.has_global_mocs = 1, \
.display.has_dsb = 1
-static const struct intel_device_info intel_tigerlake_12_info = {
+static const struct intel_device_info tgl_12_info = {
GEN12_FEATURES,
PLATFORM(INTEL_TIGERLAKE),
.pipe_mask = BIT(PIPE_A) | BIT(PIPE_B) | BIT(PIPE_C) | BIT(PIPE_D),
@@ -840,70 +840,70 @@ static const struct intel_device_info intel_tigerlake_12_info = {
* PCI ID matches, otherwise we'll use the wrong info struct above.
*/
static const struct pci_device_id pciidlist[] = {
- INTEL_I830_IDS(&intel_i830_info),
- INTEL_I845G_IDS(&intel_i845g_info),
- INTEL_I85X_IDS(&intel_i85x_info),
- INTEL_I865G_IDS(&intel_i865g_info),
- INTEL_I915G_IDS(&intel_i915g_info),
- INTEL_I915GM_IDS(&intel_i915gm_info),
- INTEL_I945G_IDS(&intel_i945g_info),
- INTEL_I945GM_IDS(&intel_i945gm_info),
- INTEL_I965G_IDS(&intel_i965g_info),
- INTEL_G33_IDS(&intel_g33_info),
- INTEL_I965GM_IDS(&intel_i965gm_info),
- INTEL_GM45_IDS(&intel_gm45_info),
- INTEL_G45_IDS(&intel_g45_info),
- INTEL_PINEVIEW_G_IDS(&intel_pineview_g_info),
- INTEL_PINEVIEW_M_IDS(&intel_pineview_m_info),
- INTEL_IRONLAKE_D_IDS(&intel_ironlake_d_info),
- INTEL_IRONLAKE_M_IDS(&intel_ironlake_m_info),
- INTEL_SNB_D_GT1_IDS(&intel_sandybridge_d_gt1_info),
- INTEL_SNB_D_GT2_IDS(&intel_sandybridge_d_gt2_info),
- INTEL_SNB_M_GT1_IDS(&intel_sandybridge_m_gt1_info),
- INTEL_SNB_M_GT2_IDS(&intel_sandybridge_m_gt2_info),
- INTEL_IVB_Q_IDS(&intel_ivybridge_q_info), /* must be first IVB */
- INTEL_IVB_M_GT1_IDS(&intel_ivybridge_m_gt1_info),
- INTEL_IVB_M_GT2_IDS(&intel_ivybridge_m_gt2_info),
- INTEL_IVB_D_GT1_IDS(&intel_ivybridge_d_gt1_info),
- INTEL_IVB_D_GT2_IDS(&intel_ivybridge_d_gt2_info),
- INTEL_HSW_GT1_IDS(&intel_haswell_gt1_info),
- INTEL_HSW_GT2_IDS(&intel_haswell_gt2_info),
- INTEL_HSW_GT3_IDS(&intel_haswell_gt3_info),
- INTEL_VLV_IDS(&intel_valleyview_info),
- INTEL_BDW_GT1_IDS(&intel_broadwell_gt1_info),
- INTEL_BDW_GT2_IDS(&intel_broadwell_gt2_info),
- INTEL_BDW_GT3_IDS(&intel_broadwell_gt3_info),
- INTEL_BDW_RSVD_IDS(&intel_broadwell_rsvd_info),
- INTEL_CHV_IDS(&intel_cherryview_info),
- INTEL_SKL_GT1_IDS(&intel_skylake_gt1_info),
- INTEL_SKL_GT2_IDS(&intel_skylake_gt2_info),
- INTEL_SKL_GT3_IDS(&intel_skylake_gt3_info),
- INTEL_SKL_GT4_IDS(&intel_skylake_gt4_info),
- INTEL_BXT_IDS(&intel_broxton_info),
- INTEL_GLK_IDS(&intel_geminilake_info),
- INTEL_KBL_GT1_IDS(&intel_kabylake_gt1_info),
- INTEL_KBL_GT2_IDS(&intel_kabylake_gt2_info),
- INTEL_KBL_GT3_IDS(&intel_kabylake_gt3_info),
- INTEL_KBL_GT4_IDS(&intel_kabylake_gt3_info),
- INTEL_AML_KBL_GT2_IDS(&intel_kabylake_gt2_info),
- INTEL_CFL_S_GT1_IDS(&intel_coffeelake_gt1_info),
- INTEL_CFL_S_GT2_IDS(&intel_coffeelake_gt2_info),
- INTEL_CFL_H_GT1_IDS(&intel_coffeelake_gt1_info),
- INTEL_CFL_H_GT2_IDS(&intel_coffeelake_gt2_info),
- INTEL_CFL_U_GT2_IDS(&intel_coffeelake_gt2_info),
- INTEL_CFL_U_GT3_IDS(&intel_coffeelake_gt3_info),
- INTEL_WHL_U_GT1_IDS(&intel_coffeelake_gt1_info),
- INTEL_WHL_U_GT2_IDS(&intel_coffeelake_gt2_info),
- INTEL_AML_CFL_GT2_IDS(&intel_coffeelake_gt2_info),
- INTEL_WHL_U_GT3_IDS(&intel_coffeelake_gt3_info),
- INTEL_CML_GT1_IDS(&intel_coffeelake_gt1_info),
- INTEL_CML_GT2_IDS(&intel_coffeelake_gt2_info),
- INTEL_CML_U_GT1_IDS(&intel_coffeelake_gt1_info),
- INTEL_CML_U_GT2_IDS(&intel_coffeelake_gt2_info),
- INTEL_CNL_IDS(&intel_cannonlake_info),
- INTEL_ICL_11_IDS(&intel_icelake_11_info),
- INTEL_EHL_IDS(&intel_elkhartlake_info),
- INTEL_TGL_12_IDS(&intel_tigerlake_12_info),
+ INTEL_I830_IDS(&i830_info),
+ INTEL_I845G_IDS(&i845g_info),
+ INTEL_I85X_IDS(&i85x_info),
+ INTEL_I865G_IDS(&i865g_info),
+ INTEL_I915G_IDS(&i915g_info),
+ INTEL_I915GM_IDS(&i915gm_info),
+ INTEL_I945G_IDS(&i945g_info),
+ INTEL_I945GM_IDS(&i945gm_info),
+ INTEL_I965G_IDS(&i965g_info),
+ INTEL_G33_IDS(&g33_info),
+ INTEL_I965GM_IDS(&i965gm_info),
+ INTEL_GM45_IDS(&gm45_info),
+ INTEL_G45_IDS(&g45_info),
+ INTEL_PINEVIEW_G_IDS(&pnv_g_info),
+ INTEL_PINEVIEW_M_IDS(&pnv_m_info),
+ INTEL_IRONLAKE_D_IDS(&ilk_d_info),
+ INTEL_IRONLAKE_M_IDS(&ilk_m_info),
+ INTEL_SNB_D_GT1_IDS(&snb_d_gt1_info),
+ INTEL_SNB_D_GT2_IDS(&snb_d_gt2_info),
+ INTEL_SNB_M_GT1_IDS(&snb_m_gt1_info),
+ INTEL_SNB_M_GT2_IDS(&snb_m_gt2_info),
+ INTEL_IVB_Q_IDS(&ivb_q_info), /* must be first IVB */
+ INTEL_IVB_M_GT1_IDS(&ivb_m_gt1_info),
+ INTEL_IVB_M_GT2_IDS(&ivb_m_gt2_info),
+ INTEL_IVB_D_GT1_IDS(&ivb_d_gt1_info),
+ INTEL_IVB_D_GT2_IDS(&ivb_d_gt2_info),
+ INTEL_HSW_GT1_IDS(&hsw_gt1_info),
+ INTEL_HSW_GT2_IDS(&hsw_gt2_info),
+ INTEL_HSW_GT3_IDS(&hsw_gt3_info),
+ INTEL_VLV_IDS(&vlv_info),
+ INTEL_BDW_GT1_IDS(&bdw_gt1_info),
+ INTEL_BDW_GT2_IDS(&bdw_gt2_info),
+ INTEL_BDW_GT3_IDS(&bdw_gt3_info),
+ INTEL_BDW_RSVD_IDS(&bdw_rsvd_info),
+ INTEL_CHV_IDS(&chv_info),
+ INTEL_SKL_GT1_IDS(&skl_gt1_info),
+ INTEL_SKL_GT2_IDS(&skl_gt2_info),
+ INTEL_SKL_GT3_IDS(&skl_gt3_info),
+ INTEL_SKL_GT4_IDS(&skl_gt4_info),
+ INTEL_BXT_IDS(&bxt_info),
+ INTEL_GLK_IDS(&glk_info),
+ INTEL_KBL_GT1_IDS(&kbl_gt1_info),
+ INTEL_KBL_GT2_IDS(&kbl_gt2_info),
+ INTEL_KBL_GT3_IDS(&kbl_gt3_info),
+ INTEL_KBL_GT4_IDS(&kbl_gt3_info),
+ INTEL_AML_KBL_GT2_IDS(&kbl_gt2_info),
+ INTEL_CFL_S_GT1_IDS(&cfl_gt1_info),
+ INTEL_CFL_S_GT2_IDS(&cfl_gt2_info),
+ INTEL_CFL_H_GT1_IDS(&cfl_gt1_info),
+ INTEL_CFL_H_GT2_IDS(&cfl_gt2_info),
+ INTEL_CFL_U_GT2_IDS(&cfl_gt2_info),
+ INTEL_CFL_U_GT3_IDS(&cfl_gt3_info),
+ INTEL_WHL_U_GT1_IDS(&cfl_gt1_info),
+ INTEL_WHL_U_GT2_IDS(&cfl_gt2_info),
+ INTEL_AML_CFL_GT2_IDS(&cfl_gt2_info),
+ INTEL_WHL_U_GT3_IDS(&cfl_gt3_info),
+ INTEL_CML_GT1_IDS(&cfl_gt1_info),
+ INTEL_CML_GT2_IDS(&cfl_gt2_info),
+ INTEL_CML_U_GT1_IDS(&cfl_gt1_info),
+ INTEL_CML_U_GT2_IDS(&cfl_gt2_info),
+ INTEL_CNL_IDS(&cnl_info),
+ INTEL_ICL_11_IDS(&icl_11_info),
+ INTEL_EHL_IDS(&ehl_info),
+ INTEL_TGL_12_IDS(&tgl_12_info),
{0, 0, 0}
};
MODULE_DEVICE_TABLE(pci, pciidlist);
--
2.24.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 17+ messages in thread* Re: [Intel-gfx] [PATCH 01/10] drm/i915: simplify prefixes on device_info
2019-12-20 17:27 ` [Intel-gfx] [PATCH 01/10] drm/i915: simplify prefixes on device_info Lucas De Marchi
@ 2019-12-20 17:45 ` Ville Syrjälä
2019-12-20 18:01 ` Lucas De Marchi
0 siblings, 1 reply; 17+ messages in thread
From: Ville Syrjälä @ 2019-12-20 17:45 UTC (permalink / raw)
To: Lucas De Marchi; +Cc: intel-gfx
On Fri, Dec 20, 2019 at 09:27:45AM -0800, Lucas De Marchi wrote:
> Drop the intel prefix since all these structs are static and prefer
> using the 3-letter prefix for each platform.
>
> Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
> ---
> drivers/gpu/drm/i915/i915_pci.c | 230 ++++++++++++++++----------------
> 1 file changed, 115 insertions(+), 115 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_pci.c b/drivers/gpu/drm/i915/i915_pci.c
> index 9571611b4b16..4cd25a83e6f1 100644
> --- a/drivers/gpu/drm/i915/i915_pci.c
> +++ b/drivers/gpu/drm/i915/i915_pci.c
> @@ -193,23 +193,23 @@
> GEN_DEFAULT_PAGE_SIZES, \
> GEN_DEFAULT_REGIONS
>
> -static const struct intel_device_info intel_i830_info = {
> +static const struct intel_device_info i830_info = {
> I830_FEATURES,
> PLATFORM(INTEL_I830),
> };
>
> -static const struct intel_device_info intel_i845g_info = {
> +static const struct intel_device_info i845g_info = {
> I845_FEATURES,
> PLATFORM(INTEL_I845G),
> };
>
> -static const struct intel_device_info intel_i85x_info = {
> +static const struct intel_device_info i85x_info = {
> I830_FEATURES,
> PLATFORM(INTEL_I85X),
> .display.has_fbc = 1,
> };
>
> -static const struct intel_device_info intel_i865g_info = {
> +static const struct intel_device_info i865g_info = {
> I845_FEATURES,
> PLATFORM(INTEL_I865G),
> };
> @@ -228,7 +228,7 @@ static const struct intel_device_info intel_i865g_info = {
> GEN_DEFAULT_PAGE_SIZES, \
> GEN_DEFAULT_REGIONS
>
> -static const struct intel_device_info intel_i915g_info = {
> +static const struct intel_device_info i915g_info = {
> GEN3_FEATURES,
> PLATFORM(INTEL_I915G),
> .has_coherent_ggtt = false,
> @@ -239,7 +239,7 @@ static const struct intel_device_info intel_i915g_info = {
> .unfenced_needs_alignment = 1,
> };
>
> -static const struct intel_device_info intel_i915gm_info = {
> +static const struct intel_device_info i915gm_info = {
> GEN3_FEATURES,
> PLATFORM(INTEL_I915GM),
> .is_mobile = 1,
> @@ -252,7 +252,7 @@ static const struct intel_device_info intel_i915gm_info = {
> .unfenced_needs_alignment = 1,
> };
>
> -static const struct intel_device_info intel_i945g_info = {
> +static const struct intel_device_info i945g_info = {
> GEN3_FEATURES,
> PLATFORM(INTEL_I945G),
> .display.has_hotplug = 1,
> @@ -263,7 +263,7 @@ static const struct intel_device_info intel_i945g_info = {
> .unfenced_needs_alignment = 1,
> };
>
> -static const struct intel_device_info intel_i945gm_info = {
> +static const struct intel_device_info i945gm_info = {
> GEN3_FEATURES,
> PLATFORM(INTEL_I945GM),
> .is_mobile = 1,
> @@ -277,21 +277,21 @@ static const struct intel_device_info intel_i945gm_info = {
> .unfenced_needs_alignment = 1,
> };
>
> -static const struct intel_device_info intel_g33_info = {
> +static const struct intel_device_info g33_info = {
> GEN3_FEATURES,
> PLATFORM(INTEL_G33),
> .display.has_hotplug = 1,
> .display.has_overlay = 1,
> };
>
> -static const struct intel_device_info intel_pineview_g_info = {
> +static const struct intel_device_info pnv_g_info = {
> GEN3_FEATURES,
> PLATFORM(INTEL_PINEVIEW),
> .display.has_hotplug = 1,
> .display.has_overlay = 1,
> };
>
> -static const struct intel_device_info intel_pineview_m_info = {
> +static const struct intel_device_info pnv_m_info = {
> GEN3_FEATURES,
> PLATFORM(INTEL_PINEVIEW),
> .is_mobile = 1,
> @@ -314,7 +314,7 @@ static const struct intel_device_info intel_pineview_m_info = {
> GEN_DEFAULT_PAGE_SIZES, \
> GEN_DEFAULT_REGIONS
>
> -static const struct intel_device_info intel_i965g_info = {
> +static const struct intel_device_info i965g_info = {
> GEN4_FEATURES,
> PLATFORM(INTEL_I965G),
> .display.has_overlay = 1,
> @@ -322,7 +322,7 @@ static const struct intel_device_info intel_i965g_info = {
> .has_snoop = false,
> };
>
> -static const struct intel_device_info intel_i965gm_info = {
> +static const struct intel_device_info i965gm_info = {
> GEN4_FEATURES,
> PLATFORM(INTEL_I965GM),
> .is_mobile = 1,
> @@ -333,14 +333,14 @@ static const struct intel_device_info intel_i965gm_info = {
> .has_snoop = false,
> };
>
> -static const struct intel_device_info intel_g45_info = {
> +static const struct intel_device_info g45_info = {
> GEN4_FEATURES,
> PLATFORM(INTEL_G45),
> .engine_mask = BIT(RCS0) | BIT(VCS0),
> .gpu_reset_clobbers_display = false,
> };
>
> -static const struct intel_device_info intel_gm45_info = {
> +static const struct intel_device_info gm45_info = {
> GEN4_FEATURES,
> PLATFORM(INTEL_GM45),
> .is_mobile = 1,
> @@ -365,12 +365,12 @@ static const struct intel_device_info intel_gm45_info = {
> GEN_DEFAULT_PAGE_SIZES, \
> GEN_DEFAULT_REGIONS
>
> -static const struct intel_device_info intel_ironlake_d_info = {
> +static const struct intel_device_info ilk_d_info = {
> GEN5_FEATURES,
> PLATFORM(INTEL_IRONLAKE),
> };
>
> -static const struct intel_device_info intel_ironlake_m_info = {
> +static const struct intel_device_info ilk_m_info = {
> GEN5_FEATURES,
> PLATFORM(INTEL_IRONLAKE),
> .is_mobile = 1,
> @@ -400,12 +400,12 @@ static const struct intel_device_info intel_ironlake_m_info = {
> GEN6_FEATURES, \
> PLATFORM(INTEL_SANDYBRIDGE)
>
> -static const struct intel_device_info intel_sandybridge_d_gt1_info = {
> +static const struct intel_device_info snb_d_gt1_info = {
> SNB_D_PLATFORM,
> .gt = 1,
> };
>
> -static const struct intel_device_info intel_sandybridge_d_gt2_info = {
> +static const struct intel_device_info snb_d_gt2_info = {
> SNB_D_PLATFORM,
> .gt = 2,
> };
> @@ -416,12 +416,12 @@ static const struct intel_device_info intel_sandybridge_d_gt2_info = {
> .is_mobile = 1
>
>
> -static const struct intel_device_info intel_sandybridge_m_gt1_info = {
> +static const struct intel_device_info snb_m_gt1_info = {
> SNB_M_PLATFORM,
> .gt = 1,
> };
>
> -static const struct intel_device_info intel_sandybridge_m_gt2_info = {
> +static const struct intel_device_info snb_m_gt2_info = {
> SNB_M_PLATFORM,
> .gt = 2,
> };
> @@ -450,12 +450,12 @@ static const struct intel_device_info intel_sandybridge_m_gt2_info = {
> PLATFORM(INTEL_IVYBRIDGE), \
> .has_l3_dpf = 1
>
> -static const struct intel_device_info intel_ivybridge_d_gt1_info = {
> +static const struct intel_device_info ivb_d_gt1_info = {
> IVB_D_PLATFORM,
> .gt = 1,
> };
>
> -static const struct intel_device_info intel_ivybridge_d_gt2_info = {
> +static const struct intel_device_info ivb_d_gt2_info = {
> IVB_D_PLATFORM,
> .gt = 2,
> };
> @@ -466,17 +466,17 @@ static const struct intel_device_info intel_ivybridge_d_gt2_info = {
> .is_mobile = 1, \
> .has_l3_dpf = 1
>
> -static const struct intel_device_info intel_ivybridge_m_gt1_info = {
> +static const struct intel_device_info ivb_m_gt1_info = {
> IVB_M_PLATFORM,
> .gt = 1,
> };
>
> -static const struct intel_device_info intel_ivybridge_m_gt2_info = {
> +static const struct intel_device_info ivb_m_gt2_info = {
> IVB_M_PLATFORM,
> .gt = 2,
> };
>
> -static const struct intel_device_info intel_ivybridge_q_info = {
> +static const struct intel_device_info ivb_q_info = {
> GEN7_FEATURES,
> PLATFORM(INTEL_IVYBRIDGE),
> .gt = 2,
> @@ -484,7 +484,7 @@ static const struct intel_device_info intel_ivybridge_q_info = {
> .has_l3_dpf = 1,
> };
>
> -static const struct intel_device_info intel_valleyview_info = {
> +static const struct intel_device_info vlv_info = {
> PLATFORM(INTEL_VALLEYVIEW),
> GEN(7),
> .is_lp = 1,
> @@ -523,17 +523,17 @@ static const struct intel_device_info intel_valleyview_info = {
> PLATFORM(INTEL_HASWELL), \
> .has_l3_dpf = 1
>
> -static const struct intel_device_info intel_haswell_gt1_info = {
> +static const struct intel_device_info hsw_gt1_info = {
> HSW_PLATFORM,
> .gt = 1,
> };
>
> -static const struct intel_device_info intel_haswell_gt2_info = {
> +static const struct intel_device_info hsw_gt2_info = {
> HSW_PLATFORM,
> .gt = 2,
> };
>
> -static const struct intel_device_info intel_haswell_gt3_info = {
> +static const struct intel_device_info hsw_gt3_info = {
> HSW_PLATFORM,
> .gt = 3,
> };
> @@ -551,17 +551,17 @@ static const struct intel_device_info intel_haswell_gt3_info = {
> GEN8_FEATURES, \
> PLATFORM(INTEL_BROADWELL)
>
> -static const struct intel_device_info intel_broadwell_gt1_info = {
> +static const struct intel_device_info bdw_gt1_info = {
> BDW_PLATFORM,
> .gt = 1,
> };
>
> -static const struct intel_device_info intel_broadwell_gt2_info = {
> +static const struct intel_device_info bdw_gt2_info = {
> BDW_PLATFORM,
> .gt = 2,
> };
>
> -static const struct intel_device_info intel_broadwell_rsvd_info = {
> +static const struct intel_device_info bdw_rsvd_info = {
> BDW_PLATFORM,
> .gt = 3,
> /* According to the device ID those devices are GT3, they were
> @@ -569,14 +569,14 @@ static const struct intel_device_info intel_broadwell_rsvd_info = {
> */
> };
>
> -static const struct intel_device_info intel_broadwell_gt3_info = {
> +static const struct intel_device_info bdw_gt3_info = {
> BDW_PLATFORM,
> .gt = 3,
> .engine_mask =
> BIT(RCS0) | BIT(VCS0) | BIT(BCS0) | BIT(VECS0) | BIT(VCS1),
> };
>
> -static const struct intel_device_info intel_cherryview_info = {
> +static const struct intel_device_info chv_info = {
> PLATFORM(INTEL_CHERRYVIEW),
> GEN(8),
> .pipe_mask = BIT(PIPE_A) | BIT(PIPE_B) | BIT(PIPE_C),
> @@ -621,12 +621,12 @@ static const struct intel_device_info intel_cherryview_info = {
> GEN9_FEATURES, \
> PLATFORM(INTEL_SKYLAKE)
>
> -static const struct intel_device_info intel_skylake_gt1_info = {
> +static const struct intel_device_info skl_gt1_info = {
> SKL_PLATFORM,
> .gt = 1,
> };
>
> -static const struct intel_device_info intel_skylake_gt2_info = {
> +static const struct intel_device_info skl_gt2_info = {
> SKL_PLATFORM,
> .gt = 2,
> };
> @@ -637,12 +637,12 @@ static const struct intel_device_info intel_skylake_gt2_info = {
> BIT(RCS0) | BIT(VCS0) | BIT(BCS0) | BIT(VECS0) | BIT(VCS1)
>
>
> -static const struct intel_device_info intel_skylake_gt3_info = {
> +static const struct intel_device_info skl_gt3_info = {
> SKL_GT3_PLUS_PLATFORM,
> .gt = 3,
> };
>
> -static const struct intel_device_info intel_skylake_gt4_info = {
> +static const struct intel_device_info skl_gt4_info = {
> SKL_GT3_PLUS_PLATFORM,
> .gt = 4,
> };
> @@ -679,13 +679,13 @@ static const struct intel_device_info intel_skylake_gt4_info = {
> GEN9_DEFAULT_PAGE_SIZES, \
> GEN_DEFAULT_REGIONS
>
> -static const struct intel_device_info intel_broxton_info = {
> +static const struct intel_device_info bxt_info = {
> GEN9_LP_FEATURES,
> PLATFORM(INTEL_BROXTON),
> .ddb_size = 512,
> };
>
> -static const struct intel_device_info intel_geminilake_info = {
> +static const struct intel_device_info glk_info = {
> GEN9_LP_FEATURES,
> PLATFORM(INTEL_GEMINILAKE),
> .ddb_size = 1024,
> @@ -696,17 +696,17 @@ static const struct intel_device_info intel_geminilake_info = {
> GEN9_FEATURES, \
> PLATFORM(INTEL_KABYLAKE)
>
> -static const struct intel_device_info intel_kabylake_gt1_info = {
> +static const struct intel_device_info kbl_gt1_info = {
> KBL_PLATFORM,
> .gt = 1,
> };
>
> -static const struct intel_device_info intel_kabylake_gt2_info = {
> +static const struct intel_device_info kbl_gt2_info = {
> KBL_PLATFORM,
> .gt = 2,
> };
>
> -static const struct intel_device_info intel_kabylake_gt3_info = {
> +static const struct intel_device_info kbl_gt3_info = {
> KBL_PLATFORM,
> .gt = 3,
> .engine_mask =
> @@ -717,17 +717,17 @@ static const struct intel_device_info intel_kabylake_gt3_info = {
> GEN9_FEATURES, \
> PLATFORM(INTEL_COFFEELAKE)
>
> -static const struct intel_device_info intel_coffeelake_gt1_info = {
> +static const struct intel_device_info cfl_gt1_info = {
> CFL_PLATFORM,
> .gt = 1,
> };
>
> -static const struct intel_device_info intel_coffeelake_gt2_info = {
> +static const struct intel_device_info cfl_gt2_info = {
> CFL_PLATFORM,
> .gt = 2,
> };
>
> -static const struct intel_device_info intel_coffeelake_gt3_info = {
> +static const struct intel_device_info cfl_gt3_info = {
> CFL_PLATFORM,
> .gt = 3,
> .engine_mask =
> @@ -742,7 +742,7 @@ static const struct intel_device_info intel_coffeelake_gt3_info = {
> .has_coherent_ggtt = false, \
> GLK_COLORS
>
> -static const struct intel_device_info intel_cannonlake_info = {
> +static const struct intel_device_info cnl_info = {
> GEN10_FEATURES,
> PLATFORM(INTEL_CANNONLAKE),
> .gt = 2,
> @@ -777,14 +777,14 @@ static const struct intel_device_info intel_cannonlake_info = {
> .has_logical_ring_elsq = 1, \
> .color = { .degamma_lut_size = 33, .gamma_lut_size = 262145 }
>
> -static const struct intel_device_info intel_icelake_11_info = {
> +static const struct intel_device_info icl_11_info = {
Why isn't that just icl_info ?
> GEN11_FEATURES,
> PLATFORM(INTEL_ICELAKE),
> .engine_mask =
> BIT(RCS0) | BIT(BCS0) | BIT(VECS0) | BIT(VCS0) | BIT(VCS2),
> };
>
> -static const struct intel_device_info intel_elkhartlake_info = {
> +static const struct intel_device_info ehl_info = {
> GEN11_FEATURES,
> PLATFORM(INTEL_ELKHARTLAKE),
> .require_force_probe = 1,
> @@ -815,7 +815,7 @@ static const struct intel_device_info intel_elkhartlake_info = {
> .has_global_mocs = 1, \
> .display.has_dsb = 1
>
> -static const struct intel_device_info intel_tigerlake_12_info = {
> +static const struct intel_device_info tgl_12_info = {
> GEN12_FEATURES,
> PLATFORM(INTEL_TIGERLAKE),
> .pipe_mask = BIT(PIPE_A) | BIT(PIPE_B) | BIT(PIPE_C) | BIT(PIPE_D),
> @@ -840,70 +840,70 @@ static const struct intel_device_info intel_tigerlake_12_info = {
> * PCI ID matches, otherwise we'll use the wrong info struct above.
> */
> static const struct pci_device_id pciidlist[] = {
> - INTEL_I830_IDS(&intel_i830_info),
> - INTEL_I845G_IDS(&intel_i845g_info),
> - INTEL_I85X_IDS(&intel_i85x_info),
> - INTEL_I865G_IDS(&intel_i865g_info),
> - INTEL_I915G_IDS(&intel_i915g_info),
> - INTEL_I915GM_IDS(&intel_i915gm_info),
> - INTEL_I945G_IDS(&intel_i945g_info),
> - INTEL_I945GM_IDS(&intel_i945gm_info),
> - INTEL_I965G_IDS(&intel_i965g_info),
> - INTEL_G33_IDS(&intel_g33_info),
> - INTEL_I965GM_IDS(&intel_i965gm_info),
> - INTEL_GM45_IDS(&intel_gm45_info),
> - INTEL_G45_IDS(&intel_g45_info),
> - INTEL_PINEVIEW_G_IDS(&intel_pineview_g_info),
> - INTEL_PINEVIEW_M_IDS(&intel_pineview_m_info),
> - INTEL_IRONLAKE_D_IDS(&intel_ironlake_d_info),
> - INTEL_IRONLAKE_M_IDS(&intel_ironlake_m_info),
> - INTEL_SNB_D_GT1_IDS(&intel_sandybridge_d_gt1_info),
> - INTEL_SNB_D_GT2_IDS(&intel_sandybridge_d_gt2_info),
> - INTEL_SNB_M_GT1_IDS(&intel_sandybridge_m_gt1_info),
> - INTEL_SNB_M_GT2_IDS(&intel_sandybridge_m_gt2_info),
> - INTEL_IVB_Q_IDS(&intel_ivybridge_q_info), /* must be first IVB */
> - INTEL_IVB_M_GT1_IDS(&intel_ivybridge_m_gt1_info),
> - INTEL_IVB_M_GT2_IDS(&intel_ivybridge_m_gt2_info),
> - INTEL_IVB_D_GT1_IDS(&intel_ivybridge_d_gt1_info),
> - INTEL_IVB_D_GT2_IDS(&intel_ivybridge_d_gt2_info),
> - INTEL_HSW_GT1_IDS(&intel_haswell_gt1_info),
> - INTEL_HSW_GT2_IDS(&intel_haswell_gt2_info),
> - INTEL_HSW_GT3_IDS(&intel_haswell_gt3_info),
> - INTEL_VLV_IDS(&intel_valleyview_info),
> - INTEL_BDW_GT1_IDS(&intel_broadwell_gt1_info),
> - INTEL_BDW_GT2_IDS(&intel_broadwell_gt2_info),
> - INTEL_BDW_GT3_IDS(&intel_broadwell_gt3_info),
> - INTEL_BDW_RSVD_IDS(&intel_broadwell_rsvd_info),
> - INTEL_CHV_IDS(&intel_cherryview_info),
> - INTEL_SKL_GT1_IDS(&intel_skylake_gt1_info),
> - INTEL_SKL_GT2_IDS(&intel_skylake_gt2_info),
> - INTEL_SKL_GT3_IDS(&intel_skylake_gt3_info),
> - INTEL_SKL_GT4_IDS(&intel_skylake_gt4_info),
> - INTEL_BXT_IDS(&intel_broxton_info),
> - INTEL_GLK_IDS(&intel_geminilake_info),
> - INTEL_KBL_GT1_IDS(&intel_kabylake_gt1_info),
> - INTEL_KBL_GT2_IDS(&intel_kabylake_gt2_info),
> - INTEL_KBL_GT3_IDS(&intel_kabylake_gt3_info),
> - INTEL_KBL_GT4_IDS(&intel_kabylake_gt3_info),
> - INTEL_AML_KBL_GT2_IDS(&intel_kabylake_gt2_info),
> - INTEL_CFL_S_GT1_IDS(&intel_coffeelake_gt1_info),
> - INTEL_CFL_S_GT2_IDS(&intel_coffeelake_gt2_info),
> - INTEL_CFL_H_GT1_IDS(&intel_coffeelake_gt1_info),
> - INTEL_CFL_H_GT2_IDS(&intel_coffeelake_gt2_info),
> - INTEL_CFL_U_GT2_IDS(&intel_coffeelake_gt2_info),
> - INTEL_CFL_U_GT3_IDS(&intel_coffeelake_gt3_info),
> - INTEL_WHL_U_GT1_IDS(&intel_coffeelake_gt1_info),
> - INTEL_WHL_U_GT2_IDS(&intel_coffeelake_gt2_info),
> - INTEL_AML_CFL_GT2_IDS(&intel_coffeelake_gt2_info),
> - INTEL_WHL_U_GT3_IDS(&intel_coffeelake_gt3_info),
> - INTEL_CML_GT1_IDS(&intel_coffeelake_gt1_info),
> - INTEL_CML_GT2_IDS(&intel_coffeelake_gt2_info),
> - INTEL_CML_U_GT1_IDS(&intel_coffeelake_gt1_info),
> - INTEL_CML_U_GT2_IDS(&intel_coffeelake_gt2_info),
> - INTEL_CNL_IDS(&intel_cannonlake_info),
> - INTEL_ICL_11_IDS(&intel_icelake_11_info),
> - INTEL_EHL_IDS(&intel_elkhartlake_info),
> - INTEL_TGL_12_IDS(&intel_tigerlake_12_info),
> + INTEL_I830_IDS(&i830_info),
> + INTEL_I845G_IDS(&i845g_info),
> + INTEL_I85X_IDS(&i85x_info),
> + INTEL_I865G_IDS(&i865g_info),
> + INTEL_I915G_IDS(&i915g_info),
> + INTEL_I915GM_IDS(&i915gm_info),
> + INTEL_I945G_IDS(&i945g_info),
> + INTEL_I945GM_IDS(&i945gm_info),
> + INTEL_I965G_IDS(&i965g_info),
> + INTEL_G33_IDS(&g33_info),
> + INTEL_I965GM_IDS(&i965gm_info),
> + INTEL_GM45_IDS(&gm45_info),
> + INTEL_G45_IDS(&g45_info),
> + INTEL_PINEVIEW_G_IDS(&pnv_g_info),
> + INTEL_PINEVIEW_M_IDS(&pnv_m_info),
> + INTEL_IRONLAKE_D_IDS(&ilk_d_info),
> + INTEL_IRONLAKE_M_IDS(&ilk_m_info),
> + INTEL_SNB_D_GT1_IDS(&snb_d_gt1_info),
> + INTEL_SNB_D_GT2_IDS(&snb_d_gt2_info),
> + INTEL_SNB_M_GT1_IDS(&snb_m_gt1_info),
> + INTEL_SNB_M_GT2_IDS(&snb_m_gt2_info),
> + INTEL_IVB_Q_IDS(&ivb_q_info), /* must be first IVB */
> + INTEL_IVB_M_GT1_IDS(&ivb_m_gt1_info),
> + INTEL_IVB_M_GT2_IDS(&ivb_m_gt2_info),
> + INTEL_IVB_D_GT1_IDS(&ivb_d_gt1_info),
> + INTEL_IVB_D_GT2_IDS(&ivb_d_gt2_info),
> + INTEL_HSW_GT1_IDS(&hsw_gt1_info),
> + INTEL_HSW_GT2_IDS(&hsw_gt2_info),
> + INTEL_HSW_GT3_IDS(&hsw_gt3_info),
> + INTEL_VLV_IDS(&vlv_info),
> + INTEL_BDW_GT1_IDS(&bdw_gt1_info),
> + INTEL_BDW_GT2_IDS(&bdw_gt2_info),
> + INTEL_BDW_GT3_IDS(&bdw_gt3_info),
> + INTEL_BDW_RSVD_IDS(&bdw_rsvd_info),
> + INTEL_CHV_IDS(&chv_info),
> + INTEL_SKL_GT1_IDS(&skl_gt1_info),
> + INTEL_SKL_GT2_IDS(&skl_gt2_info),
> + INTEL_SKL_GT3_IDS(&skl_gt3_info),
> + INTEL_SKL_GT4_IDS(&skl_gt4_info),
> + INTEL_BXT_IDS(&bxt_info),
> + INTEL_GLK_IDS(&glk_info),
> + INTEL_KBL_GT1_IDS(&kbl_gt1_info),
> + INTEL_KBL_GT2_IDS(&kbl_gt2_info),
> + INTEL_KBL_GT3_IDS(&kbl_gt3_info),
> + INTEL_KBL_GT4_IDS(&kbl_gt3_info),
> + INTEL_AML_KBL_GT2_IDS(&kbl_gt2_info),
> + INTEL_CFL_S_GT1_IDS(&cfl_gt1_info),
> + INTEL_CFL_S_GT2_IDS(&cfl_gt2_info),
> + INTEL_CFL_H_GT1_IDS(&cfl_gt1_info),
> + INTEL_CFL_H_GT2_IDS(&cfl_gt2_info),
> + INTEL_CFL_U_GT2_IDS(&cfl_gt2_info),
> + INTEL_CFL_U_GT3_IDS(&cfl_gt3_info),
> + INTEL_WHL_U_GT1_IDS(&cfl_gt1_info),
> + INTEL_WHL_U_GT2_IDS(&cfl_gt2_info),
> + INTEL_AML_CFL_GT2_IDS(&cfl_gt2_info),
> + INTEL_WHL_U_GT3_IDS(&cfl_gt3_info),
> + INTEL_CML_GT1_IDS(&cfl_gt1_info),
> + INTEL_CML_GT2_IDS(&cfl_gt2_info),
> + INTEL_CML_U_GT1_IDS(&cfl_gt1_info),
> + INTEL_CML_U_GT2_IDS(&cfl_gt2_info),
> + INTEL_CNL_IDS(&cnl_info),
> + INTEL_ICL_11_IDS(&icl_11_info),
> + INTEL_EHL_IDS(&ehl_info),
> + INTEL_TGL_12_IDS(&tgl_12_info),
> {0, 0, 0}
> };
> MODULE_DEVICE_TABLE(pci, pciidlist);
> --
> 2.24.0
--
Ville Syrjälä
Intel
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: [Intel-gfx] [PATCH 01/10] drm/i915: simplify prefixes on device_info
2019-12-20 17:45 ` Ville Syrjälä
@ 2019-12-20 18:01 ` Lucas De Marchi
2019-12-20 18:08 ` Ville Syrjälä
0 siblings, 1 reply; 17+ messages in thread
From: Lucas De Marchi @ 2019-12-20 18:01 UTC (permalink / raw)
To: Ville Syrjälä; +Cc: intel-gfx
On Fri, Dec 20, 2019 at 07:45:38PM +0200, Ville Syrjälä wrote:
>On Fri, Dec 20, 2019 at 09:27:45AM -0800, Lucas De Marchi wrote:
>> Drop the intel prefix since all these structs are static and prefer
>> using the 3-letter prefix for each platform.
>>
>> Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
>> ---
>> drivers/gpu/drm/i915/i915_pci.c | 230 ++++++++++++++++----------------
>> 1 file changed, 115 insertions(+), 115 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/i915/i915_pci.c b/drivers/gpu/drm/i915/i915_pci.c
>> index 9571611b4b16..4cd25a83e6f1 100644
>> --- a/drivers/gpu/drm/i915/i915_pci.c
>> +++ b/drivers/gpu/drm/i915/i915_pci.c
>> @@ -193,23 +193,23 @@
>> GEN_DEFAULT_PAGE_SIZES, \
>> GEN_DEFAULT_REGIONS
>>
>> -static const struct intel_device_info intel_i830_info = {
>> +static const struct intel_device_info i830_info = {
>> I830_FEATURES,
>> PLATFORM(INTEL_I830),
>> };
>>
>> -static const struct intel_device_info intel_i845g_info = {
>> +static const struct intel_device_info i845g_info = {
>> I845_FEATURES,
>> PLATFORM(INTEL_I845G),
>> };
>>
>> -static const struct intel_device_info intel_i85x_info = {
>> +static const struct intel_device_info i85x_info = {
>> I830_FEATURES,
>> PLATFORM(INTEL_I85X),
>> .display.has_fbc = 1,
>> };
>>
>> -static const struct intel_device_info intel_i865g_info = {
>> +static const struct intel_device_info i865g_info = {
>> I845_FEATURES,
>> PLATFORM(INTEL_I865G),
>> };
>> @@ -228,7 +228,7 @@ static const struct intel_device_info intel_i865g_info = {
>> GEN_DEFAULT_PAGE_SIZES, \
>> GEN_DEFAULT_REGIONS
>>
>> -static const struct intel_device_info intel_i915g_info = {
>> +static const struct intel_device_info i915g_info = {
>> GEN3_FEATURES,
>> PLATFORM(INTEL_I915G),
>> .has_coherent_ggtt = false,
>> @@ -239,7 +239,7 @@ static const struct intel_device_info intel_i915g_info = {
>> .unfenced_needs_alignment = 1,
>> };
>>
>> -static const struct intel_device_info intel_i915gm_info = {
>> +static const struct intel_device_info i915gm_info = {
>> GEN3_FEATURES,
>> PLATFORM(INTEL_I915GM),
>> .is_mobile = 1,
>> @@ -252,7 +252,7 @@ static const struct intel_device_info intel_i915gm_info = {
>> .unfenced_needs_alignment = 1,
>> };
>>
>> -static const struct intel_device_info intel_i945g_info = {
>> +static const struct intel_device_info i945g_info = {
>> GEN3_FEATURES,
>> PLATFORM(INTEL_I945G),
>> .display.has_hotplug = 1,
>> @@ -263,7 +263,7 @@ static const struct intel_device_info intel_i945g_info = {
>> .unfenced_needs_alignment = 1,
>> };
>>
>> -static const struct intel_device_info intel_i945gm_info = {
>> +static const struct intel_device_info i945gm_info = {
>> GEN3_FEATURES,
>> PLATFORM(INTEL_I945GM),
>> .is_mobile = 1,
>> @@ -277,21 +277,21 @@ static const struct intel_device_info intel_i945gm_info = {
>> .unfenced_needs_alignment = 1,
>> };
>>
>> -static const struct intel_device_info intel_g33_info = {
>> +static const struct intel_device_info g33_info = {
>> GEN3_FEATURES,
>> PLATFORM(INTEL_G33),
>> .display.has_hotplug = 1,
>> .display.has_overlay = 1,
>> };
>>
>> -static const struct intel_device_info intel_pineview_g_info = {
>> +static const struct intel_device_info pnv_g_info = {
>> GEN3_FEATURES,
>> PLATFORM(INTEL_PINEVIEW),
>> .display.has_hotplug = 1,
>> .display.has_overlay = 1,
>> };
>>
>> -static const struct intel_device_info intel_pineview_m_info = {
>> +static const struct intel_device_info pnv_m_info = {
>> GEN3_FEATURES,
>> PLATFORM(INTEL_PINEVIEW),
>> .is_mobile = 1,
>> @@ -314,7 +314,7 @@ static const struct intel_device_info intel_pineview_m_info = {
>> GEN_DEFAULT_PAGE_SIZES, \
>> GEN_DEFAULT_REGIONS
>>
>> -static const struct intel_device_info intel_i965g_info = {
>> +static const struct intel_device_info i965g_info = {
>> GEN4_FEATURES,
>> PLATFORM(INTEL_I965G),
>> .display.has_overlay = 1,
>> @@ -322,7 +322,7 @@ static const struct intel_device_info intel_i965g_info = {
>> .has_snoop = false,
>> };
>>
>> -static const struct intel_device_info intel_i965gm_info = {
>> +static const struct intel_device_info i965gm_info = {
>> GEN4_FEATURES,
>> PLATFORM(INTEL_I965GM),
>> .is_mobile = 1,
>> @@ -333,14 +333,14 @@ static const struct intel_device_info intel_i965gm_info = {
>> .has_snoop = false,
>> };
>>
>> -static const struct intel_device_info intel_g45_info = {
>> +static const struct intel_device_info g45_info = {
>> GEN4_FEATURES,
>> PLATFORM(INTEL_G45),
>> .engine_mask = BIT(RCS0) | BIT(VCS0),
>> .gpu_reset_clobbers_display = false,
>> };
>>
>> -static const struct intel_device_info intel_gm45_info = {
>> +static const struct intel_device_info gm45_info = {
>> GEN4_FEATURES,
>> PLATFORM(INTEL_GM45),
>> .is_mobile = 1,
>> @@ -365,12 +365,12 @@ static const struct intel_device_info intel_gm45_info = {
>> GEN_DEFAULT_PAGE_SIZES, \
>> GEN_DEFAULT_REGIONS
>>
>> -static const struct intel_device_info intel_ironlake_d_info = {
>> +static const struct intel_device_info ilk_d_info = {
>> GEN5_FEATURES,
>> PLATFORM(INTEL_IRONLAKE),
>> };
>>
>> -static const struct intel_device_info intel_ironlake_m_info = {
>> +static const struct intel_device_info ilk_m_info = {
>> GEN5_FEATURES,
>> PLATFORM(INTEL_IRONLAKE),
>> .is_mobile = 1,
>> @@ -400,12 +400,12 @@ static const struct intel_device_info intel_ironlake_m_info = {
>> GEN6_FEATURES, \
>> PLATFORM(INTEL_SANDYBRIDGE)
>>
>> -static const struct intel_device_info intel_sandybridge_d_gt1_info = {
>> +static const struct intel_device_info snb_d_gt1_info = {
>> SNB_D_PLATFORM,
>> .gt = 1,
>> };
>>
>> -static const struct intel_device_info intel_sandybridge_d_gt2_info = {
>> +static const struct intel_device_info snb_d_gt2_info = {
>> SNB_D_PLATFORM,
>> .gt = 2,
>> };
>> @@ -416,12 +416,12 @@ static const struct intel_device_info intel_sandybridge_d_gt2_info = {
>> .is_mobile = 1
>>
>>
>> -static const struct intel_device_info intel_sandybridge_m_gt1_info = {
>> +static const struct intel_device_info snb_m_gt1_info = {
>> SNB_M_PLATFORM,
>> .gt = 1,
>> };
>>
>> -static const struct intel_device_info intel_sandybridge_m_gt2_info = {
>> +static const struct intel_device_info snb_m_gt2_info = {
>> SNB_M_PLATFORM,
>> .gt = 2,
>> };
>> @@ -450,12 +450,12 @@ static const struct intel_device_info intel_sandybridge_m_gt2_info = {
>> PLATFORM(INTEL_IVYBRIDGE), \
>> .has_l3_dpf = 1
>>
>> -static const struct intel_device_info intel_ivybridge_d_gt1_info = {
>> +static const struct intel_device_info ivb_d_gt1_info = {
>> IVB_D_PLATFORM,
>> .gt = 1,
>> };
>>
>> -static const struct intel_device_info intel_ivybridge_d_gt2_info = {
>> +static const struct intel_device_info ivb_d_gt2_info = {
>> IVB_D_PLATFORM,
>> .gt = 2,
>> };
>> @@ -466,17 +466,17 @@ static const struct intel_device_info intel_ivybridge_d_gt2_info = {
>> .is_mobile = 1, \
>> .has_l3_dpf = 1
>>
>> -static const struct intel_device_info intel_ivybridge_m_gt1_info = {
>> +static const struct intel_device_info ivb_m_gt1_info = {
>> IVB_M_PLATFORM,
>> .gt = 1,
>> };
>>
>> -static const struct intel_device_info intel_ivybridge_m_gt2_info = {
>> +static const struct intel_device_info ivb_m_gt2_info = {
>> IVB_M_PLATFORM,
>> .gt = 2,
>> };
>>
>> -static const struct intel_device_info intel_ivybridge_q_info = {
>> +static const struct intel_device_info ivb_q_info = {
>> GEN7_FEATURES,
>> PLATFORM(INTEL_IVYBRIDGE),
>> .gt = 2,
>> @@ -484,7 +484,7 @@ static const struct intel_device_info intel_ivybridge_q_info = {
>> .has_l3_dpf = 1,
>> };
>>
>> -static const struct intel_device_info intel_valleyview_info = {
>> +static const struct intel_device_info vlv_info = {
>> PLATFORM(INTEL_VALLEYVIEW),
>> GEN(7),
>> .is_lp = 1,
>> @@ -523,17 +523,17 @@ static const struct intel_device_info intel_valleyview_info = {
>> PLATFORM(INTEL_HASWELL), \
>> .has_l3_dpf = 1
>>
>> -static const struct intel_device_info intel_haswell_gt1_info = {
>> +static const struct intel_device_info hsw_gt1_info = {
>> HSW_PLATFORM,
>> .gt = 1,
>> };
>>
>> -static const struct intel_device_info intel_haswell_gt2_info = {
>> +static const struct intel_device_info hsw_gt2_info = {
>> HSW_PLATFORM,
>> .gt = 2,
>> };
>>
>> -static const struct intel_device_info intel_haswell_gt3_info = {
>> +static const struct intel_device_info hsw_gt3_info = {
>> HSW_PLATFORM,
>> .gt = 3,
>> };
>> @@ -551,17 +551,17 @@ static const struct intel_device_info intel_haswell_gt3_info = {
>> GEN8_FEATURES, \
>> PLATFORM(INTEL_BROADWELL)
>>
>> -static const struct intel_device_info intel_broadwell_gt1_info = {
>> +static const struct intel_device_info bdw_gt1_info = {
>> BDW_PLATFORM,
>> .gt = 1,
>> };
>>
>> -static const struct intel_device_info intel_broadwell_gt2_info = {
>> +static const struct intel_device_info bdw_gt2_info = {
>> BDW_PLATFORM,
>> .gt = 2,
>> };
>>
>> -static const struct intel_device_info intel_broadwell_rsvd_info = {
>> +static const struct intel_device_info bdw_rsvd_info = {
>> BDW_PLATFORM,
>> .gt = 3,
>> /* According to the device ID those devices are GT3, they were
>> @@ -569,14 +569,14 @@ static const struct intel_device_info intel_broadwell_rsvd_info = {
>> */
>> };
>>
>> -static const struct intel_device_info intel_broadwell_gt3_info = {
>> +static const struct intel_device_info bdw_gt3_info = {
>> BDW_PLATFORM,
>> .gt = 3,
>> .engine_mask =
>> BIT(RCS0) | BIT(VCS0) | BIT(BCS0) | BIT(VECS0) | BIT(VCS1),
>> };
>>
>> -static const struct intel_device_info intel_cherryview_info = {
>> +static const struct intel_device_info chv_info = {
>> PLATFORM(INTEL_CHERRYVIEW),
>> GEN(8),
>> .pipe_mask = BIT(PIPE_A) | BIT(PIPE_B) | BIT(PIPE_C),
>> @@ -621,12 +621,12 @@ static const struct intel_device_info intel_cherryview_info = {
>> GEN9_FEATURES, \
>> PLATFORM(INTEL_SKYLAKE)
>>
>> -static const struct intel_device_info intel_skylake_gt1_info = {
>> +static const struct intel_device_info skl_gt1_info = {
>> SKL_PLATFORM,
>> .gt = 1,
>> };
>>
>> -static const struct intel_device_info intel_skylake_gt2_info = {
>> +static const struct intel_device_info skl_gt2_info = {
>> SKL_PLATFORM,
>> .gt = 2,
>> };
>> @@ -637,12 +637,12 @@ static const struct intel_device_info intel_skylake_gt2_info = {
>> BIT(RCS0) | BIT(VCS0) | BIT(BCS0) | BIT(VECS0) | BIT(VCS1)
>>
>>
>> -static const struct intel_device_info intel_skylake_gt3_info = {
>> +static const struct intel_device_info skl_gt3_info = {
>> SKL_GT3_PLUS_PLATFORM,
>> .gt = 3,
>> };
>>
>> -static const struct intel_device_info intel_skylake_gt4_info = {
>> +static const struct intel_device_info skl_gt4_info = {
>> SKL_GT3_PLUS_PLATFORM,
>> .gt = 4,
>> };
>> @@ -679,13 +679,13 @@ static const struct intel_device_info intel_skylake_gt4_info = {
>> GEN9_DEFAULT_PAGE_SIZES, \
>> GEN_DEFAULT_REGIONS
>>
>> -static const struct intel_device_info intel_broxton_info = {
>> +static const struct intel_device_info bxt_info = {
>> GEN9_LP_FEATURES,
>> PLATFORM(INTEL_BROXTON),
>> .ddb_size = 512,
>> };
>>
>> -static const struct intel_device_info intel_geminilake_info = {
>> +static const struct intel_device_info glk_info = {
>> GEN9_LP_FEATURES,
>> PLATFORM(INTEL_GEMINILAKE),
>> .ddb_size = 1024,
>> @@ -696,17 +696,17 @@ static const struct intel_device_info intel_geminilake_info = {
>> GEN9_FEATURES, \
>> PLATFORM(INTEL_KABYLAKE)
>>
>> -static const struct intel_device_info intel_kabylake_gt1_info = {
>> +static const struct intel_device_info kbl_gt1_info = {
>> KBL_PLATFORM,
>> .gt = 1,
>> };
>>
>> -static const struct intel_device_info intel_kabylake_gt2_info = {
>> +static const struct intel_device_info kbl_gt2_info = {
>> KBL_PLATFORM,
>> .gt = 2,
>> };
>>
>> -static const struct intel_device_info intel_kabylake_gt3_info = {
>> +static const struct intel_device_info kbl_gt3_info = {
>> KBL_PLATFORM,
>> .gt = 3,
>> .engine_mask =
>> @@ -717,17 +717,17 @@ static const struct intel_device_info intel_kabylake_gt3_info = {
>> GEN9_FEATURES, \
>> PLATFORM(INTEL_COFFEELAKE)
>>
>> -static const struct intel_device_info intel_coffeelake_gt1_info = {
>> +static const struct intel_device_info cfl_gt1_info = {
>> CFL_PLATFORM,
>> .gt = 1,
>> };
>>
>> -static const struct intel_device_info intel_coffeelake_gt2_info = {
>> +static const struct intel_device_info cfl_gt2_info = {
>> CFL_PLATFORM,
>> .gt = 2,
>> };
>>
>> -static const struct intel_device_info intel_coffeelake_gt3_info = {
>> +static const struct intel_device_info cfl_gt3_info = {
>> CFL_PLATFORM,
>> .gt = 3,
>> .engine_mask =
>> @@ -742,7 +742,7 @@ static const struct intel_device_info intel_coffeelake_gt3_info = {
>> .has_coherent_ggtt = false, \
>> GLK_COLORS
>>
>> -static const struct intel_device_info intel_cannonlake_info = {
>> +static const struct intel_device_info cnl_info = {
>> GEN10_FEATURES,
>> PLATFORM(INTEL_CANNONLAKE),
>> .gt = 2,
>> @@ -777,14 +777,14 @@ static const struct intel_device_info intel_cannonlake_info = {
>> .has_logical_ring_elsq = 1, \
>> .color = { .degamma_lut_size = 33, .gamma_lut_size = 262145 }
>>
>> -static const struct intel_device_info intel_icelake_11_info = {
>> +static const struct intel_device_info icl_11_info = {
>
>Why isn't that just icl_info ?
it should be, yes... I had a separate commit to remove the gen from the
device_info, I guess I dropped it instead of squashing.
Lucas De Marchi
>
>> GEN11_FEATURES,
>> PLATFORM(INTEL_ICELAKE),
>> .engine_mask =
>> BIT(RCS0) | BIT(BCS0) | BIT(VECS0) | BIT(VCS0) | BIT(VCS2),
>> };
>>
>> -static const struct intel_device_info intel_elkhartlake_info = {
>> +static const struct intel_device_info ehl_info = {
>> GEN11_FEATURES,
>> PLATFORM(INTEL_ELKHARTLAKE),
>> .require_force_probe = 1,
>> @@ -815,7 +815,7 @@ static const struct intel_device_info intel_elkhartlake_info = {
>> .has_global_mocs = 1, \
>> .display.has_dsb = 1
>>
>> -static const struct intel_device_info intel_tigerlake_12_info = {
>> +static const struct intel_device_info tgl_12_info = {
>> GEN12_FEATURES,
>> PLATFORM(INTEL_TIGERLAKE),
>> .pipe_mask = BIT(PIPE_A) | BIT(PIPE_B) | BIT(PIPE_C) | BIT(PIPE_D),
>> @@ -840,70 +840,70 @@ static const struct intel_device_info intel_tigerlake_12_info = {
>> * PCI ID matches, otherwise we'll use the wrong info struct above.
>> */
>> static const struct pci_device_id pciidlist[] = {
>> - INTEL_I830_IDS(&intel_i830_info),
>> - INTEL_I845G_IDS(&intel_i845g_info),
>> - INTEL_I85X_IDS(&intel_i85x_info),
>> - INTEL_I865G_IDS(&intel_i865g_info),
>> - INTEL_I915G_IDS(&intel_i915g_info),
>> - INTEL_I915GM_IDS(&intel_i915gm_info),
>> - INTEL_I945G_IDS(&intel_i945g_info),
>> - INTEL_I945GM_IDS(&intel_i945gm_info),
>> - INTEL_I965G_IDS(&intel_i965g_info),
>> - INTEL_G33_IDS(&intel_g33_info),
>> - INTEL_I965GM_IDS(&intel_i965gm_info),
>> - INTEL_GM45_IDS(&intel_gm45_info),
>> - INTEL_G45_IDS(&intel_g45_info),
>> - INTEL_PINEVIEW_G_IDS(&intel_pineview_g_info),
>> - INTEL_PINEVIEW_M_IDS(&intel_pineview_m_info),
>> - INTEL_IRONLAKE_D_IDS(&intel_ironlake_d_info),
>> - INTEL_IRONLAKE_M_IDS(&intel_ironlake_m_info),
>> - INTEL_SNB_D_GT1_IDS(&intel_sandybridge_d_gt1_info),
>> - INTEL_SNB_D_GT2_IDS(&intel_sandybridge_d_gt2_info),
>> - INTEL_SNB_M_GT1_IDS(&intel_sandybridge_m_gt1_info),
>> - INTEL_SNB_M_GT2_IDS(&intel_sandybridge_m_gt2_info),
>> - INTEL_IVB_Q_IDS(&intel_ivybridge_q_info), /* must be first IVB */
>> - INTEL_IVB_M_GT1_IDS(&intel_ivybridge_m_gt1_info),
>> - INTEL_IVB_M_GT2_IDS(&intel_ivybridge_m_gt2_info),
>> - INTEL_IVB_D_GT1_IDS(&intel_ivybridge_d_gt1_info),
>> - INTEL_IVB_D_GT2_IDS(&intel_ivybridge_d_gt2_info),
>> - INTEL_HSW_GT1_IDS(&intel_haswell_gt1_info),
>> - INTEL_HSW_GT2_IDS(&intel_haswell_gt2_info),
>> - INTEL_HSW_GT3_IDS(&intel_haswell_gt3_info),
>> - INTEL_VLV_IDS(&intel_valleyview_info),
>> - INTEL_BDW_GT1_IDS(&intel_broadwell_gt1_info),
>> - INTEL_BDW_GT2_IDS(&intel_broadwell_gt2_info),
>> - INTEL_BDW_GT3_IDS(&intel_broadwell_gt3_info),
>> - INTEL_BDW_RSVD_IDS(&intel_broadwell_rsvd_info),
>> - INTEL_CHV_IDS(&intel_cherryview_info),
>> - INTEL_SKL_GT1_IDS(&intel_skylake_gt1_info),
>> - INTEL_SKL_GT2_IDS(&intel_skylake_gt2_info),
>> - INTEL_SKL_GT3_IDS(&intel_skylake_gt3_info),
>> - INTEL_SKL_GT4_IDS(&intel_skylake_gt4_info),
>> - INTEL_BXT_IDS(&intel_broxton_info),
>> - INTEL_GLK_IDS(&intel_geminilake_info),
>> - INTEL_KBL_GT1_IDS(&intel_kabylake_gt1_info),
>> - INTEL_KBL_GT2_IDS(&intel_kabylake_gt2_info),
>> - INTEL_KBL_GT3_IDS(&intel_kabylake_gt3_info),
>> - INTEL_KBL_GT4_IDS(&intel_kabylake_gt3_info),
>> - INTEL_AML_KBL_GT2_IDS(&intel_kabylake_gt2_info),
>> - INTEL_CFL_S_GT1_IDS(&intel_coffeelake_gt1_info),
>> - INTEL_CFL_S_GT2_IDS(&intel_coffeelake_gt2_info),
>> - INTEL_CFL_H_GT1_IDS(&intel_coffeelake_gt1_info),
>> - INTEL_CFL_H_GT2_IDS(&intel_coffeelake_gt2_info),
>> - INTEL_CFL_U_GT2_IDS(&intel_coffeelake_gt2_info),
>> - INTEL_CFL_U_GT3_IDS(&intel_coffeelake_gt3_info),
>> - INTEL_WHL_U_GT1_IDS(&intel_coffeelake_gt1_info),
>> - INTEL_WHL_U_GT2_IDS(&intel_coffeelake_gt2_info),
>> - INTEL_AML_CFL_GT2_IDS(&intel_coffeelake_gt2_info),
>> - INTEL_WHL_U_GT3_IDS(&intel_coffeelake_gt3_info),
>> - INTEL_CML_GT1_IDS(&intel_coffeelake_gt1_info),
>> - INTEL_CML_GT2_IDS(&intel_coffeelake_gt2_info),
>> - INTEL_CML_U_GT1_IDS(&intel_coffeelake_gt1_info),
>> - INTEL_CML_U_GT2_IDS(&intel_coffeelake_gt2_info),
>> - INTEL_CNL_IDS(&intel_cannonlake_info),
>> - INTEL_ICL_11_IDS(&intel_icelake_11_info),
>> - INTEL_EHL_IDS(&intel_elkhartlake_info),
>> - INTEL_TGL_12_IDS(&intel_tigerlake_12_info),
>> + INTEL_I830_IDS(&i830_info),
>> + INTEL_I845G_IDS(&i845g_info),
>> + INTEL_I85X_IDS(&i85x_info),
>> + INTEL_I865G_IDS(&i865g_info),
>> + INTEL_I915G_IDS(&i915g_info),
>> + INTEL_I915GM_IDS(&i915gm_info),
>> + INTEL_I945G_IDS(&i945g_info),
>> + INTEL_I945GM_IDS(&i945gm_info),
>> + INTEL_I965G_IDS(&i965g_info),
>> + INTEL_G33_IDS(&g33_info),
>> + INTEL_I965GM_IDS(&i965gm_info),
>> + INTEL_GM45_IDS(&gm45_info),
>> + INTEL_G45_IDS(&g45_info),
>> + INTEL_PINEVIEW_G_IDS(&pnv_g_info),
>> + INTEL_PINEVIEW_M_IDS(&pnv_m_info),
>> + INTEL_IRONLAKE_D_IDS(&ilk_d_info),
>> + INTEL_IRONLAKE_M_IDS(&ilk_m_info),
>> + INTEL_SNB_D_GT1_IDS(&snb_d_gt1_info),
>> + INTEL_SNB_D_GT2_IDS(&snb_d_gt2_info),
>> + INTEL_SNB_M_GT1_IDS(&snb_m_gt1_info),
>> + INTEL_SNB_M_GT2_IDS(&snb_m_gt2_info),
>> + INTEL_IVB_Q_IDS(&ivb_q_info), /* must be first IVB */
>> + INTEL_IVB_M_GT1_IDS(&ivb_m_gt1_info),
>> + INTEL_IVB_M_GT2_IDS(&ivb_m_gt2_info),
>> + INTEL_IVB_D_GT1_IDS(&ivb_d_gt1_info),
>> + INTEL_IVB_D_GT2_IDS(&ivb_d_gt2_info),
>> + INTEL_HSW_GT1_IDS(&hsw_gt1_info),
>> + INTEL_HSW_GT2_IDS(&hsw_gt2_info),
>> + INTEL_HSW_GT3_IDS(&hsw_gt3_info),
>> + INTEL_VLV_IDS(&vlv_info),
>> + INTEL_BDW_GT1_IDS(&bdw_gt1_info),
>> + INTEL_BDW_GT2_IDS(&bdw_gt2_info),
>> + INTEL_BDW_GT3_IDS(&bdw_gt3_info),
>> + INTEL_BDW_RSVD_IDS(&bdw_rsvd_info),
>> + INTEL_CHV_IDS(&chv_info),
>> + INTEL_SKL_GT1_IDS(&skl_gt1_info),
>> + INTEL_SKL_GT2_IDS(&skl_gt2_info),
>> + INTEL_SKL_GT3_IDS(&skl_gt3_info),
>> + INTEL_SKL_GT4_IDS(&skl_gt4_info),
>> + INTEL_BXT_IDS(&bxt_info),
>> + INTEL_GLK_IDS(&glk_info),
>> + INTEL_KBL_GT1_IDS(&kbl_gt1_info),
>> + INTEL_KBL_GT2_IDS(&kbl_gt2_info),
>> + INTEL_KBL_GT3_IDS(&kbl_gt3_info),
>> + INTEL_KBL_GT4_IDS(&kbl_gt3_info),
>> + INTEL_AML_KBL_GT2_IDS(&kbl_gt2_info),
>> + INTEL_CFL_S_GT1_IDS(&cfl_gt1_info),
>> + INTEL_CFL_S_GT2_IDS(&cfl_gt2_info),
>> + INTEL_CFL_H_GT1_IDS(&cfl_gt1_info),
>> + INTEL_CFL_H_GT2_IDS(&cfl_gt2_info),
>> + INTEL_CFL_U_GT2_IDS(&cfl_gt2_info),
>> + INTEL_CFL_U_GT3_IDS(&cfl_gt3_info),
>> + INTEL_WHL_U_GT1_IDS(&cfl_gt1_info),
>> + INTEL_WHL_U_GT2_IDS(&cfl_gt2_info),
>> + INTEL_AML_CFL_GT2_IDS(&cfl_gt2_info),
>> + INTEL_WHL_U_GT3_IDS(&cfl_gt3_info),
>> + INTEL_CML_GT1_IDS(&cfl_gt1_info),
>> + INTEL_CML_GT2_IDS(&cfl_gt2_info),
>> + INTEL_CML_U_GT1_IDS(&cfl_gt1_info),
>> + INTEL_CML_U_GT2_IDS(&cfl_gt2_info),
>> + INTEL_CNL_IDS(&cnl_info),
>> + INTEL_ICL_11_IDS(&icl_11_info),
>> + INTEL_EHL_IDS(&ehl_info),
>> + INTEL_TGL_12_IDS(&tgl_12_info),
>> {0, 0, 0}
>> };
>> MODULE_DEVICE_TABLE(pci, pciidlist);
>> --
>> 2.24.0
>
>--
>Ville Syrjälä
>Intel
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: [Intel-gfx] [PATCH 01/10] drm/i915: simplify prefixes on device_info
2019-12-20 18:01 ` Lucas De Marchi
@ 2019-12-20 18:08 ` Ville Syrjälä
0 siblings, 0 replies; 17+ messages in thread
From: Ville Syrjälä @ 2019-12-20 18:08 UTC (permalink / raw)
To: Lucas De Marchi; +Cc: intel-gfx
On Fri, Dec 20, 2019 at 10:01:19AM -0800, Lucas De Marchi wrote:
> On Fri, Dec 20, 2019 at 07:45:38PM +0200, Ville Syrjälä wrote:
> >On Fri, Dec 20, 2019 at 09:27:45AM -0800, Lucas De Marchi wrote:
> >> Drop the intel prefix since all these structs are static and prefer
> >> using the 3-letter prefix for each platform.
> >>
> >> Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
> >> ---
> >> drivers/gpu/drm/i915/i915_pci.c | 230 ++++++++++++++++----------------
> >> 1 file changed, 115 insertions(+), 115 deletions(-)
> >>
> >> diff --git a/drivers/gpu/drm/i915/i915_pci.c b/drivers/gpu/drm/i915/i915_pci.c
> >> index 9571611b4b16..4cd25a83e6f1 100644
> >> --- a/drivers/gpu/drm/i915/i915_pci.c
> >> +++ b/drivers/gpu/drm/i915/i915_pci.c
> >> @@ -193,23 +193,23 @@
> >> GEN_DEFAULT_PAGE_SIZES, \
> >> GEN_DEFAULT_REGIONS
> >>
> >> -static const struct intel_device_info intel_i830_info = {
> >> +static const struct intel_device_info i830_info = {
> >> I830_FEATURES,
> >> PLATFORM(INTEL_I830),
> >> };
> >>
> >> -static const struct intel_device_info intel_i845g_info = {
> >> +static const struct intel_device_info i845g_info = {
> >> I845_FEATURES,
> >> PLATFORM(INTEL_I845G),
> >> };
> >>
> >> -static const struct intel_device_info intel_i85x_info = {
> >> +static const struct intel_device_info i85x_info = {
> >> I830_FEATURES,
> >> PLATFORM(INTEL_I85X),
> >> .display.has_fbc = 1,
> >> };
> >>
> >> -static const struct intel_device_info intel_i865g_info = {
> >> +static const struct intel_device_info i865g_info = {
> >> I845_FEATURES,
> >> PLATFORM(INTEL_I865G),
> >> };
> >> @@ -228,7 +228,7 @@ static const struct intel_device_info intel_i865g_info = {
> >> GEN_DEFAULT_PAGE_SIZES, \
> >> GEN_DEFAULT_REGIONS
> >>
> >> -static const struct intel_device_info intel_i915g_info = {
> >> +static const struct intel_device_info i915g_info = {
> >> GEN3_FEATURES,
> >> PLATFORM(INTEL_I915G),
> >> .has_coherent_ggtt = false,
> >> @@ -239,7 +239,7 @@ static const struct intel_device_info intel_i915g_info = {
> >> .unfenced_needs_alignment = 1,
> >> };
> >>
> >> -static const struct intel_device_info intel_i915gm_info = {
> >> +static const struct intel_device_info i915gm_info = {
> >> GEN3_FEATURES,
> >> PLATFORM(INTEL_I915GM),
> >> .is_mobile = 1,
> >> @@ -252,7 +252,7 @@ static const struct intel_device_info intel_i915gm_info = {
> >> .unfenced_needs_alignment = 1,
> >> };
> >>
> >> -static const struct intel_device_info intel_i945g_info = {
> >> +static const struct intel_device_info i945g_info = {
> >> GEN3_FEATURES,
> >> PLATFORM(INTEL_I945G),
> >> .display.has_hotplug = 1,
> >> @@ -263,7 +263,7 @@ static const struct intel_device_info intel_i945g_info = {
> >> .unfenced_needs_alignment = 1,
> >> };
> >>
> >> -static const struct intel_device_info intel_i945gm_info = {
> >> +static const struct intel_device_info i945gm_info = {
> >> GEN3_FEATURES,
> >> PLATFORM(INTEL_I945GM),
> >> .is_mobile = 1,
> >> @@ -277,21 +277,21 @@ static const struct intel_device_info intel_i945gm_info = {
> >> .unfenced_needs_alignment = 1,
> >> };
> >>
> >> -static const struct intel_device_info intel_g33_info = {
> >> +static const struct intel_device_info g33_info = {
> >> GEN3_FEATURES,
> >> PLATFORM(INTEL_G33),
> >> .display.has_hotplug = 1,
> >> .display.has_overlay = 1,
> >> };
> >>
> >> -static const struct intel_device_info intel_pineview_g_info = {
> >> +static const struct intel_device_info pnv_g_info = {
> >> GEN3_FEATURES,
> >> PLATFORM(INTEL_PINEVIEW),
> >> .display.has_hotplug = 1,
> >> .display.has_overlay = 1,
> >> };
> >>
> >> -static const struct intel_device_info intel_pineview_m_info = {
> >> +static const struct intel_device_info pnv_m_info = {
> >> GEN3_FEATURES,
> >> PLATFORM(INTEL_PINEVIEW),
> >> .is_mobile = 1,
> >> @@ -314,7 +314,7 @@ static const struct intel_device_info intel_pineview_m_info = {
> >> GEN_DEFAULT_PAGE_SIZES, \
> >> GEN_DEFAULT_REGIONS
> >>
> >> -static const struct intel_device_info intel_i965g_info = {
> >> +static const struct intel_device_info i965g_info = {
> >> GEN4_FEATURES,
> >> PLATFORM(INTEL_I965G),
> >> .display.has_overlay = 1,
> >> @@ -322,7 +322,7 @@ static const struct intel_device_info intel_i965g_info = {
> >> .has_snoop = false,
> >> };
> >>
> >> -static const struct intel_device_info intel_i965gm_info = {
> >> +static const struct intel_device_info i965gm_info = {
> >> GEN4_FEATURES,
> >> PLATFORM(INTEL_I965GM),
> >> .is_mobile = 1,
> >> @@ -333,14 +333,14 @@ static const struct intel_device_info intel_i965gm_info = {
> >> .has_snoop = false,
> >> };
> >>
> >> -static const struct intel_device_info intel_g45_info = {
> >> +static const struct intel_device_info g45_info = {
> >> GEN4_FEATURES,
> >> PLATFORM(INTEL_G45),
> >> .engine_mask = BIT(RCS0) | BIT(VCS0),
> >> .gpu_reset_clobbers_display = false,
> >> };
> >>
> >> -static const struct intel_device_info intel_gm45_info = {
> >> +static const struct intel_device_info gm45_info = {
> >> GEN4_FEATURES,
> >> PLATFORM(INTEL_GM45),
> >> .is_mobile = 1,
> >> @@ -365,12 +365,12 @@ static const struct intel_device_info intel_gm45_info = {
> >> GEN_DEFAULT_PAGE_SIZES, \
> >> GEN_DEFAULT_REGIONS
> >>
> >> -static const struct intel_device_info intel_ironlake_d_info = {
> >> +static const struct intel_device_info ilk_d_info = {
> >> GEN5_FEATURES,
> >> PLATFORM(INTEL_IRONLAKE),
> >> };
> >>
> >> -static const struct intel_device_info intel_ironlake_m_info = {
> >> +static const struct intel_device_info ilk_m_info = {
> >> GEN5_FEATURES,
> >> PLATFORM(INTEL_IRONLAKE),
> >> .is_mobile = 1,
> >> @@ -400,12 +400,12 @@ static const struct intel_device_info intel_ironlake_m_info = {
> >> GEN6_FEATURES, \
> >> PLATFORM(INTEL_SANDYBRIDGE)
> >>
> >> -static const struct intel_device_info intel_sandybridge_d_gt1_info = {
> >> +static const struct intel_device_info snb_d_gt1_info = {
> >> SNB_D_PLATFORM,
> >> .gt = 1,
> >> };
> >>
> >> -static const struct intel_device_info intel_sandybridge_d_gt2_info = {
> >> +static const struct intel_device_info snb_d_gt2_info = {
> >> SNB_D_PLATFORM,
> >> .gt = 2,
> >> };
> >> @@ -416,12 +416,12 @@ static const struct intel_device_info intel_sandybridge_d_gt2_info = {
> >> .is_mobile = 1
> >>
> >>
> >> -static const struct intel_device_info intel_sandybridge_m_gt1_info = {
> >> +static const struct intel_device_info snb_m_gt1_info = {
> >> SNB_M_PLATFORM,
> >> .gt = 1,
> >> };
> >>
> >> -static const struct intel_device_info intel_sandybridge_m_gt2_info = {
> >> +static const struct intel_device_info snb_m_gt2_info = {
> >> SNB_M_PLATFORM,
> >> .gt = 2,
> >> };
> >> @@ -450,12 +450,12 @@ static const struct intel_device_info intel_sandybridge_m_gt2_info = {
> >> PLATFORM(INTEL_IVYBRIDGE), \
> >> .has_l3_dpf = 1
> >>
> >> -static const struct intel_device_info intel_ivybridge_d_gt1_info = {
> >> +static const struct intel_device_info ivb_d_gt1_info = {
> >> IVB_D_PLATFORM,
> >> .gt = 1,
> >> };
> >>
> >> -static const struct intel_device_info intel_ivybridge_d_gt2_info = {
> >> +static const struct intel_device_info ivb_d_gt2_info = {
> >> IVB_D_PLATFORM,
> >> .gt = 2,
> >> };
> >> @@ -466,17 +466,17 @@ static const struct intel_device_info intel_ivybridge_d_gt2_info = {
> >> .is_mobile = 1, \
> >> .has_l3_dpf = 1
> >>
> >> -static const struct intel_device_info intel_ivybridge_m_gt1_info = {
> >> +static const struct intel_device_info ivb_m_gt1_info = {
> >> IVB_M_PLATFORM,
> >> .gt = 1,
> >> };
> >>
> >> -static const struct intel_device_info intel_ivybridge_m_gt2_info = {
> >> +static const struct intel_device_info ivb_m_gt2_info = {
> >> IVB_M_PLATFORM,
> >> .gt = 2,
> >> };
> >>
> >> -static const struct intel_device_info intel_ivybridge_q_info = {
> >> +static const struct intel_device_info ivb_q_info = {
> >> GEN7_FEATURES,
> >> PLATFORM(INTEL_IVYBRIDGE),
> >> .gt = 2,
> >> @@ -484,7 +484,7 @@ static const struct intel_device_info intel_ivybridge_q_info = {
> >> .has_l3_dpf = 1,
> >> };
> >>
> >> -static const struct intel_device_info intel_valleyview_info = {
> >> +static const struct intel_device_info vlv_info = {
> >> PLATFORM(INTEL_VALLEYVIEW),
> >> GEN(7),
> >> .is_lp = 1,
> >> @@ -523,17 +523,17 @@ static const struct intel_device_info intel_valleyview_info = {
> >> PLATFORM(INTEL_HASWELL), \
> >> .has_l3_dpf = 1
> >>
> >> -static const struct intel_device_info intel_haswell_gt1_info = {
> >> +static const struct intel_device_info hsw_gt1_info = {
> >> HSW_PLATFORM,
> >> .gt = 1,
> >> };
> >>
> >> -static const struct intel_device_info intel_haswell_gt2_info = {
> >> +static const struct intel_device_info hsw_gt2_info = {
> >> HSW_PLATFORM,
> >> .gt = 2,
> >> };
> >>
> >> -static const struct intel_device_info intel_haswell_gt3_info = {
> >> +static const struct intel_device_info hsw_gt3_info = {
> >> HSW_PLATFORM,
> >> .gt = 3,
> >> };
> >> @@ -551,17 +551,17 @@ static const struct intel_device_info intel_haswell_gt3_info = {
> >> GEN8_FEATURES, \
> >> PLATFORM(INTEL_BROADWELL)
> >>
> >> -static const struct intel_device_info intel_broadwell_gt1_info = {
> >> +static const struct intel_device_info bdw_gt1_info = {
> >> BDW_PLATFORM,
> >> .gt = 1,
> >> };
> >>
> >> -static const struct intel_device_info intel_broadwell_gt2_info = {
> >> +static const struct intel_device_info bdw_gt2_info = {
> >> BDW_PLATFORM,
> >> .gt = 2,
> >> };
> >>
> >> -static const struct intel_device_info intel_broadwell_rsvd_info = {
> >> +static const struct intel_device_info bdw_rsvd_info = {
> >> BDW_PLATFORM,
> >> .gt = 3,
> >> /* According to the device ID those devices are GT3, they were
> >> @@ -569,14 +569,14 @@ static const struct intel_device_info intel_broadwell_rsvd_info = {
> >> */
> >> };
> >>
> >> -static const struct intel_device_info intel_broadwell_gt3_info = {
> >> +static const struct intel_device_info bdw_gt3_info = {
> >> BDW_PLATFORM,
> >> .gt = 3,
> >> .engine_mask =
> >> BIT(RCS0) | BIT(VCS0) | BIT(BCS0) | BIT(VECS0) | BIT(VCS1),
> >> };
> >>
> >> -static const struct intel_device_info intel_cherryview_info = {
> >> +static const struct intel_device_info chv_info = {
> >> PLATFORM(INTEL_CHERRYVIEW),
> >> GEN(8),
> >> .pipe_mask = BIT(PIPE_A) | BIT(PIPE_B) | BIT(PIPE_C),
> >> @@ -621,12 +621,12 @@ static const struct intel_device_info intel_cherryview_info = {
> >> GEN9_FEATURES, \
> >> PLATFORM(INTEL_SKYLAKE)
> >>
> >> -static const struct intel_device_info intel_skylake_gt1_info = {
> >> +static const struct intel_device_info skl_gt1_info = {
> >> SKL_PLATFORM,
> >> .gt = 1,
> >> };
> >>
> >> -static const struct intel_device_info intel_skylake_gt2_info = {
> >> +static const struct intel_device_info skl_gt2_info = {
> >> SKL_PLATFORM,
> >> .gt = 2,
> >> };
> >> @@ -637,12 +637,12 @@ static const struct intel_device_info intel_skylake_gt2_info = {
> >> BIT(RCS0) | BIT(VCS0) | BIT(BCS0) | BIT(VECS0) | BIT(VCS1)
> >>
> >>
> >> -static const struct intel_device_info intel_skylake_gt3_info = {
> >> +static const struct intel_device_info skl_gt3_info = {
> >> SKL_GT3_PLUS_PLATFORM,
> >> .gt = 3,
> >> };
> >>
> >> -static const struct intel_device_info intel_skylake_gt4_info = {
> >> +static const struct intel_device_info skl_gt4_info = {
> >> SKL_GT3_PLUS_PLATFORM,
> >> .gt = 4,
> >> };
> >> @@ -679,13 +679,13 @@ static const struct intel_device_info intel_skylake_gt4_info = {
> >> GEN9_DEFAULT_PAGE_SIZES, \
> >> GEN_DEFAULT_REGIONS
> >>
> >> -static const struct intel_device_info intel_broxton_info = {
> >> +static const struct intel_device_info bxt_info = {
> >> GEN9_LP_FEATURES,
> >> PLATFORM(INTEL_BROXTON),
> >> .ddb_size = 512,
> >> };
> >>
> >> -static const struct intel_device_info intel_geminilake_info = {
> >> +static const struct intel_device_info glk_info = {
> >> GEN9_LP_FEATURES,
> >> PLATFORM(INTEL_GEMINILAKE),
> >> .ddb_size = 1024,
> >> @@ -696,17 +696,17 @@ static const struct intel_device_info intel_geminilake_info = {
> >> GEN9_FEATURES, \
> >> PLATFORM(INTEL_KABYLAKE)
> >>
> >> -static const struct intel_device_info intel_kabylake_gt1_info = {
> >> +static const struct intel_device_info kbl_gt1_info = {
> >> KBL_PLATFORM,
> >> .gt = 1,
> >> };
> >>
> >> -static const struct intel_device_info intel_kabylake_gt2_info = {
> >> +static const struct intel_device_info kbl_gt2_info = {
> >> KBL_PLATFORM,
> >> .gt = 2,
> >> };
> >>
> >> -static const struct intel_device_info intel_kabylake_gt3_info = {
> >> +static const struct intel_device_info kbl_gt3_info = {
> >> KBL_PLATFORM,
> >> .gt = 3,
> >> .engine_mask =
> >> @@ -717,17 +717,17 @@ static const struct intel_device_info intel_kabylake_gt3_info = {
> >> GEN9_FEATURES, \
> >> PLATFORM(INTEL_COFFEELAKE)
> >>
> >> -static const struct intel_device_info intel_coffeelake_gt1_info = {
> >> +static const struct intel_device_info cfl_gt1_info = {
> >> CFL_PLATFORM,
> >> .gt = 1,
> >> };
> >>
> >> -static const struct intel_device_info intel_coffeelake_gt2_info = {
> >> +static const struct intel_device_info cfl_gt2_info = {
> >> CFL_PLATFORM,
> >> .gt = 2,
> >> };
> >>
> >> -static const struct intel_device_info intel_coffeelake_gt3_info = {
> >> +static const struct intel_device_info cfl_gt3_info = {
> >> CFL_PLATFORM,
> >> .gt = 3,
> >> .engine_mask =
> >> @@ -742,7 +742,7 @@ static const struct intel_device_info intel_coffeelake_gt3_info = {
> >> .has_coherent_ggtt = false, \
> >> GLK_COLORS
> >>
> >> -static const struct intel_device_info intel_cannonlake_info = {
> >> +static const struct intel_device_info cnl_info = {
> >> GEN10_FEATURES,
> >> PLATFORM(INTEL_CANNONLAKE),
> >> .gt = 2,
> >> @@ -777,14 +777,14 @@ static const struct intel_device_info intel_cannonlake_info = {
> >> .has_logical_ring_elsq = 1, \
> >> .color = { .degamma_lut_size = 33, .gamma_lut_size = 262145 }
> >>
> >> -static const struct intel_device_info intel_icelake_11_info = {
> >> +static const struct intel_device_info icl_11_info = {
> >
> >Why isn't that just icl_info ?
>
> it should be, yes... I had a separate commit to remove the gen from the
> device_info, I guess I dropped it instead of squashing.
You also seem to have dropped Jani's acks.
Anyways, cursory scan seemed good so ack for the seris from me as well.
Acked-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
--
Ville Syrjälä
Intel
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 17+ messages in thread
* [Intel-gfx] [PATCH 02/10] drm/i915: prefer 3-letter acronym for pineview
2019-12-20 17:27 [Intel-gfx] [PATCH 00/10] Prefer acronym for prefixes Lucas De Marchi
2019-12-20 17:27 ` [Intel-gfx] [PATCH 01/10] drm/i915: simplify prefixes on device_info Lucas De Marchi
@ 2019-12-20 17:27 ` Lucas De Marchi
2019-12-20 17:27 ` [Intel-gfx] [PATCH 03/10] drm/i915: prefer 3-letter acronym for haswell Lucas De Marchi
` (10 subsequent siblings)
12 siblings, 0 replies; 17+ messages in thread
From: Lucas De Marchi @ 2019-12-20 17:27 UTC (permalink / raw)
To: intel-gfx
We are currently using a mix of platform name and acronym to name the
functions. Let's prefer the acronym as it should be clear what platform
it's about and it's shorter, so it doesn't go over 80 columns in a few
cases. This converts pineview to pnv where appropriate.
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
---
drivers/gpu/drm/i915/display/intel_display.c | 8 ++++----
drivers/gpu/drm/i915/intel_pm.c | 4 ++--
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c
index 25af0ffe1c3a..7ad9c1f129c1 100644
--- a/drivers/gpu/drm/i915/display/intel_display.c
+++ b/drivers/gpu/drm/i915/display/intel_display.c
@@ -368,7 +368,7 @@ static const struct intel_limit intel_limits_g4x_dual_channel_lvds = {
},
};
-static const struct intel_limit intel_limits_pineview_sdvo = {
+static const struct intel_limit pnv_limits_sdvo = {
.dot = { .min = 20000, .max = 400000},
.vco = { .min = 1700000, .max = 3500000 },
/* Pineview's Ncounter is a ring counter */
@@ -383,7 +383,7 @@ static const struct intel_limit intel_limits_pineview_sdvo = {
.p2_slow = 10, .p2_fast = 5 },
};
-static const struct intel_limit intel_limits_pineview_lvds = {
+static const struct intel_limit pnv_limits_lvds = {
.dot = { .min = 20000, .max = 400000 },
.vco = { .min = 1700000, .max = 3500000 },
.n = { .min = 3, .max = 6 },
@@ -8547,9 +8547,9 @@ static int pnv_crtc_compute_clock(struct intel_crtc *crtc,
DRM_DEBUG_KMS("using SSC reference clock of %d kHz\n", refclk);
}
- limit = &intel_limits_pineview_lvds;
+ limit = &pnv_limits_lvds;
} else {
- limit = &intel_limits_pineview_sdvo;
+ limit = &pnv_limits_sdvo;
}
if (!crtc_state->clock_set &&
diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
index 7cdca06be3bd..6d9707eceddd 100644
--- a/drivers/gpu/drm/i915/intel_pm.c
+++ b/drivers/gpu/drm/i915/intel_pm.c
@@ -140,7 +140,7 @@ static void glk_init_clock_gating(struct drm_i915_private *dev_priv)
}
-static void i915_pineview_get_mem_freq(struct drm_i915_private *dev_priv)
+static void pnv_get_mem_freq(struct drm_i915_private *dev_priv)
{
u32 tmp;
@@ -7179,7 +7179,7 @@ void intel_init_pm(struct drm_i915_private *dev_priv)
{
/* For cxsr */
if (IS_PINEVIEW(dev_priv))
- i915_pineview_get_mem_freq(dev_priv);
+ pnv_get_mem_freq(dev_priv);
else if (IS_GEN(dev_priv, 5))
i915_ironlake_get_mem_freq(dev_priv);
--
2.24.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 17+ messages in thread* [Intel-gfx] [PATCH 03/10] drm/i915: prefer 3-letter acronym for haswell
2019-12-20 17:27 [Intel-gfx] [PATCH 00/10] Prefer acronym for prefixes Lucas De Marchi
2019-12-20 17:27 ` [Intel-gfx] [PATCH 01/10] drm/i915: simplify prefixes on device_info Lucas De Marchi
2019-12-20 17:27 ` [Intel-gfx] [PATCH 02/10] drm/i915: prefer 3-letter acronym for pineview Lucas De Marchi
@ 2019-12-20 17:27 ` Lucas De Marchi
2019-12-20 17:27 ` [Intel-gfx] [PATCH 04/10] drm/i915: prefer 3-letter acronym for skylake Lucas De Marchi
` (9 subsequent siblings)
12 siblings, 0 replies; 17+ messages in thread
From: Lucas De Marchi @ 2019-12-20 17:27 UTC (permalink / raw)
To: intel-gfx
We are currently using a mix of platform name and acronym to name the
functions. Let's prefer the acronym as it should be clear what platform
it's about and it's shorter, so it doesn't go over 80 columns in a few
cases. This converts haswell to hsw where appropriate.
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
---
drivers/gpu/drm/i915/display/intel_ddi.c | 4 +-
drivers/gpu/drm/i915/display/intel_display.c | 57 ++++++++++----------
drivers/gpu/drm/i915/intel_device_info.c | 4 +-
3 files changed, 32 insertions(+), 33 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/intel_ddi.c b/drivers/gpu/drm/i915/display/intel_ddi.c
index c9ba7d7f3787..d687c9503025 100644
--- a/drivers/gpu/drm/i915/display/intel_ddi.c
+++ b/drivers/gpu/drm/i915/display/intel_ddi.c
@@ -3458,14 +3458,14 @@ static void tgl_ddi_pre_enable_dp(struct intel_encoder *encoder,
* (DFLEXDPSP.DPX4TXLATC)
*
* This was done before tgl_ddi_pre_enable_dp by
- * haswell_crtc_enable()->intel_encoders_pre_pll_enable().
+ * hsw_crtc_enable()->intel_encoders_pre_pll_enable().
*/
/*
* 4. Enable the port PLL.
*
* The PLL enabling itself was already done before this function by
- * haswell_crtc_enable()->intel_enable_shared_dpll(). We need only
+ * hsw_crtc_enable()->intel_enable_shared_dpll(). We need only
* configure the PLL to port mapping here.
*/
intel_ddi_clk_select(encoder, crtc_state);
diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c
index 7ad9c1f129c1..b4ea557a4bb5 100644
--- a/drivers/gpu/drm/i915/display/intel_display.c
+++ b/drivers/gpu/drm/i915/display/intel_display.c
@@ -158,7 +158,7 @@ static void intel_cpu_transcoder_set_m_n(const struct intel_crtc_state *crtc_sta
const struct intel_link_m_n *m2_n2);
static void i9xx_set_pipeconf(const struct intel_crtc_state *crtc_state);
static void ironlake_set_pipeconf(const struct intel_crtc_state *crtc_state);
-static void haswell_set_pipeconf(const struct intel_crtc_state *crtc_state);
+static void hsw_set_pipeconf(const struct intel_crtc_state *crtc_state);
static void bdw_set_pipemisc(const struct intel_crtc_state *crtc_state);
static void vlv_prepare_pll(struct intel_crtc *crtc,
const struct intel_crtc_state *pipe_config);
@@ -6538,8 +6538,8 @@ static void hsw_set_frame_start_delay(const struct intel_crtc_state *crtc_state)
I915_WRITE(reg, val);
}
-static void haswell_crtc_enable(struct intel_atomic_state *state,
- struct intel_crtc *crtc)
+static void hsw_crtc_enable(struct intel_atomic_state *state,
+ struct intel_crtc *crtc)
{
const struct intel_crtc_state *new_crtc_state =
intel_atomic_get_new_crtc_state(state, crtc);
@@ -6580,7 +6580,7 @@ static void haswell_crtc_enable(struct intel_atomic_state *state,
if (!transcoder_is_dsi(cpu_transcoder)) {
hsw_set_frame_start_delay(new_crtc_state);
- haswell_set_pipeconf(new_crtc_state);
+ hsw_set_pipeconf(new_crtc_state);
}
if (INTEL_GEN(dev_priv) >= 9 || IS_BROADWELL(dev_priv))
@@ -6718,8 +6718,8 @@ static void ironlake_crtc_disable(struct intel_atomic_state *state,
intel_set_pch_fifo_underrun_reporting(dev_priv, pipe, true);
}
-static void haswell_crtc_disable(struct intel_atomic_state *state,
- struct intel_crtc *crtc)
+static void hsw_crtc_disable(struct intel_atomic_state *state,
+ struct intel_crtc *crtc)
{
/*
* FIXME collapse everything to one hook.
@@ -9535,7 +9535,7 @@ static void ironlake_set_pipeconf(const struct intel_crtc_state *crtc_state)
POSTING_READ(PIPECONF(pipe));
}
-static void haswell_set_pipeconf(const struct intel_crtc_state *crtc_state)
+static void hsw_set_pipeconf(const struct intel_crtc_state *crtc_state)
{
struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
@@ -10167,8 +10167,9 @@ static bool ironlake_get_pipe_config(struct intel_crtc *crtc,
return ret;
}
-static int haswell_crtc_compute_clock(struct intel_crtc *crtc,
- struct intel_crtc_state *crtc_state)
+
+static int hsw_crtc_compute_clock(struct intel_crtc *crtc,
+ struct intel_crtc_state *crtc_state)
{
struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
struct intel_atomic_state *state =
@@ -10282,9 +10283,8 @@ static void skylake_get_ddi_pll(struct drm_i915_private *dev_priv,
pipe_config->shared_dpll = intel_get_shared_dpll_by_id(dev_priv, id);
}
-static void haswell_get_ddi_pll(struct drm_i915_private *dev_priv,
- enum port port,
- struct intel_crtc_state *pipe_config)
+static void hsw_get_ddi_pll(struct drm_i915_private *dev_priv, enum port port,
+ struct intel_crtc_state *pipe_config)
{
enum intel_dpll_id id;
u32 ddi_pll_sel = I915_READ(PORT_CLK_SEL(port));
@@ -10472,8 +10472,8 @@ static bool bxt_get_dsi_transcoder_state(struct intel_crtc *crtc,
return transcoder_is_dsi(pipe_config->cpu_transcoder);
}
-static void haswell_get_ddi_port_state(struct intel_crtc *crtc,
- struct intel_crtc_state *pipe_config)
+static void hsw_get_ddi_port_state(struct intel_crtc *crtc,
+ struct intel_crtc_state *pipe_config)
{
struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
enum transcoder cpu_transcoder = pipe_config->cpu_transcoder;
@@ -10501,7 +10501,7 @@ static void haswell_get_ddi_port_state(struct intel_crtc *crtc,
else if (IS_GEN9_LP(dev_priv))
bxt_get_ddi_pll(dev_priv, port, pipe_config);
else
- haswell_get_ddi_pll(dev_priv, port, pipe_config);
+ hsw_get_ddi_pll(dev_priv, port, pipe_config);
pll = pipe_config->shared_dpll;
if (pll) {
@@ -10579,8 +10579,8 @@ static void icelake_get_trans_port_sync_config(struct intel_crtc_state *crtc_sta
crtc_state->sync_mode_slaves_mask);
}
-static bool haswell_get_pipe_config(struct intel_crtc *crtc,
- struct intel_crtc_state *pipe_config)
+static bool hsw_get_pipe_config(struct intel_crtc *crtc,
+ struct intel_crtc_state *pipe_config)
{
struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
intel_wakeref_t wakerefs[POWER_DOMAIN_NUM], wf;
@@ -10615,7 +10615,7 @@ static bool haswell_get_pipe_config(struct intel_crtc *crtc,
if (!transcoder_is_dsi(pipe_config->cpu_transcoder) ||
INTEL_GEN(dev_priv) >= 11) {
- haswell_get_ddi_port_state(crtc, pipe_config);
+ hsw_get_ddi_port_state(crtc, pipe_config);
intel_get_pipe_timings(crtc, pipe_config);
}
@@ -13787,7 +13787,7 @@ static void intel_modeset_clear_plls(struct intel_atomic_state *state)
* multiple pipes, and planes are enabled after the pipe, we need to wait at
* least 2 vblanks on the first pipe before enabling planes on the second pipe.
*/
-static int haswell_mode_set_planes_workaround(struct intel_atomic_state *state)
+static int hsw_mode_set_planes_workaround(struct intel_atomic_state *state)
{
struct intel_crtc_state *crtc_state;
struct intel_crtc *crtc;
@@ -13882,7 +13882,7 @@ static int intel_modeset_checks(struct intel_atomic_state *state)
intel_modeset_clear_plls(state);
if (IS_HASWELL(dev_priv))
- return haswell_mode_set_planes_workaround(state);
+ return hsw_mode_set_planes_workaround(state);
return 0;
}
@@ -16558,21 +16558,20 @@ void intel_init_display_hooks(struct drm_i915_private *dev_priv)
intel_init_cdclk_hooks(dev_priv);
if (INTEL_GEN(dev_priv) >= 9) {
- dev_priv->display.get_pipe_config = haswell_get_pipe_config;
+ dev_priv->display.get_pipe_config = hsw_get_pipe_config;
dev_priv->display.get_initial_plane_config =
skylake_get_initial_plane_config;
- dev_priv->display.crtc_compute_clock =
- haswell_crtc_compute_clock;
- dev_priv->display.crtc_enable = haswell_crtc_enable;
- dev_priv->display.crtc_disable = haswell_crtc_disable;
+ dev_priv->display.crtc_compute_clock = hsw_crtc_compute_clock;
+ dev_priv->display.crtc_enable = hsw_crtc_enable;
+ dev_priv->display.crtc_disable = hsw_crtc_disable;
} else if (HAS_DDI(dev_priv)) {
- dev_priv->display.get_pipe_config = haswell_get_pipe_config;
+ dev_priv->display.get_pipe_config = hsw_get_pipe_config;
dev_priv->display.get_initial_plane_config =
i9xx_get_initial_plane_config;
dev_priv->display.crtc_compute_clock =
- haswell_crtc_compute_clock;
- dev_priv->display.crtc_enable = haswell_crtc_enable;
- dev_priv->display.crtc_disable = haswell_crtc_disable;
+ hsw_crtc_compute_clock;
+ dev_priv->display.crtc_enable = hsw_crtc_enable;
+ dev_priv->display.crtc_disable = hsw_crtc_disable;
} else if (HAS_PCH_SPLIT(dev_priv)) {
dev_priv->display.get_pipe_config = ironlake_get_pipe_config;
dev_priv->display.get_initial_plane_config =
diff --git a/drivers/gpu/drm/i915/intel_device_info.c b/drivers/gpu/drm/i915/intel_device_info.c
index 1acb5db77431..ca7a42e1d769 100644
--- a/drivers/gpu/drm/i915/intel_device_info.c
+++ b/drivers/gpu/drm/i915/intel_device_info.c
@@ -600,7 +600,7 @@ static void broadwell_sseu_info_init(struct drm_i915_private *dev_priv)
sseu->has_eu_pg = 0;
}
-static void haswell_sseu_info_init(struct drm_i915_private *dev_priv)
+static void hsw_sseu_info_init(struct drm_i915_private *dev_priv)
{
struct sseu_dev_info *sseu = &RUNTIME_INFO(dev_priv)->sseu;
u32 fuse1;
@@ -1021,7 +1021,7 @@ void intel_device_info_runtime_init(struct drm_i915_private *dev_priv)
/* Initialize slice/subslice/EU info */
if (IS_HASWELL(dev_priv))
- haswell_sseu_info_init(dev_priv);
+ hsw_sseu_info_init(dev_priv);
else if (IS_CHERRYVIEW(dev_priv))
cherryview_sseu_info_init(dev_priv);
else if (IS_BROADWELL(dev_priv))
--
2.24.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 17+ messages in thread* [Intel-gfx] [PATCH 04/10] drm/i915: prefer 3-letter acronym for skylake
2019-12-20 17:27 [Intel-gfx] [PATCH 00/10] Prefer acronym for prefixes Lucas De Marchi
` (2 preceding siblings ...)
2019-12-20 17:27 ` [Intel-gfx] [PATCH 03/10] drm/i915: prefer 3-letter acronym for haswell Lucas De Marchi
@ 2019-12-20 17:27 ` Lucas De Marchi
2019-12-20 17:27 ` [Intel-gfx] [PATCH 05/10] drm/i915: prefer 3-letter acronym for cannonlake Lucas De Marchi
` (8 subsequent siblings)
12 siblings, 0 replies; 17+ messages in thread
From: Lucas De Marchi @ 2019-12-20 17:27 UTC (permalink / raw)
To: intel-gfx
We are currently using a mix of platform name and acronym to name the
functions. Let's prefer the acronym as it should be clear what platform
it's about and it's shorter, so it doesn't go over 80 columns in a few
cases. This converts skylake to skl where appropriate.
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
---
drivers/gpu/drm/i915/display/icl_dsi.c | 2 +-
drivers/gpu/drm/i915/display/intel_ddi.c | 2 +-
drivers/gpu/drm/i915/display/intel_display.c | 29 ++++++++++----------
drivers/gpu/drm/i915/display/intel_display.h | 2 +-
drivers/gpu/drm/i915/display/intel_dp_mst.c | 2 +-
drivers/gpu/drm/i915/display/vlv_dsi.c | 2 +-
drivers/gpu/drm/i915/gt/intel_mocs.c | 6 ++--
7 files changed, 22 insertions(+), 23 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/icl_dsi.c b/drivers/gpu/drm/i915/display/icl_dsi.c
index 006b1a297e6f..8435bc5a7a74 100644
--- a/drivers/gpu/drm/i915/display/icl_dsi.c
+++ b/drivers/gpu/drm/i915/display/icl_dsi.c
@@ -1259,7 +1259,7 @@ static void gen11_dsi_post_disable(struct intel_encoder *encoder,
intel_dsc_disable(old_crtc_state);
- skylake_scaler_disable(old_crtc_state);
+ skl_scaler_disable(old_crtc_state);
}
static enum drm_mode_status gen11_dsi_mode_valid(struct drm_connector *connector,
diff --git a/drivers/gpu/drm/i915/display/intel_ddi.c b/drivers/gpu/drm/i915/display/intel_ddi.c
index d687c9503025..b52c31721755 100644
--- a/drivers/gpu/drm/i915/display/intel_ddi.c
+++ b/drivers/gpu/drm/i915/display/intel_ddi.c
@@ -3896,7 +3896,7 @@ static void intel_ddi_post_disable(struct intel_encoder *encoder,
intel_dsc_disable(old_crtc_state);
if (INTEL_GEN(dev_priv) >= 9)
- skylake_scaler_disable(old_crtc_state);
+ skl_scaler_disable(old_crtc_state);
else
ironlake_pfit_disable(old_crtc_state);
diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c
index b4ea557a4bb5..066e2bb2a400 100644
--- a/drivers/gpu/drm/i915/display/intel_display.c
+++ b/drivers/gpu/drm/i915/display/intel_display.c
@@ -164,7 +164,7 @@ static void vlv_prepare_pll(struct intel_crtc *crtc,
const struct intel_crtc_state *pipe_config);
static void chv_prepare_pll(struct intel_crtc *crtc,
const struct intel_crtc_state *pipe_config);
-static void skylake_pfit_enable(const struct intel_crtc_state *crtc_state);
+static void skl_pfit_enable(const struct intel_crtc_state *crtc_state);
static void ironlake_pfit_enable(const struct intel_crtc_state *crtc_state);
static void intel_modeset_setup_hw_state(struct drm_device *dev,
struct drm_modeset_acquire_ctx *ctx);
@@ -5749,7 +5749,7 @@ static int skl_update_scaler_plane(struct intel_crtc_state *crtc_state,
return 0;
}
-void skylake_scaler_disable(const struct intel_crtc_state *old_crtc_state)
+void skl_scaler_disable(const struct intel_crtc_state *old_crtc_state)
{
struct intel_crtc *crtc = to_intel_crtc(old_crtc_state->uapi.crtc);
int i;
@@ -5758,7 +5758,7 @@ void skylake_scaler_disable(const struct intel_crtc_state *old_crtc_state)
skl_detach_scaler(crtc, i);
}
-static void skylake_pfit_enable(const struct intel_crtc_state *crtc_state)
+static void skl_pfit_enable(const struct intel_crtc_state *crtc_state)
{
struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
@@ -6595,7 +6595,7 @@ static void hsw_crtc_enable(struct intel_atomic_state *state,
glk_pipe_scaler_clock_gating_wa(dev_priv, pipe, true);
if (INTEL_GEN(dev_priv) >= 9)
- skylake_pfit_enable(new_crtc_state);
+ skl_pfit_enable(new_crtc_state);
else
ironlake_pfit_enable(new_crtc_state);
@@ -9868,8 +9868,8 @@ static void ironlake_get_fdi_m_n_config(struct intel_crtc *crtc,
&pipe_config->fdi_m_n, NULL);
}
-static void skylake_get_pfit_config(struct intel_crtc *crtc,
- struct intel_crtc_state *pipe_config)
+static void skl_get_pfit_config(struct intel_crtc *crtc,
+ struct intel_crtc_state *pipe_config)
{
struct drm_device *dev = crtc->base.dev;
struct drm_i915_private *dev_priv = to_i915(dev);
@@ -9900,8 +9900,8 @@ static void skylake_get_pfit_config(struct intel_crtc *crtc,
}
static void
-skylake_get_initial_plane_config(struct intel_crtc *crtc,
- struct intel_initial_plane_config *plane_config)
+skl_get_initial_plane_config(struct intel_crtc *crtc,
+ struct intel_initial_plane_config *plane_config)
{
struct drm_device *dev = crtc->base.dev;
struct drm_i915_private *dev_priv = to_i915(dev);
@@ -10267,9 +10267,8 @@ static void bxt_get_ddi_pll(struct drm_i915_private *dev_priv,
pipe_config->shared_dpll = intel_get_shared_dpll_by_id(dev_priv, id);
}
-static void skylake_get_ddi_pll(struct drm_i915_private *dev_priv,
- enum port port,
- struct intel_crtc_state *pipe_config)
+static void skl_get_ddi_pll(struct drm_i915_private *dev_priv, enum port port,
+ struct intel_crtc_state *pipe_config)
{
enum intel_dpll_id id;
u32 temp;
@@ -10497,7 +10496,7 @@ static void hsw_get_ddi_port_state(struct intel_crtc *crtc,
else if (IS_CANNONLAKE(dev_priv))
cannonlake_get_ddi_pll(dev_priv, port, pipe_config);
else if (IS_GEN9_BC(dev_priv))
- skylake_get_ddi_pll(dev_priv, port, pipe_config);
+ skl_get_ddi_pll(dev_priv, port, pipe_config);
else if (IS_GEN9_LP(dev_priv))
bxt_get_ddi_pll(dev_priv, port, pipe_config);
else
@@ -10672,7 +10671,7 @@ static bool hsw_get_pipe_config(struct intel_crtc *crtc,
power_domain_mask |= BIT_ULL(power_domain);
if (INTEL_GEN(dev_priv) >= 9)
- skylake_get_pfit_config(crtc, pipe_config);
+ skl_get_pfit_config(crtc, pipe_config);
else
ironlake_get_pfit_config(crtc, pipe_config);
}
@@ -14211,7 +14210,7 @@ static void intel_pipe_fastset(const struct intel_crtc_state *old_crtc_state,
skl_detach_scalers(new_crtc_state);
if (new_crtc_state->pch_pfit.enabled)
- skylake_pfit_enable(new_crtc_state);
+ skl_pfit_enable(new_crtc_state);
} else if (HAS_PCH_SPLIT(dev_priv)) {
if (new_crtc_state->pch_pfit.enabled)
ironlake_pfit_enable(new_crtc_state);
@@ -16560,7 +16559,7 @@ void intel_init_display_hooks(struct drm_i915_private *dev_priv)
if (INTEL_GEN(dev_priv) >= 9) {
dev_priv->display.get_pipe_config = hsw_get_pipe_config;
dev_priv->display.get_initial_plane_config =
- skylake_get_initial_plane_config;
+ skl_get_initial_plane_config;
dev_priv->display.crtc_compute_clock = hsw_crtc_compute_clock;
dev_priv->display.crtc_enable = hsw_crtc_enable;
dev_priv->display.crtc_disable = hsw_crtc_disable;
diff --git a/drivers/gpu/drm/i915/display/intel_display.h b/drivers/gpu/drm/i915/display/intel_display.h
index ff496cfbd4ab..5093ea5dcd95 100644
--- a/drivers/gpu/drm/i915/display/intel_display.h
+++ b/drivers/gpu/drm/i915/display/intel_display.h
@@ -578,7 +578,7 @@ void intel_crtc_arm_fifo_underrun(struct intel_crtc *crtc,
u16 skl_scaler_calc_phase(int sub, int scale, bool chroma_center);
int skl_update_scaler_crtc(struct intel_crtc_state *crtc_state);
-void skylake_scaler_disable(const struct intel_crtc_state *old_crtc_state);
+void skl_scaler_disable(const struct intel_crtc_state *old_crtc_state);
void ironlake_pfit_disable(const struct intel_crtc_state *old_crtc_state);
u32 glk_plane_color_ctl(const struct intel_crtc_state *crtc_state,
const struct intel_plane_state *plane_state);
diff --git a/drivers/gpu/drm/i915/display/intel_dp_mst.c b/drivers/gpu/drm/i915/display/intel_dp_mst.c
index 7aa0975c33b7..01b4608ab56c 100644
--- a/drivers/gpu/drm/i915/display/intel_dp_mst.c
+++ b/drivers/gpu/drm/i915/display/intel_dp_mst.c
@@ -248,7 +248,7 @@ static void intel_mst_post_disable_dp(struct intel_encoder *encoder,
intel_ddi_disable_transcoder_func(old_crtc_state);
if (INTEL_GEN(dev_priv) >= 9)
- skylake_scaler_disable(old_crtc_state);
+ skl_scaler_disable(old_crtc_state);
else
ironlake_pfit_disable(old_crtc_state);
diff --git a/drivers/gpu/drm/i915/display/vlv_dsi.c b/drivers/gpu/drm/i915/display/vlv_dsi.c
index 21e820299107..70ab378803c4 100644
--- a/drivers/gpu/drm/i915/display/vlv_dsi.c
+++ b/drivers/gpu/drm/i915/display/vlv_dsi.c
@@ -895,7 +895,7 @@ static void intel_dsi_post_disable(struct intel_encoder *encoder,
if (IS_GEN9_LP(dev_priv)) {
intel_crtc_vblank_off(old_crtc_state);
- skylake_scaler_disable(old_crtc_state);
+ skl_scaler_disable(old_crtc_state);
}
if (is_vid_mode(intel_dsi)) {
diff --git a/drivers/gpu/drm/i915/gt/intel_mocs.c b/drivers/gpu/drm/i915/gt/intel_mocs.c
index 893249ea48d4..cbdeda608359 100644
--- a/drivers/gpu/drm/i915/gt/intel_mocs.c
+++ b/drivers/gpu/drm/i915/gt/intel_mocs.c
@@ -127,7 +127,7 @@ struct drm_i915_mocs_table {
LE_0_PAGETABLE | LE_TC_2_LLC_ELLC | LE_LRUM(3), \
L3_3_WB)
-static const struct drm_i915_mocs_entry skylake_mocs_table[] = {
+static const struct drm_i915_mocs_entry skl_mocs_table[] = {
GEN9_MOCS_ENTRIES,
MOCS_ENTRY(I915_MOCS_CACHED,
LE_3_WB | LE_TC_2_LLC_ELLC | LE_LRUM(3),
@@ -292,9 +292,9 @@ static bool get_mocs_settings(const struct drm_i915_private *i915,
table->table = icelake_mocs_table;
table->n_entries = GEN11_NUM_MOCS_ENTRIES;
} else if (IS_GEN9_BC(i915) || IS_CANNONLAKE(i915)) {
- table->size = ARRAY_SIZE(skylake_mocs_table);
+ table->size = ARRAY_SIZE(skl_mocs_table);
table->n_entries = GEN9_NUM_MOCS_ENTRIES;
- table->table = skylake_mocs_table;
+ table->table = skl_mocs_table;
} else if (IS_GEN9_LP(i915)) {
table->size = ARRAY_SIZE(broxton_mocs_table);
table->n_entries = GEN9_NUM_MOCS_ENTRIES;
--
2.24.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 17+ messages in thread* [Intel-gfx] [PATCH 05/10] drm/i915: prefer 3-letter acronym for cannonlake
2019-12-20 17:27 [Intel-gfx] [PATCH 00/10] Prefer acronym for prefixes Lucas De Marchi
` (3 preceding siblings ...)
2019-12-20 17:27 ` [Intel-gfx] [PATCH 04/10] drm/i915: prefer 3-letter acronym for skylake Lucas De Marchi
@ 2019-12-20 17:27 ` Lucas De Marchi
2019-12-20 17:27 ` [Intel-gfx] [PATCH 06/10] drm/i915: prefer 3-letter acronym for icelake Lucas De Marchi
` (7 subsequent siblings)
12 siblings, 0 replies; 17+ messages in thread
From: Lucas De Marchi @ 2019-12-20 17:27 UTC (permalink / raw)
To: intel-gfx
We are currently using a mix of platform name and acronym to name the
functions. Let's prefer the acronym as it should be clear what platform
it's about and it's shorter, so it doesn't go over 80 columns in a few
cases. This converts cannonlake to cnl where appropriate.
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
---
drivers/gpu/drm/i915/display/intel_display.c | 7 +++----
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c
index 066e2bb2a400..9983113ff050 100644
--- a/drivers/gpu/drm/i915/display/intel_display.c
+++ b/drivers/gpu/drm/i915/display/intel_display.c
@@ -10190,9 +10190,8 @@ static int hsw_crtc_compute_clock(struct intel_crtc *crtc,
return 0;
}
-static void cannonlake_get_ddi_pll(struct drm_i915_private *dev_priv,
- enum port port,
- struct intel_crtc_state *pipe_config)
+static void cnl_get_ddi_pll(struct drm_i915_private *dev_priv, enum port port,
+ struct intel_crtc_state *pipe_config)
{
enum intel_dpll_id id;
u32 temp;
@@ -10494,7 +10493,7 @@ static void hsw_get_ddi_port_state(struct intel_crtc *crtc,
if (INTEL_GEN(dev_priv) >= 11)
icelake_get_ddi_pll(dev_priv, port, pipe_config);
else if (IS_CANNONLAKE(dev_priv))
- cannonlake_get_ddi_pll(dev_priv, port, pipe_config);
+ cnl_get_ddi_pll(dev_priv, port, pipe_config);
else if (IS_GEN9_BC(dev_priv))
skl_get_ddi_pll(dev_priv, port, pipe_config);
else if (IS_GEN9_LP(dev_priv))
--
2.24.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 17+ messages in thread* [Intel-gfx] [PATCH 06/10] drm/i915: prefer 3-letter acronym for icelake
2019-12-20 17:27 [Intel-gfx] [PATCH 00/10] Prefer acronym for prefixes Lucas De Marchi
` (4 preceding siblings ...)
2019-12-20 17:27 ` [Intel-gfx] [PATCH 05/10] drm/i915: prefer 3-letter acronym for cannonlake Lucas De Marchi
@ 2019-12-20 17:27 ` Lucas De Marchi
2019-12-20 17:27 ` [Intel-gfx] [PATCH 07/10] drm/i915: prefer 3-letter acronym for ironlake Lucas De Marchi
` (6 subsequent siblings)
12 siblings, 0 replies; 17+ messages in thread
From: Lucas De Marchi @ 2019-12-20 17:27 UTC (permalink / raw)
To: intel-gfx
We are currently using a mix of platform name and acronym to name the
functions. Let's prefer the acronym as it should be clear what platform
it's about and it's shorter, so it doesn't go over 80 columns in a few
cases. This converts icelake to icl where appropriate.
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
---
drivers/gpu/drm/i915/display/intel_display.c | 11 +++++------
drivers/gpu/drm/i915/gt/intel_mocs.c | 6 +++---
2 files changed, 8 insertions(+), 9 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c
index 9983113ff050..2213f9505d49 100644
--- a/drivers/gpu/drm/i915/display/intel_display.c
+++ b/drivers/gpu/drm/i915/display/intel_display.c
@@ -10205,9 +10205,8 @@ static void cnl_get_ddi_pll(struct drm_i915_private *dev_priv, enum port port,
pipe_config->shared_dpll = intel_get_shared_dpll_by_id(dev_priv, id);
}
-static void icelake_get_ddi_pll(struct drm_i915_private *dev_priv,
- enum port port,
- struct intel_crtc_state *pipe_config)
+static void icl_get_ddi_pll(struct drm_i915_private *dev_priv, enum port port,
+ struct intel_crtc_state *pipe_config)
{
enum phy phy = intel_port_to_phy(dev_priv, port);
enum icl_port_dpll_id port_dpll_id;
@@ -10491,7 +10490,7 @@ static void hsw_get_ddi_port_state(struct intel_crtc *crtc,
}
if (INTEL_GEN(dev_priv) >= 11)
- icelake_get_ddi_pll(dev_priv, port, pipe_config);
+ icl_get_ddi_pll(dev_priv, port, pipe_config);
else if (IS_CANNONLAKE(dev_priv))
cnl_get_ddi_pll(dev_priv, port, pipe_config);
else if (IS_GEN9_BC(dev_priv))
@@ -10542,7 +10541,7 @@ static enum transcoder transcoder_master_readout(struct drm_i915_private *dev_pr
return master_select - 1;
}
-static void icelake_get_trans_port_sync_config(struct intel_crtc_state *crtc_state)
+static void icl_get_trans_port_sync_config(struct intel_crtc_state *crtc_state)
{
struct drm_i915_private *dev_priv = to_i915(crtc_state->uapi.crtc->dev);
u32 transcoders;
@@ -10698,7 +10697,7 @@ static bool hsw_get_pipe_config(struct intel_crtc *crtc,
if (INTEL_GEN(dev_priv) >= 11 &&
!transcoder_is_dsi(pipe_config->cpu_transcoder))
- icelake_get_trans_port_sync_config(pipe_config);
+ icl_get_trans_port_sync_config(pipe_config);
out:
for_each_power_domain(power_domain, power_domain_mask)
diff --git a/drivers/gpu/drm/i915/gt/intel_mocs.c b/drivers/gpu/drm/i915/gt/intel_mocs.c
index cbdeda608359..95f1bc45953b 100644
--- a/drivers/gpu/drm/i915/gt/intel_mocs.c
+++ b/drivers/gpu/drm/i915/gt/intel_mocs.c
@@ -267,7 +267,7 @@ static const struct drm_i915_mocs_entry tigerlake_mocs_table[] = {
L3_3_WB),
};
-static const struct drm_i915_mocs_entry icelake_mocs_table[] = {
+static const struct drm_i915_mocs_entry icl_mocs_table[] = {
/* Base - Uncached (Deprecated) */
MOCS_ENTRY(I915_MOCS_UNCACHED,
LE_1_UC | LE_TC_1_LLC,
@@ -288,8 +288,8 @@ static bool get_mocs_settings(const struct drm_i915_private *i915,
table->table = tigerlake_mocs_table;
table->n_entries = GEN11_NUM_MOCS_ENTRIES;
} else if (IS_GEN(i915, 11)) {
- table->size = ARRAY_SIZE(icelake_mocs_table);
- table->table = icelake_mocs_table;
+ table->size = ARRAY_SIZE(icl_mocs_table);
+ table->table = icl_mocs_table;
table->n_entries = GEN11_NUM_MOCS_ENTRIES;
} else if (IS_GEN9_BC(i915) || IS_CANNONLAKE(i915)) {
table->size = ARRAY_SIZE(skl_mocs_table);
--
2.24.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 17+ messages in thread* [Intel-gfx] [PATCH 07/10] drm/i915: prefer 3-letter acronym for ironlake
2019-12-20 17:27 [Intel-gfx] [PATCH 00/10] Prefer acronym for prefixes Lucas De Marchi
` (5 preceding siblings ...)
2019-12-20 17:27 ` [Intel-gfx] [PATCH 06/10] drm/i915: prefer 3-letter acronym for icelake Lucas De Marchi
@ 2019-12-20 17:27 ` Lucas De Marchi
2019-12-20 17:27 ` [Intel-gfx] [PATCH 08/10] drm/i915: prefer 3-letter acronym for broadwell Lucas De Marchi
` (5 subsequent siblings)
12 siblings, 0 replies; 17+ messages in thread
From: Lucas De Marchi @ 2019-12-20 17:27 UTC (permalink / raw)
To: intel-gfx
We are currently using a mix of platform name and acronym to name the
functions. Let's prefer the acronym as it should be clear what platform
it's about and it's shorter, so it doesn't go over 80 columns in a few
cases. This converts ironlake to ilk where appropriate.
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
---
drivers/gpu/drm/i915/display/intel_crt.c | 8 +-
drivers/gpu/drm/i915/display/intel_ddi.c | 2 +-
drivers/gpu/drm/i915/display/intel_display.c | 168 +++++++++---------
drivers/gpu/drm/i915/display/intel_display.h | 4 +-
drivers/gpu/drm/i915/display/intel_dp.c | 34 ++--
drivers/gpu/drm/i915/display/intel_dp_mst.c | 2 +-
.../drm/i915/display/intel_fifo_underrun.c | 6 +-
drivers/gpu/drm/i915/gt/intel_reset.c | 7 +-
drivers/gpu/drm/i915/i915_debugfs.c | 4 +-
drivers/gpu/drm/i915/i915_irq.c | 12 +-
drivers/gpu/drm/i915/intel_pm.c | 4 +-
11 files changed, 125 insertions(+), 126 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/intel_crt.c b/drivers/gpu/drm/i915/display/intel_crt.c
index b2b1336ecdb6..cbe5978e7fb5 100644
--- a/drivers/gpu/drm/i915/display/intel_crt.c
+++ b/drivers/gpu/drm/i915/display/intel_crt.c
@@ -247,7 +247,7 @@ static void hsw_post_disable_crt(struct intel_encoder *encoder,
intel_ddi_disable_transcoder_func(old_crtc_state);
- ironlake_pfit_disable(old_crtc_state);
+ ilk_pfit_disable(old_crtc_state);
intel_ddi_disable_pipe_clock(old_crtc_state);
@@ -351,7 +351,7 @@ intel_crt_mode_valid(struct drm_connector *connector,
/* The FDI receiver on LPT only supports 8bpc and only has 2 lanes. */
if (HAS_PCH_LPT(dev_priv) &&
- (ironlake_get_lanes_required(mode->clock, 270000, 24) > 2))
+ ilk_get_lanes_required(mode->clock, 270000, 24) > 2)
return MODE_CLOCK_HIGH;
/* HSW/BDW FDI limited to 4k */
@@ -427,7 +427,7 @@ static int hsw_crt_compute_config(struct intel_encoder *encoder,
return 0;
}
-static bool intel_ironlake_crt_detect_hotplug(struct drm_connector *connector)
+static bool ilk_crt_detect_hotplug(struct drm_connector *connector)
{
struct drm_device *dev = connector->dev;
struct intel_crt *crt = intel_attached_crt(connector);
@@ -535,7 +535,7 @@ static bool intel_crt_detect_hotplug(struct drm_connector *connector)
int i, tries = 0;
if (HAS_PCH_SPLIT(dev_priv))
- return intel_ironlake_crt_detect_hotplug(connector);
+ return ilk_crt_detect_hotplug(connector);
if (IS_VALLEYVIEW(dev_priv))
return valleyview_crt_detect_hotplug(connector);
diff --git a/drivers/gpu/drm/i915/display/intel_ddi.c b/drivers/gpu/drm/i915/display/intel_ddi.c
index b52c31721755..62fa73815d8a 100644
--- a/drivers/gpu/drm/i915/display/intel_ddi.c
+++ b/drivers/gpu/drm/i915/display/intel_ddi.c
@@ -3898,7 +3898,7 @@ static void intel_ddi_post_disable(struct intel_encoder *encoder,
if (INTEL_GEN(dev_priv) >= 9)
skl_scaler_disable(old_crtc_state);
else
- ironlake_pfit_disable(old_crtc_state);
+ ilk_pfit_disable(old_crtc_state);
/*
* When called from DP MST code:
diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c
index 2213f9505d49..5c14f1e5a871 100644
--- a/drivers/gpu/drm/i915/display/intel_display.c
+++ b/drivers/gpu/drm/i915/display/intel_display.c
@@ -145,8 +145,8 @@ static const u64 cursor_format_modifiers[] = {
static void i9xx_crtc_clock_get(struct intel_crtc *crtc,
struct intel_crtc_state *pipe_config);
-static void ironlake_pch_clock_get(struct intel_crtc *crtc,
- struct intel_crtc_state *pipe_config);
+static void ilk_pch_clock_get(struct intel_crtc *crtc,
+ struct intel_crtc_state *pipe_config);
static int intel_framebuffer_init(struct intel_framebuffer *ifb,
struct drm_i915_gem_object *obj,
@@ -157,7 +157,7 @@ static void intel_cpu_transcoder_set_m_n(const struct intel_crtc_state *crtc_sta
const struct intel_link_m_n *m_n,
const struct intel_link_m_n *m2_n2);
static void i9xx_set_pipeconf(const struct intel_crtc_state *crtc_state);
-static void ironlake_set_pipeconf(const struct intel_crtc_state *crtc_state);
+static void ilk_set_pipeconf(const struct intel_crtc_state *crtc_state);
static void hsw_set_pipeconf(const struct intel_crtc_state *crtc_state);
static void bdw_set_pipemisc(const struct intel_crtc_state *crtc_state);
static void vlv_prepare_pll(struct intel_crtc *crtc,
@@ -165,7 +165,7 @@ static void vlv_prepare_pll(struct intel_crtc *crtc,
static void chv_prepare_pll(struct intel_crtc *crtc,
const struct intel_crtc_state *pipe_config);
static void skl_pfit_enable(const struct intel_crtc_state *crtc_state);
-static void ironlake_pfit_enable(const struct intel_crtc_state *crtc_state);
+static void ilk_pfit_enable(const struct intel_crtc_state *crtc_state);
static void intel_modeset_setup_hw_state(struct drm_device *dev,
struct drm_modeset_acquire_ctx *ctx);
@@ -401,7 +401,7 @@ static const struct intel_limit pnv_limits_lvds = {
* We calculate clock using (register_value + 2) for N/M1/M2, so here
* the range value for them is (actual_value - 2).
*/
-static const struct intel_limit intel_limits_ironlake_dac = {
+static const struct intel_limit ilk_limits_dac = {
.dot = { .min = 25000, .max = 350000 },
.vco = { .min = 1760000, .max = 3510000 },
.n = { .min = 1, .max = 5 },
@@ -414,7 +414,7 @@ static const struct intel_limit intel_limits_ironlake_dac = {
.p2_slow = 10, .p2_fast = 5 },
};
-static const struct intel_limit intel_limits_ironlake_single_lvds = {
+static const struct intel_limit ilk_limits_single_lvds = {
.dot = { .min = 25000, .max = 350000 },
.vco = { .min = 1760000, .max = 3510000 },
.n = { .min = 1, .max = 3 },
@@ -427,7 +427,7 @@ static const struct intel_limit intel_limits_ironlake_single_lvds = {
.p2_slow = 14, .p2_fast = 14 },
};
-static const struct intel_limit intel_limits_ironlake_dual_lvds = {
+static const struct intel_limit ilk_limits_dual_lvds = {
.dot = { .min = 25000, .max = 350000 },
.vco = { .min = 1760000, .max = 3510000 },
.n = { .min = 1, .max = 3 },
@@ -441,7 +441,7 @@ static const struct intel_limit intel_limits_ironlake_dual_lvds = {
};
/* LVDS 100mhz refclk limits. */
-static const struct intel_limit intel_limits_ironlake_single_lvds_100m = {
+static const struct intel_limit ilk_limits_single_lvds_100m = {
.dot = { .min = 25000, .max = 350000 },
.vco = { .min = 1760000, .max = 3510000 },
.n = { .min = 1, .max = 2 },
@@ -454,7 +454,7 @@ static const struct intel_limit intel_limits_ironlake_single_lvds_100m = {
.p2_slow = 14, .p2_fast = 14 },
};
-static const struct intel_limit intel_limits_ironlake_dual_lvds_100m = {
+static const struct intel_limit ilk_limits_dual_lvds_100m = {
.dot = { .min = 25000, .max = 350000 },
.vco = { .min = 1760000, .max = 3510000 },
.n = { .min = 1, .max = 3 },
@@ -1636,7 +1636,7 @@ void vlv_wait_port_ready(struct drm_i915_private *dev_priv,
I915_READ(dpll_reg) & port_mask, expected_mask);
}
-static void ironlake_enable_pch_transcoder(const struct intel_crtc_state *crtc_state)
+static void ilk_enable_pch_transcoder(const struct intel_crtc_state *crtc_state)
{
struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
@@ -1734,8 +1734,8 @@ static void lpt_enable_pch_transcoder(struct drm_i915_private *dev_priv,
DRM_ERROR("Failed to enable PCH transcoder\n");
}
-static void ironlake_disable_pch_transcoder(struct drm_i915_private *dev_priv,
- enum pipe pipe)
+static void ilk_disable_pch_transcoder(struct drm_i915_private *dev_priv,
+ enum pipe pipe)
{
i915_reg_t reg;
u32 val;
@@ -4621,8 +4621,8 @@ static void intel_fdi_normal_train(struct intel_crtc *crtc)
}
/* The FDI link training functions for ILK/Ibexpeak. */
-static void ironlake_fdi_link_train(struct intel_crtc *crtc,
- const struct intel_crtc_state *crtc_state)
+static void ilk_fdi_link_train(struct intel_crtc *crtc,
+ const struct intel_crtc_state *crtc_state)
{
struct drm_device *dev = crtc->base.dev;
struct drm_i915_private *dev_priv = to_i915(dev);
@@ -4974,7 +4974,7 @@ static void ivb_manual_fdi_link_train(struct intel_crtc *crtc,
DRM_DEBUG_KMS("FDI train done.\n");
}
-static void ironlake_fdi_pll_enable(const struct intel_crtc_state *crtc_state)
+static void ilk_fdi_pll_enable(const struct intel_crtc_state *crtc_state)
{
struct intel_crtc *intel_crtc = to_intel_crtc(crtc_state->uapi.crtc);
struct drm_i915_private *dev_priv = to_i915(intel_crtc->base.dev);
@@ -5011,7 +5011,7 @@ static void ironlake_fdi_pll_enable(const struct intel_crtc_state *crtc_state)
}
}
-static void ironlake_fdi_pll_disable(struct intel_crtc *intel_crtc)
+static void ilk_fdi_pll_disable(struct intel_crtc *intel_crtc)
{
struct drm_device *dev = intel_crtc->base.dev;
struct drm_i915_private *dev_priv = to_i915(dev);
@@ -5041,7 +5041,7 @@ static void ironlake_fdi_pll_disable(struct intel_crtc *intel_crtc)
udelay(100);
}
-static void ironlake_fdi_disable(struct intel_crtc *crtc)
+static void ilk_fdi_disable(struct intel_crtc *crtc)
{
struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
enum pipe pipe = crtc->pipe;
@@ -5248,8 +5248,8 @@ int lpt_get_iclkip(struct drm_i915_private *dev_priv)
desired_divisor << auxdiv);
}
-static void ironlake_pch_transcoder_set_timings(const struct intel_crtc_state *crtc_state,
- enum pipe pch_transcoder)
+static void ilk_pch_transcoder_set_timings(const struct intel_crtc_state *crtc_state,
+ enum pipe pch_transcoder)
{
struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
@@ -5353,8 +5353,8 @@ intel_get_crtc_new_encoder(const struct intel_atomic_state *state,
* - DP transcoding bits
* - transcoder
*/
-static void ironlake_pch_enable(const struct intel_atomic_state *state,
- const struct intel_crtc_state *crtc_state)
+static void ilk_pch_enable(const struct intel_atomic_state *state,
+ const struct intel_crtc_state *crtc_state)
{
struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
struct drm_device *dev = crtc->base.dev;
@@ -5402,7 +5402,7 @@ static void ironlake_pch_enable(const struct intel_atomic_state *state,
/* set transcoder timing, panel must allow it */
assert_panel_unlocked(dev_priv, pipe);
- ironlake_pch_transcoder_set_timings(crtc_state, pipe);
+ ilk_pch_transcoder_set_timings(crtc_state, pipe);
intel_fdi_normal_train(crtc);
@@ -5434,7 +5434,7 @@ static void ironlake_pch_enable(const struct intel_atomic_state *state,
I915_WRITE(reg, temp);
}
- ironlake_enable_pch_transcoder(crtc_state);
+ ilk_enable_pch_transcoder(crtc_state);
}
static void lpt_pch_enable(const struct intel_atomic_state *state,
@@ -5449,7 +5449,7 @@ static void lpt_pch_enable(const struct intel_atomic_state *state,
lpt_program_iclkip(crtc_state);
/* Set transcoder timing. */
- ironlake_pch_transcoder_set_timings(crtc_state, PIPE_A);
+ ilk_pch_transcoder_set_timings(crtc_state, PIPE_A);
lpt_enable_pch_transcoder(dev_priv, cpu_transcoder);
}
@@ -5795,7 +5795,7 @@ static void skl_pfit_enable(const struct intel_crtc_state *crtc_state)
}
}
-static void ironlake_pfit_enable(const struct intel_crtc_state *crtc_state)
+static void ilk_pfit_enable(const struct intel_crtc_state *crtc_state)
{
struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
@@ -6394,8 +6394,8 @@ static void intel_disable_primary_plane(const struct intel_crtc_state *crtc_stat
plane->disable_plane(plane, crtc_state);
}
-static void ironlake_crtc_enable(struct intel_atomic_state *state,
- struct intel_crtc *crtc)
+static void ilk_crtc_enable(struct intel_atomic_state *state,
+ struct intel_crtc *crtc)
{
const struct intel_crtc_state *new_crtc_state =
intel_atomic_get_new_crtc_state(state, crtc);
@@ -6431,7 +6431,7 @@ static void ironlake_crtc_enable(struct intel_atomic_state *state,
intel_cpu_transcoder_set_m_n(new_crtc_state,
&new_crtc_state->fdi_m_n, NULL);
- ironlake_set_pipeconf(new_crtc_state);
+ ilk_set_pipeconf(new_crtc_state);
crtc->active = true;
@@ -6441,13 +6441,13 @@ static void ironlake_crtc_enable(struct intel_atomic_state *state,
/* Note: FDI PLL enabling _must_ be done before we enable the
* cpu pipes, hence this is separate from all the other fdi/pch
* enabling. */
- ironlake_fdi_pll_enable(new_crtc_state);
+ ilk_fdi_pll_enable(new_crtc_state);
} else {
assert_fdi_tx_disabled(dev_priv, pipe);
assert_fdi_rx_disabled(dev_priv, pipe);
}
- ironlake_pfit_enable(new_crtc_state);
+ ilk_pfit_enable(new_crtc_state);
/*
* On ILK+ LUT must be loaded before the pipe is running but with
@@ -6463,7 +6463,7 @@ static void ironlake_crtc_enable(struct intel_atomic_state *state,
intel_enable_pipe(new_crtc_state);
if (new_crtc_state->has_pch_encoder)
- ironlake_pch_enable(state, new_crtc_state);
+ ilk_pch_enable(state, new_crtc_state);
intel_crtc_vblank_on(new_crtc_state);
@@ -6597,7 +6597,7 @@ static void hsw_crtc_enable(struct intel_atomic_state *state,
if (INTEL_GEN(dev_priv) >= 9)
skl_pfit_enable(new_crtc_state);
else
- ironlake_pfit_enable(new_crtc_state);
+ ilk_pfit_enable(new_crtc_state);
/*
* On ILK+ LUT must be loaded before the pipe is running but with
@@ -6646,7 +6646,7 @@ static void hsw_crtc_enable(struct intel_atomic_state *state,
}
}
-void ironlake_pfit_disable(const struct intel_crtc_state *old_crtc_state)
+void ilk_pfit_disable(const struct intel_crtc_state *old_crtc_state)
{
struct intel_crtc *crtc = to_intel_crtc(old_crtc_state->uapi.crtc);
struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
@@ -6661,8 +6661,8 @@ void ironlake_pfit_disable(const struct intel_crtc_state *old_crtc_state)
}
}
-static void ironlake_crtc_disable(struct intel_atomic_state *state,
- struct intel_crtc *crtc)
+static void ilk_crtc_disable(struct intel_atomic_state *state,
+ struct intel_crtc *crtc)
{
const struct intel_crtc_state *old_crtc_state =
intel_atomic_get_old_crtc_state(state, crtc);
@@ -6683,15 +6683,15 @@ static void ironlake_crtc_disable(struct intel_atomic_state *state,
intel_disable_pipe(old_crtc_state);
- ironlake_pfit_disable(old_crtc_state);
+ ilk_pfit_disable(old_crtc_state);
if (old_crtc_state->has_pch_encoder)
- ironlake_fdi_disable(crtc);
+ ilk_fdi_disable(crtc);
intel_encoders_post_disable(state, crtc);
if (old_crtc_state->has_pch_encoder) {
- ironlake_disable_pch_transcoder(dev_priv, pipe);
+ ilk_disable_pch_transcoder(dev_priv, pipe);
if (HAS_PCH_CPT(dev_priv)) {
i915_reg_t reg;
@@ -6711,7 +6711,7 @@ static void ironlake_crtc_disable(struct intel_atomic_state *state,
I915_WRITE(PCH_DPLL_SEL, temp);
}
- ironlake_fdi_pll_disable(crtc);
+ ilk_fdi_pll_disable(crtc);
}
intel_set_cpu_fifo_underrun_reporting(dev_priv, pipe, true);
@@ -7256,8 +7256,8 @@ static int pipe_required_fdi_lanes(struct intel_crtc_state *crtc_state)
return 0;
}
-static int ironlake_check_fdi_lanes(struct drm_device *dev, enum pipe pipe,
- struct intel_crtc_state *pipe_config)
+static int ilk_check_fdi_lanes(struct drm_device *dev, enum pipe pipe,
+ struct intel_crtc_state *pipe_config)
{
struct drm_i915_private *dev_priv = to_i915(dev);
struct drm_atomic_state *state = pipe_config->uapi.state;
@@ -7329,8 +7329,8 @@ static int ironlake_check_fdi_lanes(struct drm_device *dev, enum pipe pipe,
}
#define RETRY 1
-static int ironlake_fdi_compute_config(struct intel_crtc *intel_crtc,
- struct intel_crtc_state *pipe_config)
+static int ilk_fdi_compute_config(struct intel_crtc *intel_crtc,
+ struct intel_crtc_state *pipe_config)
{
struct drm_device *dev = intel_crtc->base.dev;
const struct drm_display_mode *adjusted_mode = &pipe_config->hw.adjusted_mode;
@@ -7349,15 +7349,15 @@ static int ironlake_fdi_compute_config(struct intel_crtc *intel_crtc,
fdi_dotclock = adjusted_mode->crtc_clock;
- lane = ironlake_get_lanes_required(fdi_dotclock, link_bw,
- pipe_config->pipe_bpp);
+ lane = ilk_get_lanes_required(fdi_dotclock, link_bw,
+ pipe_config->pipe_bpp);
pipe_config->fdi_lanes = lane;
intel_link_compute_m_n(pipe_config->pipe_bpp, lane, fdi_dotclock,
link_bw, &pipe_config->fdi_m_n, false, false);
- ret = ironlake_check_fdi_lanes(dev, intel_crtc->pipe, pipe_config);
+ ret = ilk_check_fdi_lanes(dev, intel_crtc->pipe, pipe_config);
if (ret == -EDEADLK)
return ret;
@@ -7563,7 +7563,7 @@ static int intel_crtc_compute_config(struct intel_crtc *crtc,
intel_crtc_compute_pixel_rate(pipe_config);
if (pipe_config->has_pch_encoder)
- return ironlake_fdi_compute_config(crtc, pipe_config);
+ return ilk_fdi_compute_config(crtc, pipe_config);
return 0;
}
@@ -8976,7 +8976,7 @@ static bool i9xx_get_pipe_config(struct intel_crtc *crtc,
return ret;
}
-static void ironlake_init_pch_refclk(struct drm_i915_private *dev_priv)
+static void ilk_init_pch_refclk(struct drm_i915_private *dev_priv)
{
struct intel_encoder *encoder;
int i;
@@ -9474,12 +9474,12 @@ static void lpt_init_pch_refclk(struct drm_i915_private *dev_priv)
void intel_init_pch_refclk(struct drm_i915_private *dev_priv)
{
if (HAS_PCH_IBX(dev_priv) || HAS_PCH_CPT(dev_priv))
- ironlake_init_pch_refclk(dev_priv);
+ ilk_init_pch_refclk(dev_priv);
else if (HAS_PCH_LPT(dev_priv))
lpt_init_pch_refclk(dev_priv);
}
-static void ironlake_set_pipeconf(const struct intel_crtc_state *crtc_state)
+static void ilk_set_pipeconf(const struct intel_crtc_state *crtc_state)
{
struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
@@ -9623,7 +9623,7 @@ int bdw_get_pipemisc_bpp(struct intel_crtc *crtc)
}
}
-int ironlake_get_lanes_required(int target_clock, int link_bw, int bpp)
+int ilk_get_lanes_required(int target_clock, int link_bw, int bpp)
{
/*
* Account for spread spectrum to avoid
@@ -9634,14 +9634,14 @@ int ironlake_get_lanes_required(int target_clock, int link_bw, int bpp)
return DIV_ROUND_UP(bps, link_bw * 8);
}
-static bool ironlake_needs_fb_cb_tune(struct dpll *dpll, int factor)
+static bool ilk_needs_fb_cb_tune(struct dpll *dpll, int factor)
{
return i9xx_dpll_compute_m(dpll) < factor * dpll->n;
}
-static void ironlake_compute_dpll(struct intel_crtc *crtc,
- struct intel_crtc_state *crtc_state,
- struct dpll *reduced_clock)
+static void ilk_compute_dpll(struct intel_crtc *crtc,
+ struct intel_crtc_state *crtc_state,
+ struct dpll *reduced_clock)
{
struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
u32 dpll, fp, fp2;
@@ -9661,7 +9661,7 @@ static void ironlake_compute_dpll(struct intel_crtc *crtc,
fp = i9xx_dpll_compute_fp(&crtc_state->dpll);
- if (ironlake_needs_fb_cb_tune(&crtc_state->dpll, factor))
+ if (ilk_needs_fb_cb_tune(&crtc_state->dpll, factor))
fp |= FP_CB_TUNE;
if (reduced_clock) {
@@ -9741,8 +9741,8 @@ static void ironlake_compute_dpll(struct intel_crtc *crtc,
crtc_state->dpll_hw_state.fp1 = fp2;
}
-static int ironlake_crtc_compute_clock(struct intel_crtc *crtc,
- struct intel_crtc_state *crtc_state)
+static int ilk_crtc_compute_clock(struct intel_crtc *crtc,
+ struct intel_crtc_state *crtc_state)
{
struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
struct intel_atomic_state *state =
@@ -9766,17 +9766,17 @@ static int ironlake_crtc_compute_clock(struct intel_crtc *crtc,
if (intel_is_dual_link_lvds(dev_priv)) {
if (refclk == 100000)
- limit = &intel_limits_ironlake_dual_lvds_100m;
+ limit = &ilk_limits_dual_lvds_100m;
else
- limit = &intel_limits_ironlake_dual_lvds;
+ limit = &ilk_limits_dual_lvds;
} else {
if (refclk == 100000)
- limit = &intel_limits_ironlake_single_lvds_100m;
+ limit = &ilk_limits_single_lvds_100m;
else
- limit = &intel_limits_ironlake_single_lvds;
+ limit = &ilk_limits_single_lvds;
}
} else {
- limit = &intel_limits_ironlake_dac;
+ limit = &ilk_limits_dac;
}
if (!crtc_state->clock_set &&
@@ -9786,7 +9786,7 @@ static int ironlake_crtc_compute_clock(struct intel_crtc *crtc,
return -EINVAL;
}
- ironlake_compute_dpll(crtc, crtc_state, NULL);
+ ilk_compute_dpll(crtc, crtc_state, NULL);
if (!intel_reserve_shared_dplls(state, crtc, NULL)) {
DRM_DEBUG_KMS("failed to find PLL for pipe %c\n",
@@ -9861,8 +9861,8 @@ void intel_dp_get_m_n(struct intel_crtc *crtc,
&pipe_config->dp_m2_n2);
}
-static void ironlake_get_fdi_m_n_config(struct intel_crtc *crtc,
- struct intel_crtc_state *pipe_config)
+static void ilk_get_fdi_m_n_config(struct intel_crtc *crtc,
+ struct intel_crtc_state *pipe_config)
{
intel_cpu_transcoder_get_m_n(crtc, pipe_config->cpu_transcoder,
&pipe_config->fdi_m_n, NULL);
@@ -10026,8 +10026,8 @@ skl_get_initial_plane_config(struct intel_crtc *crtc,
kfree(intel_fb);
}
-static void ironlake_get_pfit_config(struct intel_crtc *crtc,
- struct intel_crtc_state *pipe_config)
+static void ilk_get_pfit_config(struct intel_crtc *crtc,
+ struct intel_crtc_state *pipe_config)
{
struct drm_device *dev = crtc->base.dev;
struct drm_i915_private *dev_priv = to_i915(dev);
@@ -10050,8 +10050,8 @@ static void ironlake_get_pfit_config(struct intel_crtc *crtc,
}
}
-static bool ironlake_get_pipe_config(struct intel_crtc *crtc,
- struct intel_crtc_state *pipe_config)
+static bool ilk_get_pipe_config(struct intel_crtc *crtc,
+ struct intel_crtc_state *pipe_config)
{
struct drm_device *dev = crtc->base.dev;
struct drm_i915_private *dev_priv = to_i915(dev);
@@ -10122,7 +10122,7 @@ static bool ironlake_get_pipe_config(struct intel_crtc *crtc,
pipe_config->fdi_lanes = ((FDI_DP_PORT_WIDTH_MASK & tmp) >>
FDI_DP_PORT_WIDTH_SHIFT) + 1;
- ironlake_get_fdi_m_n_config(crtc, pipe_config);
+ ilk_get_fdi_m_n_config(crtc, pipe_config);
if (HAS_PCH_IBX(dev_priv)) {
/*
@@ -10150,7 +10150,7 @@ static bool ironlake_get_pipe_config(struct intel_crtc *crtc,
((tmp & PLL_REF_SDVO_HDMI_MULTIPLIER_MASK)
>> PLL_REF_SDVO_HDMI_MULTIPLIER_SHIFT) + 1;
- ironlake_pch_clock_get(crtc, pipe_config);
+ ilk_pch_clock_get(crtc, pipe_config);
} else {
pipe_config->pixel_multiplier = 1;
}
@@ -10158,7 +10158,7 @@ static bool ironlake_get_pipe_config(struct intel_crtc *crtc,
intel_get_pipe_timings(crtc, pipe_config);
intel_get_pipe_src_size(crtc, pipe_config);
- ironlake_get_pfit_config(crtc, pipe_config);
+ ilk_get_pfit_config(crtc, pipe_config);
ret = true;
@@ -10519,7 +10519,7 @@ static void hsw_get_ddi_port_state(struct intel_crtc *crtc,
pipe_config->fdi_lanes = ((FDI_DP_PORT_WIDTH_MASK & tmp) >>
FDI_DP_PORT_WIDTH_SHIFT) + 1;
- ironlake_get_fdi_m_n_config(crtc, pipe_config);
+ ilk_get_fdi_m_n_config(crtc, pipe_config);
}
}
@@ -10671,7 +10671,7 @@ static bool hsw_get_pipe_config(struct intel_crtc *crtc,
if (INTEL_GEN(dev_priv) >= 9)
skl_get_pfit_config(crtc, pipe_config);
else
- ironlake_get_pfit_config(crtc, pipe_config);
+ ilk_get_pfit_config(crtc, pipe_config);
}
if (hsw_crtc_supports_ips(crtc)) {
@@ -11614,8 +11614,8 @@ int intel_dotclock_calculate(int link_freq,
return div_u64(mul_u32_u32(m_n->link_m, link_freq), m_n->link_n);
}
-static void ironlake_pch_clock_get(struct intel_crtc *crtc,
- struct intel_crtc_state *pipe_config)
+static void ilk_pch_clock_get(struct intel_crtc *crtc,
+ struct intel_crtc_state *pipe_config)
{
struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
@@ -14211,9 +14211,9 @@ static void intel_pipe_fastset(const struct intel_crtc_state *old_crtc_state,
skl_pfit_enable(new_crtc_state);
} else if (HAS_PCH_SPLIT(dev_priv)) {
if (new_crtc_state->pch_pfit.enabled)
- ironlake_pfit_enable(new_crtc_state);
+ ilk_pfit_enable(new_crtc_state);
else if (old_crtc_state->pch_pfit.enabled)
- ironlake_pfit_disable(old_crtc_state);
+ ilk_pfit_disable(old_crtc_state);
}
if (INTEL_GEN(dev_priv) >= 11)
@@ -16570,13 +16570,13 @@ void intel_init_display_hooks(struct drm_i915_private *dev_priv)
dev_priv->display.crtc_enable = hsw_crtc_enable;
dev_priv->display.crtc_disable = hsw_crtc_disable;
} else if (HAS_PCH_SPLIT(dev_priv)) {
- dev_priv->display.get_pipe_config = ironlake_get_pipe_config;
+ dev_priv->display.get_pipe_config = ilk_get_pipe_config;
dev_priv->display.get_initial_plane_config =
i9xx_get_initial_plane_config;
dev_priv->display.crtc_compute_clock =
- ironlake_crtc_compute_clock;
- dev_priv->display.crtc_enable = ironlake_crtc_enable;
- dev_priv->display.crtc_disable = ironlake_crtc_disable;
+ ilk_crtc_compute_clock;
+ dev_priv->display.crtc_enable = ilk_crtc_enable;
+ dev_priv->display.crtc_disable = ilk_crtc_disable;
} else if (IS_CHERRYVIEW(dev_priv)) {
dev_priv->display.get_pipe_config = i9xx_get_pipe_config;
dev_priv->display.get_initial_plane_config =
@@ -16622,7 +16622,7 @@ void intel_init_display_hooks(struct drm_i915_private *dev_priv)
}
if (IS_GEN(dev_priv, 5)) {
- dev_priv->display.fdi_link_train = ironlake_fdi_link_train;
+ dev_priv->display.fdi_link_train = ilk_fdi_link_train;
} else if (IS_GEN(dev_priv, 6)) {
dev_priv->display.fdi_link_train = gen6_fdi_link_train;
} else if (IS_IVYBRIDGE(dev_priv)) {
diff --git a/drivers/gpu/drm/i915/display/intel_display.h b/drivers/gpu/drm/i915/display/intel_display.h
index 5093ea5dcd95..2a88422e4454 100644
--- a/drivers/gpu/drm/i915/display/intel_display.h
+++ b/drivers/gpu/drm/i915/display/intel_display.h
@@ -521,7 +521,7 @@ int intel_get_pipe_from_crtc_id_ioctl(struct drm_device *dev, void *data,
u32 intel_crtc_get_vblank_counter(struct intel_crtc *crtc);
void intel_crtc_vblank_off(const struct intel_crtc_state *crtc_state);
-int ironlake_get_lanes_required(int target_clock, int link_bw, int bpp);
+int ilk_get_lanes_required(int target_clock, int link_bw, int bpp);
void vlv_wait_port_ready(struct drm_i915_private *dev_priv,
struct intel_digital_port *dport,
unsigned int expected_mask);
@@ -579,7 +579,7 @@ void intel_crtc_arm_fifo_underrun(struct intel_crtc *crtc,
u16 skl_scaler_calc_phase(int sub, int scale, bool chroma_center);
int skl_update_scaler_crtc(struct intel_crtc_state *crtc_state);
void skl_scaler_disable(const struct intel_crtc_state *old_crtc_state);
-void ironlake_pfit_disable(const struct intel_crtc_state *old_crtc_state);
+void ilk_pfit_disable(const struct intel_crtc_state *old_crtc_state);
u32 glk_plane_color_ctl(const struct intel_crtc_state *crtc_state,
const struct intel_plane_state *plane_state);
u32 glk_plane_color_ctl_crtc(const struct intel_crtc_state *crtc_state);
diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c
index 2f31d226c6eb..991f343579ef 100644
--- a/drivers/gpu/drm/i915/display/intel_dp.c
+++ b/drivers/gpu/drm/i915/display/intel_dp.c
@@ -2509,7 +2509,7 @@ static void intel_dp_prepare(struct intel_encoder *encoder,
*
* CPT PCH is quite different, having many bits moved
* to the TRANS_DP_CTL register instead. That
- * configuration happens (oddly) in ironlake_pch_enable
+ * configuration happens (oddly) in ilk_pch_enable
*/
/* Preserve the BIOS-computed detected bit. This is
@@ -2653,7 +2653,7 @@ static void edp_wait_backlight_off(struct intel_dp *intel_dp)
* is locked
*/
-static u32 ironlake_get_pp_control(struct intel_dp *intel_dp)
+static u32 ilk_get_pp_control(struct intel_dp *intel_dp)
{
struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
u32 control;
@@ -2703,7 +2703,7 @@ static bool edp_panel_vdd_on(struct intel_dp *intel_dp)
if (!edp_have_panel_power(intel_dp))
wait_panel_power_cycle(intel_dp);
- pp = ironlake_get_pp_control(intel_dp);
+ pp = ilk_get_pp_control(intel_dp);
pp |= EDP_FORCE_VDD;
pp_stat_reg = _pp_stat_reg(intel_dp);
@@ -2768,7 +2768,7 @@ static void edp_panel_vdd_off_sync(struct intel_dp *intel_dp)
intel_dig_port->base.base.base.id,
intel_dig_port->base.base.name);
- pp = ironlake_get_pp_control(intel_dp);
+ pp = ilk_get_pp_control(intel_dp);
pp &= ~EDP_FORCE_VDD;
pp_ctrl_reg = _pp_ctrl_reg(intel_dp);
@@ -2864,7 +2864,7 @@ static void edp_panel_on(struct intel_dp *intel_dp)
wait_panel_power_cycle(intel_dp);
pp_ctrl_reg = _pp_ctrl_reg(intel_dp);
- pp = ironlake_get_pp_control(intel_dp);
+ pp = ilk_get_pp_control(intel_dp);
if (IS_GEN(dev_priv, 5)) {
/* ILK workaround: disable reset around power sequence */
pp &= ~PANEL_POWER_RESET;
@@ -2919,7 +2919,7 @@ static void edp_panel_off(struct intel_dp *intel_dp)
WARN(!intel_dp->want_panel_vdd, "Need [ENCODER:%d:%s] VDD to turn off panel\n",
dig_port->base.base.base.id, dig_port->base.base.name);
- pp = ironlake_get_pp_control(intel_dp);
+ pp = ilk_get_pp_control(intel_dp);
/* We need to switch off panel power _and_ force vdd, for otherwise some
* panels get very unhappy and cease to work. */
pp &= ~(PANEL_POWER_ON | PANEL_POWER_RESET | EDP_FORCE_VDD |
@@ -2968,7 +2968,7 @@ static void _intel_edp_backlight_on(struct intel_dp *intel_dp)
i915_reg_t pp_ctrl_reg = _pp_ctrl_reg(intel_dp);
u32 pp;
- pp = ironlake_get_pp_control(intel_dp);
+ pp = ilk_get_pp_control(intel_dp);
pp |= EDP_BLC_ENABLE;
I915_WRITE(pp_ctrl_reg, pp);
@@ -3004,7 +3004,7 @@ static void _intel_edp_backlight_off(struct intel_dp *intel_dp)
i915_reg_t pp_ctrl_reg = _pp_ctrl_reg(intel_dp);
u32 pp;
- pp = ironlake_get_pp_control(intel_dp);
+ pp = ilk_get_pp_control(intel_dp);
pp &= ~EDP_BLC_ENABLE;
I915_WRITE(pp_ctrl_reg, pp);
@@ -3042,7 +3042,7 @@ static void intel_edp_backlight_power(struct intel_connector *connector,
is_enabled = false;
with_pps_lock(intel_dp, wakeref)
- is_enabled = ironlake_get_pp_control(intel_dp) & EDP_BLC_ENABLE;
+ is_enabled = ilk_get_pp_control(intel_dp) & EDP_BLC_ENABLE;
if (is_enabled == enable)
return;
@@ -3079,8 +3079,8 @@ static void assert_edp_pll(struct drm_i915_private *dev_priv, bool state)
#define assert_edp_pll_enabled(d) assert_edp_pll((d), true)
#define assert_edp_pll_disabled(d) assert_edp_pll((d), false)
-static void ironlake_edp_pll_on(struct intel_dp *intel_dp,
- const struct intel_crtc_state *pipe_config)
+static void ilk_edp_pll_on(struct intel_dp *intel_dp,
+ const struct intel_crtc_state *pipe_config)
{
struct intel_crtc *crtc = to_intel_crtc(pipe_config->uapi.crtc);
struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
@@ -3119,8 +3119,8 @@ static void ironlake_edp_pll_on(struct intel_dp *intel_dp,
udelay(200);
}
-static void ironlake_edp_pll_off(struct intel_dp *intel_dp,
- const struct intel_crtc_state *old_crtc_state)
+static void ilk_edp_pll_off(struct intel_dp *intel_dp,
+ const struct intel_crtc_state *old_crtc_state)
{
struct intel_crtc *crtc = to_intel_crtc(old_crtc_state->uapi.crtc);
struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
@@ -3410,7 +3410,7 @@ static void g4x_post_disable_dp(struct intel_encoder *encoder,
/* Only ilk+ has port A */
if (port == PORT_A)
- ironlake_edp_pll_off(intel_dp, old_crtc_state);
+ ilk_edp_pll_off(intel_dp, old_crtc_state);
}
static void vlv_post_disable_dp(struct intel_encoder *encoder,
@@ -3615,7 +3615,7 @@ static void g4x_pre_enable_dp(struct intel_encoder *encoder,
/* Only ilk+ has port A */
if (port == PORT_A)
- ironlake_edp_pll_on(intel_dp, pipe_config);
+ ilk_edp_pll_on(intel_dp, pipe_config);
}
static void vlv_detach_power_sequencer(struct intel_dp *intel_dp)
@@ -6693,7 +6693,7 @@ intel_pps_readout_hw_state(struct intel_dp *intel_dp, struct edp_power_seq *seq)
intel_pps_get_registers(intel_dp, ®s);
- pp_ctl = ironlake_get_pp_control(intel_dp);
+ pp_ctl = ilk_get_pp_control(intel_dp);
/* Ensure PPS is unlocked */
if (!HAS_DDI(dev_priv))
@@ -6863,7 +6863,7 @@ intel_dp_init_panel_power_sequencer_registers(struct intel_dp *intel_dp,
* soon as the new power sequencer gets initialized.
*/
if (force_disable_vdd) {
- u32 pp = ironlake_get_pp_control(intel_dp);
+ u32 pp = ilk_get_pp_control(intel_dp);
WARN(pp & PANEL_POWER_ON, "Panel power already on\n");
diff --git a/drivers/gpu/drm/i915/display/intel_dp_mst.c b/drivers/gpu/drm/i915/display/intel_dp_mst.c
index 01b4608ab56c..cbf623154af1 100644
--- a/drivers/gpu/drm/i915/display/intel_dp_mst.c
+++ b/drivers/gpu/drm/i915/display/intel_dp_mst.c
@@ -250,7 +250,7 @@ static void intel_mst_post_disable_dp(struct intel_encoder *encoder,
if (INTEL_GEN(dev_priv) >= 9)
skl_scaler_disable(old_crtc_state);
else
- ironlake_pfit_disable(old_crtc_state);
+ ilk_pfit_disable(old_crtc_state);
/*
* From TGL spec: "If multi-stream slave transcoder: Configure
diff --git a/drivers/gpu/drm/i915/display/intel_fifo_underrun.c b/drivers/gpu/drm/i915/display/intel_fifo_underrun.c
index ab61f88d1d33..d6e0d0be842e 100644
--- a/drivers/gpu/drm/i915/display/intel_fifo_underrun.c
+++ b/drivers/gpu/drm/i915/display/intel_fifo_underrun.c
@@ -126,8 +126,8 @@ static void i9xx_set_fifo_underrun_reporting(struct drm_device *dev,
}
}
-static void ironlake_set_fifo_underrun_reporting(struct drm_device *dev,
- enum pipe pipe, bool enable)
+static void ilk_set_fifo_underrun_reporting(struct drm_device *dev,
+ enum pipe pipe, bool enable)
{
struct drm_i915_private *dev_priv = to_i915(dev);
u32 bit = (pipe == PIPE_A) ?
@@ -264,7 +264,7 @@ static bool __intel_set_cpu_fifo_underrun_reporting(struct drm_device *dev,
if (HAS_GMCH(dev_priv))
i9xx_set_fifo_underrun_reporting(dev, pipe, enable, old);
else if (IS_GEN_RANGE(dev_priv, 5, 6))
- ironlake_set_fifo_underrun_reporting(dev, pipe, enable);
+ ilk_set_fifo_underrun_reporting(dev, pipe, enable);
else if (IS_GEN(dev_priv, 7))
ivybridge_set_fifo_underrun_reporting(dev, pipe, enable, old);
else if (INTEL_GEN(dev_priv) >= 8)
diff --git a/drivers/gpu/drm/i915/gt/intel_reset.c b/drivers/gpu/drm/i915/gt/intel_reset.c
index f3d1e921fba6..63b544f8b016 100644
--- a/drivers/gpu/drm/i915/gt/intel_reset.c
+++ b/drivers/gpu/drm/i915/gt/intel_reset.c
@@ -230,9 +230,8 @@ static int g4x_do_reset(struct intel_gt *gt,
return ret;
}
-static int ironlake_do_reset(struct intel_gt *gt,
- intel_engine_mask_t engine_mask,
- unsigned int retry)
+static int ilk_do_reset(struct intel_gt *gt, intel_engine_mask_t engine_mask,
+ unsigned int retry)
{
struct intel_uncore *uncore = gt->uncore;
int ret;
@@ -576,7 +575,7 @@ static reset_func intel_get_gpu_reset(const struct intel_gt *gt)
else if (INTEL_GEN(i915) >= 6)
return gen6_reset_engines;
else if (INTEL_GEN(i915) >= 5)
- return ironlake_do_reset;
+ return ilk_do_reset;
else if (IS_G4X(i915))
return g4x_do_reset;
else if (IS_G33(i915) || IS_PINEVIEW(i915))
diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
index d28468eaed57..0407229251bc 100644
--- a/drivers/gpu/drm/i915/i915_debugfs.c
+++ b/drivers/gpu/drm/i915/i915_debugfs.c
@@ -1001,7 +1001,7 @@ static int i915_frequency_info(struct seq_file *m, void *unused)
return ret;
}
-static int ironlake_drpc_info(struct seq_file *m)
+static int ilk_drpc_info(struct seq_file *m)
{
struct drm_i915_private *i915 = node_to_i915(m->private);
struct intel_uncore *uncore = &i915->uncore;
@@ -1209,7 +1209,7 @@ static int i915_drpc_info(struct seq_file *m, void *unused)
else if (INTEL_GEN(dev_priv) >= 6)
err = gen6_drpc_info(m);
else
- err = ironlake_drpc_info(m);
+ err = ilk_drpc_info(m);
}
return err;
diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c
index 42b79f577500..2d6324d2922a 100644
--- a/drivers/gpu/drm/i915/i915_irq.c
+++ b/drivers/gpu/drm/i915/i915_irq.c
@@ -2031,7 +2031,7 @@ static void ivb_display_irq_handler(struct drm_i915_private *dev_priv,
* 4 - Process the interrupt(s) that had bits set in the IIRs.
* 5 - Re-enable Master Interrupt Control.
*/
-static irqreturn_t ironlake_irq_handler(int irq, void *arg)
+static irqreturn_t ilk_irq_handler(int irq, void *arg)
{
struct drm_i915_private *dev_priv = arg;
u32 de_iir, gt_iir, de_ier, sde_ier = 0;
@@ -2742,7 +2742,7 @@ static void vlv_display_irq_postinstall(struct drm_i915_private *dev_priv)
/* drm_dma.h hooks
*/
-static void ironlake_irq_reset(struct drm_i915_private *dev_priv)
+static void ilk_irq_reset(struct drm_i915_private *dev_priv)
{
struct intel_uncore *uncore = &dev_priv->uncore;
@@ -3225,7 +3225,7 @@ static void ibx_irq_postinstall(struct drm_i915_private *dev_priv)
spt_hpd_detection_setup(dev_priv);
}
-static void ironlake_irq_postinstall(struct drm_i915_private *dev_priv)
+static void ilk_irq_postinstall(struct drm_i915_private *dev_priv)
{
struct intel_uncore *uncore = &dev_priv->uncore;
u32 display_mask, extra_mask;
@@ -3980,7 +3980,7 @@ static irq_handler_t intel_irq_handler(struct drm_i915_private *dev_priv)
else if (INTEL_GEN(dev_priv) >= 8)
return gen8_irq_handler;
else
- return ironlake_irq_handler;
+ return ilk_irq_handler;
}
}
@@ -4003,7 +4003,7 @@ static void intel_irq_reset(struct drm_i915_private *dev_priv)
else if (INTEL_GEN(dev_priv) >= 8)
gen8_irq_reset(dev_priv);
else
- ironlake_irq_reset(dev_priv);
+ ilk_irq_reset(dev_priv);
}
}
@@ -4026,7 +4026,7 @@ static void intel_irq_postinstall(struct drm_i915_private *dev_priv)
else if (INTEL_GEN(dev_priv) >= 8)
gen8_irq_postinstall(dev_priv);
else
- ironlake_irq_postinstall(dev_priv);
+ ilk_irq_postinstall(dev_priv);
}
}
diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
index 6d9707eceddd..6b3b2f794013 100644
--- a/drivers/gpu/drm/i915/intel_pm.c
+++ b/drivers/gpu/drm/i915/intel_pm.c
@@ -178,7 +178,7 @@ static void pnv_get_mem_freq(struct drm_i915_private *dev_priv)
dev_priv->is_ddr3 = (tmp & CSHRDDR3CTL_DDR3) ? 1 : 0;
}
-static void i915_ironlake_get_mem_freq(struct drm_i915_private *dev_priv)
+static void ilk_get_mem_freq(struct drm_i915_private *dev_priv)
{
u16 ddrpll, csipll;
@@ -7181,7 +7181,7 @@ void intel_init_pm(struct drm_i915_private *dev_priv)
if (IS_PINEVIEW(dev_priv))
pnv_get_mem_freq(dev_priv);
else if (IS_GEN(dev_priv, 5))
- i915_ironlake_get_mem_freq(dev_priv);
+ ilk_get_mem_freq(dev_priv);
if (intel_has_sagv(dev_priv))
skl_setup_sagv_block_time(dev_priv);
--
2.24.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 17+ messages in thread* [Intel-gfx] [PATCH 08/10] drm/i915: prefer 3-letter acronym for broadwell
2019-12-20 17:27 [Intel-gfx] [PATCH 00/10] Prefer acronym for prefixes Lucas De Marchi
` (6 preceding siblings ...)
2019-12-20 17:27 ` [Intel-gfx] [PATCH 07/10] drm/i915: prefer 3-letter acronym for ironlake Lucas De Marchi
@ 2019-12-20 17:27 ` Lucas De Marchi
2019-12-20 17:27 ` [Intel-gfx] [PATCH 09/10] drm/i915: prefer 3-letter acronym for ivybridge Lucas De Marchi
` (4 subsequent siblings)
12 siblings, 0 replies; 17+ messages in thread
From: Lucas De Marchi @ 2019-12-20 17:27 UTC (permalink / raw)
To: intel-gfx
We are currently using a mix of platform name and acronym to name the
functions. Let's prefer the acronym as it should be clear what platform
it's about and it's shorter, so it doesn't go over 80 columns in a few
cases. This converts broadwell to bdw where appropriate.
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
---
drivers/gpu/drm/i915/display/intel_fifo_underrun.c | 6 +++---
drivers/gpu/drm/i915/gt/intel_workarounds.c | 2 +-
drivers/gpu/drm/i915/gvt/handlers.c | 8 ++++----
drivers/gpu/drm/i915/i915_debugfs.c | 6 +++---
drivers/gpu/drm/i915/intel_device_info.c | 4 ++--
5 files changed, 13 insertions(+), 13 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/intel_fifo_underrun.c b/drivers/gpu/drm/i915/display/intel_fifo_underrun.c
index d6e0d0be842e..1f80f275f3f2 100644
--- a/drivers/gpu/drm/i915/display/intel_fifo_underrun.c
+++ b/drivers/gpu/drm/i915/display/intel_fifo_underrun.c
@@ -180,8 +180,8 @@ static void ivybridge_set_fifo_underrun_reporting(struct drm_device *dev,
}
}
-static void broadwell_set_fifo_underrun_reporting(struct drm_device *dev,
- enum pipe pipe, bool enable)
+static void bdw_set_fifo_underrun_reporting(struct drm_device *dev,
+ enum pipe pipe, bool enable)
{
struct drm_i915_private *dev_priv = to_i915(dev);
@@ -268,7 +268,7 @@ static bool __intel_set_cpu_fifo_underrun_reporting(struct drm_device *dev,
else if (IS_GEN(dev_priv, 7))
ivybridge_set_fifo_underrun_reporting(dev, pipe, enable, old);
else if (INTEL_GEN(dev_priv) >= 8)
- broadwell_set_fifo_underrun_reporting(dev, pipe, enable);
+ bdw_set_fifo_underrun_reporting(dev, pipe, enable);
return old;
}
diff --git a/drivers/gpu/drm/i915/gt/intel_workarounds.c b/drivers/gpu/drm/i915/gt/intel_workarounds.c
index 195ccf7db272..4e292d4bf7b9 100644
--- a/drivers/gpu/drm/i915/gt/intel_workarounds.c
+++ b/drivers/gpu/drm/i915/gt/intel_workarounds.c
@@ -254,7 +254,7 @@ static void bdw_ctx_workarounds_init(struct intel_engine_cs *engine,
/* WaDisableDopClockGating:bdw
*
- * Also see the related UCGTCL1 write in broadwell_init_clock_gating()
+ * Also see the related UCGTCL1 write in bdw_init_clock_gating()
* to disable EUTC clock gating.
*/
WA_SET_BIT_MASKED(GEN7_ROW_CHICKEN2,
diff --git a/drivers/gpu/drm/i915/gvt/handlers.c b/drivers/gpu/drm/i915/gvt/handlers.c
index 1043e6d564df..6d28d72e6c7e 100644
--- a/drivers/gpu/drm/i915/gvt/handlers.c
+++ b/drivers/gpu/drm/i915/gvt/handlers.c
@@ -2691,7 +2691,7 @@ static int init_generic_mmio_info(struct intel_gvt *gvt)
return 0;
}
-static int init_broadwell_mmio_info(struct intel_gvt *gvt)
+static int init_bdw_mmio_info(struct intel_gvt *gvt)
{
struct drm_i915_private *dev_priv = gvt->dev_priv;
int ret;
@@ -3380,20 +3380,20 @@ int intel_gvt_setup_mmio_info(struct intel_gvt *gvt)
goto err;
if (IS_BROADWELL(dev_priv)) {
- ret = init_broadwell_mmio_info(gvt);
+ ret = init_bdw_mmio_info(gvt);
if (ret)
goto err;
} else if (IS_SKYLAKE(dev_priv)
|| IS_KABYLAKE(dev_priv)
|| IS_COFFEELAKE(dev_priv)) {
- ret = init_broadwell_mmio_info(gvt);
+ ret = init_bdw_mmio_info(gvt);
if (ret)
goto err;
ret = init_skl_mmio_info(gvt);
if (ret)
goto err;
} else if (IS_BROXTON(dev_priv)) {
- ret = init_broadwell_mmio_info(gvt);
+ ret = init_bdw_mmio_info(gvt);
if (ret)
goto err;
ret = init_skl_mmio_info(gvt);
diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
index 0407229251bc..cb34e8c31511 100644
--- a/drivers/gpu/drm/i915/i915_debugfs.c
+++ b/drivers/gpu/drm/i915/i915_debugfs.c
@@ -3815,8 +3815,8 @@ static void gen9_sseu_device_status(struct drm_i915_private *dev_priv,
#undef SS_MAX
}
-static void broadwell_sseu_device_status(struct drm_i915_private *dev_priv,
- struct sseu_dev_info *sseu)
+static void bdw_sseu_device_status(struct drm_i915_private *dev_priv,
+ struct sseu_dev_info *sseu)
{
const struct intel_runtime_info *info = RUNTIME_INFO(dev_priv);
u32 slice_info = I915_READ(GEN8_GT_SLICE_INFO);
@@ -3901,7 +3901,7 @@ static int i915_sseu_status(struct seq_file *m, void *unused)
if (IS_CHERRYVIEW(dev_priv))
cherryview_sseu_device_status(dev_priv, &sseu);
else if (IS_BROADWELL(dev_priv))
- broadwell_sseu_device_status(dev_priv, &sseu);
+ bdw_sseu_device_status(dev_priv, &sseu);
else if (IS_GEN(dev_priv, 9))
gen9_sseu_device_status(dev_priv, &sseu);
else if (INTEL_GEN(dev_priv) >= 10)
diff --git a/drivers/gpu/drm/i915/intel_device_info.c b/drivers/gpu/drm/i915/intel_device_info.c
index ca7a42e1d769..d87c31444fa8 100644
--- a/drivers/gpu/drm/i915/intel_device_info.c
+++ b/drivers/gpu/drm/i915/intel_device_info.c
@@ -519,7 +519,7 @@ static void gen9_sseu_info_init(struct drm_i915_private *dev_priv)
}
}
-static void broadwell_sseu_info_init(struct drm_i915_private *dev_priv)
+static void bdw_sseu_info_init(struct drm_i915_private *dev_priv)
{
struct sseu_dev_info *sseu = &RUNTIME_INFO(dev_priv)->sseu;
int s, ss;
@@ -1025,7 +1025,7 @@ void intel_device_info_runtime_init(struct drm_i915_private *dev_priv)
else if (IS_CHERRYVIEW(dev_priv))
cherryview_sseu_info_init(dev_priv);
else if (IS_BROADWELL(dev_priv))
- broadwell_sseu_info_init(dev_priv);
+ bdw_sseu_info_init(dev_priv);
else if (IS_GEN(dev_priv, 9))
gen9_sseu_info_init(dev_priv);
else if (IS_GEN(dev_priv, 10))
--
2.24.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 17+ messages in thread* [Intel-gfx] [PATCH 09/10] drm/i915: prefer 3-letter acronym for ivybridge
2019-12-20 17:27 [Intel-gfx] [PATCH 00/10] Prefer acronym for prefixes Lucas De Marchi
` (7 preceding siblings ...)
2019-12-20 17:27 ` [Intel-gfx] [PATCH 08/10] drm/i915: prefer 3-letter acronym for broadwell Lucas De Marchi
@ 2019-12-20 17:27 ` Lucas De Marchi
2019-12-20 17:27 ` [Intel-gfx] [PATCH 10/10] drm/i915: prefer 3-letter acronym for tigerlake Lucas De Marchi
` (3 subsequent siblings)
12 siblings, 0 replies; 17+ messages in thread
From: Lucas De Marchi @ 2019-12-20 17:27 UTC (permalink / raw)
To: intel-gfx
We are currently using a mix of platform name and acronym to name the
functions. Let's prefer the acronym as it should be clear what platform
it's about and it's shorter, so it doesn't go over 80 columns in a few
cases. This converts ivybridge to ivb where appropriate.
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
---
drivers/gpu/drm/i915/display/intel_display.c | 4 ++--
drivers/gpu/drm/i915/display/intel_fifo_underrun.c | 12 ++++++------
drivers/gpu/drm/i915/i915_irq.c | 6 +++---
3 files changed, 11 insertions(+), 11 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c
index 5c14f1e5a871..9569187979fe 100644
--- a/drivers/gpu/drm/i915/display/intel_display.c
+++ b/drivers/gpu/drm/i915/display/intel_display.c
@@ -5292,7 +5292,7 @@ static void cpt_set_fdi_bc_bifurcation(struct drm_i915_private *dev_priv, bool e
POSTING_READ(SOUTH_CHICKEN1);
}
-static void ivybridge_update_fdi_bc_bifurcation(const struct intel_crtc_state *crtc_state)
+static void ivb_update_fdi_bc_bifurcation(const struct intel_crtc_state *crtc_state)
{
struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
@@ -5365,7 +5365,7 @@ static void ilk_pch_enable(const struct intel_atomic_state *state,
assert_pch_transcoder_disabled(dev_priv, pipe);
if (IS_IVYBRIDGE(dev_priv))
- ivybridge_update_fdi_bc_bifurcation(crtc_state);
+ ivb_update_fdi_bc_bifurcation(crtc_state);
/* Write the TU size bits before fdi link training, so that error
* detection works. */
diff --git a/drivers/gpu/drm/i915/display/intel_fifo_underrun.c b/drivers/gpu/drm/i915/display/intel_fifo_underrun.c
index 1f80f275f3f2..6c83b350525d 100644
--- a/drivers/gpu/drm/i915/display/intel_fifo_underrun.c
+++ b/drivers/gpu/drm/i915/display/intel_fifo_underrun.c
@@ -139,7 +139,7 @@ static void ilk_set_fifo_underrun_reporting(struct drm_device *dev,
ilk_disable_display_irq(dev_priv, bit);
}
-static void ivybridge_check_fifo_underruns(struct intel_crtc *crtc)
+static void ivb_check_fifo_underruns(struct intel_crtc *crtc)
{
struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
enum pipe pipe = crtc->pipe;
@@ -157,9 +157,9 @@ static void ivybridge_check_fifo_underruns(struct intel_crtc *crtc)
DRM_ERROR("fifo underrun on pipe %c\n", pipe_name(pipe));
}
-static void ivybridge_set_fifo_underrun_reporting(struct drm_device *dev,
- enum pipe pipe,
- bool enable, bool old)
+static void ivb_set_fifo_underrun_reporting(struct drm_device *dev,
+ enum pipe pipe, bool enable,
+ bool old)
{
struct drm_i915_private *dev_priv = to_i915(dev);
if (enable) {
@@ -266,7 +266,7 @@ static bool __intel_set_cpu_fifo_underrun_reporting(struct drm_device *dev,
else if (IS_GEN_RANGE(dev_priv, 5, 6))
ilk_set_fifo_underrun_reporting(dev, pipe, enable);
else if (IS_GEN(dev_priv, 7))
- ivybridge_set_fifo_underrun_reporting(dev, pipe, enable, old);
+ ivb_set_fifo_underrun_reporting(dev, pipe, enable, old);
else if (INTEL_GEN(dev_priv) >= 8)
bdw_set_fifo_underrun_reporting(dev, pipe, enable);
@@ -427,7 +427,7 @@ void intel_check_cpu_fifo_underruns(struct drm_i915_private *dev_priv)
if (HAS_GMCH(dev_priv))
i9xx_check_fifo_underruns(crtc);
else if (IS_GEN(dev_priv, 7))
- ivybridge_check_fifo_underruns(crtc);
+ ivb_check_fifo_underruns(crtc);
}
spin_unlock_irq(&dev_priv->irq_lock);
diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c
index 2d6324d2922a..afc6aad9bf8c 100644
--- a/drivers/gpu/drm/i915/i915_irq.c
+++ b/drivers/gpu/drm/i915/i915_irq.c
@@ -893,7 +893,7 @@ int intel_get_crtc_scanline(struct intel_crtc *crtc)
}
/**
- * ivybridge_parity_work - Workqueue called when a parity error interrupt
+ * ivb_parity_work - Workqueue called when a parity error interrupt
* occurred.
* @work: workqueue struct
*
@@ -901,7 +901,7 @@ int intel_get_crtc_scanline(struct intel_crtc *crtc)
* this event, userspace should try to remap the bad rows since statistically
* it is likely the same row is more likely to go bad again.
*/
-static void ivybridge_parity_work(struct work_struct *work)
+static void ivb_parity_work(struct work_struct *work)
{
struct drm_i915_private *dev_priv =
container_of(work, typeof(*dev_priv), l3_parity.error_work);
@@ -3899,7 +3899,7 @@ void intel_irq_init(struct drm_i915_private *dev_priv)
intel_hpd_init_work(dev_priv);
- INIT_WORK(&dev_priv->l3_parity.error_work, ivybridge_parity_work);
+ INIT_WORK(&dev_priv->l3_parity.error_work, ivb_parity_work);
for (i = 0; i < MAX_L3_SLICES; ++i)
dev_priv->l3_parity.remap_info[i] = NULL;
--
2.24.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 17+ messages in thread* [Intel-gfx] [PATCH 10/10] drm/i915: prefer 3-letter acronym for tigerlake
2019-12-20 17:27 [Intel-gfx] [PATCH 00/10] Prefer acronym for prefixes Lucas De Marchi
` (8 preceding siblings ...)
2019-12-20 17:27 ` [Intel-gfx] [PATCH 09/10] drm/i915: prefer 3-letter acronym for ivybridge Lucas De Marchi
@ 2019-12-20 17:27 ` Lucas De Marchi
2019-12-21 0:07 ` [Intel-gfx] ✓ Fi.CI.BAT: success for Prefer acronym for prefixes Patchwork
` (2 subsequent siblings)
12 siblings, 0 replies; 17+ messages in thread
From: Lucas De Marchi @ 2019-12-20 17:27 UTC (permalink / raw)
To: intel-gfx
We are currently using a mix of platform name and acronym to name the
functions. Let's prefer the acronym as it should be clear what platform
it's about and it's shorter, so it doesn't go over 80 columns in a few
cases. This converts tigerlake to tgl where appropriate.
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
---
drivers/gpu/drm/i915/gt/intel_mocs.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/i915/gt/intel_mocs.c b/drivers/gpu/drm/i915/gt/intel_mocs.c
index 95f1bc45953b..eeef90b55c64 100644
--- a/drivers/gpu/drm/i915/gt/intel_mocs.c
+++ b/drivers/gpu/drm/i915/gt/intel_mocs.c
@@ -233,7 +233,7 @@ static const struct drm_i915_mocs_entry broxton_mocs_table[] = {
LE_3_WB | LE_TC_1_LLC | LE_LRUM(3), \
L3_1_UC)
-static const struct drm_i915_mocs_entry tigerlake_mocs_table[] = {
+static const struct drm_i915_mocs_entry tgl_mocs_table[] = {
/* Base - Error (Reserved for Non-Use) */
MOCS_ENTRY(0, 0x0, 0x0),
/* Base - Reserved */
@@ -284,8 +284,8 @@ static bool get_mocs_settings(const struct drm_i915_private *i915,
struct drm_i915_mocs_table *table)
{
if (INTEL_GEN(i915) >= 12) {
- table->size = ARRAY_SIZE(tigerlake_mocs_table);
- table->table = tigerlake_mocs_table;
+ table->size = ARRAY_SIZE(tgl_mocs_table);
+ table->table = tgl_mocs_table;
table->n_entries = GEN11_NUM_MOCS_ENTRIES;
} else if (IS_GEN(i915, 11)) {
table->size = ARRAY_SIZE(icl_mocs_table);
--
2.24.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 17+ messages in thread* [Intel-gfx] ✓ Fi.CI.BAT: success for Prefer acronym for prefixes
2019-12-20 17:27 [Intel-gfx] [PATCH 00/10] Prefer acronym for prefixes Lucas De Marchi
` (9 preceding siblings ...)
2019-12-20 17:27 ` [Intel-gfx] [PATCH 10/10] drm/i915: prefer 3-letter acronym for tigerlake Lucas De Marchi
@ 2019-12-21 0:07 ` Patchwork
2019-12-22 20:35 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork
2019-12-23 11:05 ` [Intel-gfx] [PATCH 00/10] " Jani Nikula
12 siblings, 0 replies; 17+ messages in thread
From: Patchwork @ 2019-12-21 0:07 UTC (permalink / raw)
To: Lucas De Marchi; +Cc: intel-gfx
== Series Details ==
Series: Prefer acronym for prefixes
URL : https://patchwork.freedesktop.org/series/71224/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_7615 -> Patchwork_15863
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/index.html
Known issues
------------
Here are the changes found in Patchwork_15863 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@i915_pm_rpm@module-reload:
- fi-icl-u3: [PASS][1] -> [DMESG-WARN][2] ([i915#109])
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/fi-icl-u3/igt@i915_pm_rpm@module-reload.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/fi-icl-u3/igt@i915_pm_rpm@module-reload.html
#### Possible fixes ####
* igt@i915_selftest@live_gem_contexts:
- fi-hsw-4770: [DMESG-FAIL][3] ([i915#722]) -> [PASS][4]
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/fi-hsw-4770/igt@i915_selftest@live_gem_contexts.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/fi-hsw-4770/igt@i915_selftest@live_gem_contexts.html
- fi-hsw-peppy: [INCOMPLETE][5] ([i915#694]) -> [PASS][6]
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/fi-hsw-peppy/igt@i915_selftest@live_gem_contexts.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/fi-hsw-peppy/igt@i915_selftest@live_gem_contexts.html
#### Warnings ####
* igt@i915_selftest@live_blt:
- fi-hsw-4770: [DMESG-FAIL][7] ([i915#725]) -> [DMESG-FAIL][8] ([i915#553] / [i915#725])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/fi-hsw-4770/igt@i915_selftest@live_blt.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/fi-hsw-4770/igt@i915_selftest@live_blt.html
* igt@kms_chamelium@hdmi-hpd-fast:
- fi-kbl-7500u: [FAIL][9] ([fdo#111096] / [i915#323]) -> [FAIL][10] ([fdo#111407])
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html
* igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy:
- fi-kbl-x1275: [DMESG-WARN][11] ([i915#62] / [i915#92] / [i915#95]) -> [DMESG-WARN][12] ([i915#62] / [i915#92]) +6 similar issues
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/fi-kbl-x1275/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/fi-kbl-x1275/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html
* igt@kms_flip@basic-flip-vs-modeset:
- fi-kbl-x1275: [DMESG-WARN][13] ([i915#62] / [i915#92]) -> [DMESG-WARN][14] ([i915#62] / [i915#92] / [i915#95]) +1 similar issue
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/fi-kbl-x1275/igt@kms_flip@basic-flip-vs-modeset.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/fi-kbl-x1275/igt@kms_flip@basic-flip-vs-modeset.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#111096]: https://bugs.freedesktop.org/show_bug.cgi?id=111096
[fdo#111407]: https://bugs.freedesktop.org/show_bug.cgi?id=111407
[fdo#111736]: https://bugs.freedesktop.org/show_bug.cgi?id=111736
[i915#109]: https://gitlab.freedesktop.org/drm/intel/issues/109
[i915#323]: https://gitlab.freedesktop.org/drm/intel/issues/323
[i915#553]: https://gitlab.freedesktop.org/drm/intel/issues/553
[i915#62]: https://gitlab.freedesktop.org/drm/intel/issues/62
[i915#694]: https://gitlab.freedesktop.org/drm/intel/issues/694
[i915#722]: https://gitlab.freedesktop.org/drm/intel/issues/722
[i915#725]: https://gitlab.freedesktop.org/drm/intel/issues/725
[i915#92]: https://gitlab.freedesktop.org/drm/intel/issues/92
[i915#95]: https://gitlab.freedesktop.org/drm/intel/issues/95
Participating hosts (39 -> 40)
------------------------------
Additional (8): fi-bsw-n3050 fi-bwr-2160 fi-gdg-551 fi-blb-e6850 fi-byt-n2820 fi-bsw-nick fi-skl-6600u fi-kbl-r
Missing (7): fi-ilk-m540 fi-bdw-gvtdvm fi-byt-squawks fi-ctg-p8600 fi-skl-lmem fi-byt-clapper fi-bdw-samus
Build changes
-------------
* CI: CI-20190529 -> None
* Linux: CI_DRM_7615 -> Patchwork_15863
CI-20190529: 20190529
CI_DRM_7615: 2a3a29d061687518b6c7fa4605518fdaebac82cf @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5351: e7fdcef72d1d6b3bb9f3003bbc37571959e6e8bb @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_15863: c9c5e8aad5a4a0b30c5b913525dfeae7e7e1e8cd @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
c9c5e8aad5a4 drm/i915: prefer 3-letter acronym for tigerlake
bfcab138f5f4 drm/i915: prefer 3-letter acronym for ivybridge
e18792a17250 drm/i915: prefer 3-letter acronym for broadwell
fc31bb67431d drm/i915: prefer 3-letter acronym for ironlake
28bfd56cbaa9 drm/i915: prefer 3-letter acronym for icelake
c1d0af728a4d drm/i915: prefer 3-letter acronym for cannonlake
59bd91da8368 drm/i915: prefer 3-letter acronym for skylake
5d4e2de63a83 drm/i915: prefer 3-letter acronym for haswell
cd6a3dc4691b drm/i915: prefer 3-letter acronym for pineview
6507ba4cfb86 drm/i915: simplify prefixes on device_info
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/index.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 17+ messages in thread* [Intel-gfx] ✓ Fi.CI.IGT: success for Prefer acronym for prefixes
2019-12-20 17:27 [Intel-gfx] [PATCH 00/10] Prefer acronym for prefixes Lucas De Marchi
` (10 preceding siblings ...)
2019-12-21 0:07 ` [Intel-gfx] ✓ Fi.CI.BAT: success for Prefer acronym for prefixes Patchwork
@ 2019-12-22 20:35 ` Patchwork
2019-12-23 11:05 ` [Intel-gfx] [PATCH 00/10] " Jani Nikula
12 siblings, 0 replies; 17+ messages in thread
From: Patchwork @ 2019-12-22 20:35 UTC (permalink / raw)
To: Lucas De Marchi; +Cc: intel-gfx
== Series Details ==
Series: Prefer acronym for prefixes
URL : https://patchwork.freedesktop.org/series/71224/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_7615_full -> Patchwork_15863_full
====================================================
Summary
-------
**SUCCESS**
No regressions found.
New tests
---------
New tests have been introduced between CI_DRM_7615_full and Patchwork_15863_full:
### New Piglit tests (3) ###
* spec@arb_vertex_attrib_64bit@execution@vs_in@vs-input-double_double-position-float_mat4x2_array3:
- Statuses : 1 fail(s)
- Exec time: [0.12] s
* spec@glsl-4.20@execution@vs_in@vs-input-position-double_dmat2_array3-double_dvec2:
- Statuses : 1 fail(s)
- Exec time: [0.13] s
* spec@glsl-4.20@execution@vs_in@vs-input-position-uint_uvec2-double_dmat2x4:
- Statuses : 1 fail(s)
- Exec time: [0.16] s
Known issues
------------
Here are the changes found in Patchwork_15863_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_exec_reuse@single:
- shard-tglb: [PASS][1] -> [INCOMPLETE][2] ([i915#435])
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-tglb2/igt@gem_exec_reuse@single.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-tglb9/igt@gem_exec_reuse@single.html
* igt@gem_exec_schedule@preempt-queue-render:
- shard-tglb: [PASS][3] -> [INCOMPLETE][4] ([fdo#111606] / [fdo#111677])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-tglb4/igt@gem_exec_schedule@preempt-queue-render.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-tglb6/igt@gem_exec_schedule@preempt-queue-render.html
* igt@gem_ppgtt@flink-and-close-vma-leak:
- shard-kbl: [PASS][5] -> [FAIL][6] ([i915#644])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-kbl3/igt@gem_ppgtt@flink-and-close-vma-leak.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-kbl1/igt@gem_ppgtt@flink-and-close-vma-leak.html
* igt@gem_softpin@noreloc-s3:
- shard-skl: [PASS][7] -> [INCOMPLETE][8] ([i915#69])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-skl2/igt@gem_softpin@noreloc-s3.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-skl2/igt@gem_softpin@noreloc-s3.html
* igt@gem_sync@basic-all:
- shard-tglb: [PASS][9] -> [INCOMPLETE][10] ([i915#470] / [i915#472])
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-tglb3/igt@gem_sync@basic-all.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-tglb6/igt@gem_sync@basic-all.html
* igt@gem_workarounds@suspend-resume-fd:
- shard-kbl: [PASS][11] -> [DMESG-WARN][12] ([i915#180]) +8 similar issues
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-kbl7/igt@gem_workarounds@suspend-resume-fd.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-kbl7/igt@gem_workarounds@suspend-resume-fd.html
* igt@i915_pm_rpm@system-suspend-modeset:
- shard-skl: [PASS][13] -> [INCOMPLETE][14] ([i915#151] / [i915#69])
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-skl8/igt@i915_pm_rpm@system-suspend-modeset.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-skl1/igt@i915_pm_rpm@system-suspend-modeset.html
* igt@kms_color@pipe-b-ctm-0-5:
- shard-skl: [PASS][15] -> [DMESG-WARN][16] ([i915#109])
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-skl8/igt@kms_color@pipe-b-ctm-0-5.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-skl9/igt@kms_color@pipe-b-ctm-0-5.html
* igt@kms_cursor_crc@pipe-c-cursor-256x85-random:
- shard-hsw: [PASS][17] -> [DMESG-WARN][18] ([IGT#6])
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-hsw8/igt@kms_cursor_crc@pipe-c-cursor-256x85-random.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-hsw6/igt@kms_cursor_crc@pipe-c-cursor-256x85-random.html
* igt@kms_cursor_crc@pipe-c-cursor-256x85-sliding:
- shard-skl: [PASS][19] -> [FAIL][20] ([i915#54]) +2 similar issues
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-skl6/igt@kms_cursor_crc@pipe-c-cursor-256x85-sliding.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-skl4/igt@kms_cursor_crc@pipe-c-cursor-256x85-sliding.html
* igt@kms_flip@2x-flip-vs-expired-vblank-interruptible:
- shard-glk: [PASS][21] -> [FAIL][22] ([i915#79])
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-glk2/igt@kms_flip@2x-flip-vs-expired-vblank-interruptible.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-glk3/igt@kms_flip@2x-flip-vs-expired-vblank-interruptible.html
* igt@kms_flip@2x-plain-flip-fb-recreate-interruptible:
- shard-glk: [PASS][23] -> [FAIL][24] ([i915#34])
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-glk2/igt@kms_flip@2x-plain-flip-fb-recreate-interruptible.html
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-glk3/igt@kms_flip@2x-plain-flip-fb-recreate-interruptible.html
* igt@kms_frontbuffer_tracking@psr-1p-primscrn-cur-indfb-draw-mmap-wc:
- shard-skl: [PASS][25] -> [INCOMPLETE][26] ([i915#123] / [i915#667])
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-skl8/igt@kms_frontbuffer_tracking@psr-1p-primscrn-cur-indfb-draw-mmap-wc.html
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-skl3/igt@kms_frontbuffer_tracking@psr-1p-primscrn-cur-indfb-draw-mmap-wc.html
* igt@kms_frontbuffer_tracking@psr-1p-primscrn-cur-indfb-draw-pwrite:
- shard-tglb: [PASS][27] -> [INCOMPLETE][28] ([i915#667])
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-tglb5/igt@kms_frontbuffer_tracking@psr-1p-primscrn-cur-indfb-draw-pwrite.html
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-tglb9/igt@kms_frontbuffer_tracking@psr-1p-primscrn-cur-indfb-draw-pwrite.html
* igt@kms_plane@plane-panning-bottom-right-suspend-pipe-b-planes:
- shard-apl: [PASS][29] -> [DMESG-WARN][30] ([i915#180]) +1 similar issue
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-apl2/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-b-planes.html
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-apl4/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-b-planes.html
* igt@kms_plane_alpha_blend@pipe-c-coverage-7efc:
- shard-skl: [PASS][31] -> [FAIL][32] ([fdo#108145] / [i915#265])
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-skl10/igt@kms_plane_alpha_blend@pipe-c-coverage-7efc.html
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-skl2/igt@kms_plane_alpha_blend@pipe-c-coverage-7efc.html
#### Possible fixes ####
* igt@gem_ctx_persistence@processes:
- shard-tglb: [FAIL][33] ([i915#570]) -> [PASS][34]
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-tglb1/igt@gem_ctx_persistence@processes.html
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-tglb3/igt@gem_ctx_persistence@processes.html
* igt@gem_ctx_shared@q-smoketest-bsd:
- shard-tglb: [INCOMPLETE][35] ([i915#461]) -> [PASS][36]
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-tglb3/igt@gem_ctx_shared@q-smoketest-bsd.html
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-tglb2/igt@gem_ctx_shared@q-smoketest-bsd.html
* igt@gem_ctx_shared@q-smoketest-bsd1:
- shard-tglb: [INCOMPLETE][37] ([fdo#111735]) -> [PASS][38]
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-tglb6/igt@gem_ctx_shared@q-smoketest-bsd1.html
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-tglb5/igt@gem_ctx_shared@q-smoketest-bsd1.html
* igt@gem_eio@kms:
- shard-snb: [INCOMPLETE][39] ([i915#82]) -> [PASS][40]
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-snb1/igt@gem_eio@kms.html
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-snb4/igt@gem_eio@kms.html
* igt@gem_exec_nop@basic-sequential:
- shard-tglb: [INCOMPLETE][41] ([i915#435]) -> [PASS][42]
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-tglb7/igt@gem_exec_nop@basic-sequential.html
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-tglb2/igt@gem_exec_nop@basic-sequential.html
* igt@gem_exec_parallel@basic:
- shard-tglb: [INCOMPLETE][43] ([i915#476]) -> [PASS][44]
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-tglb9/igt@gem_exec_parallel@basic.html
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-tglb1/igt@gem_exec_parallel@basic.html
* igt@gem_exec_parallel@vcs0:
- shard-tglb: [INCOMPLETE][45] ([fdo#111593]) -> [PASS][46]
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-tglb1/igt@gem_exec_parallel@vcs0.html
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-tglb2/igt@gem_exec_parallel@vcs0.html
* igt@gem_persistent_relocs@forked-thrashing:
- shard-tglb: [INCOMPLETE][47] ([i915#530]) -> [PASS][48]
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-tglb6/igt@gem_persistent_relocs@forked-thrashing.html
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-tglb5/igt@gem_persistent_relocs@forked-thrashing.html
* igt@gem_pipe_control_store_loop@reused-buffer:
- shard-skl: [TIMEOUT][49] ([fdo#111732]) -> [PASS][50]
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-skl3/igt@gem_pipe_control_store_loop@reused-buffer.html
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-skl3/igt@gem_pipe_control_store_loop@reused-buffer.html
* igt@gem_ppgtt@flink-and-close-vma-leak:
- shard-glk: [FAIL][51] ([i915#644]) -> [PASS][52]
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-glk1/igt@gem_ppgtt@flink-and-close-vma-leak.html
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-glk4/igt@gem_ppgtt@flink-and-close-vma-leak.html
* igt@i915_suspend@fence-restore-tiled2untiled:
- shard-tglb: [INCOMPLETE][53] ([i915#456] / [i915#460]) -> [PASS][54] +1 similar issue
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-tglb5/igt@i915_suspend@fence-restore-tiled2untiled.html
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-tglb1/igt@i915_suspend@fence-restore-tiled2untiled.html
* igt@kms_color@pipe-a-ctm-0-5:
- shard-skl: [DMESG-WARN][55] ([i915#109]) -> [PASS][56]
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-skl8/igt@kms_color@pipe-a-ctm-0-5.html
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-skl8/igt@kms_color@pipe-a-ctm-0-5.html
* igt@kms_cursor_crc@pipe-a-cursor-128x42-sliding:
- shard-skl: [FAIL][57] ([i915#54]) -> [PASS][58] +1 similar issue
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-skl2/igt@kms_cursor_crc@pipe-a-cursor-128x42-sliding.html
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-skl8/igt@kms_cursor_crc@pipe-a-cursor-128x42-sliding.html
* igt@kms_cursor_crc@pipe-d-cursor-suspend:
- shard-tglb: [INCOMPLETE][59] ([i915#460]) -> [PASS][60]
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-tglb1/igt@kms_cursor_crc@pipe-d-cursor-suspend.html
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-tglb6/igt@kms_cursor_crc@pipe-d-cursor-suspend.html
* igt@kms_draw_crc@draw-method-xrgb8888-blt-xtiled:
- shard-snb: [SKIP][61] ([fdo#109271]) -> [PASS][62]
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-snb6/igt@kms_draw_crc@draw-method-xrgb8888-blt-xtiled.html
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-snb1/igt@kms_draw_crc@draw-method-xrgb8888-blt-xtiled.html
* igt@kms_draw_crc@draw-method-xrgb8888-mmap-gtt-xtiled:
- shard-skl: [FAIL][63] ([i915#52] / [i915#54]) -> [PASS][64]
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-skl10/igt@kms_draw_crc@draw-method-xrgb8888-mmap-gtt-xtiled.html
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-skl2/igt@kms_draw_crc@draw-method-xrgb8888-mmap-gtt-xtiled.html
* igt@kms_flip@flip-vs-suspend-interruptible:
- shard-apl: [DMESG-WARN][65] ([i915#180]) -> [PASS][66] +1 similar issue
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-apl1/igt@kms_flip@flip-vs-suspend-interruptible.html
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-apl7/igt@kms_flip@flip-vs-suspend-interruptible.html
* igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-pwrite:
- shard-tglb: [FAIL][67] ([i915#49]) -> [PASS][68] +1 similar issue
[67]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-tglb1/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-pwrite.html
[68]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-tglb4/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-pwrite.html
* igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-spr-indfb-move:
- shard-tglb: [DMESG-WARN][69] ([i915#766]) -> [PASS][70]
[69]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-tglb5/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-spr-indfb-move.html
[70]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-tglb2/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-spr-indfb-move.html
* igt@kms_frontbuffer_tracking@psr-1p-primscrn-cur-indfb-move:
- shard-tglb: [INCOMPLETE][71] ([i915#667]) -> [PASS][72]
[71]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-tglb7/igt@kms_frontbuffer_tracking@psr-1p-primscrn-cur-indfb-move.html
[72]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-tglb7/igt@kms_frontbuffer_tracking@psr-1p-primscrn-cur-indfb-move.html
* igt@kms_psr@psr2_suspend:
- shard-tglb: [DMESG-WARN][73] ([i915#402]) -> [PASS][74]
[73]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-tglb7/igt@kms_psr@psr2_suspend.html
[74]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-tglb4/igt@kms_psr@psr2_suspend.html
* igt@kms_vblank@pipe-a-ts-continuation-suspend:
- shard-kbl: [DMESG-WARN][75] ([i915#180]) -> [PASS][76]
[75]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-kbl7/igt@kms_vblank@pipe-a-ts-continuation-suspend.html
[76]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-kbl2/igt@kms_vblank@pipe-a-ts-continuation-suspend.html
- shard-skl: [INCOMPLETE][77] ([i915#69]) -> [PASS][78]
[77]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-skl2/igt@kms_vblank@pipe-a-ts-continuation-suspend.html
[78]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-skl5/igt@kms_vblank@pipe-a-ts-continuation-suspend.html
#### Warnings ####
* igt@i915_pm_dc@dc6-psr:
- shard-tglb: [FAIL][79] ([i915#454]) -> [SKIP][80] ([i915#468])
[79]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7615/shard-tglb4/igt@i915_pm_dc@dc6-psr.html
[80]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/shard-tglb6/igt@i915_pm_dc@dc6-psr.html
[IGT#6]: https://gitlab.freedesktop.org/drm/igt-gpu-tools/issues/6
[fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#111593]: https://bugs.freedesktop.org/show_bug.cgi?id=111593
[fdo#111606]: https://bugs.freedesktop.org/show_bug.cgi?id=111606
[fdo#111677]: https://bugs.freedesktop.org/show_bug.cgi?id=111677
[fdo#111732]: https://bugs.freedesktop.org/show_bug.cgi?id=111732
[fdo#111735]: https://bugs.freedesktop.org/show_bug.cgi?id=111735
[i915#109]: https://gitlab.freedesktop.org/drm/intel/issues/109
[i915#123]: https://gitlab.freedesktop.org/drm/intel/issues/123
[i915#151]: https://gitlab.freedesktop.org/drm/intel/issues/151
[i915#180]: https://gitlab.freedesktop.org/drm/intel/issues/180
[i915#265]: https://gitlab.freedesktop.org/drm/intel/issues/265
[i915#34]: https://gitlab.freedesktop.org/drm/intel/issues/34
[i915#402]: https://gitlab.freedesktop.org/drm/intel/issues/402
[i915#435]: https://gitlab.freedesktop.org/drm/intel/issues/435
[i915#454]: https://gitlab.freedesktop.org/drm/intel/issues/454
[i915#456]: https://gitlab.freedesktop.org/drm/intel/issues/456
[i915#460]: https://gitlab.freedesktop.org/drm/intel/issues/460
[i915#461]: https://gitlab.freedesktop.org/drm/intel/issues/461
[i915#468]: https://gitlab.freedesktop.org/drm/intel/issues/468
[i915#470]: https://gitlab.freedesktop.org/drm/intel/issues/470
[i915#472]: https://gitlab.freedesktop.org/drm/intel/issues/472
[i915#476]: https://gitlab.freedesktop.org/drm/intel/issues/476
[i915#49]: https://gitlab.freedesktop.org/drm/intel/issues/49
[i915#52]: https://gitlab.freedesktop.org/drm/intel/issues/52
[i915#530]: https://gitlab.freedesktop.org/drm/intel/issues/530
[i915#54]: https://gitlab.freedesktop.org/drm/intel/issues/54
[i915#570]: https://gitlab.freedesktop.org/drm/intel/issues/570
[i915#644]: https://gitlab.freedesktop.org/drm/intel/issues/644
[i915#667]: https://gitlab.freedesktop.org/drm/intel/issues/667
[i915#69]: https://gitlab.freedesktop.org/drm/intel/issues/69
[i915#766]: https://gitlab.freedesktop.org/drm/intel/issues/766
[i915#79]: https://gitlab.freedesktop.org/drm/intel/issues/79
[i915#82]: https://gitlab.freedesktop.org/drm/intel/issues/82
Participating hosts (9 -> 10)
------------------------------
Additional (1): pig-hsw-4770r
Build changes
-------------
* CI: CI-20190529 -> None
* Linux: CI_DRM_7615 -> Patchwork_15863
CI-20190529: 20190529
CI_DRM_7615: 2a3a29d061687518b6c7fa4605518fdaebac82cf @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5351: e7fdcef72d1d6b3bb9f3003bbc37571959e6e8bb @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_15863: c9c5e8aad5a4a0b30c5b913525dfeae7e7e1e8cd @ git://anongit.freedesktop.org/gfx-ci/linux
piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15863/index.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: [Intel-gfx] [PATCH 00/10] Prefer acronym for prefixes
2019-12-20 17:27 [Intel-gfx] [PATCH 00/10] Prefer acronym for prefixes Lucas De Marchi
` (11 preceding siblings ...)
2019-12-22 20:35 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork
@ 2019-12-23 11:05 ` Jani Nikula
12 siblings, 0 replies; 17+ messages in thread
From: Jani Nikula @ 2019-12-23 11:05 UTC (permalink / raw)
To: Lucas De Marchi, intel-gfx
On Fri, 20 Dec 2019, Lucas De Marchi <lucas.demarchi@intel.com> wrote:
> This is "v2" https://patchwork.freedesktop.org/series/71079/, but
> extended to the rest of the driver, not only display, and trying to
> cover all platforms.
Acked-by: Jani Nikula <jani.nikula@intel.com>
on the series, but I think still needs detailed review, I only had a
cursory glance.
BR,
Jani.
--
Jani Nikula, Intel Open Source Graphics Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 17+ messages in thread