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

2018-04-11 Thread Chris Wilson
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

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

2018-04-10 Thread Oscar Mateo
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

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

2018-04-10 Thread Oscar Mateo
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

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

2018-04-10 Thread Chris Wilson
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

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

2018-04-10 Thread Oscar Mateo
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