From: Ville Syrjälä <[email protected]>

We have the same code to determine the MMIO BAR in
two places. Collect it to a single place.

Signed-off-by: Ville Syrjälä <[email protected]>
---
 drivers/gpu/drm/i915/gt/intel_gt.c      | 2 +-
 drivers/gpu/drm/i915/i915_pci.c         | 4 +---
 drivers/gpu/drm/i915/intel_pci_config.h | 8 ++++++++
 3 files changed, 10 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/i915/gt/intel_gt.c 
b/drivers/gpu/drm/i915/gt/intel_gt.c
index b367cfff48d5..8995a21c2209 100644
--- a/drivers/gpu/drm/i915/gt/intel_gt.c
+++ b/drivers/gpu/drm/i915/gt/intel_gt.c
@@ -838,7 +838,7 @@ int intel_gt_probe_all(struct drm_i915_private *i915)
        unsigned int i;
        int ret;
 
-       mmio_bar = GRAPHICS_VER(i915) == 2 ? GEN2_GTTMMADR_BAR : GTTMMADR_BAR;
+       mmio_bar = intel_mmio_bar(GRAPHICS_VER(i915));
        phys_addr = pci_resource_start(pdev, mmio_bar);
 
        /*
diff --git a/drivers/gpu/drm/i915/i915_pci.c b/drivers/gpu/drm/i915/i915_pci.c
index 38460a0bd7cb..19ae173937bc 100644
--- a/drivers/gpu/drm/i915/i915_pci.c
+++ b/drivers/gpu/drm/i915/i915_pci.c
@@ -1297,9 +1297,7 @@ bool i915_pci_resource_valid(struct pci_dev *pdev, int 
bar)
 
 static bool intel_mmio_bar_valid(struct pci_dev *pdev, struct 
intel_device_info *intel_info)
 {
-       int gttmmaddr_bar = intel_info->__runtime.graphics.ip.ver == 2 ? 
GEN2_GTTMMADR_BAR : GTTMMADR_BAR;
-
-       return i915_pci_resource_valid(pdev, gttmmaddr_bar);
+       return i915_pci_resource_valid(pdev, 
intel_mmio_bar(intel_info->__runtime.graphics.ip.ver));
 }
 
 static int i915_pci_probe(struct pci_dev *pdev, const struct pci_device_id 
*ent)
diff --git a/drivers/gpu/drm/i915/intel_pci_config.h 
b/drivers/gpu/drm/i915/intel_pci_config.h
index 4977a524ce6f..305f137d2ebd 100644
--- a/drivers/gpu/drm/i915/intel_pci_config.h
+++ b/drivers/gpu/drm/i915/intel_pci_config.h
@@ -13,6 +13,14 @@
 #define GTT_APERTURE_BAR                       GFXMEM_BAR
 #define GEN12_LMEM_BAR                         GFXMEM_BAR
 
+static inline int intel_mmio_bar(int graphics_ver)
+{
+       switch (graphics_ver) {
+       case 2: return GEN2_GTTMMADR_BAR;
+       default: return GTTMMADR_BAR;
+       }
+}
+
 /* BSM in include/drm/i915_drm.h */
 
 #define MCHBAR_I915                            0x44
-- 
2.35.1

Reply via email to