[Intel-gfx] [PATCH] drm/i915/guc: Print CTL params passed to Guc

[Michel Thierry]

On 06/15/2018 07:10 AM, Michal Wajdeczko wrote:
> While debugging we may want to examine params passed to GuC.
> Print them all if config I915_DEBUG_GUC is enabled.
> 
> Signed-off-by: Michal Wajdeczko <michal.wajdeczko at intel.com>
> Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio at intel.com>
> Cc: Michel Thierry <michel.thierry at intel.com>
> ---
>   drivers/gpu/drm/i915/intel_guc.c | 5 +++++
>   1 file changed, 5 insertions(+)
> 
> diff --git a/drivers/gpu/drm/i915/intel_guc.c b/drivers/gpu/drm/i915/intel_guc.c
> index 1aff30b..f84fbde 100644
> --- a/drivers/gpu/drm/i915/intel_guc.c
> +++ b/drivers/gpu/drm/i915/intel_guc.c
> @@ -327,6 +327,11 @@ void intel_guc_init_params(struct intel_guc *guc)
>   	params[GUC_CTL_DEBUG] = guc_ctl_debug_flags(guc);
>   	params[GUC_CTL_CTXINFO] = guc_ctl_ctxinfo_flags(guc);
>   
> +#ifdef CONFIG_DRM_I915_DEBUG_GUC
> +	for (i = 0; i < GUC_CTL_MAX_DWORDS; i++)
> +		DRM_DEBUG_DRIVER("param[%2d] = %#x\n", i, params[i]);
> +#endif
> +
>   	/*
>   	 * All SOFT_SCRATCH registers are in FORCEWAKE_BLITTER domain and
>   	 * they are power context saved so it's ok to release forcewake
> 

Reviewed-by: Michel Thierry <michel.thierry at intel.com>