Warning: Can't synchronize with repository "(default)" (No changeset 96d22ec3fa3ef6de3ea8dc0d7d398adc9aa071cf in the repository). Look in the Trac log for more information.

Ticket #398 (closed enhancement: fixed)

Opened 4 years ago

Last modified 4 years ago

FSO should max out the handset speaker volume by default

Reported by: dkogan Owned by: mickey
Priority: major Milestone:
Component: framework/ogsmd Version:
Keywords: Cc:

Description

On my freerunner, I need to max out the speaker volume both with the calypso (AT+CLVL) and with ALSA to make phone conversations loud enough. FSO should max out the calypso volume so that the ALSA control alone is used to control volume.

Change History

comment:1 Changed 4 years ago by daniel

If I recall correctly there were some issues with this approach as it would overdrive the input for some GTA02_Ax. I too have the problem that the speaker volume is too low rather than too high, but I have encountered some phones (MP devices?) which have really high volume by default.

Anyone else know more about this situation?

comment:2 Changed 4 years ago by daniel

From an IRC conversation with PaulFertser?:

<PaulFertser?> alphaone: Joerg says that the mystic capacitors (R300[45]) come and go, motivated by some mythical gsm certification issues. When i tried to use an A7 statefile (posted by Angus on OM trac) on my A6, the earpiece sound was extremely loud. I haven't checked the components. <alphaone> PaulFertser?: Okay, thanks <PaulFertser?> alphaone: he says there's no really a way to find out whether they are in or not, BOM changing all the time without any clear traces. Anyway, earpiece volume is something that should be easily user-tweakable, imho. <PaulFertser?> alphaone: anyway, those strange caps are in the earpiece path, not in the wolfson input.

Mickey, I think I remember you tried +CLVL and it had some bad effects for you. Can we apply this fix? There are quite some phones that suffer from low volume.

comment:3 Changed 4 years ago by mickey

  • Status changed from new to closed
  • Resolution set to fixed

comment:4 Changed 4 years ago by dkogan

  • Status changed from closed to reopened
  • Resolution fixed deleted

There are 2 problems with the current implementation:

  1. channel.py now contains: c.append( "AT+CLVL=255" )

Since the framework automatically prepends "AT", this generates a command "ATAT+CLVL=255"

  1. It appears that turning on the gsm with AT+CFUN=1 resets this setting. Since the current code sets +CLVL before trying to register to the network, the setting doesn't take effect even if issue #1 above is addressed. Reopening bug.

comment:5 Changed 4 years ago by mickey

Thanks for spotting. Should both be fixed by [540f58f64045f4a55c788473da68c2102697da43], please close if you agree.

comment:6 Changed 4 years ago by dkogan

Hmmm. Not yet. Now the AT+CLVL command isn't run at all (as far as I can tell from the frameworkd.log with DEBUG logging enabled).

comment:7 Changed 4 years ago by mickey

comment:8 Changed 4 years ago by dkogan

  • Status changed from reopened to closed
  • Resolution set to fixed

It works now. Thank you, Mickey.

Note: See TracTickets for help on using tickets.