[Intel-gfx] [PATCH 1/1] Revert "drm/i915/tgl: Add extra hdc flush workaround"

[Lucas De Marchi]

On Wed, Mar 4, 2020 at 2:33 PM Caz Yokoyama <caz.yokoyama at intel.com> wrote:
>
> This reverts commit 36a6b5d964d995b536b1925ec42052ee40ba92c4.
> Fixes: 36a6b5d964d9 ("drm/i915/tgl: Add extra hdc flush workaround")
>
> The commit takes care Wa_1604544889 which was fixed on a0 stepping based on
> a0 replan. So no SW workaround is required on any stepping now.
>
> Reviewed-by: Matt Roper <matthew.d.roper at intel.com>
> Signed-off-by: Caz Yokoyama <caz.yokoyama at intel.com>

You didn't add what the reviewers asked for. Please go back to the
reviews and check that you
still have things to change in this commit in order to add his r-b.
Also check what José said.


Lucas De Marchi

> ---
>  drivers/gpu/drm/i915/gt/intel_lrc.c | 20 --------------------
>  1 file changed, 20 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/gt/intel_lrc.c b/drivers/gpu/drm/i915/gt/intel_lrc.c
> index b9b3f78f1324..f9425e5ed7ea 100644
> --- a/drivers/gpu/drm/i915/gt/intel_lrc.c
> +++ b/drivers/gpu/drm/i915/gt/intel_lrc.c
> @@ -4145,26 +4145,6 @@ static int gen12_emit_flush_render(struct i915_request *request,
>
>                 *cs++ = preparser_disable(false);
>                 intel_ring_advance(request, cs);
> -
> -               /*
> -                * Wa_1604544889:tgl
> -                */
> -               if (IS_TGL_REVID(request->i915, TGL_REVID_A0, TGL_REVID_A0)) {
> -                       flags = 0;
> -                       flags |= PIPE_CONTROL_CS_STALL;
> -                       flags |= PIPE_CONTROL_HDC_PIPELINE_FLUSH;
> -
> -                       flags |= PIPE_CONTROL_STORE_DATA_INDEX;
> -                       flags |= PIPE_CONTROL_QW_WRITE;
> -
> -                       cs = intel_ring_begin(request, 6);
> -                       if (IS_ERR(cs))
> -                               return PTR_ERR(cs);
> -
> -                       cs = gen8_emit_pipe_control(cs, flags,
> -                                                   LRC_PPHWSP_SCRATCH_ADDR);
> -                       intel_ring_advance(request, cs);
> -               }
>         }
>
>         return 0;
> --
> 2.21.0.5.gaeb582a983
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx



-- 
Lucas De Marchi