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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Sajeesh,

Thanks for trying to contribute to PulseAudio. While going through old 
patches I found this one, but as Pierre-Louis says below, I'm not sure 
how this would reduce power consumption.

Could you explain? Or we could drop your patch in case you changed your 
mind.

Thanks,

On 2014-08-05 23:59, Pierre-Louis Bossart wrote:
> On 8/5/14, 12:35 AM, Sajeesh Sidharthan wrote:
>> ---
>>   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 */
>
> Why would this reduce power consumption, one would think that you want
> to buffer up by specifying a large latency. If you disable timers you'll
> have to force a specific buffer size for this sink.
> Thanks,
> -Pierre
>
> _______________________________________________
> pulseaudio-discuss mailing list
> pulseaudio-discuss at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss
>

-- 
David Henningsson, Canonical Ltd.
https://launchpad.net/~diwic


[Index of Archives]     [Linux Audio Users]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux