Re: [PATCH v8 02/26] drm/xe/forcewake: Change awake_domain datatype

["Nilawar, Badal" <badal.nilawar@intel.com>]

On 08-10-2024 12:40, Himal Prasad Ghimiray wrote:
> Change the datatype of awake_domains to unsigned int to accommodate
> values that differ from the enum xe_force_wake_domains.
> 
> Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
> Cc: Badal Nilawar <badal.nilawar@intel.com>
> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> Signed-off-by: Himal Prasad Ghimiray <himal.prasad.ghimiray@intel.com>
> ---
>   drivers/gpu/drm/xe/xe_force_wake_types.h | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_force_wake_types.h b/drivers/gpu/drm/xe/xe_force_wake_types.h
> index 494240777d03..fde17dc3d01e 100644
> --- a/drivers/gpu/drm/xe/xe_force_wake_types.h
> +++ b/drivers/gpu/drm/xe/xe_force_wake_types.h
> @@ -78,7 +78,7 @@ struct xe_force_wake {
>   	/** @lock: protects everything force wake struct */
>   	spinlock_t lock;
>   	/** @awake_domains: mask of all domains awake */
> -	enum xe_force_wake_domains awake_domains;
> +	unsigned int awake_domains;

Reviewed-by: Badal Nilawar <badal.nilawar@intel.com>

>   	/** @initialized_domains: mask of all initialized domains */
>   	unsigned int initialized_domains;
>   	/** @domains: force wake domains */