Skip to content

Commit b7e26c8

Browse files
matte-schwartztiwai
authored andcommitted
ALSA: hda/tas2781: Skip UEFI calibration on ASUS ROG Xbox Ally X
There is currently an issue with UEFI calibration data parsing for some TAS devices, like the ASUS ROG Xbox Ally X (RC73XA), that causes audio quality issues such as gaps in playback. Until the issue is root caused and fixed, add a quirk to skip using the UEFI calibration data and fall back to using the calibration data provided by the DSP firmware, which restores full speaker functionality on affected devices. Cc: stable@vger.kernel.org # 6.18 Link: https://lore.kernel.org/all/160aef32646c4d5498cbfd624fd683cc@ti.com/ Closes: https://lore.kernel.org/all/0ba100d0-9b6f-4a3b-bffa-61abe1b46cd5@linux.dev/ Suggested-by: Baojun Xu <baojun.xu@ti.com> Signed-off-by: Matthew Schwartz <matthew.schwartz@linux.dev> Reviewed-by: Antheas Kapenekakis <lkml@antheas.dev> Link: https://patch.msgid.link/20260108093650.1142176-1-matthew.schwartz@linux.dev Signed-off-by: Takashi Iwai <tiwai@suse.de>
1 parent 47c27c9 commit b7e26c8

1 file changed

Lines changed: 12 additions & 1 deletion

File tree

sound/hda/codecs/side-codecs/tas2781_hda_i2c.c

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,6 +60,7 @@ struct tas2781_hda_i2c_priv {
6060
int (*save_calibration)(struct tas2781_hda *h);
6161

6262
int hda_chip_id;
63+
bool skip_calibration;
6364
};
6465

6566
static int tas2781_get_i2c_res(struct acpi_resource *ares, void *data)
@@ -491,7 +492,8 @@ static void tasdevice_dspfw_init(void *context)
491492
/* If calibrated data occurs error, dsp will still works with default
492493
* calibrated data inside algo.
493494
*/
494-
hda_priv->save_calibration(tas_hda);
495+
if (!hda_priv->skip_calibration)
496+
hda_priv->save_calibration(tas_hda);
495497
}
496498

497499
static void tasdev_fw_ready(const struct firmware *fmw, void *context)
@@ -548,6 +550,7 @@ static int tas2781_hda_bind(struct device *dev, struct device *master,
548550
void *master_data)
549551
{
550552
struct tas2781_hda *tas_hda = dev_get_drvdata(dev);
553+
struct tas2781_hda_i2c_priv *hda_priv = tas_hda->hda_priv;
551554
struct hda_component_parent *parent = master_data;
552555
struct hda_component *comp;
553556
struct hda_codec *codec;
@@ -573,6 +576,14 @@ static int tas2781_hda_bind(struct device *dev, struct device *master,
573576
break;
574577
}
575578

579+
/*
580+
* Using ASUS ROG Xbox Ally X (RC73XA) UEFI calibration data
581+
* causes audio dropouts during playback, use fallback data
582+
* from DSP firmware as a workaround.
583+
*/
584+
if (codec->core.subsystem_id == 0x10431384)
585+
hda_priv->skip_calibration = true;
586+
576587
pm_runtime_get_sync(dev);
577588

578589
comp->dev = dev;

0 commit comments

Comments
 (0)