[Intel-gfx] [PATCH v2 2/2] drm/i915/bxt, glk: Avoid long atomic poll during CDCLK change

Chris Wilson chris at chris-wilson.co.uk
Tue Jan 30 15:24:23 UTC 2018


Quoting Imre Deak (2018-01-30 14:29:39)
> There is no requirement for doing the PCODE request polling atomically,
> so do that only for a short time switching to sleeping poll afterwards.
> The specification requires a 150usec timeout for the change notification,
> so let's use that for the atomic poll. Do the extra 2ms poll - needed as
> a workaround on BXT/GLK - in sleeping mode.
> 
> v2:
> - rebase on v2 of patchset dropping the sandybridge_pcode_read/write
>   refactoring (Chris)
> 
> Cc: Chris Wilson <chris at chris-wilson.co.uk>
> Cc: Ville Syrjälä <ville.syrjala at linux.intel.com>
> Signed-off-by: Imre Deak <imre.deak at intel.com>

Likewise, I'm also happier with _timeout.

Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>
-Chris


More information about the Intel-gfx mailing list