john_lee at openmoko.com
Thu Nov 27 10:51:14 CET 2008
On Thu, Nov 27, 2008 at 01:55:02AM +0100, "Marco Trevisan (Treviño)" wrote:
> Tick Chen wrote:
> Well, I've tested this implementation but there's something of bugged.
> In fact you use the Qtopia PhoneProfile "Volume" setting as the earphone
> setting, while that value is mainly used by Qtopia for the ringtone volume!
> So if you try to place a call with this code when your phone is in the
> "Vibrate Only" or "Silent" profile, as soon as the call starts, it gets
> As a workaround I've actually changed the callscreen.cpp file to use
> only if the currentProfile.volume() is not 0, but I guess that we should
> have a better implementation adding the support to "callvolume" to
> QPhoneProfileManager. The implementation should be really easy, but I
> had not time to write a patch to attach here... :(
> Ehm, then why in vendor_ficgta01.cpp currentVolumeLevel is set to 3 at
> the beginning and then to 153 (that i figure is 255*3/5)? I figure that
> the first line should be removed (not only because it is ignored, but
> also because in the next git commit the currentVolumeLevel is changed to
> 4, and maybe this should imply the update of currentVolumeLevel from 153
> to 204 [= 255*4/5]; isn't it?)
Thanks for pointing this out. Filed as ticket #2140.
More information about the devel