[Intel-gfx] [PATCH v4 2/2] drm/i915/gt: Move power management debug files into a gt aware debugfs
Chris Wilson
chris at chris-wilson.co.uk
Fri Dec 13 19:47:51 UTC 2019
Quoting Andi Shyti (2019-12-13 19:41:29)
> Hi Michal,
>
> > > @@ -75,6 +75,8 @@ i915-$(CONFIG_PERF_EVENTS) += i915_pmu.o
> > > # "Graphics Technology" (aka we talk to the gpu)
> > > obj-y += gt/
> > > gt-y += \
> > > + gt/debugfs_gt.o \
> > > + gt/debugfs_pm.o \
> >
> > hm, maybe this should be:
> > gt/intel_gt_debugfs.o
> > and
> > gt/intel_pm_debugfs.o
>
> this was actually the name I wanted to give it originally, but
> meantime I also wanted to have the debugfs files alphabetically
> sorted in sequence, like the selftest_* files (I can imagine in
> the future having more debugfs files).
>
> Maybe intel_debugfs_gt.c/intel_debugfs_pm.c would be a good
> compromise?
I don't mind, your argument that we will partition these files off
under gt-${CONFIG_DEBUGFS} += gt/debugfs_*.o was convincing.
>
> > > @@ -0,0 +1,22 @@
> > > +/* SPDX-License-Identifier: MIT */
> >
> > in .c SPDX shall start with //
>
> I agree this is the "official" way of doing it, and I also read
> some discussions about it in this mailing list. But however I do
> it, I know someone won't like it. I checked the style in this
> directory and tried to keep it conform to the "gt way".
Shrug. I don't like the coding style violation, so leave it up to
someone who insists to do treewide changes.
-Chris
More information about the Intel-gfx
mailing list