Quoting Oscar Mateo (2018-02-15 22:46:40)
> 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
> 
> Signed-off-by: Oscar Mateo <oscar.ma...@intel.com>
> Cc: Chris Wilson <ch...@chris-wilson.co.uk>
> Cc: Mika Kuoppala <mika.kuopp...@linux.intel.com>
> ---
> diff --git a/drivers/gpu/drm/i915/intel_workarounds.c 
> b/drivers/gpu/drm/i915/intel_workarounds.c
> new file mode 100644
> index 0000000..56a1152
> --- /dev/null
> +++ b/drivers/gpu/drm/i915/intel_workarounds.c
> @@ -0,0 +1,642 @@
> +/*
> + * SPDX-License-Identifier: MIT
> + *
> + * Copyright © 2014-2018 Intel Corporation
> + */
> +
> +#include "i915_drv.h"
> +#include "intel_workarounds.h"

We probably want some organisational notes here,
        infrastructure,
        display
          - platform in standard gen/chrono order
        gt
        ctx
? Or
        platform
         - display
         - gt
         - ctx

Reviewed-by: Chris Wilson <ch...@chris-wilson.co.uk>
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to