From: Jyri Sarha <jsa...@ti.com>

Allow HDMI audio setup even if we do not have video configured. Audio
will get configured at the same time with video if the video is
configured soon enough. If it is not the audio DMA will timeout in
couple of seconds and audio playback will be aborted.

Signed-off-by: Jyri Sarha <jsa...@ti.com>
Signed-off-by: Tomi Valkeinen <tomi.valkei...@ti.com>
---
 drivers/gpu/drm/omapdrm/dss/hdmi4.c | 33 ++++++++++++++-------------------
 drivers/gpu/drm/omapdrm/dss/hdmi5.c | 37 ++++++++++++++++---------------------
 2 files changed, 30 insertions(+), 40 deletions(-)

diff --git a/drivers/gpu/drm/omapdrm/dss/hdmi4.c 
b/drivers/gpu/drm/omapdrm/dss/hdmi4.c
index ae6401c569c4..9d5c921cbf7b 100644
--- a/drivers/gpu/drm/omapdrm/dss/hdmi4.c
+++ b/drivers/gpu/drm/omapdrm/dss/hdmi4.c
@@ -598,21 +598,16 @@ static int hdmi_audio_startup(struct device *dev,
                              void (*abort_cb)(struct device *dev))
 {
        struct omap_hdmi *hd = dev_get_drvdata(dev);
-       int ret = 0;
 
        mutex_lock(&hd->lock);
 
-       if (!hdmi_mode_has_audio(&hd->cfg) || !hd->display_enabled) {
-               ret = -EPERM;
-               goto out;
-       }
+       WARN_ON(hd->audio_abort_cb != NULL);
 
        hd->audio_abort_cb = abort_cb;
 
-out:
        mutex_unlock(&hd->lock);
 
-       return ret;
+       return 0;
 }
 
 static int hdmi_audio_shutdown(struct device *dev)
@@ -633,12 +628,14 @@ static int hdmi_audio_start(struct device *dev)
        struct omap_hdmi *hd = dev_get_drvdata(dev);
        unsigned long flags;
 
-       WARN_ON(!hdmi_mode_has_audio(&hd->cfg));
-
        spin_lock_irqsave(&hd->audio_playing_lock, flags);
 
-       if (hd->display_enabled)
+       if (hd->display_enabled) {
+               if (!hdmi_mode_has_audio(&hd->cfg))
+                       DSSERR("%s: Video mode does not support audio\n",
+                              __func__);
                hdmi_start_audio_stream(hd);
+       }
        hd->audio_playing = true;
 
        spin_unlock_irqrestore(&hd->audio_playing_lock, flags);
@@ -669,17 +666,15 @@ static int hdmi_audio_config(struct device *dev,
 
        mutex_lock(&hd->lock);
 
-       if (!hdmi_mode_has_audio(&hd->cfg) || !hd->display_enabled) {
-               ret = -EPERM;
-               goto out;
+       if (hd->display_enabled) {
+               ret = hdmi4_audio_config(&hd->core, &hd->wp, dss_audio,
+                                        hd->cfg.vm.pixelclock);
+               if (ret)
+                       goto out;
        }
 
-       ret = hdmi4_audio_config(&hd->core, &hd->wp, dss_audio,
-                                hd->cfg.vm.pixelclock);
-       if (!ret) {
-               hd->audio_configured = true;
-               hd->audio_config = *dss_audio;
-       }
+       hd->audio_configured = true;
+       hd->audio_config = *dss_audio;
 out:
        mutex_unlock(&hd->lock);
 
diff --git a/drivers/gpu/drm/omapdrm/dss/hdmi5.c 
b/drivers/gpu/drm/omapdrm/dss/hdmi5.c
index 9571be938d81..33297d282a61 100644
--- a/drivers/gpu/drm/omapdrm/dss/hdmi5.c
+++ b/drivers/gpu/drm/omapdrm/dss/hdmi5.c
@@ -593,21 +593,16 @@ static int hdmi_audio_startup(struct device *dev,
                              void (*abort_cb)(struct device *dev))
 {
        struct omap_hdmi *hd = dev_get_drvdata(dev);
-       int ret = 0;
 
        mutex_lock(&hd->lock);
 
-       if (!hdmi_mode_has_audio(&hd->cfg) || !hd->display_enabled) {
-               ret = -EPERM;
-               goto out;
-       }
+       WARN_ON(hd->audio_abort_cb != NULL);
 
        hd->audio_abort_cb = abort_cb;
 
-out:
        mutex_unlock(&hd->lock);
 
-       return ret;
+       return 0;
 }
 
 static int hdmi_audio_shutdown(struct device *dev)
@@ -628,12 +623,14 @@ static int hdmi_audio_start(struct device *dev)
        struct omap_hdmi *hd = dev_get_drvdata(dev);
        unsigned long flags;
 
-       WARN_ON(!hdmi_mode_has_audio(&hd->cfg));
-
        spin_lock_irqsave(&hd->audio_playing_lock, flags);
 
-       if (hd->display_enabled)
+       if (hd->display_enabled) {
+               if (!hdmi_mode_has_audio(&hd->cfg))
+                       DSSERR("%s: Video mode does not support audio\n",
+                              __func__);
                hdmi_start_audio_stream(hd);
+       }
        hd->audio_playing = true;
 
        spin_unlock_irqrestore(&hd->audio_playing_lock, flags);
@@ -645,7 +642,8 @@ static void hdmi_audio_stop(struct device *dev)
        struct omap_hdmi *hd = dev_get_drvdata(dev);
        unsigned long flags;
 
-       WARN_ON(!hdmi_mode_has_audio(&hd->cfg));
+       if (!hdmi_mode_has_audio(&hd->cfg))
+               DSSERR("%s: Video mode does not support audio\n", __func__);
 
        spin_lock_irqsave(&hd->audio_playing_lock, flags);
 
@@ -664,18 +662,15 @@ static int hdmi_audio_config(struct device *dev,
 
        mutex_lock(&hd->lock);
 
-       if (!hdmi_mode_has_audio(&hd->cfg) || !hd->display_enabled) {
-               ret = -EPERM;
-               goto out;
+       if (hd->display_enabled) {
+               ret = hdmi5_audio_config(&hd->core, &hd->wp, dss_audio,
+                                        hd->cfg.vm.pixelclock);
+               if (ret)
+                       goto out;
        }
 
-       ret = hdmi5_audio_config(&hd->core, &hd->wp, dss_audio,
-                                hd->cfg.vm.pixelclock);
-
-       if (!ret) {
-               hd->audio_configured = true;
-               hd->audio_config = *dss_audio;
-       }
+       hd->audio_configured = true;
+       hd->audio_config = *dss_audio;
 out:
        mutex_unlock(&hd->lock);
 
-- 
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to