Re: [2/2] drm/xe/pf: Print configuration KLVs using debug printer

["Laguna, Lukasz" <lukasz.laguna@intel.com>]

On 6/4/2025 21:00, Michal Wajdeczko wrote:
> While we print VF's configuration KLVs only under DEBUG_SRIOV
> config, we should be doing it at debug level, not info level.
>
> Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
> ---
>   drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c | 8 ++++----
>   1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c b/drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c
> index 3556c41c041b..3128d9adc10e 100644
> --- a/drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c
> +++ b/drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c
> @@ -104,13 +104,13 @@ static int pf_push_vf_buf_klvs(struct xe_gt *gt, unsigned int vfid, u32 num_klvs
>   	}
>   
>   	if (IS_ENABLED(CONFIG_DRM_XE_DEBUG_SRIOV)) {
> -		struct drm_printer p = xe_gt_info_printer(gt);
> +		struct drm_printer p = xe_gt_dbg_printer(gt);
>   		void *klvs = xe_guc_buf_cpu_ptr(buf);
>   		char name[8];
>   
> -		xe_gt_sriov_info(gt, "pushed %s config with %u KLV%s:\n",
> -				 xe_sriov_function_name(vfid, name, sizeof(name)),
> -				 num_klvs, str_plural(num_klvs));
> +		xe_gt_sriov_dbg(gt, "pushed %s config with %u KLV%s:\n",
> +				xe_sriov_function_name(vfid, name, sizeof(name)),
> +				num_klvs, str_plural(num_klvs));
>   		xe_guc_klv_print(klvs, num_dwords, &p);
>   	}
>   
Shouldn't xe_gt_sriov_pf_config_restore() be updated in the same way on 
the occasion?