* [PATCH 1/3] ASoC: tlv320aic32x4: prepare driver for different device variants [not found] <20210615094933.3076392-1-ch@denx.de> @ 2021-06-15 9:49 ` Claudius Heine 2021-06-15 11:54 ` Mark Brown 2021-06-15 9:49 ` [PATCH 2/3] ASoC: tlv320aic32x4: add support for TAS2505 Claudius Heine 2021-06-15 9:49 ` [PATCH 3/3] ASoC: tlv320aic32x4: dt-bindings: add TAS2505 to compatible Claudius Heine 2 siblings, 1 reply; 9+ messages in thread From: Claudius Heine @ 2021-06-15 9:49 UTC (permalink / raw) To: Liam Girdwood, Mark Brown, Jaroslav Kysela, Takashi Iwai, alsa-devel, Marek Vasut, Claudius Heine, Kuninori Morimoto, Sia Jee Heng, Pierre-Louis Bossart, Michael Sit Wei Hong, Miquel Raynal, Annaliese McDermond, Matthias Schiffer, open list With this change it will be possible to add different code paths for similar devices. Signed-off-by: Claudius Heine <ch@denx.de> --- sound/soc/codecs/tlv320aic32x4-i2c.c | 20 ++++++++++++++++---- sound/soc/codecs/tlv320aic32x4-spi.c | 23 +++++++++++++++++++---- sound/soc/codecs/tlv320aic32x4.c | 3 +++ sound/soc/codecs/tlv320aic32x4.h | 5 +++++ 4 files changed, 43 insertions(+), 8 deletions(-) diff --git a/sound/soc/codecs/tlv320aic32x4-i2c.c b/sound/soc/codecs/tlv320aic32x4-i2c.c index 6d54cbf70a0b..247fb1e13674 100644 --- a/sound/soc/codecs/tlv320aic32x4-i2c.c +++ b/sound/soc/codecs/tlv320aic32x4-i2c.c @@ -16,6 +16,8 @@ #include "tlv320aic32x4.h" +static const struct of_device_id aic32x4_of_id[]; + static int aic32x4_i2c_probe(struct i2c_client *i2c, const struct i2c_device_id *id) { @@ -27,6 +29,16 @@ static int aic32x4_i2c_probe(struct i2c_client *i2c, config.val_bits = 8; regmap = devm_regmap_init_i2c(i2c, &config); + + if (i2c->dev.of_node) { + const struct of_device_id *oid; + + oid = of_match_node(aic32x4_of_id, i2c->dev.of_node); + dev_set_drvdata(&i2c->dev, (void *)oid->data); + } else if (id) { + dev_set_drvdata(&i2c->dev, (void *)id->driver_data); + } + return aic32x4_probe(&i2c->dev, regmap); } @@ -36,15 +48,15 @@ static int aic32x4_i2c_remove(struct i2c_client *i2c) } static const struct i2c_device_id aic32x4_i2c_id[] = { - { "tlv320aic32x4", 0 }, - { "tlv320aic32x6", 1 }, + { "tlv320aic32x4", (kernel_ulong_t)AIC32X4_TYPE_AIC32X4 }, + { "tlv320aic32x6", (kernel_ulong_t)AIC32X4_TYPE_AIC32X6 }, { /* sentinel */ } }; MODULE_DEVICE_TABLE(i2c, aic32x4_i2c_id); static const struct of_device_id aic32x4_of_id[] = { - { .compatible = "ti,tlv320aic32x4", }, - { .compatible = "ti,tlv320aic32x6", }, + { .compatible = "ti,tlv320aic32x4", .data = (void *)AIC32X4_TYPE_AIC32X4 }, + { .compatible = "ti,tlv320aic32x6", .data = (void *)AIC32X4_TYPE_AIC32X6 }, { /* senitel */ } }; MODULE_DEVICE_TABLE(of, aic32x4_of_id); diff --git a/sound/soc/codecs/tlv320aic32x4-spi.c b/sound/soc/codecs/tlv320aic32x4-spi.c index a22e7700bfc8..e81c72958a82 100644 --- a/sound/soc/codecs/tlv320aic32x4-spi.c +++ b/sound/soc/codecs/tlv320aic32x4-spi.c @@ -16,6 +16,8 @@ #include "tlv320aic32x4.h" +static const struct of_device_id aic32x4_of_id[]; + static int aic32x4_spi_probe(struct spi_device *spi) { struct regmap *regmap; @@ -28,6 +30,19 @@ static int aic32x4_spi_probe(struct spi_device *spi) config.read_flag_mask = 0x01; regmap = devm_regmap_init_spi(spi, &config); + + if (spi->dev.of_node) { + const struct of_device_id *oid; + + oid = of_match_node(aic32x4_of_id, spi->dev.of_node); + dev_set_drvdata(&spi->dev, (void *)oid->data); + } else { + const struct spi_device_id *id_entry; + + id_entry = spi_get_device_id(spi); + dev_set_drvdata(&spi->dev, (void *)id_entry->driver_data); + } + return aic32x4_probe(&spi->dev, regmap); } @@ -37,15 +52,15 @@ static int aic32x4_spi_remove(struct spi_device *spi) } static const struct spi_device_id aic32x4_spi_id[] = { - { "tlv320aic32x4", 0 }, - { "tlv320aic32x6", 1 }, + { "tlv320aic32x4", (kernel_ulong_t)AIC32X4_TYPE_AIC32X4 }, + { "tlv320aic32x6", (kernel_ulong_t)AIC32X4_TYPE_AIC32X6 }, { /* sentinel */ } }; MODULE_DEVICE_TABLE(spi, aic32x4_spi_id); static const struct of_device_id aic32x4_of_id[] = { - { .compatible = "ti,tlv320aic32x4", }, - { .compatible = "ti,tlv320aic32x6", }, + { .compatible = "ti,tlv320aic32x4", .data = (void *)AIC32X4_TYPE_AIC32X4 }, + { .compatible = "ti,tlv320aic32x6", .data = (void *)AIC32X4_TYPE_AIC32X6 }, { /* senitel */ } }; MODULE_DEVICE_TABLE(of, aic32x4_of_id); diff --git a/sound/soc/codecs/tlv320aic32x4.c b/sound/soc/codecs/tlv320aic32x4.c index b689f26fc4be..70a1574fb72a 100644 --- a/sound/soc/codecs/tlv320aic32x4.c +++ b/sound/soc/codecs/tlv320aic32x4.c @@ -48,6 +48,7 @@ struct aic32x4_priv { struct aic32x4_setup_data *setup; struct device *dev; + enum aic32x4_type type; }; static int aic32x4_reset_adc(struct snd_soc_dapm_widget *w, @@ -1198,6 +1199,8 @@ int aic32x4_probe(struct device *dev, struct regmap *regmap) return -ENOMEM; aic32x4->dev = dev; + aic32x4->type = (enum aic32x4_type)dev_get_drvdata(dev); + dev_set_drvdata(dev, aic32x4); if (pdata) { diff --git a/sound/soc/codecs/tlv320aic32x4.h b/sound/soc/codecs/tlv320aic32x4.h index 7550122e9f8a..8a18dbec76a6 100644 --- a/sound/soc/codecs/tlv320aic32x4.h +++ b/sound/soc/codecs/tlv320aic32x4.h @@ -10,6 +10,11 @@ struct device; struct regmap_config; +enum aic32x4_type { + AIC32X4_TYPE_AIC32X4 = 0, + AIC32X4_TYPE_AIC32X6, +}; + extern const struct regmap_config aic32x4_regmap_config; int aic32x4_probe(struct device *dev, struct regmap *regmap); int aic32x4_remove(struct device *dev); -- 2.32.0 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH 1/3] ASoC: tlv320aic32x4: prepare driver for different device variants 2021-06-15 9:49 ` [PATCH 1/3] ASoC: tlv320aic32x4: prepare driver for different device variants Claudius Heine @ 2021-06-15 11:54 ` Mark Brown 2021-06-15 12:06 ` Claudius Heine 0 siblings, 1 reply; 9+ messages in thread From: Mark Brown @ 2021-06-15 11:54 UTC (permalink / raw) To: Claudius Heine Cc: Liam Girdwood, Jaroslav Kysela, Takashi Iwai, alsa-devel, Marek Vasut, Kuninori Morimoto, Sia Jee Heng, Pierre-Louis Bossart, Michael Sit Wei Hong, Miquel Raynal, Annaliese McDermond, Matthias Schiffer, open list [-- Attachment #1: Type: text/plain, Size: 483 bytes --] On Tue, Jun 15, 2021 at 11:49:30AM +0200, Claudius Heine wrote: > With this change it will be possible to add different code paths for > similar devices. > static const struct i2c_device_id aic32x4_i2c_id[] = { > - { "tlv320aic32x4", 0 }, > - { "tlv320aic32x6", 1 }, > + { "tlv320aic32x4", (kernel_ulong_t)AIC32X4_TYPE_AIC32X4 }, > + { "tlv320aic32x6", (kernel_ulong_t)AIC32X4_TYPE_AIC32X6 }, > { /* sentinel */ } It appears that the device already supports multiple variants? [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/3] ASoC: tlv320aic32x4: prepare driver for different device variants 2021-06-15 11:54 ` Mark Brown @ 2021-06-15 12:06 ` Claudius Heine 2021-06-15 12:30 ` Mark Brown 0 siblings, 1 reply; 9+ messages in thread From: Claudius Heine @ 2021-06-15 12:06 UTC (permalink / raw) To: Mark Brown Cc: Liam Girdwood, Jaroslav Kysela, Takashi Iwai, alsa-devel, Marek Vasut, Kuninori Morimoto, Sia Jee Heng, Pierre-Louis Bossart, Michael Sit Wei Hong, Miquel Raynal, Annaliese McDermond, Matthias Schiffer, open list Hi Mark, On 2021-06-15 13:54, Mark Brown wrote: > On Tue, Jun 15, 2021 at 11:49:30AM +0200, Claudius Heine wrote: > >> With this change it will be possible to add different code paths for >> similar devices. > >> static const struct i2c_device_id aic32x4_i2c_id[] = { >> - { "tlv320aic32x4", 0 }, >> - { "tlv320aic32x6", 1 }, >> + { "tlv320aic32x4", (kernel_ulong_t)AIC32X4_TYPE_AIC32X4 }, >> + { "tlv320aic32x6", (kernel_ulong_t)AIC32X4_TYPE_AIC32X6 }, >> { /* sentinel */ } > > It appears that the device already supports multiple variants? Those values aren't used anywhere as far as I can see. regards, Claudius ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/3] ASoC: tlv320aic32x4: prepare driver for different device variants 2021-06-15 12:06 ` Claudius Heine @ 2021-06-15 12:30 ` Mark Brown 0 siblings, 0 replies; 9+ messages in thread From: Mark Brown @ 2021-06-15 12:30 UTC (permalink / raw) To: Claudius Heine Cc: Liam Girdwood, Jaroslav Kysela, Takashi Iwai, alsa-devel, Marek Vasut, Kuninori Morimoto, Sia Jee Heng, Pierre-Louis Bossart, Michael Sit Wei Hong, Miquel Raynal, Annaliese McDermond, Matthias Schiffer, open list [-- Attachment #1: Type: text/plain, Size: 790 bytes --] On Tue, Jun 15, 2021 at 02:06:30PM +0200, Claudius Heine wrote: > On 2021-06-15 13:54, Mark Brown wrote: > > On Tue, Jun 15, 2021 at 11:49:30AM +0200, Claudius Heine wrote: > > > With this change it will be possible to add different code paths for > > > similar devices. > > > - { "tlv320aic32x4", 0 }, > > > - { "tlv320aic32x6", 1 }, > > > + { "tlv320aic32x4", (kernel_ulong_t)AIC32X4_TYPE_AIC32X4 }, > > > + { "tlv320aic32x6", (kernel_ulong_t)AIC32X4_TYPE_AIC32X6 }, > > > { /* sentinel */ } > > It appears that the device already supports multiple variants? > Those values aren't used anywhere as far as I can see. The point here is that you need a better changelog, the driver clearly already supports multiple devices so we need a few more words to explain what this is doing. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 2/3] ASoC: tlv320aic32x4: add support for TAS2505 [not found] <20210615094933.3076392-1-ch@denx.de> 2021-06-15 9:49 ` [PATCH 1/3] ASoC: tlv320aic32x4: prepare driver for different device variants Claudius Heine @ 2021-06-15 9:49 ` Claudius Heine 2021-06-15 11:59 ` Claudius Heine 2021-06-15 12:22 ` Mark Brown 2021-06-15 9:49 ` [PATCH 3/3] ASoC: tlv320aic32x4: dt-bindings: add TAS2505 to compatible Claudius Heine 2 siblings, 2 replies; 9+ messages in thread From: Claudius Heine @ 2021-06-15 9:49 UTC (permalink / raw) To: Liam Girdwood, Mark Brown, Jaroslav Kysela, Takashi Iwai, alsa-devel, Marek Vasut, Claudius Heine, Michael Sit Wei Hong, Kuninori Morimoto, Pierre-Louis Bossart, Sia Jee Heng, Miquel Raynal, Annaliese McDermond, Matthias Schiffer, open list This adds support for TAS2505 and TAS2521 to the tlv320aic32x4 driver. Signed-off-by: Claudius Heine <ch@denx.de> --- sound/soc/codecs/tlv320aic32x4-i2c.c | 2 + sound/soc/codecs/tlv320aic32x4.c | 170 ++++++++++++++++++++++++++- sound/soc/codecs/tlv320aic32x4.h | 5 + 3 files changed, 175 insertions(+), 2 deletions(-) diff --git a/sound/soc/codecs/tlv320aic32x4-i2c.c b/sound/soc/codecs/tlv320aic32x4-i2c.c index 247fb1e13674..04ad38311360 100644 --- a/sound/soc/codecs/tlv320aic32x4-i2c.c +++ b/sound/soc/codecs/tlv320aic32x4-i2c.c @@ -50,6 +50,7 @@ static int aic32x4_i2c_remove(struct i2c_client *i2c) static const struct i2c_device_id aic32x4_i2c_id[] = { { "tlv320aic32x4", (kernel_ulong_t)AIC32X4_TYPE_AIC32X4 }, { "tlv320aic32x6", (kernel_ulong_t)AIC32X4_TYPE_AIC32X6 }, + { "tas2505", (kernel_ulong_t)AIC32X4_TYPE_TAS2505 }, { /* sentinel */ } }; MODULE_DEVICE_TABLE(i2c, aic32x4_i2c_id); @@ -57,6 +58,7 @@ MODULE_DEVICE_TABLE(i2c, aic32x4_i2c_id); static const struct of_device_id aic32x4_of_id[] = { { .compatible = "ti,tlv320aic32x4", .data = (void *)AIC32X4_TYPE_AIC32X4 }, { .compatible = "ti,tlv320aic32x6", .data = (void *)AIC32X4_TYPE_AIC32X6 }, + { .compatible = "ti,tas2505", .data = (void *)AIC32X4_TYPE_TAS2505 }, { /* senitel */ } }; MODULE_DEVICE_TABLE(of, aic32x4_of_id); diff --git a/sound/soc/codecs/tlv320aic32x4.c b/sound/soc/codecs/tlv320aic32x4.c index 70a1574fb72a..58e773d682d2 100644 --- a/sound/soc/codecs/tlv320aic32x4.c +++ b/sound/soc/codecs/tlv320aic32x4.c @@ -221,6 +221,38 @@ static int aic32x4_set_mfp5_gpio(struct snd_kcontrol *kcontrol, return 0; }; +static int aic32x4_tas2505_spkdrv_getvol(struct snd_kcontrol *kcontrol, + struct snd_ctl_elem_value *ucontrol) +{ + struct snd_soc_component *component = snd_kcontrol_chip(kcontrol); + struct soc_mixer_control *mc = + (struct soc_mixer_control *)kcontrol->private_value; + unsigned int val; + + val = snd_soc_component_read(component, TAS2505_SPKVOL1); + val = (val > mc->max) ? mc->max : val; + val = mc->invert ? mc->max - val : val; + ucontrol->value.integer.value[0] = val; + + return 0; +} + +static int aic32x4_tas2505_spkdrv_putvol(struct snd_kcontrol *kcontrol, + struct snd_ctl_elem_value *ucontrol) +{ + struct snd_soc_component *component = snd_kcontrol_chip(kcontrol); + struct soc_mixer_control *mc = + (struct soc_mixer_control *)kcontrol->private_value; + u8 val; + + val = (ucontrol->value.integer.value[0] & 0x7f); + val = mc->invert ? mc->max - val : val; + val = (val < 0) ? 0 : val; + snd_soc_component_write(component, TAS2505_SPKVOL1, val); + + return 0; +} + static const struct snd_kcontrol_new aic32x4_mfp1[] = { SOC_SINGLE_BOOL_EXT("MFP1 GPIO", 0, aic32x4_get_mfp1_gpio, NULL), }; @@ -251,6 +283,9 @@ static DECLARE_TLV_DB_SCALE(tlv_driver_gain, -600, 100, 0); /* -12dB min, 0.5dB steps */ static DECLARE_TLV_DB_SCALE(tlv_adc_vol, -1200, 50, 0); +static DECLARE_TLV_DB_LINEAR(tlv_spk_vol, TLV_DB_GAIN_MUTE, 0); +static DECLARE_TLV_DB_SCALE(tlv_amp_vol, 0, 600, 1); + static const char * const lo_cm_text[] = { "Full Chip", "1.65V", }; @@ -1059,6 +1094,130 @@ static const struct snd_soc_component_driver soc_component_dev_aic32x4 = { .non_legacy_dai_naming = 1, }; +static const struct snd_kcontrol_new aic32x4_tas2505_snd_controls[] = { + SOC_DOUBLE_R_S_TLV("PCM Playback Volume", AIC32X4_LDACVOL, + AIC32X4_LDACVOL, 0, -0x7f, 0x30, 7, 0, tlv_pcm), + SOC_ENUM("DAC Playback PowerTune Switch", l_ptm_enum), + SOC_DOUBLE_R_S_TLV("HP Driver Gain Playback Volume", AIC32X4_HPLGAIN, + AIC32X4_HPLGAIN, 0, -0x6, 0x1d, 5, 0, + tlv_driver_gain), + SOC_DOUBLE_R("HP DAC Playback Switch", AIC32X4_HPLGAIN, + AIC32X4_HPLGAIN, 6, 0x01, 1), + + SOC_SINGLE("Auto-mute Switch", AIC32X4_DACMUTE, 4, 7, 0), + + SOC_SINGLE_RANGE_EXT_TLV("Speaker Driver Playback Volume", TAS2505_SPKVOL1, + 0, 0, 117, 1, aic32x4_tas2505_spkdrv_getvol, + aic32x4_tas2505_spkdrv_putvol, tlv_spk_vol), + SOC_SINGLE_TLV("Speaker Amplifier Playback Volume", TAS2505_SPKVOL2, + 4, 5, 0, tlv_amp_vol), +}; + +static const struct snd_kcontrol_new hp_output_mixer_controls[] = { + SOC_DAPM_SINGLE("DAC Switch", AIC32X4_HPLROUTE, 3, 1, 0), +}; + +static const struct snd_soc_dapm_widget aic32x4_tas2505_dapm_widgets[] = { + SND_SOC_DAPM_DAC("DAC", "Playback", AIC32X4_DACSETUP, 7, 0), + SND_SOC_DAPM_MIXER("HP Output Mixer", SND_SOC_NOPM, 0, 0, + &hp_output_mixer_controls[0], + ARRAY_SIZE(hp_output_mixer_controls)), + SND_SOC_DAPM_PGA("HP Power", AIC32X4_OUTPWRCTL, 5, 0, NULL, 0), + + SND_SOC_DAPM_PGA("Speaker Driver", TAS2505_SPK, 1, 0, NULL, 0), + + SND_SOC_DAPM_OUTPUT("HP"), + SND_SOC_DAPM_OUTPUT("Speaker"), +}; + +static const struct snd_soc_dapm_route aic32x4_tas2505_dapm_routes[] = { + /* Left Output */ + {"HP Output Mixer", "DAC Switch", "DAC"}, + + {"HP Power", NULL, "HP Output Mixer"}, + {"HP", NULL, "HP Power"}, + + {"Speaker Driver", NULL, "DAC"}, + {"Speaker", NULL, "Speaker Driver"}, +}; + +static struct snd_soc_dai_driver aic32x4_tas2505_dai = { + .name = "tas2505-hifi", + .playback = { + .stream_name = "Playback", + .channels_min = 1, + .channels_max = 1, + .rates = SNDRV_PCM_RATE_8000_96000, + .formats = AIC32X4_FORMATS,}, + .ops = &aic32x4_ops, + .symmetric_rates = 1, +}; + +static int aic32x4_tas2505_component_probe(struct snd_soc_component *component) +{ + struct aic32x4_priv *aic32x4 = snd_soc_component_get_drvdata(component); + u32 tmp_reg; + int ret; + + struct clk_bulk_data clocks[] = { + { .id = "codec_clkin" }, + { .id = "pll" }, + { .id = "bdiv" }, + { .id = "mdac" }, + }; + + ret = devm_clk_bulk_get(component->dev, ARRAY_SIZE(clocks), clocks); + if (ret) + return ret; + + if (aic32x4->setup) + aic32x4_setup_gpios(component); + + clk_set_parent(clocks[0].clk, clocks[1].clk); + clk_set_parent(clocks[2].clk, clocks[3].clk); + + /* Power platform configuration */ + if (aic32x4->power_cfg & AIC32X4_PWR_AVDD_DVDD_WEAK_DISABLE) + snd_soc_component_write(component, AIC32X4_PWRCFG, AIC32X4_AVDDWEAKDISABLE); + + tmp_reg = (aic32x4->power_cfg & AIC32X4_PWR_AIC32X4_LDO_ENABLE) ? + AIC32X4_LDOCTLEN : 0; + snd_soc_component_write(component, AIC32X4_LDOCTL, tmp_reg); + + tmp_reg = snd_soc_component_read(component, AIC32X4_CMMODE); + if (aic32x4->power_cfg & AIC32X4_PWR_CMMODE_LDOIN_RANGE_18_36) + tmp_reg |= AIC32X4_LDOIN_18_36; + if (aic32x4->power_cfg & AIC32X4_PWR_CMMODE_HP_LDOIN_POWERED) + tmp_reg |= AIC32X4_LDOIN2HP; + snd_soc_component_write(component, AIC32X4_CMMODE, tmp_reg); + + /* + * Enable the fast charging feature and ensure the needed 40ms ellapsed + * before using the analog circuits. + */ + snd_soc_component_write(component, TAS2505_REFPOWERUP, + AIC32X4_REFPOWERUP_40MS); + msleep(40); + + return 0; +} + +static const struct snd_soc_component_driver soc_component_dev_aic32x4_tas2505 = { + .probe = aic32x4_tas2505_component_probe, + .set_bias_level = aic32x4_set_bias_level, + .controls = aic32x4_tas2505_snd_controls, + .num_controls = ARRAY_SIZE(aic32x4_tas2505_snd_controls), + .dapm_widgets = aic32x4_tas2505_dapm_widgets, + .num_dapm_widgets = ARRAY_SIZE(aic32x4_tas2505_dapm_widgets), + .dapm_routes = aic32x4_tas2505_dapm_routes, + .num_dapm_routes = ARRAY_SIZE(aic32x4_tas2505_dapm_routes), + .suspend_bias_off = 1, + .idle_bias_on = 1, + .use_pmdown_time = 1, + .endianness = 1, + .non_legacy_dai_naming = 1, +}; + static int aic32x4_parse_dt(struct aic32x4_priv *aic32x4, struct device_node *np) { @@ -1250,8 +1409,15 @@ int aic32x4_probe(struct device *dev, struct regmap *regmap) if (ret) goto err_disable_regulators; - ret = devm_snd_soc_register_component(dev, - &soc_component_dev_aic32x4, &aic32x4_dai, 1); + switch (aic32x4->type) { + case AIC32X4_TYPE_TAS2505: + ret = devm_snd_soc_register_component(dev, + &soc_component_dev_aic32x4_tas2505, &aic32x4_tas2505_dai, 1); + break; + default: + ret = devm_snd_soc_register_component(dev, + } + if (ret) { dev_err(dev, "Failed to register component\n"); goto err_disable_regulators; diff --git a/sound/soc/codecs/tlv320aic32x4.h b/sound/soc/codecs/tlv320aic32x4.h index 8a18dbec76a6..e9fd2e55d6c3 100644 --- a/sound/soc/codecs/tlv320aic32x4.h +++ b/sound/soc/codecs/tlv320aic32x4.h @@ -13,6 +13,7 @@ struct regmap_config; enum aic32x4_type { AIC32X4_TYPE_AIC32X4 = 0, AIC32X4_TYPE_AIC32X6, + AIC32X4_TYPE_TAS2505, }; extern const struct regmap_config aic32x4_regmap_config; @@ -93,6 +94,9 @@ int aic32x4_register_clocks(struct device *dev, const char *mclk_name); #define AIC32X4_LOLGAIN AIC32X4_REG(1, 18) #define AIC32X4_LORGAIN AIC32X4_REG(1, 19) #define AIC32X4_HEADSTART AIC32X4_REG(1, 20) +#define TAS2505_SPK AIC32X4_REG(1, 45) +#define TAS2505_SPKVOL1 AIC32X4_REG(1, 46) +#define TAS2505_SPKVOL2 AIC32X4_REG(1, 48) #define AIC32X4_MICBIAS AIC32X4_REG(1, 51) #define AIC32X4_LMICPGAPIN AIC32X4_REG(1, 52) #define AIC32X4_LMICPGANIN AIC32X4_REG(1, 54) @@ -101,6 +105,7 @@ int aic32x4_register_clocks(struct device *dev, const char *mclk_name); #define AIC32X4_FLOATINGINPUT AIC32X4_REG(1, 58) #define AIC32X4_LMICPGAVOL AIC32X4_REG(1, 59) #define AIC32X4_RMICPGAVOL AIC32X4_REG(1, 60) +#define TAS2505_REFPOWERUP AIC32X4_REG(1, 122) #define AIC32X4_REFPOWERUP AIC32X4_REG(1, 123) /* Bits, masks, and shifts */ -- 2.32.0 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH 2/3] ASoC: tlv320aic32x4: add support for TAS2505 2021-06-15 9:49 ` [PATCH 2/3] ASoC: tlv320aic32x4: add support for TAS2505 Claudius Heine @ 2021-06-15 11:59 ` Claudius Heine 2021-06-15 12:22 ` Mark Brown 1 sibling, 0 replies; 9+ messages in thread From: Claudius Heine @ 2021-06-15 11:59 UTC (permalink / raw) To: Liam Girdwood, Mark Brown, Jaroslav Kysela, Takashi Iwai, alsa-devel, Marek Vasut, Michael Sit Wei Hong, Kuninori Morimoto, Pierre-Louis Bossart, Sia Jee Heng, Miquel Raynal, Annaliese McDermond, Matthias Schiffer, open list On 2021-06-15 11:49, Claudius Heine wrote: > This adds support for TAS2505 and TAS2521 to the tlv320aic32x4 driver. There are some rebase issues here, I will have to fix up. regards, Claudius > > Signed-off-by: Claudius Heine <ch@denx.de> > --- > sound/soc/codecs/tlv320aic32x4-i2c.c | 2 + > sound/soc/codecs/tlv320aic32x4.c | 170 ++++++++++++++++++++++++++- > sound/soc/codecs/tlv320aic32x4.h | 5 + > 3 files changed, 175 insertions(+), 2 deletions(-) > > diff --git a/sound/soc/codecs/tlv320aic32x4-i2c.c b/sound/soc/codecs/tlv320aic32x4-i2c.c > index 247fb1e13674..04ad38311360 100644 > --- a/sound/soc/codecs/tlv320aic32x4-i2c.c > +++ b/sound/soc/codecs/tlv320aic32x4-i2c.c > @@ -50,6 +50,7 @@ static int aic32x4_i2c_remove(struct i2c_client *i2c) > static const struct i2c_device_id aic32x4_i2c_id[] = { > { "tlv320aic32x4", (kernel_ulong_t)AIC32X4_TYPE_AIC32X4 }, > { "tlv320aic32x6", (kernel_ulong_t)AIC32X4_TYPE_AIC32X6 }, > + { "tas2505", (kernel_ulong_t)AIC32X4_TYPE_TAS2505 }, > { /* sentinel */ } > }; > MODULE_DEVICE_TABLE(i2c, aic32x4_i2c_id); > @@ -57,6 +58,7 @@ MODULE_DEVICE_TABLE(i2c, aic32x4_i2c_id); > static const struct of_device_id aic32x4_of_id[] = { > { .compatible = "ti,tlv320aic32x4", .data = (void *)AIC32X4_TYPE_AIC32X4 }, > { .compatible = "ti,tlv320aic32x6", .data = (void *)AIC32X4_TYPE_AIC32X6 }, > + { .compatible = "ti,tas2505", .data = (void *)AIC32X4_TYPE_TAS2505 }, > { /* senitel */ } > }; > MODULE_DEVICE_TABLE(of, aic32x4_of_id); > diff --git a/sound/soc/codecs/tlv320aic32x4.c b/sound/soc/codecs/tlv320aic32x4.c > index 70a1574fb72a..58e773d682d2 100644 > --- a/sound/soc/codecs/tlv320aic32x4.c > +++ b/sound/soc/codecs/tlv320aic32x4.c > @@ -221,6 +221,38 @@ static int aic32x4_set_mfp5_gpio(struct snd_kcontrol *kcontrol, > return 0; > }; > > +static int aic32x4_tas2505_spkdrv_getvol(struct snd_kcontrol *kcontrol, > + struct snd_ctl_elem_value *ucontrol) > +{ > + struct snd_soc_component *component = snd_kcontrol_chip(kcontrol); > + struct soc_mixer_control *mc = > + (struct soc_mixer_control *)kcontrol->private_value; > + unsigned int val; > + > + val = snd_soc_component_read(component, TAS2505_SPKVOL1); > + val = (val > mc->max) ? mc->max : val; > + val = mc->invert ? mc->max - val : val; > + ucontrol->value.integer.value[0] = val; > + > + return 0; > +} > + > +static int aic32x4_tas2505_spkdrv_putvol(struct snd_kcontrol *kcontrol, > + struct snd_ctl_elem_value *ucontrol) > +{ > + struct snd_soc_component *component = snd_kcontrol_chip(kcontrol); > + struct soc_mixer_control *mc = > + (struct soc_mixer_control *)kcontrol->private_value; > + u8 val; > + > + val = (ucontrol->value.integer.value[0] & 0x7f); > + val = mc->invert ? mc->max - val : val; > + val = (val < 0) ? 0 : val; > + snd_soc_component_write(component, TAS2505_SPKVOL1, val); > + > + return 0; > +} > + > static const struct snd_kcontrol_new aic32x4_mfp1[] = { > SOC_SINGLE_BOOL_EXT("MFP1 GPIO", 0, aic32x4_get_mfp1_gpio, NULL), > }; > @@ -251,6 +283,9 @@ static DECLARE_TLV_DB_SCALE(tlv_driver_gain, -600, 100, 0); > /* -12dB min, 0.5dB steps */ > static DECLARE_TLV_DB_SCALE(tlv_adc_vol, -1200, 50, 0); > > +static DECLARE_TLV_DB_LINEAR(tlv_spk_vol, TLV_DB_GAIN_MUTE, 0); > +static DECLARE_TLV_DB_SCALE(tlv_amp_vol, 0, 600, 1); > + > static const char * const lo_cm_text[] = { > "Full Chip", "1.65V", > }; > @@ -1059,6 +1094,130 @@ static const struct snd_soc_component_driver soc_component_dev_aic32x4 = { > .non_legacy_dai_naming = 1, > }; > > +static const struct snd_kcontrol_new aic32x4_tas2505_snd_controls[] = { > + SOC_DOUBLE_R_S_TLV("PCM Playback Volume", AIC32X4_LDACVOL, > + AIC32X4_LDACVOL, 0, -0x7f, 0x30, 7, 0, tlv_pcm), > + SOC_ENUM("DAC Playback PowerTune Switch", l_ptm_enum), > + SOC_DOUBLE_R_S_TLV("HP Driver Gain Playback Volume", AIC32X4_HPLGAIN, > + AIC32X4_HPLGAIN, 0, -0x6, 0x1d, 5, 0, > + tlv_driver_gain), > + SOC_DOUBLE_R("HP DAC Playback Switch", AIC32X4_HPLGAIN, > + AIC32X4_HPLGAIN, 6, 0x01, 1), > + > + SOC_SINGLE("Auto-mute Switch", AIC32X4_DACMUTE, 4, 7, 0), > + > + SOC_SINGLE_RANGE_EXT_TLV("Speaker Driver Playback Volume", TAS2505_SPKVOL1, > + 0, 0, 117, 1, aic32x4_tas2505_spkdrv_getvol, > + aic32x4_tas2505_spkdrv_putvol, tlv_spk_vol), > + SOC_SINGLE_TLV("Speaker Amplifier Playback Volume", TAS2505_SPKVOL2, > + 4, 5, 0, tlv_amp_vol), > +}; > + > +static const struct snd_kcontrol_new hp_output_mixer_controls[] = { > + SOC_DAPM_SINGLE("DAC Switch", AIC32X4_HPLROUTE, 3, 1, 0), > +}; > + > +static const struct snd_soc_dapm_widget aic32x4_tas2505_dapm_widgets[] = { > + SND_SOC_DAPM_DAC("DAC", "Playback", AIC32X4_DACSETUP, 7, 0), > + SND_SOC_DAPM_MIXER("HP Output Mixer", SND_SOC_NOPM, 0, 0, > + &hp_output_mixer_controls[0], > + ARRAY_SIZE(hp_output_mixer_controls)), > + SND_SOC_DAPM_PGA("HP Power", AIC32X4_OUTPWRCTL, 5, 0, NULL, 0), > + > + SND_SOC_DAPM_PGA("Speaker Driver", TAS2505_SPK, 1, 0, NULL, 0), > + > + SND_SOC_DAPM_OUTPUT("HP"), > + SND_SOC_DAPM_OUTPUT("Speaker"), > +}; > + > +static const struct snd_soc_dapm_route aic32x4_tas2505_dapm_routes[] = { > + /* Left Output */ > + {"HP Output Mixer", "DAC Switch", "DAC"}, > + > + {"HP Power", NULL, "HP Output Mixer"}, > + {"HP", NULL, "HP Power"}, > + > + {"Speaker Driver", NULL, "DAC"}, > + {"Speaker", NULL, "Speaker Driver"}, > +}; > + > +static struct snd_soc_dai_driver aic32x4_tas2505_dai = { > + .name = "tas2505-hifi", > + .playback = { > + .stream_name = "Playback", > + .channels_min = 1, > + .channels_max = 1, > + .rates = SNDRV_PCM_RATE_8000_96000, > + .formats = AIC32X4_FORMATS,}, > + .ops = &aic32x4_ops, > + .symmetric_rates = 1, > +}; > + > +static int aic32x4_tas2505_component_probe(struct snd_soc_component *component) > +{ > + struct aic32x4_priv *aic32x4 = snd_soc_component_get_drvdata(component); > + u32 tmp_reg; > + int ret; > + > + struct clk_bulk_data clocks[] = { > + { .id = "codec_clkin" }, > + { .id = "pll" }, > + { .id = "bdiv" }, > + { .id = "mdac" }, > + }; > + > + ret = devm_clk_bulk_get(component->dev, ARRAY_SIZE(clocks), clocks); > + if (ret) > + return ret; > + > + if (aic32x4->setup) > + aic32x4_setup_gpios(component); > + > + clk_set_parent(clocks[0].clk, clocks[1].clk); > + clk_set_parent(clocks[2].clk, clocks[3].clk); > + > + /* Power platform configuration */ > + if (aic32x4->power_cfg & AIC32X4_PWR_AVDD_DVDD_WEAK_DISABLE) > + snd_soc_component_write(component, AIC32X4_PWRCFG, AIC32X4_AVDDWEAKDISABLE); > + > + tmp_reg = (aic32x4->power_cfg & AIC32X4_PWR_AIC32X4_LDO_ENABLE) ? > + AIC32X4_LDOCTLEN : 0; > + snd_soc_component_write(component, AIC32X4_LDOCTL, tmp_reg); > + > + tmp_reg = snd_soc_component_read(component, AIC32X4_CMMODE); > + if (aic32x4->power_cfg & AIC32X4_PWR_CMMODE_LDOIN_RANGE_18_36) > + tmp_reg |= AIC32X4_LDOIN_18_36; > + if (aic32x4->power_cfg & AIC32X4_PWR_CMMODE_HP_LDOIN_POWERED) > + tmp_reg |= AIC32X4_LDOIN2HP; > + snd_soc_component_write(component, AIC32X4_CMMODE, tmp_reg); > + > + /* > + * Enable the fast charging feature and ensure the needed 40ms ellapsed > + * before using the analog circuits. > + */ > + snd_soc_component_write(component, TAS2505_REFPOWERUP, > + AIC32X4_REFPOWERUP_40MS); > + msleep(40); > + > + return 0; > +} > + > +static const struct snd_soc_component_driver soc_component_dev_aic32x4_tas2505 = { > + .probe = aic32x4_tas2505_component_probe, > + .set_bias_level = aic32x4_set_bias_level, > + .controls = aic32x4_tas2505_snd_controls, > + .num_controls = ARRAY_SIZE(aic32x4_tas2505_snd_controls), > + .dapm_widgets = aic32x4_tas2505_dapm_widgets, > + .num_dapm_widgets = ARRAY_SIZE(aic32x4_tas2505_dapm_widgets), > + .dapm_routes = aic32x4_tas2505_dapm_routes, > + .num_dapm_routes = ARRAY_SIZE(aic32x4_tas2505_dapm_routes), > + .suspend_bias_off = 1, > + .idle_bias_on = 1, > + .use_pmdown_time = 1, > + .endianness = 1, > + .non_legacy_dai_naming = 1, > +}; > + > static int aic32x4_parse_dt(struct aic32x4_priv *aic32x4, > struct device_node *np) > { > @@ -1250,8 +1409,15 @@ int aic32x4_probe(struct device *dev, struct regmap *regmap) > if (ret) > goto err_disable_regulators; > > - ret = devm_snd_soc_register_component(dev, > - &soc_component_dev_aic32x4, &aic32x4_dai, 1); > + switch (aic32x4->type) { > + case AIC32X4_TYPE_TAS2505: > + ret = devm_snd_soc_register_component(dev, > + &soc_component_dev_aic32x4_tas2505, &aic32x4_tas2505_dai, 1); > + break; > + default: > + ret = devm_snd_soc_register_component(dev, > + } > + > if (ret) { > dev_err(dev, "Failed to register component\n"); > goto err_disable_regulators; > diff --git a/sound/soc/codecs/tlv320aic32x4.h b/sound/soc/codecs/tlv320aic32x4.h > index 8a18dbec76a6..e9fd2e55d6c3 100644 > --- a/sound/soc/codecs/tlv320aic32x4.h > +++ b/sound/soc/codecs/tlv320aic32x4.h > @@ -13,6 +13,7 @@ struct regmap_config; > enum aic32x4_type { > AIC32X4_TYPE_AIC32X4 = 0, > AIC32X4_TYPE_AIC32X6, > + AIC32X4_TYPE_TAS2505, > }; > > extern const struct regmap_config aic32x4_regmap_config; > @@ -93,6 +94,9 @@ int aic32x4_register_clocks(struct device *dev, const char *mclk_name); > #define AIC32X4_LOLGAIN AIC32X4_REG(1, 18) > #define AIC32X4_LORGAIN AIC32X4_REG(1, 19) > #define AIC32X4_HEADSTART AIC32X4_REG(1, 20) > +#define TAS2505_SPK AIC32X4_REG(1, 45) > +#define TAS2505_SPKVOL1 AIC32X4_REG(1, 46) > +#define TAS2505_SPKVOL2 AIC32X4_REG(1, 48) > #define AIC32X4_MICBIAS AIC32X4_REG(1, 51) > #define AIC32X4_LMICPGAPIN AIC32X4_REG(1, 52) > #define AIC32X4_LMICPGANIN AIC32X4_REG(1, 54) > @@ -101,6 +105,7 @@ int aic32x4_register_clocks(struct device *dev, const char *mclk_name); > #define AIC32X4_FLOATINGINPUT AIC32X4_REG(1, 58) > #define AIC32X4_LMICPGAVOL AIC32X4_REG(1, 59) > #define AIC32X4_RMICPGAVOL AIC32X4_REG(1, 60) > +#define TAS2505_REFPOWERUP AIC32X4_REG(1, 122) > #define AIC32X4_REFPOWERUP AIC32X4_REG(1, 123) > > /* Bits, masks, and shifts */ > -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-54 Fax: (+49)-8142-66989-80 Email: ch@denx.de PGP key: 6FF2 E59F 00C6 BC28 31D8 64C1 1173 CB19 9808 B153 Keyserver: hkp://pool.sks-keyservers.net ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 2/3] ASoC: tlv320aic32x4: add support for TAS2505 2021-06-15 9:49 ` [PATCH 2/3] ASoC: tlv320aic32x4: add support for TAS2505 Claudius Heine 2021-06-15 11:59 ` Claudius Heine @ 2021-06-15 12:22 ` Mark Brown 2021-06-15 12:39 ` Claudius Heine 1 sibling, 1 reply; 9+ messages in thread From: Mark Brown @ 2021-06-15 12:22 UTC (permalink / raw) To: Claudius Heine Cc: Liam Girdwood, Jaroslav Kysela, Takashi Iwai, alsa-devel, Marek Vasut, Michael Sit Wei Hong, Kuninori Morimoto, Pierre-Louis Bossart, Sia Jee Heng, Miquel Raynal, Annaliese McDermond, Matthias Schiffer, open list [-- Attachment #1: Type: text/plain, Size: 1012 bytes --] On Tue, Jun 15, 2021 at 11:49:31AM +0200, Claudius Heine wrote: > +static int aic32x4_tas2505_spkdrv_putvol(struct snd_kcontrol *kcontrol, > + struct snd_ctl_elem_value *ucontrol) > +{ > + struct snd_soc_component *component = snd_kcontrol_chip(kcontrol); > + struct soc_mixer_control *mc = > + (struct soc_mixer_control *)kcontrol->private_value; > + u8 val; > + > + val = (ucontrol->value.integer.value[0] & 0x7f); > + val = mc->invert ? mc->max - val : val; > + val = (val < 0) ? 0 : val; > + snd_soc_component_write(component, TAS2505_SPKVOL1, val); > + > + return 0; > +} Controls should return a boolean indicating if they changed their value when written. Other than the hard coded register what's device specific here? It looks like a normal control with a maximum value, it is unclear why this is being open coded. > + SOC_DOUBLE_R_S_TLV("HP Driver Gain Playback Volume", AIC32X4_HPLGAIN, > + AIC32X4_HPLGAIN, 0, -0x6, 0x1d, 5, 0, > + tlv_driver_gain), Drop the Gain. See control-names.rst. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 2/3] ASoC: tlv320aic32x4: add support for TAS2505 2021-06-15 12:22 ` Mark Brown @ 2021-06-15 12:39 ` Claudius Heine 0 siblings, 0 replies; 9+ messages in thread From: Claudius Heine @ 2021-06-15 12:39 UTC (permalink / raw) To: Mark Brown Cc: Liam Girdwood, Jaroslav Kysela, Takashi Iwai, alsa-devel, Marek Vasut, Michael Sit Wei Hong, Kuninori Morimoto, Pierre-Louis Bossart, Sia Jee Heng, Miquel Raynal, Annaliese McDermond, Matthias Schiffer, open list On 2021-06-15 14:22, Mark Brown wrote: > On Tue, Jun 15, 2021 at 11:49:31AM +0200, Claudius Heine wrote: > >> +static int aic32x4_tas2505_spkdrv_putvol(struct snd_kcontrol *kcontrol, >> + struct snd_ctl_elem_value *ucontrol) >> +{ >> + struct snd_soc_component *component = snd_kcontrol_chip(kcontrol); >> + struct soc_mixer_control *mc = >> + (struct soc_mixer_control *)kcontrol->private_value; >> + u8 val; >> + >> + val = (ucontrol->value.integer.value[0] & 0x7f); >> + val = mc->invert ? mc->max - val : val; >> + val = (val < 0) ? 0 : val; >> + snd_soc_component_write(component, TAS2505_SPKVOL1, val); >> + >> + return 0; >> +} > > Controls should return a boolean indicating if they changed their value > when written. Other than the hard coded register what's device specific > here? It looks like a normal control with a maximum value, it is > unclear why this is being open coded. Well probably because I didn't knew any better. Will look into it. Thx! > >> + SOC_DOUBLE_R_S_TLV("HP Driver Gain Playback Volume", AIC32X4_HPLGAIN, >> + AIC32X4_HPLGAIN, 0, -0x6, 0x1d, 5, 0, >> + tlv_driver_gain), > > Drop the Gain. See control-names.rst. > Ok. Thanks! Claudius ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 3/3] ASoC: tlv320aic32x4: dt-bindings: add TAS2505 to compatible [not found] <20210615094933.3076392-1-ch@denx.de> 2021-06-15 9:49 ` [PATCH 1/3] ASoC: tlv320aic32x4: prepare driver for different device variants Claudius Heine 2021-06-15 9:49 ` [PATCH 2/3] ASoC: tlv320aic32x4: add support for TAS2505 Claudius Heine @ 2021-06-15 9:49 ` Claudius Heine 2 siblings, 0 replies; 9+ messages in thread From: Claudius Heine @ 2021-06-15 9:49 UTC (permalink / raw) To: Liam Girdwood, Mark Brown, Jaroslav Kysela, Takashi Iwai, alsa-devel, Marek Vasut, Rob Herring, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, open list Cc: Claudius Heine This adds 'ti,tas2505' for TAS2505 to the list of allowed compatible strings. Signed-off-by: Claudius Heine <ch@denx.de> --- Documentation/devicetree/bindings/sound/tlv320aic32x4.txt | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/sound/tlv320aic32x4.txt b/Documentation/devicetree/bindings/sound/tlv320aic32x4.txt index ca75890f0d07..f59125bc79d1 100644 --- a/Documentation/devicetree/bindings/sound/tlv320aic32x4.txt +++ b/Documentation/devicetree/bindings/sound/tlv320aic32x4.txt @@ -6,6 +6,7 @@ Required properties: - compatible - "string" - One of: "ti,tlv320aic32x4" TLV320AIC3204 "ti,tlv320aic32x6" TLV320AIC3206, TLV320AIC3256 + "ti,tas2505" TAS2505, TAS2521 - reg: I2C slave address - supply-*: Required supply regulators are: "iov" - digital IO power supply -- 2.32.0 ^ permalink raw reply related [flat|nested] 9+ messages in thread
end of thread, other threads:[~2021-06-15 12:39 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20210615094933.3076392-1-ch@denx.de>
2021-06-15 9:49 ` [PATCH 1/3] ASoC: tlv320aic32x4: prepare driver for different device variants Claudius Heine
2021-06-15 11:54 ` Mark Brown
2021-06-15 12:06 ` Claudius Heine
2021-06-15 12:30 ` Mark Brown
2021-06-15 9:49 ` [PATCH 2/3] ASoC: tlv320aic32x4: add support for TAS2505 Claudius Heine
2021-06-15 11:59 ` Claudius Heine
2021-06-15 12:22 ` Mark Brown
2021-06-15 12:39 ` Claudius Heine
2021-06-15 9:49 ` [PATCH 3/3] ASoC: tlv320aic32x4: dt-bindings: add TAS2505 to compatible Claudius Heine
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox