[Intel-gfx] [PATCH 0/2] Couple of patches on top of the SKL latency retrieval

Damien Lespiau damien.lespiau at intel.com
Fri Sep 5 14:53:10 CEST 2014


Ville had 2 comments on the SKL memory latency patch that are better addressed
as separate patches on top.

Patch 1: SKL unifies the plane hw and their latencies to fech data from memory
is identical. So SKL only needs a single array. We save some room by using
anymous union/structs. However, last time I tried that (shared DPLL state),
Daniel didn't like it. So separate that into its own patch so it can be easily
rejected.

Patch 2: While it shouldn't cause any harm and DATA1 is likely unused by the
pcode firmware, we don't actually know. So it's better to have as a separate
patch, just in case a revert is needed (for testing or otherwise).

-- 
Damien

Damien Lespiau (2):
  drm/i915: Use anonymous union/struct to save space taken by latency
    values
  drm/i915: Clear PCODE_DATA1 on SNB+

 drivers/gpu/drm/i915/i915_drv.h | 38 +++++++++++++++++++++-----------------
 drivers/gpu/drm/i915/i915_reg.h |  2 +-
 drivers/gpu/drm/i915/intel_pm.c |  3 +--
 3 files changed, 23 insertions(+), 20 deletions(-)

-- 
1.8.3.1




More information about the Intel-gfx mailing list