diff options
author | Tvrtko Ursulin <tvrtko.ursulin@intel.com> | 2016-10-13 06:03:02 -0400 |
---|---|---|
committer | Tvrtko Ursulin <tvrtko.ursulin@intel.com> | 2016-10-14 07:23:19 -0400 |
commit | 0853723b8956acc9690ace7ab6ea1f50bb997b52 (patch) | |
tree | 2b8d4afc56e49a101f21efa3bb17c87f94df089c /drivers/gpu/drm | |
parent | 772c2a519cec26546b070aed71fc1ff0c5f31459 (diff) |
drm/i915: Make IS_KABYLAKE only take dev_priv
Saves 1320 bytes of .rodata strings.
v2: Add parantheses around dev_priv. (Ville Syrjala)
Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Reviewed-by: David Weinehall <david.weinehall@linux.intel.com>
Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Acked-by: Jani Nikula <jani.nikula@linux.intel.com>
Acked-by: Chris Wilson <chris@chris-wilson.co.uk>
Acked-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Diffstat (limited to 'drivers/gpu/drm')
-rw-r--r-- | drivers/gpu/drm/i915/i915_drv.c | 10 | ||||
-rw-r--r-- | drivers/gpu/drm/i915/i915_drv.h | 6 | ||||
-rw-r--r-- | drivers/gpu/drm/i915/i915_gem_stolen.c | 2 | ||||
-rw-r--r-- | drivers/gpu/drm/i915/intel_ddi.c | 16 | ||||
-rw-r--r-- | drivers/gpu/drm/i915/intel_display.c | 10 | ||||
-rw-r--r-- | drivers/gpu/drm/i915/intel_dpll_mgr.c | 2 | ||||
-rw-r--r-- | drivers/gpu/drm/i915/intel_guc_loader.c | 2 | ||||
-rw-r--r-- | drivers/gpu/drm/i915/intel_runtime_pm.c | 2 |
8 files changed, 25 insertions, 25 deletions
diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c index ef7bed81c6a0..5dd56d9ea905 100644 --- a/drivers/gpu/drm/i915/i915_drv.c +++ b/drivers/gpu/drm/i915/i915_drv.c | |||
@@ -203,17 +203,17 @@ static void intel_detect_pch(struct drm_device *dev) | |||
203 | } else if (id == INTEL_PCH_SPT_DEVICE_ID_TYPE) { | 203 | } else if (id == INTEL_PCH_SPT_DEVICE_ID_TYPE) { |
204 | dev_priv->pch_type = PCH_SPT; | 204 | dev_priv->pch_type = PCH_SPT; |
205 | DRM_DEBUG_KMS("Found SunrisePoint PCH\n"); | 205 | DRM_DEBUG_KMS("Found SunrisePoint PCH\n"); |
206 | WARN_ON(!IS_SKYLAKE(dev) && | 206 | WARN_ON(!IS_SKYLAKE(dev_priv) && |
207 | !IS_KABYLAKE(dev)); | 207 | !IS_KABYLAKE(dev_priv)); |
208 | } else if (id == INTEL_PCH_SPT_LP_DEVICE_ID_TYPE) { | 208 | } else if (id == INTEL_PCH_SPT_LP_DEVICE_ID_TYPE) { |
209 | dev_priv->pch_type = PCH_SPT; | 209 | dev_priv->pch_type = PCH_SPT; |
210 | DRM_DEBUG_KMS("Found SunrisePoint LP PCH\n"); | 210 | DRM_DEBUG_KMS("Found SunrisePoint LP PCH\n"); |
211 | WARN_ON(!IS_SKYLAKE(dev) && | 211 | WARN_ON(!IS_SKYLAKE(dev_priv) && |
212 | !IS_KABYLAKE(dev)); | 212 | !IS_KABYLAKE(dev_priv)); |
213 | } else if (id == INTEL_PCH_KBP_DEVICE_ID_TYPE) { | 213 | } else if (id == INTEL_PCH_KBP_DEVICE_ID_TYPE) { |
214 | dev_priv->pch_type = PCH_KBP; | 214 | dev_priv->pch_type = PCH_KBP; |
215 | DRM_DEBUG_KMS("Found KabyPoint PCH\n"); | 215 | DRM_DEBUG_KMS("Found KabyPoint PCH\n"); |
216 | WARN_ON(!IS_KABYLAKE(dev)); | 216 | WARN_ON(!IS_KABYLAKE(dev_priv)); |
217 | } else if ((id == INTEL_PCH_P2X_DEVICE_ID_TYPE) || | 217 | } else if ((id == INTEL_PCH_P2X_DEVICE_ID_TYPE) || |
218 | (id == INTEL_PCH_P3X_DEVICE_ID_TYPE) || | 218 | (id == INTEL_PCH_P3X_DEVICE_ID_TYPE) || |
219 | ((id == INTEL_PCH_QEMU_DEVICE_ID_TYPE) && | 219 | ((id == INTEL_PCH_QEMU_DEVICE_ID_TYPE) && |
diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h index 6fa237739fdc..40b86dcb781b 100644 --- a/drivers/gpu/drm/i915/i915_drv.h +++ b/drivers/gpu/drm/i915/i915_drv.h | |||
@@ -2661,7 +2661,7 @@ struct drm_i915_cmd_table { | |||
2661 | #define IS_BROADWELL(dev_priv) ((dev_priv)->info.is_broadwell) | 2661 | #define IS_BROADWELL(dev_priv) ((dev_priv)->info.is_broadwell) |
2662 | #define IS_SKYLAKE(dev) (INTEL_INFO(dev)->is_skylake) | 2662 | #define IS_SKYLAKE(dev) (INTEL_INFO(dev)->is_skylake) |
2663 | #define IS_BROXTON(dev) (INTEL_INFO(dev)->is_broxton) | 2663 | #define IS_BROXTON(dev) (INTEL_INFO(dev)->is_broxton) |
2664 | #define IS_KABYLAKE(dev) (INTEL_INFO(dev)->is_kabylake) | 2664 | #define IS_KABYLAKE(dev_priv) ((dev_priv)->info.is_kabylake) |
2665 | #define IS_MOBILE(dev) (INTEL_INFO(dev)->is_mobile) | 2665 | #define IS_MOBILE(dev) (INTEL_INFO(dev)->is_mobile) |
2666 | #define IS_HSW_EARLY_SDV(dev_priv) (IS_HASWELL(dev_priv) && \ | 2666 | #define IS_HSW_EARLY_SDV(dev_priv) (IS_HASWELL(dev_priv) && \ |
2667 | (INTEL_DEVID(dev_priv) & 0xFF00) == 0x0C00) | 2667 | (INTEL_DEVID(dev_priv) & 0xFF00) == 0x0C00) |
@@ -2728,8 +2728,8 @@ struct drm_i915_cmd_table { | |||
2728 | #define KBL_REVID_D0 0x3 | 2728 | #define KBL_REVID_D0 0x3 |
2729 | #define KBL_REVID_E0 0x4 | 2729 | #define KBL_REVID_E0 0x4 |
2730 | 2730 | ||
2731 | #define IS_KBL_REVID(p, since, until) \ | 2731 | #define IS_KBL_REVID(dev_priv, since, until) \ |
2732 | (IS_KABYLAKE(p) && IS_REVID(p, since, until)) | 2732 | (IS_KABYLAKE(dev_priv) && IS_REVID(dev_priv, since, until)) |
2733 | 2733 | ||
2734 | /* | 2734 | /* |
2735 | * The genX designation typically refers to the render engine, so render | 2735 | * The genX designation typically refers to the render engine, so render |
diff --git a/drivers/gpu/drm/i915/i915_gem_stolen.c b/drivers/gpu/drm/i915/i915_gem_stolen.c index cbea6fb83ce5..3508120b8c90 100644 --- a/drivers/gpu/drm/i915/i915_gem_stolen.c +++ b/drivers/gpu/drm/i915/i915_gem_stolen.c | |||
@@ -456,7 +456,7 @@ int i915_gem_init_stolen(struct drm_device *dev) | |||
456 | break; | 456 | break; |
457 | default: | 457 | default: |
458 | if (IS_BROADWELL(dev_priv) || | 458 | if (IS_BROADWELL(dev_priv) || |
459 | IS_SKYLAKE(dev_priv) || IS_KABYLAKE(dev)) | 459 | IS_SKYLAKE(dev_priv) || IS_KABYLAKE(dev_priv)) |
460 | bdw_get_stolen_reserved(dev_priv, &reserved_base, | 460 | bdw_get_stolen_reserved(dev_priv, &reserved_base, |
461 | &reserved_size); | 461 | &reserved_size); |
462 | else | 462 | else |
diff --git a/drivers/gpu/drm/i915/intel_ddi.c b/drivers/gpu/drm/i915/intel_ddi.c index cd7128b89b4d..07164e250adf 100644 --- a/drivers/gpu/drm/i915/intel_ddi.c +++ b/drivers/gpu/drm/i915/intel_ddi.c | |||
@@ -1020,13 +1020,13 @@ static void bxt_ddi_clock_get(struct intel_encoder *encoder, | |||
1020 | void intel_ddi_clock_get(struct intel_encoder *encoder, | 1020 | void intel_ddi_clock_get(struct intel_encoder *encoder, |
1021 | struct intel_crtc_state *pipe_config) | 1021 | struct intel_crtc_state *pipe_config) |
1022 | { | 1022 | { |
1023 | struct drm_device *dev = encoder->base.dev; | 1023 | struct drm_i915_private *dev_priv = to_i915(encoder->base.dev); |
1024 | 1024 | ||
1025 | if (INTEL_INFO(dev)->gen <= 8) | 1025 | if (INTEL_GEN(dev_priv) <= 8) |
1026 | hsw_ddi_clock_get(encoder, pipe_config); | 1026 | hsw_ddi_clock_get(encoder, pipe_config); |
1027 | else if (IS_SKYLAKE(dev) || IS_KABYLAKE(dev)) | 1027 | else if (IS_SKYLAKE(dev_priv) || IS_KABYLAKE(dev_priv)) |
1028 | skl_ddi_clock_get(encoder, pipe_config); | 1028 | skl_ddi_clock_get(encoder, pipe_config); |
1029 | else if (IS_BROXTON(dev)) | 1029 | else if (IS_BROXTON(dev_priv)) |
1030 | bxt_ddi_clock_get(encoder, pipe_config); | 1030 | bxt_ddi_clock_get(encoder, pipe_config); |
1031 | } | 1031 | } |
1032 | 1032 | ||
@@ -1081,14 +1081,14 @@ bxt_ddi_pll_select(struct intel_crtc *intel_crtc, | |||
1081 | bool intel_ddi_pll_select(struct intel_crtc *intel_crtc, | 1081 | bool intel_ddi_pll_select(struct intel_crtc *intel_crtc, |
1082 | struct intel_crtc_state *crtc_state) | 1082 | struct intel_crtc_state *crtc_state) |
1083 | { | 1083 | { |
1084 | struct drm_device *dev = intel_crtc->base.dev; | 1084 | struct drm_i915_private *dev_priv = to_i915(intel_crtc->base.dev); |
1085 | struct intel_encoder *intel_encoder = | 1085 | struct intel_encoder *intel_encoder = |
1086 | intel_ddi_get_crtc_new_encoder(crtc_state); | 1086 | intel_ddi_get_crtc_new_encoder(crtc_state); |
1087 | 1087 | ||
1088 | if (IS_SKYLAKE(dev) || IS_KABYLAKE(dev)) | 1088 | if (IS_SKYLAKE(dev_priv) || IS_KABYLAKE(dev_priv)) |
1089 | return skl_ddi_pll_select(intel_crtc, crtc_state, | 1089 | return skl_ddi_pll_select(intel_crtc, crtc_state, |
1090 | intel_encoder); | 1090 | intel_encoder); |
1091 | else if (IS_BROXTON(dev)) | 1091 | else if (IS_BROXTON(dev_priv)) |
1092 | return bxt_ddi_pll_select(intel_crtc, crtc_state, | 1092 | return bxt_ddi_pll_select(intel_crtc, crtc_state, |
1093 | intel_encoder); | 1093 | intel_encoder); |
1094 | else | 1094 | else |
@@ -1742,7 +1742,7 @@ static void intel_ddi_post_disable(struct intel_encoder *intel_encoder, | |||
1742 | intel_edp_panel_off(intel_dp); | 1742 | intel_edp_panel_off(intel_dp); |
1743 | } | 1743 | } |
1744 | 1744 | ||
1745 | if (IS_SKYLAKE(dev) || IS_KABYLAKE(dev)) | 1745 | if (IS_SKYLAKE(dev_priv) || IS_KABYLAKE(dev_priv)) |
1746 | I915_WRITE(DPLL_CTRL2, (I915_READ(DPLL_CTRL2) | | 1746 | I915_WRITE(DPLL_CTRL2, (I915_READ(DPLL_CTRL2) | |
1747 | DPLL_CTRL2_DDI_CLK_OFF(port))); | 1747 | DPLL_CTRL2_DDI_CLK_OFF(port))); |
1748 | else if (INTEL_INFO(dev)->gen < 9) | 1748 | else if (INTEL_INFO(dev)->gen < 9) |
diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index e057b5480b49..1a2b3be4d882 100644 --- a/drivers/gpu/drm/i915/intel_display.c +++ b/drivers/gpu/drm/i915/intel_display.c | |||
@@ -5828,7 +5828,7 @@ static void intel_update_max_cdclk(struct drm_device *dev) | |||
5828 | { | 5828 | { |
5829 | struct drm_i915_private *dev_priv = to_i915(dev); | 5829 | struct drm_i915_private *dev_priv = to_i915(dev); |
5830 | 5830 | ||
5831 | if (IS_SKYLAKE(dev) || IS_KABYLAKE(dev)) { | 5831 | if (IS_SKYLAKE(dev_priv) || IS_KABYLAKE(dev_priv)) { |
5832 | u32 limit = I915_READ(SKL_DFSM) & SKL_DFSM_CDCLK_LIMIT_MASK; | 5832 | u32 limit = I915_READ(SKL_DFSM) & SKL_DFSM_CDCLK_LIMIT_MASK; |
5833 | int max_cdclk, vco; | 5833 | int max_cdclk, vco; |
5834 | 5834 | ||
@@ -10646,7 +10646,7 @@ static void haswell_get_ddi_port_state(struct intel_crtc *crtc, | |||
10646 | 10646 | ||
10647 | port = (tmp & TRANS_DDI_PORT_MASK) >> TRANS_DDI_PORT_SHIFT; | 10647 | port = (tmp & TRANS_DDI_PORT_MASK) >> TRANS_DDI_PORT_SHIFT; |
10648 | 10648 | ||
10649 | if (IS_SKYLAKE(dev) || IS_KABYLAKE(dev)) | 10649 | if (IS_SKYLAKE(dev_priv) || IS_KABYLAKE(dev_priv)) |
10650 | skylake_get_ddi_pll(dev_priv, port, pipe_config); | 10650 | skylake_get_ddi_pll(dev_priv, port, pipe_config); |
10651 | else if (IS_BROXTON(dev)) | 10651 | else if (IS_BROXTON(dev)) |
10652 | bxt_get_ddi_pll(dev_priv, port, pipe_config); | 10652 | bxt_get_ddi_pll(dev_priv, port, pipe_config); |
@@ -12821,7 +12821,7 @@ static void intel_dump_pipe_config(struct intel_crtc *crtc, | |||
12821 | pipe_config->dpll_hw_state.pll9, | 12821 | pipe_config->dpll_hw_state.pll9, |
12822 | pipe_config->dpll_hw_state.pll10, | 12822 | pipe_config->dpll_hw_state.pll10, |
12823 | pipe_config->dpll_hw_state.pcsdw12); | 12823 | pipe_config->dpll_hw_state.pcsdw12); |
12824 | } else if (IS_SKYLAKE(dev) || IS_KABYLAKE(dev)) { | 12824 | } else if (IS_SKYLAKE(dev_priv) || IS_KABYLAKE(dev_priv)) { |
12825 | DRM_DEBUG_KMS("dpll_hw_state: " | 12825 | DRM_DEBUG_KMS("dpll_hw_state: " |
12826 | "ctrl1: 0x%x, cfgcr1: 0x%x, cfgcr2: 0x%x\n", | 12826 | "ctrl1: 0x%x, cfgcr1: 0x%x, cfgcr2: 0x%x\n", |
12827 | pipe_config->dpll_hw_state.ctrl1, | 12827 | pipe_config->dpll_hw_state.ctrl1, |
@@ -15420,7 +15420,7 @@ static void intel_setup_outputs(struct drm_device *dev) | |||
15420 | */ | 15420 | */ |
15421 | found = I915_READ(DDI_BUF_CTL(PORT_A)) & DDI_INIT_DISPLAY_DETECTED; | 15421 | found = I915_READ(DDI_BUF_CTL(PORT_A)) & DDI_INIT_DISPLAY_DETECTED; |
15422 | /* WaIgnoreDDIAStrap: skl */ | 15422 | /* WaIgnoreDDIAStrap: skl */ |
15423 | if (found || IS_SKYLAKE(dev) || IS_KABYLAKE(dev)) | 15423 | if (found || IS_SKYLAKE(dev_priv) || IS_KABYLAKE(dev_priv)) |
15424 | intel_ddi_init(dev, PORT_A); | 15424 | intel_ddi_init(dev, PORT_A); |
15425 | 15425 | ||
15426 | /* DDI B, C and D detection is indicated by the SFUSE_STRAP | 15426 | /* DDI B, C and D detection is indicated by the SFUSE_STRAP |
@@ -15436,7 +15436,7 @@ static void intel_setup_outputs(struct drm_device *dev) | |||
15436 | /* | 15436 | /* |
15437 | * On SKL we don't have a way to detect DDI-E so we rely on VBT. | 15437 | * On SKL we don't have a way to detect DDI-E so we rely on VBT. |
15438 | */ | 15438 | */ |
15439 | if ((IS_SKYLAKE(dev) || IS_KABYLAKE(dev)) && | 15439 | if ((IS_SKYLAKE(dev_priv) || IS_KABYLAKE(dev_priv)) && |
15440 | (dev_priv->vbt.ddi_port_info[PORT_E].supports_dp || | 15440 | (dev_priv->vbt.ddi_port_info[PORT_E].supports_dp || |
15441 | dev_priv->vbt.ddi_port_info[PORT_E].supports_dvi || | 15441 | dev_priv->vbt.ddi_port_info[PORT_E].supports_dvi || |
15442 | dev_priv->vbt.ddi_port_info[PORT_E].supports_hdmi)) | 15442 | dev_priv->vbt.ddi_port_info[PORT_E].supports_hdmi)) |
diff --git a/drivers/gpu/drm/i915/intel_dpll_mgr.c b/drivers/gpu/drm/i915/intel_dpll_mgr.c index c37ce1263142..7cf9d91c0746 100644 --- a/drivers/gpu/drm/i915/intel_dpll_mgr.c +++ b/drivers/gpu/drm/i915/intel_dpll_mgr.c | |||
@@ -1851,7 +1851,7 @@ void intel_shared_dpll_init(struct drm_device *dev) | |||
1851 | const struct dpll_info *dpll_info; | 1851 | const struct dpll_info *dpll_info; |
1852 | int i; | 1852 | int i; |
1853 | 1853 | ||
1854 | if (IS_SKYLAKE(dev) || IS_KABYLAKE(dev)) | 1854 | if (IS_SKYLAKE(dev_priv) || IS_KABYLAKE(dev_priv)) |
1855 | dpll_mgr = &skl_pll_mgr; | 1855 | dpll_mgr = &skl_pll_mgr; |
1856 | else if (IS_BROXTON(dev)) | 1856 | else if (IS_BROXTON(dev)) |
1857 | dpll_mgr = &bxt_pll_mgr; | 1857 | dpll_mgr = &bxt_pll_mgr; |
diff --git a/drivers/gpu/drm/i915/intel_guc_loader.c b/drivers/gpu/drm/i915/intel_guc_loader.c index 95c6751d598d..8eb49685dfdd 100644 --- a/drivers/gpu/drm/i915/intel_guc_loader.c +++ b/drivers/gpu/drm/i915/intel_guc_loader.c | |||
@@ -736,7 +736,7 @@ void intel_guc_init(struct drm_device *dev) | |||
736 | fw_path = I915_BXT_GUC_UCODE; | 736 | fw_path = I915_BXT_GUC_UCODE; |
737 | guc_fw->guc_fw_major_wanted = BXT_FW_MAJOR; | 737 | guc_fw->guc_fw_major_wanted = BXT_FW_MAJOR; |
738 | guc_fw->guc_fw_minor_wanted = BXT_FW_MINOR; | 738 | guc_fw->guc_fw_minor_wanted = BXT_FW_MINOR; |
739 | } else if (IS_KABYLAKE(dev)) { | 739 | } else if (IS_KABYLAKE(dev_priv)) { |
740 | fw_path = I915_KBL_GUC_UCODE; | 740 | fw_path = I915_KBL_GUC_UCODE; |
741 | guc_fw->guc_fw_major_wanted = KBL_FW_MAJOR; | 741 | guc_fw->guc_fw_major_wanted = KBL_FW_MAJOR; |
742 | guc_fw->guc_fw_minor_wanted = KBL_FW_MINOR; | 742 | guc_fw->guc_fw_minor_wanted = KBL_FW_MINOR; |
diff --git a/drivers/gpu/drm/i915/intel_runtime_pm.c b/drivers/gpu/drm/i915/intel_runtime_pm.c index 77ef03cb6163..025fbd522819 100644 --- a/drivers/gpu/drm/i915/intel_runtime_pm.c +++ b/drivers/gpu/drm/i915/intel_runtime_pm.c | |||
@@ -2594,7 +2594,7 @@ void intel_power_domains_init_hw(struct drm_i915_private *dev_priv, bool resume) | |||
2594 | 2594 | ||
2595 | power_domains->initializing = true; | 2595 | power_domains->initializing = true; |
2596 | 2596 | ||
2597 | if (IS_SKYLAKE(dev) || IS_KABYLAKE(dev)) { | 2597 | if (IS_SKYLAKE(dev_priv) || IS_KABYLAKE(dev_priv)) { |
2598 | skl_display_core_init(dev_priv, resume); | 2598 | skl_display_core_init(dev_priv, resume); |
2599 | } else if (IS_BROXTON(dev)) { | 2599 | } else if (IS_BROXTON(dev)) { |
2600 | bxt_display_core_init(dev_priv, resume); | 2600 | bxt_display_core_init(dev_priv, resume); |