Skip to content

Commit d96536f

Browse files
ideakjlahtine-intel
authored andcommitted
drm/i915: Fix AUX power domain toggling across TypeC mode resets
Make sure to select the port's AUX power domain while holding the TC port lock. The domain depends on the port's current TC mode, which may get changed under us if we're not holding the lock. This was left out from commit 8c10e22 ("drm/i915: Keep the TypeC port mode fixed for detect/AUX transfers") Cc: <[email protected]> # v5.4+ Signed-off-by: Imre Deak <[email protected]> Reviewed-by: José Roberto de Souza <[email protected]> Link: https://patchwork.freedesktop.org/patch/msgid/[email protected] (cherry picked from commit ae9b6cf) Signed-off-by: Joonas Lahtinen <[email protected]>
1 parent f159c64 commit d96536f

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

drivers/gpu/drm/i915/display/intel_dp.c

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1359,8 +1359,7 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp,
13591359
bool is_tc_port = intel_phy_is_tc(i915, phy);
13601360
i915_reg_t ch_ctl, ch_data[5];
13611361
u32 aux_clock_divider;
1362-
enum intel_display_power_domain aux_domain =
1363-
intel_aux_power_domain(intel_dig_port);
1362+
enum intel_display_power_domain aux_domain;
13641363
intel_wakeref_t aux_wakeref;
13651364
intel_wakeref_t pps_wakeref;
13661365
int i, ret, recv_bytes;
@@ -1375,6 +1374,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp,
13751374
if (is_tc_port)
13761375
intel_tc_port_lock(intel_dig_port);
13771376

1377+
aux_domain = intel_aux_power_domain(intel_dig_port);
1378+
13781379
aux_wakeref = intel_display_power_get(i915, aux_domain);
13791380
pps_wakeref = pps_lock(intel_dp);
13801381

0 commit comments

Comments
 (0)