From: Niranjan H Y <niranjan.hy@ti.com>
To: <linux-kernel@vger.kernel.org>, <linux-gpio@vger.kernel.org>,
<linux-sound@vger.kernel.org>, <devicetree@vger.kernel.org>
Cc: <lee@kernel.org>, <linusw@kernel.org>, <lgirdwood@gmail.com>,
<broonie@kernel.org>, <perex@perex.cz>, <tiwai@suse.com>,
<robh@kernel.org>, <krzk+dt@kernel.org>, <conor+dt@kernel.org>,
<niranjan.hy@ti.com>, <nb@tipi-net.de>, <navada@ti.com>,
<v-hampiholi@ti.com>, <sandeepk@ti.com>, <baojun.xu@ti.com>,
<shenghao-ding@ti.com>
Subject: [PATCH v1 8/8] ASoC: pcm6240: remove support for taac5x1x family
Date: Fri, 13 Mar 2026 00:18:33 +0530 [thread overview]
Message-ID: <20260312184833.263-9-niranjan.hy@ti.com> (raw)
In-Reply-To: <20260312184833.263-1-niranjan.hy@ti.com>
Remove TAC5X1X support from the legacy pcm6240 codec driver. The TAC5X1X
family now uses a dedicated MFD-based driver for better code organization
and maintainability.
Signed-off-by: Niranjan H Y <niranjan.hy@ti.com>
---
sound/soc/codecs/pcm6240.c | 126 +------------------------------------
sound/soc/codecs/pcm6240.h | 4 --
2 files changed, 3 insertions(+), 127 deletions(-)
diff --git a/sound/soc/codecs/pcm6240.c b/sound/soc/codecs/pcm6240.c
index 78b21fbfad50..b1902b9e9f32 100644
--- a/sound/soc/codecs/pcm6240.c
+++ b/sound/soc/codecs/pcm6240.c
@@ -44,10 +44,6 @@ static const struct i2c_device_id pcmdevice_i2c_id[] = {
{ "pcmd3140", PCMD3140 },
{ "pcmd3180", PCMD3180 },
{ "pcmd512x", PCMD512X },
- { "taa5212", TAA5212 },
- { "taa5412", TAA5412 },
- { "tad5212", TAD5212 },
- { "tad5412", TAD5412 },
{}
};
MODULE_DEVICE_TABLE(i2c, pcmdevice_i2c_id);
@@ -442,60 +438,6 @@ static const struct pcmdevice_mixer_control pcmd3180_fine_gain_ctl[] = {
}
};
-static const struct pcmdevice_mixer_control taa5412_digi_vol_ctl[] = {
- {
- .shift = 0,
- .reg = TAA5412_REG_CH1_DIGITAL_VOLUME,
- .max = 0xff,
- .invert = 0,
- },
- {
- .shift = 0,
- .reg = TAA5412_REG_CH2_DIGITAL_VOLUME,
- .max = 0xff,
- .invert = 0,
- },
- {
- .shift = 0,
- .reg = TAA5412_REG_CH3_DIGITAL_VOLUME,
- .max = 0xff,
- .invert = 0,
- },
- {
- .shift = 0,
- .reg = TAA5412_REG_CH4_DIGITAL_VOLUME,
- .max = 0xff,
- .invert = 0,
- }
-};
-
-static const struct pcmdevice_mixer_control taa5412_fine_gain_ctl[] = {
- {
- .shift = 4,
- .reg = TAA5412_REG_CH1_FINE_GAIN,
- .max = 0xf,
- .invert = 0,
- },
- {
- .shift = 4,
- .reg = TAA5412_REG_CH2_FINE_GAIN,
- .max = 0xf,
- .invert = 0,
- },
- {
- .shift = 4,
- .reg = TAA5412_REG_CH3_FINE_GAIN,
- .max = 0xf,
- .invert = 4,
- },
- {
- .shift = 0,
- .reg = TAA5412_REG_CH4_FINE_GAIN,
- .max = 0xf,
- .invert = 4,
- }
-};
-
static const DECLARE_TLV_DB_MINMAX_MUTE(pcmd3140_dig_gain_tlv,
-10000, 2700);
static const DECLARE_TLV_DB_MINMAX_MUTE(pcm1690_fine_dig_gain_tlv,
@@ -510,9 +452,7 @@ static const DECLARE_TLV_DB_LINEAR(adc5120_chgain_tlv, 0, 4200);
static const DECLARE_TLV_DB_MINMAX_MUTE(pcm6260_fgain_tlv,
-10000, 2700);
static const DECLARE_TLV_DB_LINEAR(pcm6260_chgain_tlv, 0, 4200);
-static const DECLARE_TLV_DB_MINMAX_MUTE(taa5412_dig_vol_tlv,
- -8050, 4700);
-static const DECLARE_TLV_DB_LINEAR(taa5412_fine_gain_tlv,
+static const DECLARE_TLV_DB_LINEAR(pcmd31x0_fine_gain_tlv,
-80, 70);
static int pcmdev_change_dev(struct pcmdevice_priv *pcm_priv,
@@ -981,7 +921,7 @@ static const struct pcmdev_ctrl_info pcmdev_gain_ctl_info[][2] = {
// PCMD3140
{
{
- .gain = taa5412_fine_gain_tlv,
+ .gain = pcmd31x0_fine_gain_tlv,
.pcmdev_ctrl = pcmd3140_fine_gain_ctl,
.ctrl_array_size = ARRAY_SIZE(pcmd3140_fine_gain_ctl),
.get = pcmdevice_get_volsw,
@@ -1000,7 +940,7 @@ static const struct pcmdev_ctrl_info pcmdev_gain_ctl_info[][2] = {
// PCMD3180
{
{
- .gain = taa5412_fine_gain_tlv,
+ .gain = pcmd31x0_fine_gain_tlv,
.pcmdev_ctrl = pcmd3180_fine_gain_ctl,
.ctrl_array_size = ARRAY_SIZE(pcmd3180_fine_gain_ctl),
.get = pcmdevice_get_volsw,
@@ -1025,62 +965,6 @@ static const struct pcmdev_ctrl_info pcmdev_gain_ctl_info[][2] = {
.ctrl_array_size = 0,
},
},
- // TAA5212
- {
- {
- .gain = taa5412_fine_gain_tlv,
- .pcmdev_ctrl = taa5412_fine_gain_ctl,
- .ctrl_array_size = ARRAY_SIZE(taa5412_fine_gain_ctl),
- .get = pcmdevice_get_volsw,
- .put = pcmdevice_put_volsw,
- .pcmdev_ctrl_name_id = 2,
- },
- {
- .gain = taa5412_dig_vol_tlv,
- .pcmdev_ctrl = taa5412_digi_vol_ctl,
- .ctrl_array_size = ARRAY_SIZE(taa5412_digi_vol_ctl),
- .get = pcmdevice_get_volsw,
- .put = pcmdevice_put_volsw,
- .pcmdev_ctrl_name_id = 1,
- },
- },
- // TAA5412
- {
- {
- .gain = taa5412_fine_gain_tlv,
- .pcmdev_ctrl = taa5412_fine_gain_ctl,
- .ctrl_array_size = ARRAY_SIZE(taa5412_fine_gain_ctl),
- .get = pcmdevice_get_volsw,
- .put = pcmdevice_put_volsw,
- .pcmdev_ctrl_name_id = 2,
- },
- {
- .gain = taa5412_dig_vol_tlv,
- .pcmdev_ctrl = taa5412_digi_vol_ctl,
- .ctrl_array_size = ARRAY_SIZE(taa5412_digi_vol_ctl),
- .get = pcmdevice_get_volsw,
- .put = pcmdevice_put_volsw,
- .pcmdev_ctrl_name_id = 1,
- },
- },
- // TAD5212
- {
- {
- .ctrl_array_size = 0,
- },
- {
- .ctrl_array_size = 0,
- },
- },
- // TAD5412
- {
- {
- .ctrl_array_size = 0,
- },
- {
- .ctrl_array_size = 0,
- },
- },
};
static int pcmdev_dev_bulk_write(struct pcmdevice_priv *pcm_dev,
@@ -1998,10 +1882,6 @@ static const struct of_device_id pcmdevice_of_match[] = {
{ .compatible = "ti,pcmd3140" },
{ .compatible = "ti,pcmd3180" },
{ .compatible = "ti,pcmd512x" },
- { .compatible = "ti,taa5212" },
- { .compatible = "ti,taa5412" },
- { .compatible = "ti,tad5212" },
- { .compatible = "ti,tad5412" },
{},
};
MODULE_DEVICE_TABLE(of, pcmdevice_of_match);
diff --git a/sound/soc/codecs/pcm6240.h b/sound/soc/codecs/pcm6240.h
index 2d8f9e798139..86b1ef734a3d 100644
--- a/sound/soc/codecs/pcm6240.h
+++ b/sound/soc/codecs/pcm6240.h
@@ -33,10 +33,6 @@ enum pcm_device {
PCMD3140,
PCMD3180,
PCMD512X,
- TAA5212,
- TAA5412,
- TAD5212,
- TAD5412,
MAX_DEVICE,
};
--
2.34.1
prev parent reply other threads:[~2026-03-12 18:50 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-03-12 18:48 [PATCH v1 0/8] ASoC: support TI's TAC5x1x audio codec family Niranjan H Y
2026-03-12 18:48 ` [PATCH v1 1/8] dt-bindings: mfd: Add bindings for TI TAC5x1x MFD core Niranjan H Y
2026-03-13 20:43 ` Rob Herring
2026-03-14 10:13 ` Krzysztof Kozlowski
2026-03-12 18:48 ` [PATCH v1 2/8] dt-bindings: pinctrl: Add bindings for TI TAC5x1x pinctrl Niranjan H Y
2026-03-14 10:10 ` Krzysztof Kozlowski
2026-03-14 10:12 ` Krzysztof Kozlowski
2026-03-12 18:48 ` [PATCH v1 3/8] dt-bindings: sound: Add bindings for TI TAC5x1x codec Niranjan H Y
2026-03-14 10:14 ` Krzysztof Kozlowski
2026-03-12 18:48 ` [PATCH v1 4/8] dt-bindings: sound: Update ti,pcm6240.yaml to remove TAC5x1x family Niranjan H Y
2026-03-14 10:15 ` Krzysztof Kozlowski
2026-03-12 18:48 ` [PATCH v1 5/8] mfd: tac5x1x: Add TI TAC5x1x MFD core driver Niranjan H Y
2026-03-14 10:20 ` Krzysztof Kozlowski
2026-03-12 18:48 ` [PATCH v1 6/8] pinctrl: pinctrl-tac5x1x: Add TI TAC5x1x pinctrl driver Niranjan H Y
2026-03-14 10:11 ` Krzysztof Kozlowski
2026-03-12 18:48 ` [PATCH v1 7/8] ASoC: tac5x1x: Add TI TAC5x1x codec driver Niranjan H Y
2026-03-12 19:56 ` Mark Brown
2026-03-12 18:48 ` Niranjan H Y [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260312184833.263-9-niranjan.hy@ti.com \
--to=niranjan.hy@ti.com \
--cc=baojun.xu@ti.com \
--cc=broonie@kernel.org \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=krzk+dt@kernel.org \
--cc=lee@kernel.org \
--cc=lgirdwood@gmail.com \
--cc=linusw@kernel.org \
--cc=linux-gpio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-sound@vger.kernel.org \
--cc=navada@ti.com \
--cc=nb@tipi-net.de \
--cc=perex@perex.cz \
--cc=robh@kernel.org \
--cc=sandeepk@ti.com \
--cc=shenghao-ding@ti.com \
--cc=tiwai@suse.com \
--cc=v-hampiholi@ti.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox