On Tue, Jan 16, 2024 at 11:11:47AM +0000, Srinivas Kandagatla wrote: > On 16/01/2024 09:38, Johan Hovold wrote: > > The current UCM configuration sets the speaker PA volume to 15 dB when > > enabling the speakers but this does not prevent the user from increasing > > the volume further. > > > > Limit the PA volume to 15 dB in the machine driver to reduce the risk of > > speaker damage until we have active speaker protection in place. > LGTM, We can get rid of this limit once we have Speaker protection inplace. There should be a userspace component for speaker protection so you'll need to limit things when that's not running.
Attachment:
signature.asc
Description: PGP signature
- References:
- [PATCH 0/7] ASoC: qcom: volume fixes and codec cleanups
- From: Johan Hovold
- [PATCH 1/7] ASoC: qcom: sc8280xp: limit speaker volumes
- From: Johan Hovold
- Re: [PATCH 1/7] ASoC: qcom: sc8280xp: limit speaker volumes
- From: Srinivas Kandagatla
- [PATCH 0/7] ASoC: qcom: volume fixes and codec cleanups
- Prev by Date: Re: [PATCH 0/2] ASoC: SOF: ipc4: Kernel log noise reduction for 6.8
- Next by Date: Re: [PATCH v3 5/5] i2c: muxes: pca954x: Allow sharing reset GPIO
- Previous by thread: Re: [PATCH 1/7] ASoC: qcom: sc8280xp: limit speaker volumes
- Next by thread: [PATCH 7/7] ASoC: codecs: msm8916-wcd-digital: drop dead gain hacks
- Index(es):