In preparation for splitting common bits of this driver, move the
mutex-protected cable plugged callback and codec device setting
to a new function called mtk_hdmi_audio_set_plugged_cb().

Reviewed-by: CK Hu <ck...@mediatek.com>
Signed-off-by: AngeloGioacchino Del Regno 
<angelogioacchino.delre...@collabora.com>
---
 drivers/gpu/drm/mediatek/mtk_hdmi.c | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/mediatek/mtk_hdmi.c 
b/drivers/gpu/drm/mediatek/mtk_hdmi.c
index 25ea3f3d982b..69db6b19017d 100644
--- a/drivers/gpu/drm/mediatek/mtk_hdmi.c
+++ b/drivers/gpu/drm/mediatek/mtk_hdmi.c
@@ -1597,17 +1597,22 @@ static int mtk_hdmi_audio_get_eld(struct device *dev, 
void *data, uint8_t *buf,
        return 0;
 }
 
-static int mtk_hdmi_audio_hook_plugged_cb(struct device *dev, void *data,
-                                         hdmi_codec_plugged_cb fn,
+static void mtk_hdmi_audio_set_plugged_cb(struct mtk_hdmi *hdmi, 
hdmi_codec_plugged_cb fn,
                                          struct device *codec_dev)
 {
-       struct mtk_hdmi *hdmi = data;
-
        mutex_lock(&hdmi->update_plugged_status_lock);
        hdmi->plugged_cb = fn;
        hdmi->codec_dev = codec_dev;
        mutex_unlock(&hdmi->update_plugged_status_lock);
+}
+
+static int mtk_hdmi_audio_hook_plugged_cb(struct device *dev, void *data,
+                                         hdmi_codec_plugged_cb fn,
+                                         struct device *codec_dev)
+{
+       struct mtk_hdmi *hdmi = data;
 
+       mtk_hdmi_audio_set_plugged_cb(hdmi, fn, codec_dev);
        mtk_hdmi_update_plugged_status(hdmi);
 
        return 0;
-- 
2.48.1

Reply via email to