[PATCH 1/5] drm/xe: Move XEHP_MTCFG_ADDR register definition to xe_regs.h
Matt Roper
matthew.d.roper at intel.com
Tue May 28 21:14:43 UTC 2024
On Mon, May 27, 2024 at 07:35:50PM +0200, Michal Wajdeczko wrote:
> We should not define registers directly in the code while we have
> dedicated files for all register definitions. Move XEHP_MTCFG_ADDR
> to regs/xe_regs.h
>
> Signed-off-by: Michal Wajdeczko <michal.wajdeczko at intel.com>
> Cc: Matt Roper <matthew.d.roper at intel.com>
Reviewed-by: Matt Roper <matthew.d.roper at intel.com>
> ---
> drivers/gpu/drm/xe/regs/xe_regs.h | 3 +++
> drivers/gpu/drm/xe/xe_mmio.c | 3 ---
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/xe/regs/xe_regs.h b/drivers/gpu/drm/xe/regs/xe_regs.h
> index 722fb6dbb72e..23e33ec84902 100644
> --- a/drivers/gpu/drm/xe/regs/xe_regs.h
> +++ b/drivers/gpu/drm/xe/regs/xe_regs.h
> @@ -30,6 +30,9 @@
> #define XEHP_CLOCK_GATE_DIS XE_REG(0x101014)
> #define SGSI_SIDECLK_DIS REG_BIT(17)
>
> +#define XEHP_MTCFG_ADDR XE_REG(0x101800)
> +#define TILE_COUNT REG_GENMASK(15, 8)
> +
> #define GGC XE_REG(0x108040)
> #define GMS_MASK REG_GENMASK(15, 8)
> #define GGMS_MASK REG_GENMASK(7, 6)
> diff --git a/drivers/gpu/drm/xe/xe_mmio.c b/drivers/gpu/drm/xe/xe_mmio.c
> index 248e93ec6df7..44bff104c011 100644
> --- a/drivers/gpu/drm/xe/xe_mmio.c
> +++ b/drivers/gpu/drm/xe/xe_mmio.c
> @@ -28,9 +28,6 @@
> #include "xe_sriov.h"
> #include "xe_tile.h"
>
> -#define XEHP_MTCFG_ADDR XE_REG(0x101800)
> -#define TILE_COUNT REG_GENMASK(15, 8)
> -
> #define BAR_SIZE_SHIFT 20
>
> static void
> --
> 2.43.0
>
--
Matt Roper
Graphics Software Engineer
Linux GPU Platform Enablement
Intel Corporation
More information about the Intel-xe
mailing list