Commit 3797783b authored by Ville Syrjälä's avatar Ville Syrjälä

drm/i915: Remove counter productive REGION_* wrappers

This extra macro level between the region IDs and their bitmasks
just makes it harder to see what is used where. Get rid of the
wrappers.
Signed-off-by: default avatarVille Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20240502121423.1002-3-ville.syrjala@linux.intel.comReviewed-by: default avatarRodrigo Vivi <rodrigo.vivi@intel.com>
parent d082c05a
...@@ -81,7 +81,7 @@ ...@@ -81,7 +81,7 @@
.__runtime.page_sizes = I915_GTT_PAGE_SIZE_4K .__runtime.page_sizes = I915_GTT_PAGE_SIZE_4K
#define GEN_DEFAULT_REGIONS \ #define GEN_DEFAULT_REGIONS \
.memory_regions = REGION_SMEM | REGION_STOLEN_SMEM .memory_regions = BIT(INTEL_REGION_SMEM) | BIT(INTEL_REGION_STOLEN_SMEM)
#define I830_FEATURES \ #define I830_FEATURES \
GEN(2), \ GEN(2), \
...@@ -660,7 +660,7 @@ static const struct intel_device_info rkl_info = { ...@@ -660,7 +660,7 @@ static const struct intel_device_info rkl_info = {
}; };
#define DGFX_FEATURES \ #define DGFX_FEATURES \
.memory_regions = REGION_SMEM | REGION_LMEM | REGION_STOLEN_LMEM, \ .memory_regions = BIT(INTEL_REGION_SMEM) | BIT(INTEL_REGION_LMEM_0) | BIT(INTEL_REGION_STOLEN_LMEM), \
.has_llc = 0, \ .has_llc = 0, \
.has_pxp = 0, \ .has_pxp = 0, \
.has_snoop = 1, \ .has_snoop = 1, \
...@@ -835,7 +835,7 @@ static const struct intel_device_info mtl_info = { ...@@ -835,7 +835,7 @@ static const struct intel_device_info mtl_info = {
.has_snoop = 1, .has_snoop = 1,
.max_pat_index = 4, .max_pat_index = 4,
.has_pxp = 1, .has_pxp = 1,
.memory_regions = REGION_SMEM | REGION_STOLEN_LMEM, .memory_regions = BIT(INTEL_REGION_SMEM) | BIT(INTEL_REGION_STOLEN_LMEM),
.platform_engine_mask = BIT(RCS0) | BIT(BCS0) | BIT(CCS0), .platform_engine_mask = BIT(RCS0) | BIT(BCS0) | BIT(CCS0),
MTL_CACHELEVEL, MTL_CACHELEVEL,
}; };
......
...@@ -38,11 +38,6 @@ enum intel_region_id { ...@@ -38,11 +38,6 @@ enum intel_region_id {
INTEL_REGION_UNKNOWN, /* Should be last */ INTEL_REGION_UNKNOWN, /* Should be last */
}; };
#define REGION_SMEM BIT(INTEL_REGION_SMEM)
#define REGION_LMEM BIT(INTEL_REGION_LMEM_0)
#define REGION_STOLEN_SMEM BIT(INTEL_REGION_STOLEN_SMEM)
#define REGION_STOLEN_LMEM BIT(INTEL_REGION_STOLEN_LMEM)
#define I915_ALLOC_CONTIGUOUS BIT(0) #define I915_ALLOC_CONTIGUOUS BIT(0)
#define for_each_memory_region(mr, i915, id) \ #define for_each_memory_region(mr, i915, id) \
......
...@@ -122,7 +122,7 @@ static const struct intel_device_info mock_info = { ...@@ -122,7 +122,7 @@ static const struct intel_device_info mock_info = {
.__runtime.page_sizes = (I915_GTT_PAGE_SIZE_4K | .__runtime.page_sizes = (I915_GTT_PAGE_SIZE_4K |
I915_GTT_PAGE_SIZE_64K | I915_GTT_PAGE_SIZE_64K |
I915_GTT_PAGE_SIZE_2M), I915_GTT_PAGE_SIZE_2M),
.memory_regions = REGION_SMEM, .memory_regions = BIT(INTEL_REGION_SMEM),
.platform_engine_mask = BIT(0), .platform_engine_mask = BIT(0),
/* simply use legacy cache level for mock device */ /* simply use legacy cache level for mock device */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment