[v1] ALSA: hda/tas2781: Fix index issue in tas2781 hda SPI driver

Commit Message

Baojun Xu Feb. 14, 2025, 1:30 a.m. UTC

Correct wrong mask for device index.

Signed-off-by: Baojun Xu <[e-mail]>
---
 sound/pci/hda/tas2781_spi_fwlib.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

Comments

Takashi Iwai Feb. 14, 2025, 7:40 a.m. UTC | #1

On Fri, 14 Feb 2025 02:30:21 +0100,
Baojun Xu wrote:
> 
> Correct wrong mask for device index.
> 
> Signed-off-by: Baojun Xu <[e-mail]>

Applied now with Fixes tag to commit bb5f86ea50ff ("ALSA: hda/tas2781:
Add tas2781 hda SPI driver").

Please put the proper Fix tag at the next time.


thanks,

Takashi

Patch

diff --git a/sound/pci/hda/tas2781_spi_fwlib.c b/sound/pci/hda/tas2781_spi_fwlib.c
index 0e2acbc3c900..131d9a77d140 100644
--- a/sound/pci/hda/tas2781_spi_fwlib.c
+++ b/sound/pci/hda/tas2781_spi_fwlib.c
@@ -2,7 +2,7 @@ 
 //
 // TAS2781 HDA SPI driver
 //
-// Copyright 2024 Texas Instruments, Inc.
+// Copyright 2024-2025 Texas Instruments, Inc.
 //
 // Author: Baojun Xu <[e-mail]>
 
@@ -771,19 +771,19 @@  static int tasdevice_process_block(void *context, unsigned char *data,
 	switch (subblk_typ) {
 	case TASDEVICE_CMD_SING_W:
 		subblk_offset = tasdevice_single_byte_wr(tas_priv,
-			dev_idx & 0x4f, data, sublocksize);
+			dev_idx & 0x3f, data, sublocksize);
 		break;
 	case TASDEVICE_CMD_BURST:
 		subblk_offset = tasdevice_burst_wr(tas_priv,
-			dev_idx & 0x4f, data, sublocksize);
+			dev_idx & 0x3f, data, sublocksize);
 		break;
 	case TASDEVICE_CMD_DELAY:
 		subblk_offset = tasdevice_delay(tas_priv,
-			dev_idx & 0x4f, data, sublocksize);
+			dev_idx & 0x3f, data, sublocksize);
 		break;
 	case TASDEVICE_CMD_FIELD_W:
 		subblk_offset = tasdevice_field_wr(tas_priv,
-			dev_idx & 0x4f, data, sublocksize);
+			dev_idx & 0x3f, data, sublocksize);
 		break;
 	default:
 		subblk_offset = 2;