[PATCH 1/2] drm/xe: Fix END redefinition

[Guenter Roeck]

On Thu, Mar 21, 2024 at 03:17:25PM -0700, Lucas De Marchi wrote:
> mips declares an END macro in its headers so it can't be used without
> namespace in a driver like xe.
> 
> Instead of coming up with a longer name, just remove the macro and
> replace its use with 0 since it's still clear what that means:
> set_offsets() was already using that implicitly when checking the data
> variable.
> 
> Closes: http://kisskb.ellerman.id.au/kisskb/buildresult/15143996/
> Reported-by: Guenter Roeck <linux at roeck-us.net>
> Signed-off-by: Lucas De Marchi <lucas.demarchi at intel.com>

Tested-by: Guenter Roeck <linux at roeck-us.net>

Thanks!
Guenter

> ---
>  drivers/gpu/drm/xe/xe_lrc.c | 20 +++++++++-----------
>  1 file changed, 9 insertions(+), 11 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_lrc.c b/drivers/gpu/drm/xe/xe_lrc.c
> index 95e0f7b1ec3f..db0793273de0 100644
> --- a/drivers/gpu/drm/xe/xe_lrc.c
> +++ b/drivers/gpu/drm/xe/xe_lrc.c
> @@ -115,7 +115,6 @@ static void set_offsets(u32 *regs,
>  #define REG16(x) \
>  	(((x) >> 9) | BIT(7) | BUILD_BUG_ON_ZERO(x >= 0x10000)), \
>  	(((x) >> 2) & 0x7f)
> -#define END 0
>  {
>  	const u32 base = hwe->mmio_base;
>  
> @@ -186,7 +185,7 @@ static const u8 gen12_xcs_offsets[] = {
>  	REG16(0x274),
>  	REG16(0x270),
>  
> -	END
> +	0
>  };
>  
>  static const u8 dg2_xcs_offsets[] = {
> @@ -220,7 +219,7 @@ static const u8 dg2_xcs_offsets[] = {
>  	REG16(0x274),
>  	REG16(0x270),
>  
> -	END
> +	0
>  };
>  
>  static const u8 gen12_rcs_offsets[] = {
> @@ -316,7 +315,7 @@ static const u8 gen12_rcs_offsets[] = {
>  	REG(0x084),
>  	NOP(1),
>  
> -	END
> +	0
>  };
>  
>  static const u8 xehp_rcs_offsets[] = {
> @@ -357,7 +356,7 @@ static const u8 xehp_rcs_offsets[] = {
>  	LRI(1, 0),
>  	REG(0x0c8),
>  
> -	END
> +	0
>  };
>  
>  static const u8 dg2_rcs_offsets[] = {
> @@ -400,7 +399,7 @@ static const u8 dg2_rcs_offsets[] = {
>  	LRI(1, 0),
>  	REG(0x0c8),
>  
> -	END
> +	0
>  };
>  
>  static const u8 mtl_rcs_offsets[] = {
> @@ -443,7 +442,7 @@ static const u8 mtl_rcs_offsets[] = {
>  	LRI(1, 0),
>  	REG(0x0c8),
>  
> -	END
> +	0
>  };
>  
>  #define XE2_CTX_COMMON \
> @@ -489,7 +488,7 @@ static const u8 xe2_rcs_offsets[] = {
>  	LRI(1, 0),              /* [0x47] */
>  	REG(0x0c8),             /* [0x48] R_PWR_CLK_STATE */
>  
> -	END
> +	0
>  };
>  
>  static const u8 xe2_bcs_offsets[] = {
> @@ -500,16 +499,15 @@ static const u8 xe2_bcs_offsets[] = {
>  	REG16(0x200),           /* [0x42] BCS_SWCTRL */
>  	REG16(0x204),           /* [0x44] BLIT_CCTL */
>  
> -	END
> +	0
>  };
>  
>  static const u8 xe2_xcs_offsets[] = {
>  	XE2_CTX_COMMON,
>  
> -	END
> +	0
>  };
>  
> -#undef END
>  #undef REG16
>  #undef REG
>  #undef LRI
> -- 
> 2.43.0
>