]> code.ossystems Code Review - meta-freescale.git/commitdiff
alsa-lib: Add patch pcm-Don-t-store-the-state-for-SND_PCM_STATE_SUSPENDE.patch
authorLauren Post <lauren.post@nxp.com>
Fri, 30 Sep 2016 18:34:51 +0000 (13:34 -0500)
committerOtavio Salvador <otavio@ossystems.com.br>
Tue, 18 Oct 2016 18:26:15 +0000 (16:26 -0200)
Patch fixes the resume funcion which does not update teh dmix state

Signed-off-by: Lauren Post <lauren.post@nxp.com>
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
recipes-core/packagegroup/nativesdk-packagegroup-sdk-host.bbappend
recipes-multimedia/alsa/alsa-lib/0004-pcm-Don-t-store-the-state-for-SND_PCM_STATE_SUSPENDE.patch [new file with mode: 0755]
recipes-multimedia/alsa/alsa-lib_%.bbappend

index bd22b9b5d37577dcbd30398466884aa30d368d45..b3fdd680161c466265c909ef5c089cc8dbc61ff1 100644 (file)
@@ -3,4 +3,5 @@ RDEPENDS_${PN} += " \
     nativesdk-mxsldr \
     nativesdk-u-boot-mkimage \
     nativesdk-imx-usb-loader \
+    ${@bb.utils.contains('DISTRO_FEATURES', 'wayland', 'nativesdk-wayland', '', d)} \
 "
diff --git a/recipes-multimedia/alsa/alsa-lib/0004-pcm-Don-t-store-the-state-for-SND_PCM_STATE_SUSPENDE.patch b/recipes-multimedia/alsa/alsa-lib/0004-pcm-Don-t-store-the-state-for-SND_PCM_STATE_SUSPENDE.patch
new file mode 100755 (executable)
index 0000000..3cbd55a
--- /dev/null
@@ -0,0 +1,69 @@
+From af227bfcf02c5dd446e670f8367761a63c1b531f Mon Sep 17 00:00:00 2001
+From: Shengjiu Wang <shengjiu.wang@freescale.com>
+Date: Tue, 10 May 2016 15:09:26 +0800
+Subject: [PATCH 2/2] pcm: Don't store the state for SND_PCM_STATE_SUSPENDED
+
+The resume function don't update the dmix->state, if store SUSPENDED
+state in snd_pcm_dmix_state, the write function after resume will
+return error -ESTRPIPE, because the snd_pcm_write_areas() will check
+the state of the pcm device.
+This patch remove the store SND_PCM_STATE_SUSPENDED state operation
+for dmix,dshare,dsnoop.
+
+Upstream-Status: Pending
+
+Signed-off-by: Shengjiu Wang <shengjiu.wang@freescale.com>
+---
+ src/pcm/pcm_dmix.c   | 2 +-
+ src/pcm/pcm_dshare.c | 2 +-
+ src/pcm/pcm_dsnoop.c | 2 +-
+ 3 files changed, 3 insertions(+), 3 deletions(-)
+
+diff --git a/src/pcm/pcm_dmix.c b/src/pcm/pcm_dmix.c
+index b26a5c7..bd35d73 100644
+--- a/src/pcm/pcm_dmix.c
++++ b/src/pcm/pcm_dmix.c
+@@ -451,9 +451,9 @@ static snd_pcm_state_t snd_pcm_dmix_state(snd_pcm_t *pcm)
+       state = snd_pcm_state(dmix->spcm);
+       switch (state) {
+       case SND_PCM_STATE_XRUN:
+-      case SND_PCM_STATE_SUSPENDED:
+       case SND_PCM_STATE_DISCONNECTED:
+               dmix->state = state;
++      case SND_PCM_STATE_SUSPENDED:
+               return state;
+       default:
+               break;
+diff --git a/src/pcm/pcm_dshare.c b/src/pcm/pcm_dshare.c
+index 58e47bb..433b100 100644
+--- a/src/pcm/pcm_dshare.c
++++ b/src/pcm/pcm_dshare.c
+@@ -241,9 +241,9 @@ static snd_pcm_state_t snd_pcm_dshare_state(snd_pcm_t *pcm)
+       state = snd_pcm_state(dshare->spcm);
+       switch (state) {
+       case SND_PCM_STATE_XRUN:
+-      case SND_PCM_STATE_SUSPENDED:
+       case SND_PCM_STATE_DISCONNECTED:
+               dshare->state = state;
++      case SND_PCM_STATE_SUSPENDED:
+               return state;
+       default:
+               break;
+diff --git a/src/pcm/pcm_dsnoop.c b/src/pcm/pcm_dsnoop.c
+index 576c35b..a3b221f 100644
+--- a/src/pcm/pcm_dsnoop.c
++++ b/src/pcm/pcm_dsnoop.c
+@@ -205,9 +205,9 @@ static snd_pcm_state_t snd_pcm_dsnoop_state(snd_pcm_t *pcm)
+       state = snd_pcm_state(dsnoop->spcm);
+       switch (state) {
+       case SND_PCM_STATE_XRUN:
+-      case SND_PCM_STATE_SUSPENDED:
+       case SND_PCM_STATE_DISCONNECTED:
+               dsnoop->state = state;
++      case SND_PCM_STATE_SUSPENDED:
+               return state;
+       default:
+               break;
+-- 
+1.9.1
+
index 92b613c7e0f95af377e650ea599d7b8c3dead282..0bb2c1cdb3636fc5c737eaa89702acac7b80aa28 100644 (file)
@@ -1,8 +1,11 @@
 FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
 
-SRC_URI_append_mx6 = " file://0001-add-conf-for-multichannel-support-in-imx.patch"
-SRC_URI_append_mx6ul = " file://0001-add-conf-for-multichannel-support-in-imx.patch"
-SRC_URI_append_mx7 = " file://0001-add-conf-for-multichannel-support-in-imx.patch"
+IMX_PATCH = " file://0001-add-conf-for-multichannel-support-in-imx.patch \
+              file://0004-pcm-Don-t-store-the-state-for-SND_PCM_STATE_SUSPENDE.patch"
+
+SRC_URI_append_mx6 = "${IMX_PATCH}"
+SRC_URI_append_mx6ul = "${IMX_PATCH}"
+SRC_URI_append_mx7 = "${IMX_PATCH}"
 
 PACKAGE_ARCH_mx6 = "${MACHINE_SOCARCH}"
 PACKAGE_ARCH_mx6ul = "${MACHINE_SOCARCH}"