18 Jan
2024
18 Jan
'24
6:21 p.m.
On Thu, Jan 18, 2024 at 05:58:08PM +0100, Johan Hovold wrote:
The default PA gain is set to a pretty high level of 15 dB. Initialise the register to the minimum -3 dB level instead.
This is specifically needed to allow machine drivers to use the lowest level as a volume limit.
@@ -722,7 +722,7 @@ static struct reg_default wsa883x_defaults[] = { { WSA883X_WAVG_PER_6_7, 0x88 }, { WSA883X_WAVG_STA, 0x00 }, { WSA883X_DRE_CTL_0, 0x70 },
- { WSA883X_DRE_CTL_1, 0x08 },
- { WSA883X_DRE_CTL_1, 0x1e },
This is broken, the register defaults provided to regmap need to correspond to whatever the hardware default is since for example a register cache sync will not write back any default values (as they should already be there in the hardware). Anything like this would need to be done by writes during init.