* [PATCH] ASoC: Fix setting update bits for WM8741_DACRMSB_ATTENUATION
@ 2011-10-03 3:09 Axel Lin
2011-10-03 13:15 ` Mark Brown
0 siblings, 1 reply; 2+ messages in thread
From: Axel Lin @ 2011-10-03 3:09 UTC (permalink / raw)
To: linux-kernel; +Cc: alsa-devel, Mark Brown, Ian Lartey, Liam Girdwood
After checking the code and datasheet, I think what we want in the second
snd_soc_update_bits call is to update WM8741_DACRMSB_ATTENUATION register
instead of WM8741_DACRLSB_ATTENUATION.
Signed-off-by: Axel Lin <axel.lin@gmail.com>
---
I don't have this hardware handy, thus I only check the the code and datasheet.
The bug is there since commit 992bee40 "ASoC: Initial WM8741 CODEC driver".
I'd appreciate if someone can test this patch.
Regards,
Axel
sound/soc/codecs/wm8741.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/sound/soc/codecs/wm8741.c b/sound/soc/codecs/wm8741.c
index 78c9e5a..a42b282 100644
--- a/sound/soc/codecs/wm8741.c
+++ b/sound/soc/codecs/wm8741.c
@@ -462,7 +462,7 @@ static int wm8741_probe(struct snd_soc_codec *codec)
WM8741_UPDATELM, WM8741_UPDATELM);
snd_soc_update_bits(codec, WM8741_DACRLSB_ATTENUATION,
WM8741_UPDATERL, WM8741_UPDATERL);
- snd_soc_update_bits(codec, WM8741_DACRLSB_ATTENUATION,
+ snd_soc_update_bits(codec, WM8741_DACRMSB_ATTENUATION,
WM8741_UPDATERM, WM8741_UPDATERM);
snd_soc_add_controls(codec, wm8741_snd_controls,
--
1.7.4.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] ASoC: Fix setting update bits for WM8741_DACRMSB_ATTENUATION
2011-10-03 3:09 [PATCH] ASoC: Fix setting update bits for WM8741_DACRMSB_ATTENUATION Axel Lin
@ 2011-10-03 13:15 ` Mark Brown
0 siblings, 0 replies; 2+ messages in thread
From: Mark Brown @ 2011-10-03 13:15 UTC (permalink / raw)
To: Axel Lin; +Cc: linux-kernel, Ian Lartey, Liam Girdwood, alsa-devel
On Mon, Oct 03, 2011 at 11:09:01AM +0800, Axel Lin wrote:
> After checking the code and datasheet, I think what we want in the second
> snd_soc_update_bits call is to update WM8741_DACRMSB_ATTENUATION register
> instead of WM8741_DACRLSB_ATTENUATION.
Applied, thanks.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2011-10-03 13:15 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-10-03 3:09 [PATCH] ASoC: Fix setting update bits for WM8741_DACRMSB_ATTENUATION Axel Lin
2011-10-03 13:15 ` Mark Brown
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).