]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commitdiff
drm/i915: Just use icl+ definition for PLANE_WM blocks field
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Tue, 5 Feb 2019 20:50:56 +0000 (22:50 +0200)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Wed, 6 Feb 2019 13:13:51 +0000 (15:13 +0200)
The unused bits on PLANE_WM & co. are hardwired to zero. So no
need to worry about reading the extra bit on pre-icl.

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20190205205056.30081-2-ville.syrjala@linux.intel.com
Reviewed-by: José Roberto de Souza <jose.souza@intel.com>
drivers/gpu/drm/i915/i915_reg.h
drivers/gpu/drm/i915/intel_pm.c

index 2be34e13af781b30296db3cd13f8e795f27c6e12..638a586469f97be9fb83bbbcb152c518e7d46e1e 100644 (file)
@@ -6784,8 +6784,7 @@ enum {
 
 #define _PLANE_BUF_CFG_1_B                     0x7127c
 #define _PLANE_BUF_CFG_2_B                     0x7137c
-#define  SKL_DDB_ENTRY_MASK                    0x3FF
-#define  ICL_DDB_ENTRY_MASK                    0x7FF
+#define  DDB_ENTRY_MASK                                0x7FF /* skl+: 10 bits, icl+ 11 bits */
 #define  DDB_ENTRY_END_SHIFT                   16
 #define _PLANE_BUF_CFG_1(pipe) \
        _PIPE(pipe, _PLANE_BUF_CFG_1_A, _PLANE_BUF_CFG_1_B)
index 737005bf6816abdc5872386fe555ea300800d1f0..0f15685529a01a4e10e337d957062d544b645f0d 100644 (file)
@@ -3927,14 +3927,9 @@ static unsigned int skl_cursor_allocation(int num_active)
 static void skl_ddb_entry_init_from_hw(struct drm_i915_private *dev_priv,
                                       struct skl_ddb_entry *entry, u32 reg)
 {
-       u16 mask;
 
-       if (INTEL_GEN(dev_priv) >= 11)
-               mask = ICL_DDB_ENTRY_MASK;
-       else
-               mask = SKL_DDB_ENTRY_MASK;
-       entry->start = reg & mask;
-       entry->end = (reg >> DDB_ENTRY_END_SHIFT) & mask;
+       entry->start = reg & DDB_ENTRY_MASK;
+       entry->end = (reg >> DDB_ENTRY_END_SHIFT) & DDB_ENTRY_MASK;
 
        if (entry->end)
                entry->end += 1;