[Intel-gfx] [PATCH 1/2] drm/i915: Move a bunch of workaround-related code to its own file

Oscar Mateo oscar.mateo at intel.com
Tue Apr 10 16:24:42 UTC 2018



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).

Thanks for asking!
Oscar


More information about the Intel-gfx mailing list