Title: Samsung Enterprise Portal mySingle

Hi,

I have disabled timer scheduling for a2dp playback with this patch to reduce power consumption.

Could you please review this change and let me know if it can cause any side effect.

If its worth giving an option to disable timer scheduling similar to alsa-sink ( tsched param ), I would like to do the same.

Thank You.

Sajeesh

 

------- Original Message -------

Sender : Sajeesh Sidharthan<[email protected]> Chief Engineer/SRI-Bangalore-MM & Applications/Samsung Electronics

Date : Aug 05, 2014 14:35 (GMT+09:00)

Title : [pulseaudio-discuss] [PATCH] modules: Disable timer scheduling for a2dp playback to reduce power consumption.

 

---
src/modules/bluetooth/module-bluez5-device.c |    8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/modules/bluetooth/module-bluez5-device.c b/src/modules/bluetooth/module-bluez5-device.c
index 57b2791..eda7a9d 100644
--- a/src/modules/bluetooth/module-bluez5-device.c
+++ b/src/modules/bluetooth/module-bluez5-device.c
@@ -1170,10 +1170,10 @@ static void thread_func(void *userdata) {
                                     a2dp_reduce_bitpool(u);
                             }
                         }
-
-                        do_write = 1;
-                        pending_read_bytes = 0;
                     }
+
+                    do_write = 1;
+                    pending_read_bytes = 0;
                 }

                 if (writable && do_write > 0) {
@@ -1208,7 +1208,7 @@ static void thread_func(void *userdata) {
                         sleep_for = PA_USEC_PER_MSEC * 500;

                     pa_rtpoll_set_timer_relative(u->rtpoll, sleep_for);
-                    disable_timer = false;
+                    /* disable_timer = false; *//* Disable timer to reduce power consumption */
                 }
             }
         }
--
1.7.9.5

_______________________________________________
pulseaudio-discuss mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss

 

 

_______________________________________________
pulseaudio-discuss mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss

Reply via email to