Commit 02efac0f authored by Russell King's avatar Russell King

drm/i2c: tda998x: remove complexity from tda998x_audio_get_eld()

tda998x_audio_get_eld() is needlessly complex - the connector associated
with the encoder is always our own priv->connector.  Remove this
complexity, but ensure that there are no races when copying out the ELD.
Tested-by: default avatarJon Medhurst <tixy@linaro.org>
Acked-by: default avatarJon Medhurst <tixy@linaro.org>
Tested-by: default avatarJyri Sarha <jsarha@ti.com>
Signed-off-by: default avatarRussell King <rmk+kernel@armlinux.org.uk>
parent ad975f93
...@@ -910,21 +910,13 @@ static int tda998x_audio_get_eld(struct device *dev, void *data, ...@@ -910,21 +910,13 @@ static int tda998x_audio_get_eld(struct device *dev, void *data,
uint8_t *buf, size_t len) uint8_t *buf, size_t len)
{ {
struct tda998x_priv *priv = dev_get_drvdata(dev); struct tda998x_priv *priv = dev_get_drvdata(dev);
struct drm_mode_config *config = &priv->encoder.dev->mode_config;
struct drm_connector *connector;
int ret = -ENODEV;
mutex_lock(&config->mutex);
list_for_each_entry(connector, &config->connector_list, head) {
if (&priv->encoder == connector->encoder) {
memcpy(buf, connector->eld,
min(sizeof(connector->eld), len));
ret = 0;
}
}
mutex_unlock(&config->mutex);
return ret; mutex_lock(&priv->audio_mutex);
memcpy(buf, priv->connector.eld,
min(sizeof(priv->connector.eld), len));
mutex_unlock(&priv->audio_mutex);
return 0;
} }
static const struct hdmi_codec_ops audio_codec_ops = { static const struct hdmi_codec_ops audio_codec_ops = {
...@@ -975,6 +967,7 @@ static int tda998x_connector_fill_modes(struct drm_connector *connector, ...@@ -975,6 +967,7 @@ static int tda998x_connector_fill_modes(struct drm_connector *connector,
struct tda998x_priv *priv = conn_to_tda998x_priv(connector); struct tda998x_priv *priv = conn_to_tda998x_priv(connector);
int ret; int ret;
mutex_lock(&priv->audio_mutex);
ret = drm_helper_probe_single_connector_modes(connector, maxX, maxY); ret = drm_helper_probe_single_connector_modes(connector, maxX, maxY);
if (connector->edid_blob_ptr) { if (connector->edid_blob_ptr) {
...@@ -984,6 +977,7 @@ static int tda998x_connector_fill_modes(struct drm_connector *connector, ...@@ -984,6 +977,7 @@ static int tda998x_connector_fill_modes(struct drm_connector *connector,
} else { } else {
priv->sink_has_audio = false; priv->sink_has_audio = false;
} }
mutex_unlock(&priv->audio_mutex);
return ret; return ret;
} }
......
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