[Intel-gfx] [PATCH 1/2] drm/i915: Move a bunch of workaround-related code to its own file
Chris Wilson
chris at chris-wilson.co.uk
Thu Apr 12 00:55:10 UTC 2018
Quoting Oscar Mateo (2018-04-10 17:24:42)
>
>
> On 4/10/2018 9:16 AM, Chris Wilson wrote:
> > Quoting Oscar Mateo (2018-04-10 17:12:46)
> >> This has grown to be a sizable amount of code, so move it to
> >> its own file before we try to refactor anything. For the moment,
> >> we are leaving behind the WA BB code and the WAs that get applied
> >> (incorrectly) in init_clock_gating, but we will deal with it later.
> >>
> >> v2: Use intel_ prefix for code that deals with the hardware (Chris)
> >> v3: Rebased
> >> v4:
> >> - Rebased
> >> - New license header
> >> v5:
> >> - Rebased
> >> - Added some organisational notes to the file (Chris)
> >> v6: Include DOC section in the documentation build (Jani)
> >>
> >> Signed-off-by: Oscar Mateo <oscar.mateo at intel.com>
> >> Cc: Mika Kuoppala <mika.kuoppala at linux.intel.com>
> >> Cc: Jani Nikula <jani.nikula at intel.com>
> >> Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>
> > I presume this impacts your work. Would you rather have this patch in
> > place or land your w/a series first? (Assuming all goes well!)
> > -Chris
>
> I can do this patch first and I'll rebase the Gen11 WAs series later, no
> problem (in fact, this refactoring started because it was quite
> problematic to decide where each Gen11 WA should go).
I did a pass to try and make checkpatch happier and pushed. Thanks for
the patches, hopefully not causing too much work for you!
-Chris
More information about the Intel-gfx
mailing list