Skip to content

Commit 4d79ce4

Browse files
committed
Merge tag 'drm-intel-next-fixes-2017-06-27' of git://anongit.freedesktop.org/git/drm-intel into drm-next
Just three minor fixups for stuff in -next. * tag 'drm-intel-next-fixes-2017-06-27' of git://anongit.freedesktop.org/git/drm-intel: drm/i915: Clear execbuf's vma backpointer upon release drm/i915: Pass the right flags to i915_vma_move_to_active() drm/i915/cnl: Fix RMW on ddi vswing sequence.
2 parents 12d0166 + bdbbf7d commit 4d79ce4

File tree

3 files changed

+18
-1
lines changed

3 files changed

+18
-1
lines changed

drivers/gpu/drm/i915/i915_gem_execbuffer.c

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -878,6 +878,7 @@ static void eb_release_vmas(const struct i915_execbuffer *eb)
878878

879879
GEM_BUG_ON(vma->exec_entry != entry);
880880
vma->exec_entry = NULL;
881+
__exec_to_vma(entry) = 0;
881882

882883
if (entry->flags & __EXEC_OBJECT_HAS_PIN)
883884
__eb_unreserve_vma(vma, entry);
@@ -1199,7 +1200,7 @@ static int __reloc_gpu_alloc(struct i915_execbuffer *eb,
11991200
reservation_object_unlock(batch->resv);
12001201
i915_vma_unpin(batch);
12011202

1202-
i915_vma_move_to_active(vma, rq, true);
1203+
i915_vma_move_to_active(vma, rq, EXEC_OBJECT_WRITE);
12031204
reservation_object_lock(vma->resv, NULL);
12041205
reservation_object_add_excl_fence(vma->resv, &rq->fence);
12051206
reservation_object_unlock(vma->resv);

drivers/gpu/drm/i915/i915_reg.h

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1764,8 +1764,11 @@ enum skl_disp_power_wells {
17641764
_CNL_PORT_TX_DW2_LN0_AE, \
17651765
_CNL_PORT_TX_DW2_LN0_F)
17661766
#define SWING_SEL_UPPER(x) ((x >> 3) << 15)
1767+
#define SWING_SEL_UPPER_MASK (1 << 15)
17671768
#define SWING_SEL_LOWER(x) ((x & 0x7) << 11)
1769+
#define SWING_SEL_LOWER_MASK (0x7 << 11)
17681770
#define RCOMP_SCALAR(x) ((x) << 0)
1771+
#define RCOMP_SCALAR_MASK (0xFF << 0)
17691772

17701773
#define _CNL_PORT_TX_DW4_GRP_AE 0x162350
17711774
#define _CNL_PORT_TX_DW4_GRP_B 0x1623D0
@@ -1795,8 +1798,11 @@ enum skl_disp_power_wells {
17951798
_CNL_PORT_TX_DW4_LN0_F)
17961799
#define LOADGEN_SELECT (1 << 31)
17971800
#define POST_CURSOR_1(x) ((x) << 12)
1801+
#define POST_CURSOR_1_MASK (0x3F << 12)
17981802
#define POST_CURSOR_2(x) ((x) << 6)
1803+
#define POST_CURSOR_2_MASK (0x3F << 6)
17991804
#define CURSOR_COEFF(x) ((x) << 0)
1805+
#define CURSOR_COEFF_MASK (0x3F << 6)
18001806

18011807
#define _CNL_PORT_TX_DW5_GRP_AE 0x162354
18021808
#define _CNL_PORT_TX_DW5_GRP_B 0x1623D4
@@ -1825,7 +1831,9 @@ enum skl_disp_power_wells {
18251831
#define TX_TRAINING_EN (1 << 31)
18261832
#define TAP3_DISABLE (1 << 29)
18271833
#define SCALING_MODE_SEL(x) ((x) << 18)
1834+
#define SCALING_MODE_SEL_MASK (0x7 << 18)
18281835
#define RTERM_SELECT(x) ((x) << 3)
1836+
#define RTERM_SELECT_MASK (0x7 << 3)
18291837

18301838
#define _CNL_PORT_TX_DW7_GRP_AE 0x16235C
18311839
#define _CNL_PORT_TX_DW7_GRP_B 0x1623DC
@@ -1852,6 +1860,7 @@ enum skl_disp_power_wells {
18521860
_CNL_PORT_TX_DW7_LN0_AE, \
18531861
_CNL_PORT_TX_DW7_LN0_F)
18541862
#define N_SCALAR(x) ((x) << 24)
1863+
#define N_SCALAR_MASK (0x7F << 24)
18551864

18561865
/* The spec defines this only for BXT PHY0, but lets assume that this
18571866
* would exist for PHY1 too if it had a second channel.

drivers/gpu/drm/i915/intel_ddi.c

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1813,11 +1813,14 @@ static void cnl_ddi_vswing_program(struct drm_i915_private *dev_priv,
18131813

18141814
/* Set PORT_TX_DW5 Scaling Mode Sel to 010b. */
18151815
val = I915_READ(CNL_PORT_TX_DW5_LN0(port));
1816+
val &= ~SCALING_MODE_SEL_MASK;
18161817
val |= SCALING_MODE_SEL(2);
18171818
I915_WRITE(CNL_PORT_TX_DW5_GRP(port), val);
18181819

18191820
/* Program PORT_TX_DW2 */
18201821
val = I915_READ(CNL_PORT_TX_DW2_LN0(port));
1822+
val &= ~(SWING_SEL_LOWER_MASK | SWING_SEL_UPPER_MASK |
1823+
RCOMP_SCALAR_MASK);
18211824
val |= SWING_SEL_UPPER(ddi_translations[level].dw2_swing_sel);
18221825
val |= SWING_SEL_LOWER(ddi_translations[level].dw2_swing_sel);
18231826
/* Rcomp scalar is fixed as 0x98 for every table entry */
@@ -1828,6 +1831,8 @@ static void cnl_ddi_vswing_program(struct drm_i915_private *dev_priv,
18281831
/* We cannot write to GRP. It would overrite individual loadgen */
18291832
for (ln = 0; ln < 4; ln++) {
18301833
val = I915_READ(CNL_PORT_TX_DW4_LN(port, ln));
1834+
val &= ~(POST_CURSOR_1_MASK | POST_CURSOR_2_MASK |
1835+
CURSOR_COEFF_MASK);
18311836
val |= POST_CURSOR_1(ddi_translations[level].dw4_post_cursor_1);
18321837
val |= POST_CURSOR_2(ddi_translations[level].dw4_post_cursor_2);
18331838
val |= CURSOR_COEFF(ddi_translations[level].dw4_cursor_coeff);
@@ -1837,12 +1842,14 @@ static void cnl_ddi_vswing_program(struct drm_i915_private *dev_priv,
18371842
/* Program PORT_TX_DW5 */
18381843
/* All DW5 values are fixed for every table entry */
18391844
val = I915_READ(CNL_PORT_TX_DW5_LN0(port));
1845+
val &= ~RTERM_SELECT_MASK;
18401846
val |= RTERM_SELECT(6);
18411847
val |= TAP3_DISABLE;
18421848
I915_WRITE(CNL_PORT_TX_DW5_GRP(port), val);
18431849

18441850
/* Program PORT_TX_DW7 */
18451851
val = I915_READ(CNL_PORT_TX_DW7_LN0(port));
1852+
val &= ~N_SCALAR_MASK;
18461853
val |= N_SCALAR(ddi_translations[level].dw7_n_scalar);
18471854
I915_WRITE(CNL_PORT_TX_DW7_GRP(port), val);
18481855
}

0 commit comments

Comments
 (0)