Re: [PATCH v3 1/2] drm/xe/guc: Remove spurious line feed in debug print

[Michal Wajdeczko <michal.wajdeczko@intel.com>]

On 09.05.2024 00:49, John.C.Harrison@Intel.com wrote:
> From: John Harrison <John.C.Harrison@Intel.com>
> 
> Including line feeds at the start of a debug print messes up the
> output when sent to dmesg. The break actually appears between all the
> usefu prefix information and the actual string being printed. In this

typo

> case, each block of data has a very clear start line and an extra
> delimeter is really not necessary. So don't do it.
> 

this was also on my to-fix list,

Reviewed-by: Michal Wajdeczko <michal.wajdeczko@intel.com>

> Signed-off-by: John Harrison <John.C.Harrison@Intel.com>
> ---
>  drivers/gpu/drm/xe/xe_guc_ct.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_guc_ct.c b/drivers/gpu/drm/xe/xe_guc_ct.c
> index 0151d29b3c58..ecbf0b116cc3 100644
> --- a/drivers/gpu/drm/xe/xe_guc_ct.c
> +++ b/drivers/gpu/drm/xe/xe_guc_ct.c
> @@ -1435,7 +1435,7 @@ void xe_guc_ct_snapshot_print(struct xe_guc_ct_snapshot *snapshot,
>  		drm_puts(p, "H2G CTB (all sizes in DW):\n");
>  		guc_ctb_snapshot_print(&snapshot->h2g, p);
>  
> -		drm_puts(p, "\nG2H CTB (all sizes in DW):\n");
> +		drm_puts(p, "G2H CTB (all sizes in DW):\n");
>  		guc_ctb_snapshot_print(&snapshot->g2h, p);
>  
>  		drm_printf(p, "\tg2h outstanding: %d\n",