Re: [PATCH v2 2/2] ASoC: tas2783-sdw: do not cache read-only Controls

From: Mark Brown

Date: Fri Aug 14 2026 - 11:14:44 EST


On Fri, Aug 14, 2026 at 09:40:00AM +0300, Andrey Golovko wrote:
> The read-only SDCA Controls are readings of device state, but the regmap
> caches them and tas2783_reg_default[] gives each one a placeholder, so a
> read is answered from the cache and never reaches the peripheral. The

> static bool tas2783_volatile_register(struct device *dev, u32 reg)
> {
> + if (tas2783_read_only_control(reg))
> + return true;
> +

Does this work for multibyte reads? The regmap has val_size set to 8
but some of the read only registers are multibyte. For SoundWire
val_bits doesn't really matter so you should just be able to set that to
32.

Attachment: signature.asc
Description: PGP signature