Re: [PATCH v3 3/4] drm/xe/guc: Release GuC v70.29.2 for BMG

[Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>]

"Release" in the commit title is a bit of a misdirection, because this 
patch does not release any GuC, we just define it. Maybe just 
s/Release/Define/ ?
With that (or a similar reword):

Reviewed-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>

Daniele

On 8/1/2024 3:42 PM, Julia Filipchuk wrote:
> UAPI version 1.13.4
>
> Signed-off-by: Julia Filipchuk <julia.filipchuk@intel.com>
> ---
>   drivers/gpu/drm/xe/xe_uc_fw.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/drivers/gpu/drm/xe/xe_uc_fw.c b/drivers/gpu/drm/xe/xe_uc_fw.c
> index fe1b97d3d2a0..05a3c5b80b5b 100644
> --- a/drivers/gpu/drm/xe/xe_uc_fw.c
> +++ b/drivers/gpu/drm/xe/xe_uc_fw.c
> @@ -106,6 +106,7 @@ struct fw_blobs_by_type {
>   };
>   
>   #define XE_GUC_FIRMWARE_DEFS(fw_def, mmp_ver, major_ver)			\
> +	fw_def(BATTLEMAGE,	major_ver(xe,	guc,	bmg,	70, 29, 2))	\
>   	fw_def(LUNARLAKE,	major_ver(xe,	guc,	lnl,	70, 29, 2))	\
>   	fw_def(METEORLAKE,	major_ver(i915,	guc,	mtl,	70, 29, 2))	\
>   	fw_def(DG2,		major_ver(i915,	guc,	dg2,	70, 29, 2))	\