[PATCH 2/2] drm/i915/gmbus: Swap locks vs. wakeref in intel_gmbus_output_aksv()

[Ville Syrjala <ville.syrjala@linux.intel.com>]

From: Ville Syrjälä <ville.syrjala@linux.intel.com>

Normal GMBUS transfers lock the mutexes first, the grab the power
domain wakeref. Do it the same way around for the HDCP stuff,
for consistency.

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
 drivers/gpu/drm/i915/display/intel_gmbus.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_gmbus.c b/drivers/gpu/drm/i915/display/intel_gmbus.c
index 4d0b8c82f6ff..66f1366e74ef 100644
--- a/drivers/gpu/drm/i915/display/intel_gmbus.c
+++ b/drivers/gpu/drm/i915/display/intel_gmbus.c
@@ -833,10 +833,10 @@ int intel_gmbus_output_aksv(struct i2c_adapter *adapter)
 	struct ref_tracker *wakeref;
 	int ret;
 
-	wakeref = intel_display_power_get(display, POWER_DOMAIN_GMBUS);
-
 	adapter->lock_ops->lock_bus(adapter, 0);
 
+	wakeref = intel_display_power_get(display, POWER_DOMAIN_GMBUS);
+
 	/*
 	 * In order to output Aksv to the receiver, use an indexed write to
 	 * pass the i2c command, and tell GMBUS to use the HW-provided value
@@ -844,10 +844,10 @@ int intel_gmbus_output_aksv(struct i2c_adapter *adapter)
 	 */
 	ret = do_gmbus_xfer(adapter, msgs, ARRAY_SIZE(msgs), GMBUS_AKSV_SELECT);
 
+	intel_display_power_put(display, POWER_DOMAIN_GMBUS, wakeref);
+
 	adapter->lock_ops->unlock_bus(adapter, 0);
 
-	intel_display_power_put(display, POWER_DOMAIN_GMBUS, wakeref);
-
 	return ret;
 }
 
-- 
2.52.0