Re: [PATCH v1] drm/xe/drm_ras: Move has_drm_ras check to drm_ras layer

On 03-08-2026 13:38, Raag Jadav wrote:
> has_drm_ras flag is meant to facilitate drm_ras feature. Move it to the
> correct layer where it belogs.

typo..

>
> Fixes: 63dfab5786ca ("drm/xe/xe_ras: Add drm_ras feature flag")
> Signed-off-by: Raag Jadav <[email protected]>

With that fixed
Reviewed-by: Riana Tauro <[email protected]>

Thanks
Riana

> ---
>   drivers/gpu/drm/xe/xe_drm_ras.c | 3 +++
>   drivers/gpu/drm/xe/xe_ras.c     | 3 ---
>   2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/xe/xe_drm_ras.c b/drivers/gpu/drm/xe/xe_drm_ras.c
> index 4afa2ad98300..a23cfb49361b 100644
> --- a/drivers/gpu/drm/xe/xe_drm_ras.c
> +++ b/drivers/gpu/drm/xe/xe_drm_ras.c
> @@ -233,6 +233,9 @@ int xe_drm_ras_init(struct xe_device *xe)
>   	struct drm_ras_node *node;
>   	int err;
>   
> +	if (!xe->info.has_drm_ras)
> +		return 0;
> +
>   	node = drmm_kcalloc(&xe->drm, DRM_XE_RAS_ERR_SEV_MAX, sizeof(*node), GFP_KERNEL);
>   	if (!node)
>   		return -ENOMEM;
> diff --git a/drivers/gpu/drm/xe/xe_ras.c b/drivers/gpu/drm/xe/xe_ras.c
> index 28123f08e513..f7af4da8172c 100644
> --- a/drivers/gpu/drm/xe/xe_ras.c
> +++ b/drivers/gpu/drm/xe/xe_ras.c
> @@ -816,9 +816,6 @@ void xe_ras_init(struct xe_device *xe)
>   {
>   	int ret;
>   
> -	if (!xe->info.has_drm_ras)
> -		return;
> -
>   	xe_drm_ras_init(xe);
>   
>   	if (!xe->info.has_sysctrl)