Re: [PATCH 03/10] drm/xe/sriov: Add support for enabling scheduler groups

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

<snip>

>>> @@ -214,6 +228,9 @@ enum  {
>>>   #define GUC_KLV_VGT_POLICY_ADVERSE_SAMPLE_PERIOD_KEY 0x8002
>>>   #define GUC_KLV_VGT_POLICY_ADVERSE_SAMPLE_PERIOD_LEN    1u
>>>   +#define GUC_KLV_VGT_POLICY_ENGINE_GROUP_CONFIG_KEY 0x8004
>> maybe we should add some _LEN macros for completeness?
>>
>>     #define GUC_KLV_VGT_POLICY_ENGINE_GROUP_CONFIG_MIN_LEN    0u
>>     #define GUC_KLV_VGT_POLICY_ENGINE_GROUP_CONFIG_MAX_LEN \
>>     (GUC_MAX_ENGINE_CLASSES * GUC_KLV_VGT_POLICY_ENGINE_GROUP_MAX_COUNT)
>>
>> which then can be used in some asserts where we prepare KLV payloads
>
> ok
>
Can't actually do this in an easy way because GUC_MAX_ENGINE_CLASSES is 
defined in guc_fwif.h, which already includes guc_klvs_abi.h . Easier to 
just do the multiplication in the .c file.

Daniele