On Fri, Nov 22, 2013 at 10:29:18AM -0700, Stephen Warren wrote:
From: Stephen Warren swarren@nvidia.com
SND_SOC_DAPM_MUX() doesn't currently initialize the .mask field. This results in the mux never affecting HW, since no bits are ever set or cleared. Fix SND_SOC_DAPM_MUX() to use SND_SOC_DAPM_INIT_REG_VAL() to set up the reg, shift, on_val, and off_val fields like almost all other SND_SOC_xxx() macros. It looks like this was a "typo" in the fixed commit linked below.
Hrm. Why has nobody else noticed this? I've been doing plenty of testing that involved changing muxes... The patch and reasoning makes sense but I can't immediately see why any of the testing I've been doing recently would've worked without it since it all relies on muxes being configured to make any noise.