ASoC: WM8903: Fix platform data gpio_cfg confusion
authorStephen Warren <swarren@nvidia.com>
Fri, 2 Dec 2011 22:08:37 +0000 (15:08 -0700)
committerMark Brown <broonie@opensource.wolfsonmicro.com>
Tue, 6 Dec 2011 10:29:22 +0000 (10:29 +0000)
commita0f203d384fadacba514748cd0095efeadeed96c
tree30fa69763a7eb33567b92625a81819ea01d538e0
parent102477b6380978b1532e5d1f769155026a332aff
ASoC: WM8903: Fix platform data gpio_cfg confusion

wm8903_platform_data.gpio_cfg[] was intended to be interpreted as follows:
0:       Don't touch this GPIO's configuration register
1..7fff: Write that value to the GPIO's configuration register
8000:    Write zero to the GPIO's configuration register
other:   Undefined (invalid)

The rationale is that platform data is usually global data, and a value of
zero means that the field wasn't explicitly set to anything (e.g. because
the field was new to the pdata type, and existing users weren't update to
initialize it) and hence the value zero should be ignored. 0x8000 is an
explicit way to get 0 in the register.

The code worked this way until commit 7cfe561 "ASoC: wm8903: Expose GPIOs
through gpiolib", where the behaviour was changed due to my lack of
awareness of the above rationale.

This patch reverts to the intended behaviour, and updates all in-tree users
to use the correct scheme. This also makes WM8903 consistent with other
devices that use a similar scheme.

WM8903_GPIO_NO_CONFIG is also renamed to WM8903_GPIO_CONFIG_ZERO so that
its name accurately reflects its purpose.

Signed-off-by: Stephen Warren <swarren@nvidia.com>
Cc: Olof Johansson <olof@lixom.net>
Cc: Colin Cross <ccross@android.com>
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
arch/arm/mach-tegra/board-harmony.c
arch/arm/mach-tegra/board-seaboard.c
include/sound/wm8903.h
sound/soc/codecs/wm8903.c
This page took 0.027064 seconds and 5 git commands to generate.