* [PATCH 4/5] ALSA: cs5535audio: fix ACC_BM[x]_CMD register handling
@ 2007-08-30 3:30 Andres Salomon
2007-08-30 14:45 ` Jordan Crouse
0 siblings, 1 reply; 2+ messages in thread
From: Andres Salomon @ 2007-08-30 3:30 UTC (permalink / raw)
To: jayakumar.alsa; +Cc: akpm, jordan.crouse, linux-kernel, info-linux
ACcording to 6.3.2.7 of the cs5535/cs5536 data sheets, the ACC_BM[x]_CMD
registers are only 8 bits wide. This driver treats them as 32 bits wide,
and also has bits in the wrong place. Simple fix to the definitions.
Signed-off-by: Andres Salomon <dilinger@debian.org>
---
sound/pci/cs5535audio/cs5535audio.h | 10 +++++-----
1 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/sound/pci/cs5535audio/cs5535audio.h b/sound/pci/cs5535audio/cs5535audio.h
index c7a2044..516219a 100644
--- a/sound/pci/cs5535audio/cs5535audio.h
+++ b/sound/pci/cs5535audio/cs5535audio.h
@@ -62,11 +62,11 @@
#define EOP (1<<0)
#define BM_EOP_ERR (1<<1)
/* ACC_BMX_CTL */
-#define BM_CTL_EN 0x00000001
-#define BM_CTL_PAUSE 0x00000011
-#define BM_CTL_DIS 0x00000000
-#define BM_CTL_BYTE_ORD_LE 0x00000000
-#define BM_CTL_BYTE_ORD_BE 0x00000100
+#define BM_CTL_EN 0x01
+#define BM_CTL_PAUSE 0x03
+#define BM_CTL_DIS 0x00
+#define BM_CTL_BYTE_ORD_LE 0x00
+#define BM_CTL_BYTE_ORD_BE 0x04
/* cs5535 specific ac97 codec register defines */
#define CMD_MASK 0xFF00FFFF
#define CMD_NEW 0x00010000
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: ALSA: cs5535audio: fix ACC_BM[x]_CMD register handling
2007-08-30 3:30 [PATCH 4/5] ALSA: cs5535audio: fix ACC_BM[x]_CMD register handling Andres Salomon
@ 2007-08-30 14:45 ` Jordan Crouse
0 siblings, 0 replies; 2+ messages in thread
From: Jordan Crouse @ 2007-08-30 14:45 UTC (permalink / raw)
To: Andres Salomon; +Cc: jayakumar.alsa, akpm, linux-kernel, info-linux
On 29/08/07 23:30 -0400, Andres Salomon wrote:
>
> ACcording to 6.3.2.7 of the cs5535/cs5536 data sheets, the ACC_BM[x]_CMD
> registers are only 8 bits wide. This driver treats them as 32 bits wide,
> and also has bits in the wrong place. Simple fix to the definitions.
>
> Signed-off-by: Andres Salomon <dilinger@debian.org>
Acked-by: Jordan Crouse <jordan.crouse@amd.com>
> ---
>
> sound/pci/cs5535audio/cs5535audio.h | 10 +++++-----
> 1 files changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/sound/pci/cs5535audio/cs5535audio.h b/sound/pci/cs5535audio/cs5535audio.h
> index c7a2044..516219a 100644
> --- a/sound/pci/cs5535audio/cs5535audio.h
> +++ b/sound/pci/cs5535audio/cs5535audio.h
> @@ -62,11 +62,11 @@
> #define EOP (1<<0)
> #define BM_EOP_ERR (1<<1)
> /* ACC_BMX_CTL */
> -#define BM_CTL_EN 0x00000001
> -#define BM_CTL_PAUSE 0x00000011
> -#define BM_CTL_DIS 0x00000000
> -#define BM_CTL_BYTE_ORD_LE 0x00000000
> -#define BM_CTL_BYTE_ORD_BE 0x00000100
> +#define BM_CTL_EN 0x01
> +#define BM_CTL_PAUSE 0x03
> +#define BM_CTL_DIS 0x00
> +#define BM_CTL_BYTE_ORD_LE 0x00
> +#define BM_CTL_BYTE_ORD_BE 0x04
> /* cs5535 specific ac97 codec register defines */
> #define CMD_MASK 0xFF00FFFF
> #define CMD_NEW 0x00010000
>
>
--
Jordan Crouse
Systems Software Development Engineer
Advanced Micro Devices, Inc.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2007-08-30 14:45 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-08-30 3:30 [PATCH 4/5] ALSA: cs5535audio: fix ACC_BM[x]_CMD register handling Andres Salomon
2007-08-30 14:45 ` Jordan Crouse
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox