:), i think you mean should use CPU column head? because the reg value just 
reflect CPU related information

















At 2024-09-10 17:24:32, "Andi Shyti" <andi.sh...@linux.intel.com> wrote:
>Hi Zhanghe,
>
>Thanks for your patch. Please next time check from
>get_maintainers.pl the mailing lists that need to be included in
>your patches.
>
>In this case you should have included at least the
>intel-gfx <intel-gfx@lists.freedesktop.org> and the
>dri-devel <dri-de...@lists.freedesktop.org> mailing lists.
>
>On Sat, Sep 07, 2024 at 05:24:43PM +0800, zhanghe9702 wrote:
>> column header should be GPU, not CPU
>> 
>> Signed-off-by: zhanghe9702 <zhanghe9...@163.com>
>
>Do you really want your name to appear as zhanghe9702? If you git
>log the linux directory you will se that people normally use
>the "Name Surname <email>" style. As you wish.
>
>> ---
>>  drivers/gpu/drm/i915/gt/intel_gt_pm_debugfs.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>> 
>> diff --git a/drivers/gpu/drm/i915/gt/intel_gt_pm_debugfs.c 
>> b/drivers/gpu/drm/i915/gt/intel_gt_pm_debugfs.c
>> index 8d08b38874ef..b635aa2820d9 100644
>> --- a/drivers/gpu/drm/i915/gt/intel_gt_pm_debugfs.c
>> +++ b/drivers/gpu/drm/i915/gt/intel_gt_pm_debugfs.c
>> @@ -431,7 +431,7 @@ static int llc_show(struct seq_file *m, void *data)
>>              max_gpu_freq /= GEN9_FREQ_SCALER;
>>      }
>>  
>> -    seq_puts(m, "GPU freq (MHz)\tEffective CPU freq (MHz)\tEffective Ring 
>> freq (MHz)\n");
>> +    seq_puts(m, "GPU freq (MHz)\tEffective GPU freq (MHz)\tEffective Ring 
>> freq (MHz)\n");
>
>This is correct:
>
>Reviewed-by: Andi Shyti <andi.sh...@linux.intel.com>
>
>Thanks,
>Andi

Reply via email to