Re: [PATCH] drm/xe: Allow const pointer when checking SR-IOV mode

["Piotr Piórkowski" <piotr.piorkowski@intel.com>]

Michal Wajdeczko <michal.wajdeczko@intel.com> wrote on pon [2024-cze-17 14:26:13 +0200]:
> It seems that more and more parts of the driver code rely on
> having only a const pointer to the xe_device. Allow to check
> SR-IOV mode in that code as well.
> 
> Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
> ---
>  drivers/gpu/drm/xe/xe_sriov.h | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_sriov.h b/drivers/gpu/drm/xe/xe_sriov.h
> index 486bb21c3256..688fbabf08f1 100644
> --- a/drivers/gpu/drm/xe/xe_sriov.h
> +++ b/drivers/gpu/drm/xe/xe_sriov.h
> @@ -19,18 +19,18 @@ void xe_sriov_probe_early(struct xe_device *xe);
>  void xe_sriov_print_info(struct xe_device *xe, struct drm_printer *p);
>  int xe_sriov_init(struct xe_device *xe);
>  
> -static inline enum xe_sriov_mode xe_device_sriov_mode(struct xe_device *xe)
> +static inline enum xe_sriov_mode xe_device_sriov_mode(const struct xe_device *xe)
>  {
>  	xe_assert(xe, xe->sriov.__mode);
>  	return xe->sriov.__mode;
>  }
>  
> -static inline bool xe_device_is_sriov_pf(struct xe_device *xe)
> +static inline bool xe_device_is_sriov_pf(const struct xe_device *xe)
>  {
>  	return xe_device_sriov_mode(xe) == XE_SRIOV_MODE_PF;
>  }
>  
> -static inline bool xe_device_is_sriov_vf(struct xe_device *xe)
> +static inline bool xe_device_is_sriov_vf(const struct xe_device *xe)
>  {
>  	return xe_device_sriov_mode(xe) == XE_SRIOV_MODE_VF;
>  }

Reviewed-by: Piotr Piórkowski <piotr.piorkowski@intel.com>

> -- 
> 2.43.0
> 

--