:), 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