Re: [PATCH 1/2] drm/xe: Add GSC to powergate_info

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

On 16-01-2026 04:10, Vinay Belgaumkar wrote:
> Add GSC powergate status to the existing debugfs.
>
> Signed-off-by: Vinay Belgaumkar <vinay.belgaumkar@intel.com>
> ---
>   drivers/gpu/drm/xe/regs/xe_gt_regs.h | 1 +
>   drivers/gpu/drm/xe/xe_gt_idle.c      | 5 +++++
>   2 files changed, 6 insertions(+)
>
> diff --git a/drivers/gpu/drm/xe/regs/xe_gt_regs.h b/drivers/gpu/drm/xe/regs/xe_gt_regs.h
> index 24fc64fc832e..7d3ec1fe4f7f 100644
> --- a/drivers/gpu/drm/xe/regs/xe_gt_regs.h
> +++ b/drivers/gpu/drm/xe/regs/xe_gt_regs.h
> @@ -367,6 +367,7 @@
>   #define FORCEWAKE_RENDER			XE_REG(0xa278)
>   
>   #define POWERGATE_DOMAIN_STATUS			XE_REG(0xa2a0)
> +#define   GSC_AWAKE_STATUS			REG_BIT(8)
>   #define   MEDIA_SLICE3_AWAKE_STATUS		REG_BIT(4)
>   #define   MEDIA_SLICE2_AWAKE_STATUS		REG_BIT(3)
>   #define   MEDIA_SLICE1_AWAKE_STATUS		REG_BIT(2)
> diff --git a/drivers/gpu/drm/xe/xe_gt_idle.c b/drivers/gpu/drm/xe/xe_gt_idle.c
> index c1c9bec3c487..52436dcb6381 100644
> --- a/drivers/gpu/drm/xe/xe_gt_idle.c
> +++ b/drivers/gpu/drm/xe/xe_gt_idle.c
> @@ -255,6 +255,11 @@ int xe_gt_idle_pg_print(struct xe_gt *gt, struct drm_printer *p)
>   		drm_printf(p, "Media Samplers Power Gating Enabled: %s\n",
>   			   str_yes_no(pg_enabled & MEDIA_SAMPLERS_POWERGATE_ENABLE));
>   
> +	if (gt->info.engine_mask & BIT(XE_HW_ENGINE_GSCCS0)) {
> +		drm_printf(p, "GSC Power Gate Status: %s\n",
> +			   str_up_down(pg_status & GSC_AWAKE_STATUS));
> +	}
> +

Looks good to me.
Reviewed-by: Badal Nilawar <badal.nilawar@intel.com>

Thanks,
Badal

>   	return 0;
>   }
>