From nobody Mon Oct 6 17:09:08 2025 Received: from fllvem-ot03.ext.ti.com (fllvem-ot03.ext.ti.com [198.47.19.245]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6A4233C01; Fri, 18 Jul 2025 09:45:57 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=198.47.19.245 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1752831959; cv=none; b=k7HjXcvtRMHeCGhrpM13Y6fwBhCEyrLGm0sQD+ANZJgypyyXu5QL3YBDmgUbFiXdGdv/VaxFkMgVBxBGr1No0ERHfR1yrH5SrhdzlTJhh8G6eo0teVvxF+tVVpPwJZIYKV0/yRx+i7veSAJZ+1eoORUa4GNh6TA3cVs0mqb8m90= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1752831959; c=relaxed/simple; bh=pY0MV4mpYdYjPZWc2VFKwAFPw4Xd+79vKStlUueZr5k=; h=From:To:CC:Subject:Date:Message-ID:MIME-Version:Content-Type; b=XYm9WsZL+uSEQ9RTYtIYbTgwGoxVVvR7VYIAe/HGoEnqyL1TFs4i41Q+KsPFjqBxzwvyQtsm5voLjZL0fbeA88xl9mZvLHvZCLszi6Q2QgmPxYpDMZPFcb/3WyniaQuZVgGQa0igtfMQZGGfmpYLOAF7b7PebCihHjG+uvBhqVE= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=ti.com; spf=pass smtp.mailfrom=ti.com; dkim=pass (1024-bit key) header.d=ti.com header.i=@ti.com header.b=xkTl9IOt; arc=none smtp.client-ip=198.47.19.245 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=ti.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=ti.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=ti.com header.i=@ti.com header.b="xkTl9IOt" Received: from lelvem-sh02.itg.ti.com ([10.180.78.226]) by fllvem-ot03.ext.ti.com (8.15.2/8.15.2) with ESMTP id 56I9jSEE226499; Fri, 18 Jul 2025 04:45:28 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1752831928; bh=J0S1VbKJLH31RiIy3lX0zXmUf3R0ULFs66hqGBDEbTo=; h=From:To:CC:Subject:Date; b=xkTl9IOtCTPKVLfekzPLQh82tamAXKw+mlv+TVDSKT5DHEH/8XtFK84TaIosg3HtN 7Bj1j0GAi50EjTmNDnunu2C4u0Hs7rU3rs4QW18kMWHyXQFGdbqI4HGfbTB5nOg+AT TAcWt/bMA9hMosLHrPiHNLgXhLmN6NDmswlxty9U= Received: from DFLE112.ent.ti.com (dfle112.ent.ti.com [10.64.6.33]) by lelvem-sh02.itg.ti.com (8.18.1/8.18.1) with ESMTPS id 56I9jSaW3421028 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-SHA256 bits=128 verify=FAIL); Fri, 18 Jul 2025 04:45:28 -0500 Received: from DFLE114.ent.ti.com (10.64.6.35) by DFLE112.ent.ti.com (10.64.6.33) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.55; Fri, 18 Jul 2025 04:45:27 -0500 Received: from lelvem-mr05.itg.ti.com (10.180.75.9) by DFLE114.ent.ti.com (10.64.6.35) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.55 via Frontend Transport; Fri, 18 Jul 2025 04:45:27 -0500 Received: from lelvem-mr05.itg.ti.com ([10.250.165.138]) by lelvem-mr05.itg.ti.com (8.18.1/8.18.1) with ESMTP id 56I9jLCH3643100; Fri, 18 Jul 2025 04:45:22 -0500 From: Baojun Xu To: CC: , , , , , <13916275206@139.com>, , , , , Subject: [PATCH v1] ALSA: hda: Add TAS2770 support Date: Fri, 18 Jul 2025 17:44:54 +0800 Message-ID: <20250718094454.26574-1-baojun.xu@ti.com> X-Mailer: git-send-email 2.43.0.windows.1 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-C2ProcessedOrg: 333ef613-75bf-4e12-a4b1-8e3623f5dcea Content-Type: text/plain; charset="utf-8" Add TAS2770 support in TI's HDA driver. And add hda_chip_id for more product support in the future. Separated DSP and non-DSP in firmware load function. Signed-off-by: Baojun Xu --- include/sound/tas2770-tlv.h | 23 +++++ sound/pci/hda/tas2781_hda_i2c.c | 157 ++++++++++++++++++++++---------- 2 files changed, 133 insertions(+), 47 deletions(-) create mode 100644 include/sound/tas2770-tlv.h diff --git a/include/sound/tas2770-tlv.h b/include/sound/tas2770-tlv.h new file mode 100644 index 000000000000..c0bd495b4a07 --- /dev/null +++ b/include/sound/tas2770-tlv.h @@ -0,0 +1,23 @@ +/* SPDX-License-Identifier: GPL-2.0 */ +// +// ALSA SoC Texas Instruments TAS2770 Audio Smart Amplifier +// +// Copyright (C) 2025 Texas Instruments Incorporated +// https://www.ti.com +// +// The TAS2770 hda driver implements for one, two, or even multiple +// TAS2770 chips. +// +// Author: Baojun Xu +// + +#ifndef __TAS2770_TLV_H__ +#define __TAS2770_TLV_H__ + +#define TAS2770_DVC_LEVEL TASDEVICE_REG(0x0, 0x0, 0x17) +#define TAS2770_AMP_LEVEL TASDEVICE_REG(0x0, 0x0, 0x03) + +static const __maybe_unused DECLARE_TLV_DB_SCALE(tas2770_dvc_tlv, 1650, 50= , 0); +static const __maybe_unused DECLARE_TLV_DB_SCALE(tas2770_amp_tlv, 1100, 50= , 0); + +#endif diff --git a/sound/pci/hda/tas2781_hda_i2c.c b/sound/pci/hda/tas2781_hda_i2= c.c index d91eed9f7804..2fdff0f88af7 100644 --- a/sound/pci/hda/tas2781_hda_i2c.c +++ b/sound/pci/hda/tas2781_hda_i2c.c @@ -24,6 +24,7 @@ #include #include #include +#include #include =20 #include "hda_local.h" @@ -45,9 +46,18 @@ #define TAS2563_CAL_TLIM TASDEVICE_REG(0, 0x10, 0x14) #define TAS2563_CAL_R0 TASDEVICE_REG(0, 0x0f, 0x34) =20 +enum device_chip_id { + HDA_TAS2563, + HDA_TAS2770, + HDA_TAS2781, + HDA_OTHERS +}; + struct tas2781_hda_i2c_priv { struct snd_kcontrol *snd_ctls[2]; int (*save_calibration)(struct tas2781_hda *h); + + int hda_chip_id; }; =20 static int tas2781_get_i2c_res(struct acpi_resource *ares, void *data) @@ -246,6 +256,15 @@ static int tas2781_force_fwload_put(struct snd_kcontro= l *kcontrol, return change; } =20 +static const struct snd_kcontrol_new tas2770_snd_controls[] =3D { + ACARD_SINGLE_RANGE_EXT_TLV("Speaker Analog Volume", TAS2770_AMP_LEVEL, + 0, 0, 20, 0, tas2781_amp_getvol, + tas2781_amp_putvol, tas2770_amp_tlv), + ACARD_SINGLE_RANGE_EXT_TLV("Speaker Digital Volume", TAS2770_DVC_LEVEL, + 0, 0, 31, 0, tas2781_amp_getvol, + tas2781_amp_putvol, tas2770_dvc_tlv), +}; + static const struct snd_kcontrol_new tas2781_snd_controls[] =3D { ACARD_SINGLE_RANGE_EXT_TLV("Speaker Analog Gain", TAS2781_AMP_LEVEL, 1, 0, 20, 0, tas2781_amp_getvol, @@ -254,7 +273,7 @@ static const struct snd_kcontrol_new tas2781_snd_contro= ls[] =3D { tas2781_force_fwload_get, tas2781_force_fwload_put), }; =20 -static const struct snd_kcontrol_new tas2781_prof_ctrl =3D { +static const struct snd_kcontrol_new tasdevice_prof_ctrl =3D { .name =3D "Speaker Profile Id", .iface =3D SNDRV_CTL_ELEM_IFACE_CARD, .info =3D tasdevice_info_profile, @@ -262,7 +281,7 @@ static const struct snd_kcontrol_new tas2781_prof_ctrl = =3D { .put =3D tasdevice_set_profile_id, }; =20 -static const struct snd_kcontrol_new tas2781_dsp_prog_ctrl =3D { +static const struct snd_kcontrol_new tasdevice_dsp_prog_ctrl =3D { .name =3D "Speaker Program Id", .iface =3D SNDRV_CTL_ELEM_IFACE_CARD, .info =3D tasdevice_info_programs, @@ -270,7 +289,7 @@ static const struct snd_kcontrol_new tas2781_dsp_prog_c= trl =3D { .put =3D tasdevice_program_put, }; =20 -static const struct snd_kcontrol_new tas2781_dsp_conf_ctrl =3D { +static const struct snd_kcontrol_new tasdevice_dsp_conf_ctrl =3D { .name =3D "Speaker Config Id", .iface =3D SNDRV_CTL_ELEM_IFACE_CARD, .info =3D tasdevice_info_config, @@ -379,44 +398,15 @@ static void tas2781_hda_remove_controls(struct tas278= 1_hda *tas_hda) snd_ctl_remove(codec->card, tas_hda->prof_ctl); } =20 -static void tasdev_fw_ready(const struct firmware *fmw, void *context) +static void tasdevice_dspfw_init(void *context) { struct tasdevice_priv *tas_priv =3D context; struct tas2781_hda *tas_hda =3D dev_get_drvdata(tas_priv->dev); struct tas2781_hda_i2c_priv *hda_priv =3D tas_hda->hda_priv; struct hda_codec *codec =3D tas_priv->codec; - int i, ret, spk_id; - - pm_runtime_get_sync(tas_priv->dev); - mutex_lock(&tas_priv->codec_lock); - - ret =3D tasdevice_rca_parser(tas_priv, fmw); - if (ret) - goto out; - - tas_hda->prof_ctl =3D snd_ctl_new1(&tas2781_prof_ctrl, tas_priv); - ret =3D snd_ctl_add(codec->card, tas_hda->prof_ctl); - if (ret) { - dev_err(tas_priv->dev, - "Failed to add KControl %s =3D %d\n", - tas2781_prof_ctrl.name, ret); - goto out; - } - - for (i =3D 0; i < ARRAY_SIZE(tas2781_snd_controls); i++) { - hda_priv->snd_ctls[i] =3D snd_ctl_new1(&tas2781_snd_controls[i], - tas_priv); - ret =3D snd_ctl_add(codec->card, hda_priv->snd_ctls[i]); - if (ret) { - dev_err(tas_priv->dev, - "Failed to add KControl %s =3D %d\n", - tas2781_snd_controls[i].name, ret); - goto out; - } - } + int ret, spk_id; =20 tasdevice_dsp_remove(tas_priv); - tas_priv->fw_state =3D TASDEVICE_DSP_FW_PENDING; if (tas_priv->speaker_id !=3D NULL) { // Speaker id need to be checked for ASUS only. @@ -442,48 +432,101 @@ static void tasdev_fw_ready(const struct firmware *f= mw, void *context) dev_err(tas_priv->dev, "dspfw load %s error\n", tas_priv->coef_binaryname); tas_priv->fw_state =3D TASDEVICE_DSP_FW_FAIL; - goto out; + return; } =20 - tas_hda->dsp_prog_ctl =3D snd_ctl_new1(&tas2781_dsp_prog_ctrl, + tas_hda->dsp_prog_ctl =3D snd_ctl_new1(&tasdevice_dsp_prog_ctrl, tas_priv); ret =3D snd_ctl_add(codec->card, tas_hda->dsp_prog_ctl); if (ret) { dev_err(tas_priv->dev, "Failed to add KControl %s =3D %d\n", - tas2781_dsp_prog_ctrl.name, ret); - goto out; + tasdevice_dsp_prog_ctrl.name, ret); + return; } =20 - tas_hda->dsp_conf_ctl =3D snd_ctl_new1(&tas2781_dsp_conf_ctrl, + tas_hda->dsp_conf_ctl =3D snd_ctl_new1(&tasdevice_dsp_conf_ctrl, tas_priv); ret =3D snd_ctl_add(codec->card, tas_hda->dsp_conf_ctl); if (ret) { dev_err(tas_priv->dev, "Failed to add KControl %s =3D %d\n", - tas2781_dsp_conf_ctrl.name, ret); - goto out; + tasdevice_dsp_conf_ctrl.name, ret); + return; } - tas_priv->fw_state =3D TASDEVICE_DSP_FW_ALL_OK; + tasdevice_prmg_load(tas_priv, 0); if (tas_priv->fmw->nr_programs > 0) tas_priv->cur_prog =3D 0; if (tas_priv->fmw->nr_configurations > 0) tas_priv->cur_conf =3D 0; - /* If calibrated data occurs error, dsp will still works with default * calibrated data inside algo. */ hda_priv->save_calibration(tas_hda); +} + +static void tasdev_add_kcontrols(struct tasdevice_priv *tas_priv, + struct snd_kcontrol **ctls, struct hda_codec *codec, + const struct snd_kcontrol_new *tas_snd_ctrls, int num_ctls) +{ + int i, ret; + + for (i =3D 0; i < num_ctls; i++) { + ctls[i] =3D snd_ctl_new1( + &tas_snd_ctrls[i], tas_priv); + ret =3D snd_ctl_add(codec->card, ctls[i]); + if (ret) { + dev_err(tas_priv->dev, + "Failed to add KControl %s =3D %d\n", + tas_snd_ctrls[i].name, ret); + break; + } + } +} + +static void tasdev_fw_ready(const struct firmware *fmw, void *context) +{ + struct tasdevice_priv *tas_priv =3D context; + struct tas2781_hda *tas_hda =3D dev_get_drvdata(tas_priv->dev); + struct tas2781_hda_i2c_priv *hda_priv =3D tas_hda->hda_priv; + struct hda_codec *codec =3D tas_priv->codec; + int ret; + + pm_runtime_get_sync(tas_priv->dev); + mutex_lock(&tas_priv->codec_lock); =20 - tasdevice_tuning_switch(tas_hda->priv, 0); - tas_hda->priv->playback_started =3D true; + ret =3D tasdevice_rca_parser(tas_priv, fmw); + if (ret) + goto out; + + tas_priv->fw_state =3D TASDEVICE_RCA_FW_OK; + tasdev_add_kcontrols(tas_priv, &tas_hda->prof_ctl, codec, + &tasdevice_prof_ctrl, 1); + + switch (hda_priv->hda_chip_id) { + case HDA_TAS2770: + tasdev_add_kcontrols(tas_priv, hda_priv->snd_ctls, codec, + &tas2770_snd_controls[0], + ARRAY_SIZE(tas2770_snd_controls)); + break; + case HDA_TAS2781: + tasdev_add_kcontrols(tas_priv, hda_priv->snd_ctls, codec, + &tas2781_snd_controls[0], + ARRAY_SIZE(tas2781_snd_controls)); + tasdevice_dspfw_init(context); + break; + case HDA_TAS2563: + tasdevice_dspfw_init(context); + break; + default: + break; + } =20 out: mutex_unlock(&tas_hda->priv->codec_lock); release_firmware(fmw); - pm_runtime_mark_last_busy(tas_hda->dev); pm_runtime_put_autosuspend(tas_hda->dev); } =20 @@ -584,15 +627,34 @@ static int tas2781_hda_i2c_probe(struct i2c_client *c= lt) return -ENOMEM; =20 if (strstr(dev_name(&clt->dev), "TIAS2781")) { + /* + * TAS2781, integrated on-chip DSP with + * global I2C address supported. + */ device_name =3D "TIAS2781"; + hda_priv->hda_chip_id =3D HDA_TAS2781; hda_priv->save_calibration =3D tas2781_save_calibration; tas_hda->priv->global_addr =3D TAS2781_GLOBAL_ADDR; + } else if (strstarts(dev_name(&clt->dev), "i2c-TXNW2770")) { + /* + * TAS2770, has no on-chip DSP, so no calibration data + * required; has no global I2C address supported. + */ + device_name =3D "TXNW2770"; + hda_priv->hda_chip_id =3D HDA_TAS2770; } else if (strstr(dev_name(&clt->dev), "INT8866")) { + /* + * TAS2563, integrated on-chip DSP with + * global I2C address supported. + */ device_name =3D "INT8866"; + hda_priv->hda_chip_id =3D HDA_TAS2563; hda_priv->save_calibration =3D tas2563_save_calibration; tas_hda->priv->global_addr =3D TAS2563_GLOBAL_ADDR; - } else + } else { return -ENODEV; + } + =20 tas_hda->priv->irq =3D clt->irq; ret =3D tas2781_read_acpi(tas_hda->priv, device_name); @@ -724,6 +786,7 @@ static const struct i2c_device_id tas2781_hda_i2c_id[] = =3D { static const struct acpi_device_id tas2781_acpi_hda_match[] =3D { {"TIAS2781", 0 }, {"INT8866", 0 }, + {"TXNW2770", 0 }, {} }; MODULE_DEVICE_TABLE(acpi, tas2781_acpi_hda_match); --=20 2.34.1