[QT Extended] Observersation on QtE 4.4.2

Chris Samuel chris at csamuel.org
Sat Nov 8 15:00:09 CET 2008


On Fri, 7 Nov 2008 11:40:15 pm Tha_Man wrote:

> According to Lorn Potter, it does. And as you wrote earlier in this thread,
> it only appears to send these commands with outgoing calls, since incoming
> calls still have the echo (and the phone set to handsfree/speaker instead
> of handset).

Yup, just seen that in the QT Extended sources in:

  devices/neo/src/plugins/phonevendors/neo/vendor_neo.cpp

Looking at the code in that file I'm wondering if a better place to put the
command to set the echo suppression is in NeoSimInfo::requestIdentity()
as that seems to be called occasionally to get the SIM info.

Alternatively it could go into the NeoCallProvider::cpiNotification() code 
after it calls hangupRemote() to set it ready for the next call and make sure 
it gets set in NeoModemService::initialize() ready for the first use.

The reason I say that is because the original patch put this into the QTE 
libraries rather than in the Neo specific code.

> I'm curious about the alsa-settings that were posted by Kishore, what do
> they exactly fix? As I understand it would fix incoming calls being set to
> speaker (instead of handset), right?

Apparently so, as well as reducing the maximum volume on the handset.

Thing is looking at the diff of the original and fixed versions I can't see how 
it fixes it to use the handset rather than the speakerphone!

--- gsmhandset.state.orig       Wed Sep  3 10:14:56 2008
+++ gsmhandset.state    Fri Nov  7 06:55:33 2008
@@ -36,8 +36,8 @@
                comment.range '0 - 127'
                iface MIXER
                name 'Speaker Playback Volume'
-               value.0 120
-               value.1 120
+               value.0 100
+               value.1 100
        }
        control.5 {
                comment.access 'read write'
@@ -46,7 +46,7 @@
                comment.range '0 - 127'
                iface MIXER
                name 'Mono Playback Volume'
-               value 103
+               value 85
        }
        control.6 {
                comment.access 'read write'
@@ -456,7 +456,7 @@
                comment.range '0 - 3'
                iface MIXER
                name 'Mic2 Capture Volume'
-               value 0
+               value 1
        }
        control.49 {
                comment.access 'read write'


cheers,
Chris
-- 
 Chris Samuel  :  http://www.csamuel.org/  :  Melbourne, VIC

This email may come with a PGP signature as a file. Do not panic.
For more info see: http://en.wikipedia.org/wiki/OpenPGP

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 481 bytes
Desc: This is a digitally signed message part.
Url : http://lists.openmoko.org/pipermail/community/attachments/20081109/f620e692/attachment.pgp 


More information about the community mailing list