Re: [PATCH v3 01/14] drm/i915/guc: remove runtime info printing from time stamp logging

[John Harrison <john.c.harrison@xxxxxxxxx>]

On 8/19/2022 05:02, Jani Nikula wrote:
> Commit 368d179adbac ("drm/i915/guc: Add GuC <-> kernel time stamp
> translation information") added intel_device_info_print_runtime() in the
> time info dump for no obvious reason or explanation in the commit
> message. It only logs the rawclk freq. Remove it.
>
> Cc: John Harrison <John.C.Harrison@xxxxxxxxx>
> Cc: Alan Previn <alan.previn.teres.alexis@xxxxxxxxx>
> Signed-off-by: Jani Nikula <jani.nikula@xxxxxxxxx>
Reviewed-by: John Harrison <John.C.Harrison@xxxxxxxxx>

> ---
>   drivers/gpu/drm/i915/gt/uc/intel_guc.c | 2 --
>   1 file changed, 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc.c b/drivers/gpu/drm/i915/gt/uc/intel_guc.c
> index 01f2705cb94a..24451d000a6a 100644
> --- a/drivers/gpu/drm/i915/gt/uc/intel_guc.c
> +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc.c
> @@ -365,8 +365,6 @@ void intel_guc_dump_time_info(struct intel_guc *guc, struct drm_printer *p)
>   	u32 stamp = 0;
>   	u64 ktime;
>   
> -	intel_device_info_print_runtime(RUNTIME_INFO(gt->i915), p);
> -
>   	with_intel_runtime_pm(&gt->i915->runtime_pm, wakeref)
>   		stamp = intel_uncore_read(gt->uncore, GUCPMTIMESTAMP);
>   	ktime = ktime_get_boottime_ns();