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

[Jani Nikula <jani.nikula@xxxxxxxxx>]

On Tue, 23 Aug 2022, John Harrison <john.c.harrison@xxxxxxxxx> wrote:
> 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>

Thanks, this one pushed to drm-intel-gt-next.

BR,
Jani.

>
>> ---
>>   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();
>

-- 
Jani Nikula, Intel Open Source Graphics Center