From d874aca0c8bb1b0ef2d71de585175374f1f6ef3c Mon Sep 17 00:00:00 2001 From: Denys Dmytriyenko Date: Thu, 27 Apr 2023 17:51:07 +0000 Subject: [PATCH] mpd: remove patches that were upstreamed Latest meta-openembedded/meta-multimedia updates mpd to newer version which has these 3 patches already included, remove them locally. Bug-AGL: SPEC-4768 Change-Id: Ifbb07c01f8f5cbc75935b005da473bdce3139ff0 Signed-off-by: Denys Dmytriyenko (cherry picked from commit 6d435b0bbfa3ab11e571a284c01f5735edcba49f) --- ...t-PipeWire-skip-Cancel-if-already-drained.patch | 26 --------------- ...t-PipeWire-call-pw_stream_flush-in-Cancel.patch | 35 ------------------- ...Wire-after-Cancel-refill-buffer-before-re.patch | 39 ---------------------- recipes-multimedia/musicpd/mpd_agldemo.inc | 3 -- 4 files changed, 103 deletions(-) delete mode 100644 recipes-multimedia/musicpd/files/0002-output-PipeWire-skip-Cancel-if-already-drained.patch delete mode 100644 recipes-multimedia/musicpd/files/0003-output-PipeWire-call-pw_stream_flush-in-Cancel.patch delete mode 100644 recipes-multimedia/musicpd/files/0004-output-PipeWire-after-Cancel-refill-buffer-before-re.patch diff --git a/recipes-multimedia/musicpd/files/0002-output-PipeWire-skip-Cancel-if-already-drained.patch b/recipes-multimedia/musicpd/files/0002-output-PipeWire-skip-Cancel-if-already-drained.patch deleted file mode 100644 index a01a04a02..000000000 --- a/recipes-multimedia/musicpd/files/0002-output-PipeWire-skip-Cancel-if-already-drained.patch +++ /dev/null @@ -1,26 +0,0 @@ -From 493677ff81e708133f87e15157c5dd5131adad48 Mon Sep 17 00:00:00 2001 -From: Max Kellermann -Date: Sat, 9 Jul 2022 00:53:52 +0200 -Subject: [PATCH] output/PipeWire: skip Cancel() if already drained - -Upstream-Status: Backport [https://github.com/MusicPlayerDaemon/MPD/commit/493677ff81e708133f87e15157c5dd5131adad48] -Signed-off-by: Scott Murray - ---- -diff --git a/src/output/plugins/PipeWireOutputPlugin.cxx b/src/output/plugins/PipeWireOutputPlugin.cxx -index ccdf5c0e4..7184edfa5 100644 ---- a/src/output/plugins/PipeWireOutputPlugin.cxx -+++ b/src/output/plugins/PipeWireOutputPlugin.cxx -@@ -892,6 +892,9 @@ PipeWireOutput::Cancel() noexcept - const PipeWire::ThreadLoopLock lock(thread_loop); - interrupted = false; - -+ if (drained) -+ return; -+ - ring_buffer->reset(); - } - --- -2.39.0 - diff --git a/recipes-multimedia/musicpd/files/0003-output-PipeWire-call-pw_stream_flush-in-Cancel.patch b/recipes-multimedia/musicpd/files/0003-output-PipeWire-call-pw_stream_flush-in-Cancel.patch deleted file mode 100644 index c63d86155..000000000 --- a/recipes-multimedia/musicpd/files/0003-output-PipeWire-call-pw_stream_flush-in-Cancel.patch +++ /dev/null @@ -1,35 +0,0 @@ -From 547a084c7ed95c09136159623240b7c92f6a2f5e Mon Sep 17 00:00:00 2001 -From: Max Kellermann -Date: Fri, 8 Jul 2022 23:38:45 +0200 -Subject: [PATCH] output/PipeWire: call pw_stream_flush() in Cancel() - -Clear not only MPD's ring buffer, but also libpipewire's buffers, to -avoid playing some audio from the previous song after a manual song -change. - -Fixes part 1 of https://github.com/MusicPlayerDaemon/MPD/issues/1354 - -Upstream-Status: Backport [https://github.com/MusicPlayerDaemon/MPD/commit/547a084c7ed95c09136159623240b7c92f6a2f5e] -Signed-off-by: Scott Murray - ---- -diff --git a/src/output/plugins/PipeWireOutputPlugin.cxx b/src/output/plugins/PipeWireOutputPlugin.cxx -index 7184edfa5..0f8550d41 100644 ---- a/src/output/plugins/PipeWireOutputPlugin.cxx -+++ b/src/output/plugins/PipeWireOutputPlugin.cxx -@@ -895,7 +895,12 @@ PipeWireOutput::Cancel() noexcept - if (drained) - return; - -+ /* clear MPD's ring buffer */ - ring_buffer->reset(); -+ -+ /* clear libpipewire's buffer */ -+ pw_stream_flush(stream, false); -+ drained = true; - } - - bool --- -2.39.0 - diff --git a/recipes-multimedia/musicpd/files/0004-output-PipeWire-after-Cancel-refill-buffer-before-re.patch b/recipes-multimedia/musicpd/files/0004-output-PipeWire-after-Cancel-refill-buffer-before-re.patch deleted file mode 100644 index 6f7d39420..000000000 --- a/recipes-multimedia/musicpd/files/0004-output-PipeWire-after-Cancel-refill-buffer-before-re.patch +++ /dev/null @@ -1,39 +0,0 @@ -From c8dae95eff60419fdff88d55400b6cbaacac137d Mon Sep 17 00:00:00 2001 -From: Max Kellermann -Date: Sat, 9 Jul 2022 00:59:35 +0200 -Subject: [PATCH] output/PipeWire: after Cancel(), refill buffer before - resuming playback - -Deactivate the stream in Cancel(). This fixes stuttering after a -manual song change by refilling the whole ring buffer before -reactivating the stream. - -Closes https://github.com/MusicPlayerDaemon/MPD/issues/1354 - -Upstream-Status: Backport [https://github.com/MusicPlayerDaemon/MPD/commit/c8dae95eff60419fdff88d55400b6cbaacac137d] -Signed-off-by: Scott Murray - ---- -diff --git a/src/output/plugins/PipeWireOutputPlugin.cxx b/src/output/plugins/PipeWireOutputPlugin.cxx -index 0f8550d41..16febe909 100644 ---- a/src/output/plugins/PipeWireOutputPlugin.cxx -+++ b/src/output/plugins/PipeWireOutputPlugin.cxx -@@ -901,6 +901,15 @@ PipeWireOutput::Cancel() noexcept - /* clear libpipewire's buffer */ - pw_stream_flush(stream, false); - drained = true; -+ -+ /* pause the PipeWire stream so libpipewire ceases invoking -+ the "process" callback (we have no data until our Play() -+ method gets called again); the stream will be resume by -+ Play() after the ring_buffer has been refilled */ -+ if (active) { -+ active = false; -+ pw_stream_set_active(stream, false); -+ } - } - - bool --- -2.39.0 - diff --git a/recipes-multimedia/musicpd/mpd_agldemo.inc b/recipes-multimedia/musicpd/mpd_agldemo.inc index 990fdecf5..e096e30ae 100644 --- a/recipes-multimedia/musicpd/mpd_agldemo.inc +++ b/recipes-multimedia/musicpd/mpd_agldemo.inc @@ -6,9 +6,6 @@ FILESEXTRAPATHS:prepend := "${THISDIR}/files:" DEPENDS:remove = "curl" SRC_URI += "file://0001-output-PipeWire-activate-stream-in-Drain.patch \ - file://0002-output-PipeWire-skip-Cancel-if-already-drained.patch \ - file://0003-output-PipeWire-call-pw_stream_flush-in-Cancel.patch \ - file://0004-output-PipeWire-after-Cancel-refill-buffer-before-re.patch \ file://mpd-volatiles.conf \ " -- 2.16.6