[Helix-client-dev] Re: RESEND: [datatype-dev] [Client-dev] [Clientapps-dev] [Nokia-private-dev]CR: Symbian Add Power Save Support to Brizo420 and head

This looks fine.

Moving IHXPowerSave definition into a separate header probably breaks 
the Intel Morrestown DSP decoding work we've done but since that project 
is idle, we can fix it later.

fxd

[email protected] wrote:
>  
>
> *From:* [email protected] 
> [mailto:[email protected]] *On Behalf Of *Lliu 
> Xiaolin (Nokia-MS/Dallas)
> *Sent:* Friday, September 10, 2010 1:31 PM
> *To:* [email protected]; 
> [email protected]; [email protected]
> *Subject:* RESEND: [datatype-dev] [Client-dev] [Clientapps-dev] CR: 
> Symbian Add Power Save Support to Brizo420 and head
>
>  
>
> *From:* [email protected] 
> [mailto:[email protected]] *On Behalf Of *Lliu 
> Xiaolin (Nokia-MS/Dallas)
> *Sent:* Tuesday, September 07, 2010 2:54 PM
> *To:* [email protected]; 
> [email protected]; [email protected]
> *Subject:* [datatype-dev] [Client-dev] [Clientapps-dev] CR: Symbian 
> Add Power Save Support to Brizo420 and head
>
>  
> "Nokia submits this code under the terms of a commercial contribution 
> agreement with Real Networks, and I am authorized to contribute this 
> code under said agreement."
>  
>  
> Modified by:  [email protected]
>  
> Reviewed by:
>  
> Date: 08/31/2010
>  
> Project: symbian_client_apps
>  
> Synopsis: Power Save
>  
> Overview: Reduce the power consumption for music playback
>  
> Solution: Helix goes to the power save mode when the following 
> conditions are met:
> -One source is available
> -local audio playback
> -Power save feature is enabled
> Symbian audio device controls the scheduler during the power save 
> mode. Audio device makes the decision when the scheduler should be 
> paused or resumed. PowerSave feature is implemented to allow the 
> playback engine to be in sleep between high water mark and low water 
> mark.
>  
>  
> Files Added:
> \common\include\hxTimelineLimit.h
> \common\include\hxPowerSave.h
>  
>  
> Files Modified:
>  
> \datatype\mdf\audio\dsp\mdfauddevice.h 
> \datatype\mdf\audio\dsp\mdfauddevice.cpp 
> \datatype\mdf\audio\dsp\mdfdevsound.h 
> \datatype\mdf\audio\dsp\mdfdevsound.cpp 
>  
> \client\audiosvc\pub\hxaudply.h 
> \client\audiosvc\hxaudply.cpp
> \client\audiosvc\pub\hxaudsec.h
> \client\audiosvc\hxaudsec.cpp
>  
> \client\core\pub\hxcleng.h 
> \client\core\pub\hxcleng.cpp
> \client\core\pub\hxplay.h
> \client\core\hxplay.cpp 
> \client\medpltfm\pub\chxmedpltfmsched.h 
> \client\medpltfm\chxmedpltfmsched.cpp 
> \client\medpltfm\chxmedpltfm.cpp
>  
> \clientapps\symbiancommon\config\R1_Mobile_4_0_Factory.cfg
> \clientapps\symbianMmf\hxmmfstatectrl.h
> \clientapps\symbianMmf\hxmmfstatectrl.cpp 
> \clientapps\symbianMmf\audiocontroller\hxmmfaudioctrl.h 
> \clientapps\symbianMmf\audiocontroller\hxmmfaudioctrl.cpp 
> \clientapps\symbianMmf\videocontroller\hxmmfctrlimpl.cpp 
>  
> \common\include\hxcore.h 
> \common\include\hxiids.h 
> \common\include\hxengin.h
>  
> \ribosome\build\umakepf\helix-client-symbian-4-integration-common.pfi
> \ribosome\build\umakepf\helix-client-symbian-4-common.pfi
>  
>  
> Image Size and Heap Use impact: minor
>  
> Module Release testing :
>  
> Test case(s) Added  : 
>  
> Memory leak : No
>  
> Platforms and Profiles Build Verified:
> helix-client-s60-52-mmf-mdf-dsp
>  
> Platforms and Profiles Functionality verified: armv5
>  
> Branch: 420Brizo, HEAD.
>  
>  
>