[Intel-gfx] [PATCH 2/2] drm/i915: Eliminate has_4tile feature flag

[Kalvala, Haridhar]

On 8/11/2023 5:16 AM, Matt Roper wrote:
> We don't really need a feature flag for has_4tile since there's a
> well-defined cutover point (DG2) at which all new platforms started
> using Tile4 as their Y-major tiling layout.  The GT side of the code
> already handles Tile4 vs legacy TileY with checks on the IP version
> rather than looking at the feature flag, and we can simplify the display
> code similarly (which will also make it more self-contained for re-use
> in the Xe driver).

Hi Matt,

Looks good to me.

> Signed-off-by: Matt Roper <matthew.d.roper at intel.com>
> ---
>   drivers/gpu/drm/i915/display/intel_display_device.h | 1 +
>   drivers/gpu/drm/i915/i915_drv.h                     | 1 -
>   drivers/gpu/drm/i915/i915_pci.c                     | 1 -
>   drivers/gpu/drm/i915/intel_device_info.h            | 1 -
>   4 files changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_display_device.h b/drivers/gpu/drm/i915/display/intel_display_device.h
> index 215e682bd8b7..eb630a946343 100644
> --- a/drivers/gpu/drm/i915/display/intel_display_device.h
> +++ b/drivers/gpu/drm/i915/display/intel_display_device.h
> @@ -32,6 +32,7 @@ struct drm_printer;
>   	func(overlay_needs_physical); \
>   	func(supports_tv);
>   
> +#define HAS_4TILE(i915)			(IS_DG2(i915) || DISPLAY_VER(i915) >= 14)
is this tab space aligned with below lines ?
>   #define HAS_ASYNC_FLIPS(i915)		(DISPLAY_VER(i915) >= 5)
>   #define HAS_CDCLK_CRAWL(i915)		(DISPLAY_INFO(i915)->has_cdclk_crawl)
>   #define HAS_CDCLK_SQUASH(i915)		(DISPLAY_INFO(i915)->has_cdclk_squash)
> diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
> index 7a8ce7239bc9..4c6852f0a61c 100644
> --- a/drivers/gpu/drm/i915/i915_drv.h
> +++ b/drivers/gpu/drm/i915/i915_drv.h
> @@ -737,7 +737,6 @@ IS_SUBPLATFORM(const struct drm_i915_private *i915,
>   #define CMDPARSER_USES_GGTT(i915) (GRAPHICS_VER(i915) == 7)
>   
>   #define HAS_LLC(i915)	(INTEL_INFO(i915)->has_llc)
> -#define HAS_4TILE(i915)	(INTEL_INFO(i915)->has_4tile)
>   #define HAS_SNOOP(i915)	(INTEL_INFO(i915)->has_snoop)
>   #define HAS_EDRAM(i915)	((i915)->edram_size_mb)
>   #define HAS_SECURE_BATCHES(i915) (GRAPHICS_VER(i915) < 6)
> diff --git a/drivers/gpu/drm/i915/i915_pci.c b/drivers/gpu/drm/i915/i915_pci.c
> index fcacdc21643c..df7c261410f7 100644
> --- a/drivers/gpu/drm/i915/i915_pci.c
> +++ b/drivers/gpu/drm/i915/i915_pci.c
> @@ -713,7 +713,6 @@ static const struct intel_device_info adl_p_info = {
>   	.has_3d_pipeline = 1, \
>   	.has_64bit_reloc = 1, \
>   	.has_flat_ccs = 1, \
> -	.has_4tile = 1, \
>   	.has_global_mocs = 1, \
>   	.has_gt_uc = 1, \
>   	.has_llc = 1, \
> diff --git a/drivers/gpu/drm/i915/intel_device_info.h b/drivers/gpu/drm/i915/intel_device_info.h
> index dbfe6443457b..19d120728ff1 100644
> --- a/drivers/gpu/drm/i915/intel_device_info.h
> +++ b/drivers/gpu/drm/i915/intel_device_info.h
> @@ -150,7 +150,6 @@ enum intel_ppgtt_type {
>   	func(gpu_reset_clobbers_display); \
>   	func(has_reset_engine); \
>   	func(has_3d_pipeline); \
> -	func(has_4tile); \
>   	func(has_flat_ccs); \
>   	func(has_global_mocs); \
>   	func(has_gmd_id); \

-- 
Regards,
Haridhar Kalvala