Commit aac5db82 authored by Harry Wentland's avatar Harry Wentland Committed by Alex Deucher

drm/amd/display: Serialize is_dp_sink_present

Access to GPIO needs to be serialized. Aux transactions are already
serialized in DRM but we also need to serialize access to the GPIO pin
for purposes of DP dongle detection.

Call is_dp_sink_present through DM so we can lock correctly. This
follows the same pattern used for DPCD transactions.
Signed-off-by: default avatarHarry Wentland <harry.wentland@amd.com>
Reviewed-by: default avatarJun Lei <Jun.Lei@amd.com>
Acked-by: default avatarHarry Wentland <harry.wentland@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 1bd493e3
...@@ -454,6 +454,22 @@ bool dm_helpers_submit_i2c( ...@@ -454,6 +454,22 @@ bool dm_helpers_submit_i2c(
return result; return result;
} }
bool dm_helpers_is_dp_sink_present(struct dc_link *link)
{
bool dp_sink_present;
struct amdgpu_dm_connector *aconnector = link->priv;
if (!aconnector) {
BUG_ON("Failed to found connector for link!");
return true;
}
mutex_lock(&aconnector->dm_dp_aux.aux.hw_mutex);
dp_sink_present = dc_link_is_dp_sink_present(link);
mutex_unlock(&aconnector->dm_dp_aux.aux.hw_mutex);
return dp_sink_present;
}
enum dc_edid_status dm_helpers_read_local_edid( enum dc_edid_status dm_helpers_read_local_edid(
struct dc_context *ctx, struct dc_context *ctx,
struct dc_link *link, struct dc_link *link,
......
...@@ -313,7 +313,7 @@ static enum signal_type get_basic_signal_type( ...@@ -313,7 +313,7 @@ static enum signal_type get_basic_signal_type(
* @brief * @brief
* Check whether there is a dongle on DP connector * Check whether there is a dongle on DP connector
*/ */
static bool is_dp_sink_present(struct dc_link *link) bool dc_link_is_dp_sink_present(struct dc_link *link)
{ {
enum gpio_result gpio_result; enum gpio_result gpio_result;
uint32_t clock_pin = 0; uint32_t clock_pin = 0;
...@@ -406,7 +406,7 @@ static enum signal_type link_detect_sink( ...@@ -406,7 +406,7 @@ static enum signal_type link_detect_sink(
* we assume signal is DVI; it could be corrected * we assume signal is DVI; it could be corrected
* to HDMI after dongle detection * to HDMI after dongle detection
*/ */
if (!is_dp_sink_present(link)) if (!dm_helpers_is_dp_sink_present(link))
result = SIGNAL_TYPE_DVI_SINGLE_LINK; result = SIGNAL_TYPE_DVI_SINGLE_LINK;
} }
} }
......
...@@ -210,6 +210,8 @@ bool dc_link_dp_set_test_pattern( ...@@ -210,6 +210,8 @@ bool dc_link_dp_set_test_pattern(
void dc_link_enable_hpd_filter(struct dc_link *link, bool enable); void dc_link_enable_hpd_filter(struct dc_link *link, bool enable);
bool dc_link_is_dp_sink_present(struct dc_link *link);
/* /*
* DPCD access interfaces * DPCD access interfaces
*/ */
......
...@@ -103,6 +103,9 @@ bool dm_helpers_submit_i2c( ...@@ -103,6 +103,9 @@ bool dm_helpers_submit_i2c(
const struct dc_link *link, const struct dc_link *link,
struct i2c_command *cmd); struct i2c_command *cmd);
bool dm_helpers_is_dp_sink_present(
struct dc_link *link);
enum dc_edid_status dm_helpers_read_local_edid( enum dc_edid_status dm_helpers_read_local_edid(
struct dc_context *ctx, struct dc_context *ctx,
struct dc_link *link, struct dc_link *link,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment