Commit 3beddef8 authored by Baojun Xu's avatar Baojun Xu Committed by Takashi Iwai

ALSA: hda/tas2781: fix wrong calibrated data order

Wrong calibration data order cause sound too low in some device.
Fix wrong calibrated data order, add calibration data converssion
by get_unaligned_be32() after reading from UEFI.

Fixes: 5be27f1e ("ALSA: hda/tas2781: Add tas2781 HDA driver")
Cc: <stable@vger.kernel.org>
Signed-off-by: default avatarBaojun Xu <baojun.xu@ti.com>
Link: https://patch.msgid.link/20240813043749.108-1-shenghao-ding@ti.comSigned-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent 004eb8ba
...@@ -2,10 +2,12 @@ ...@@ -2,10 +2,12 @@
// //
// TAS2781 HDA I2C driver // TAS2781 HDA I2C driver
// //
// Copyright 2023 Texas Instruments, Inc. // Copyright 2023 - 2024 Texas Instruments, Inc.
// //
// Author: Shenghao Ding <shenghao-ding@ti.com> // Author: Shenghao Ding <shenghao-ding@ti.com>
// Current maintainer: Baojun Xu <baojun.xu@ti.com>
#include <asm/unaligned.h>
#include <linux/acpi.h> #include <linux/acpi.h>
#include <linux/crc8.h> #include <linux/crc8.h>
#include <linux/crc32.h> #include <linux/crc32.h>
...@@ -519,20 +521,22 @@ static void tas2781_apply_calib(struct tasdevice_priv *tas_priv) ...@@ -519,20 +521,22 @@ static void tas2781_apply_calib(struct tasdevice_priv *tas_priv)
static const unsigned char rgno_array[CALIB_MAX] = { static const unsigned char rgno_array[CALIB_MAX] = {
0x74, 0x0c, 0x14, 0x70, 0x7c, 0x74, 0x0c, 0x14, 0x70, 0x7c,
}; };
unsigned char *data; int offset = 0;
int i, j, rc; int i, j, rc;
__be32 data;
for (i = 0; i < tas_priv->ndev; i++) { for (i = 0; i < tas_priv->ndev; i++) {
data = tas_priv->cali_data.data +
i * TASDEVICE_SPEAKER_CALIBRATION_SIZE;
for (j = 0; j < CALIB_MAX; j++) { for (j = 0; j < CALIB_MAX; j++) {
data = get_unaligned_be32(
&tas_priv->cali_data.data[offset]);
rc = tasdevice_dev_bulk_write(tas_priv, i, rc = tasdevice_dev_bulk_write(tas_priv, i,
TASDEVICE_REG(0, page_array[j], rgno_array[j]), TASDEVICE_REG(0, page_array[j], rgno_array[j]),
&(data[4 * j]), 4); (unsigned char *)&data, 4);
if (rc < 0) if (rc < 0)
dev_err(tas_priv->dev, dev_err(tas_priv->dev,
"chn %d calib %d bulk_wr err = %d\n", "chn %d calib %d bulk_wr err = %d\n",
i, j, rc); i, j, rc);
offset += 4;
} }
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment