[igt-dev] [PATCH i-g-t v2] lib/i915: Add a helper to read mmio registers via ioctl
Teres Alexis, Alan Previn
alan.previn.teres.alexis at intel.com
Mon Sep 11 16:42:49 UTC 2023
Hi Kamil,
Apologies for the tardy response.
Thanks again for taking the time to review.
I will fix everything below as per your recommendation.
might need some trivial mods to callers.
alan:snip
I do wanna explain the below hunk specifically though, but i prefer your recommendation:
On one of my first upstream patch reviews, i was asked to re-arrange the headers of a hunk
to be alphabetical eventhough my change was just adding 1 header.
I'll fix below to add just the needed. :)
> > --- a/tools/i915-perf/i915_perf_recorder.c
> > +++ b/tools/i915-perf/i915_perf_recorder.c
> > @@ -44,14 +44,15 @@
> >
> > #include <i915_drm.h>
> >
> > -#include "i915/i915_drm_local.h"
> > -
> > +#include "i915_perf_recorder_commands.h"
>
> Why do you touch unrelated headers?
>
> > #include "igt_core.h"
> > #include "intel_chipset.h"
> > +
> > +#include "i915/i915_drm_local.h"
> > #include "i915/perf.h"
> > #include "i915/perf_data.h"
> >
> > -#include "i915_perf_recorder_commands.h"
> > +#include "lib/intel_reg.h"
> ------------ ^^^^^^^^^^^^^^^
> You need only this here.
More information about the igt-dev
mailing list