lkml.org 
[lkml]   [2021]   [Feb]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v1 2/2] ASoC: tegra: Add RT5631 machine driver
From
Date

On 31/01/2021 18:41, Ion Agorria wrote:
> From: Svyatoslav Ryhel <clamor95@gmail.com>
>
> Add Tegra ASoC driver for Realtek ALC5631/RT5631 codec. The RT5631
> codec is found on devices like ASUS Transformer TF201, TF700T and other
> Tegra-based Android tablets.
>
> Signed-off-by: Svyatoslav Ryhel <clamor95@gmail.com>
> Signed-off-by: Ion Agorria <ion@agorria.com>
> ---
> sound/soc/tegra/Kconfig | 8 +
> sound/soc/tegra/Makefile | 2 +
> sound/soc/tegra/tegra_rt5631.c | 261 +++++++++++++++++++++++++++++++++
> 3 files changed, 271 insertions(+)
> create mode 100644 sound/soc/tegra/tegra_rt5631.c
>
> diff --git a/sound/soc/tegra/Kconfig b/sound/soc/tegra/Kconfig
> index acaf7339168d..449a858f155d 100644
> --- a/sound/soc/tegra/Kconfig
> +++ b/sound/soc/tegra/Kconfig
> @@ -126,6 +126,14 @@ config SND_SOC_TEGRA_AUDIO_GRAPH_CARD
> few things for Tegra audio. Most of the code is re-used from
> audio graph driver and the same DT bindings are used.
>
> +config SND_SOC_TEGRA_RT5631
> + tristate "SoC Audio support for Tegra boards using an RT5631 codec"
> + depends on SND_SOC_TEGRA && I2C && GPIOLIB
> + select SND_SOC_RT5631
> + help
> + Say Y or M here if you want to add support for SoC audio on Tegra
> + boards using the RT5631 codec, such as Transformer.
> +
> config SND_SOC_TEGRA_RT5640
> tristate "SoC Audio support for Tegra boards using an RT5640 codec"
> depends on SND_SOC_TEGRA && I2C && GPIOLIB
> diff --git a/sound/soc/tegra/Makefile b/sound/soc/tegra/Makefile
> index af0b9889306c..11debfc03bc4 100644
> --- a/sound/soc/tegra/Makefile
> +++ b/sound/soc/tegra/Makefile
> @@ -29,6 +29,7 @@ obj-$(CONFIG_SND_SOC_TEGRA186_DSPK) += snd-soc-tegra186-dspk.o
> obj-$(CONFIG_SND_SOC_TEGRA210_ADMAIF) += snd-soc-tegra210-admaif.o
>
> # Tegra machine Support
> +snd-soc-tegra-rt5631-objs := tegra_rt5631.o
> snd-soc-tegra-rt5640-objs := tegra_rt5640.o
> snd-soc-tegra-rt5677-objs := tegra_rt5677.o
> snd-soc-tegra-wm8753-objs := tegra_wm8753.o
> @@ -41,6 +42,7 @@ snd-soc-tegra-max98090-objs := tegra_max98090.o
> snd-soc-tegra-sgtl5000-objs := tegra_sgtl5000.o
> snd-soc-tegra-audio-graph-card-objs := tegra_audio_graph_card.o
>
> +obj-$(CONFIG_SND_SOC_TEGRA_RT5631) += snd-soc-tegra-rt5631.o
> obj-$(CONFIG_SND_SOC_TEGRA_RT5640) += snd-soc-tegra-rt5640.o
> obj-$(CONFIG_SND_SOC_TEGRA_RT5677) += snd-soc-tegra-rt5677.o
> obj-$(CONFIG_SND_SOC_TEGRA_WM8753) += snd-soc-tegra-wm8753.o
> diff --git a/sound/soc/tegra/tegra_rt5631.c b/sound/soc/tegra/tegra_rt5631.c
> new file mode 100644
> index 000000000000..9034f48bcb26
> --- /dev/null
> +++ b/sound/soc/tegra/tegra_rt5631.c
> @@ -0,0 +1,261 @@
> +// SPDX-License-Identifier: GPL-2.0-only
> +/*
> + * tegra_rt5631.c - Tegra machine ASoC driver for boards using RT5631 codec.
> + *
> + * Copyright (c) 2020, Svyatoslav Ryhel and Ion Agorria

2021 now :-)

> + *
> + * Based on code copyright/by:
> + *
> + * Copyright (c) 2012, NVIDIA CORPORATION. All rights reserved.
> + * Author: Stephen Warren <swarren@nvidia.com>
> + */
> +
> +#include <linux/gpio/consumer.h>
> +#include <linux/module.h>
> +#include <linux/of.h>
> +#include <linux/of_device.h>
> +#include <linux/platform_device.h>
> +#include <linux/slab.h>
> +
> +#include <sound/core.h>
> +#include <sound/jack.h>
> +#include <sound/pcm.h>
> +#include <sound/pcm_params.h>
> +#include <sound/soc.h>
> +
> +#include "tegra_asoc_utils.h"
> +
> +#include "../codecs/rt5631.h"
> +
> +struct tegra_rt5631 {
> + struct tegra_asoc_utils_data util_data;
> + struct gpio_desc *gpiod_hp_mute;
> + struct gpio_desc *gpiod_hp_det;
> +};
> +
> +static int tegra_rt5631_hw_params(struct snd_pcm_substream *substream,
> + struct snd_pcm_hw_params *params)
> +{
> + struct snd_soc_pcm_runtime *rtd = substream->private_data;
> + struct snd_soc_dai *codec_dai = asoc_rtd_to_codec(rtd, 0);
> + struct snd_soc_card *card = rtd->card;
> + struct tegra_rt5631 *machine = snd_soc_card_get_drvdata(card);
> + unsigned int srate, mclk;
> + int err;
> +
> + srate = params_rate(params);
> + switch (srate) {
> + case 64000:
> + case 88200:
> + case 96000:
> + mclk = 128 * srate;
> + break;
> + default:
> + mclk = 256 * srate;
> + break;
> + }
> + /* FIXME: Codec only requires >= 3MHz if OSR==0 */
> + while (mclk < 6000000)
> + mclk *= 2;

I don't understand that above. You say greater or equal to 3MHz, but
then compare against 6MHz. Please explain and elborate a bit more in the
comment.

> +
> + err = tegra_asoc_utils_set_rate(&machine->util_data, srate, mclk);
> + if (err < 0) {
> + dev_err(card->dev, "Can't configure clocks\n");
> + return err;
> + }
> +
> + err = snd_soc_dai_set_sysclk(codec_dai, 0, mclk, SND_SOC_CLOCK_IN);
> + if (err < 0) {
> + dev_err(card->dev, "codec_dai clock not set\n");
> + return err;
> + }
> +
> + return 0;
> +}
> +
> +static struct snd_soc_ops tegra_rt5631_ops = {
> + .hw_params = tegra_rt5631_hw_params,
> +};
> +
> +static struct snd_soc_jack tegra_rt5631_hp_jack;
> +
> +static struct snd_soc_jack_pin tegra_rt5631_hp_jack_pins[] = {
> + {
> + .pin = "Headphone Jack",
> + .mask = SND_JACK_HEADPHONE,
> + },
> +};
> +
> +static struct snd_soc_jack_gpio tegra_rt5631_hp_jack_gpio = {
> + .name = "Headphone detection",
> + .report = SND_JACK_HEADPHONE,
> + .debounce_time = 150,
> +};
> +
> +static int tegra_rt5631_event_hp(struct snd_soc_dapm_widget *w,
> + struct snd_kcontrol *k, int event)
> +{
> + struct snd_soc_dapm_context *dapm = w->dapm;
> + struct snd_soc_card *card = dapm->card;
> + struct tegra_rt5631 *machine = snd_soc_card_get_drvdata(card);
> +
> + gpiod_set_value_cansleep(machine->gpiod_hp_mute,
> + !SND_SOC_DAPM_EVENT_ON(event));
> +
> + return 0;
> +}
> +
> +static const struct snd_soc_dapm_widget tegra_rt5631_dapm_widgets[] = {
> + SND_SOC_DAPM_SPK("Int Spk", NULL),
> + SND_SOC_DAPM_HP("Headphone Jack", tegra_rt5631_event_hp),
> + SND_SOC_DAPM_MIC("Mic Jack", NULL),
> + SND_SOC_DAPM_MIC("Int Mic", NULL),
> +};
> +
> +static const struct snd_kcontrol_new tegra_rt5631_controls[] = {
> + SOC_DAPM_PIN_SWITCH("Int Spk"),
> + SOC_DAPM_PIN_SWITCH("Int Mic"),
> +};
> +
> +static int tegra_rt5631_init(struct snd_soc_pcm_runtime *rtd)
> +{
> + struct tegra_rt5631 *machine = snd_soc_card_get_drvdata(rtd->card);
> + int ret;
> +
> + ret = snd_soc_card_jack_new(rtd->card, "Headphone Jack",
> + SND_JACK_HEADPHONE,
> + &tegra_rt5631_hp_jack,
> + tegra_rt5631_hp_jack_pins,
> + ARRAY_SIZE(tegra_rt5631_hp_jack_pins));
> + if (ret) {
> + dev_err(rtd->dev, "Headset Jack creation failed: %d\n", ret);
> + return ret;
> + }
> +
> + if (machine->gpiod_hp_det) {
> + tegra_rt5631_hp_jack_gpio.desc = machine->gpiod_hp_det;
> +
> + ret = snd_soc_jack_add_gpios(&tegra_rt5631_hp_jack, 1,
> + &tegra_rt5631_hp_jack_gpio);
> + if (ret)
> + dev_err(rtd->dev, "Jack GPIOs not added: %d\n", ret);
> + }
> +
> + return 0;
> +}
> +
> +SND_SOC_DAILINK_DEFS(hifi,
> + DAILINK_COMP_ARRAY(COMP_EMPTY()),
> + DAILINK_COMP_ARRAY(COMP_CODEC(NULL, "rt5631-hifi")),
> + DAILINK_COMP_ARRAY(COMP_EMPTY()));
> +
> +static struct snd_soc_dai_link tegra_rt5631_dai = {
> + .name = "RT5631",
> + .stream_name = "RT5631 PCM",
> + .init = tegra_rt5631_init,
> + .ops = &tegra_rt5631_ops,
> + .dai_fmt = SND_SOC_DAIFMT_I2S |
> + SND_SOC_DAIFMT_NB_NF |
> + SND_SOC_DAIFMT_CBS_CFS,
> + SND_SOC_DAILINK_REG(hifi),
> +};
> +
> +static struct snd_soc_card snd_soc_tegra_rt5631 = {
> + .name = "tegra-rt5631",
> + .owner = THIS_MODULE,
> + .dai_link = &tegra_rt5631_dai,
> + .num_links = 1,
> + .controls = tegra_rt5631_controls,
> + .num_controls = ARRAY_SIZE(tegra_rt5631_controls),
> + .dapm_widgets = tegra_rt5631_dapm_widgets,
> + .num_dapm_widgets = ARRAY_SIZE(tegra_rt5631_dapm_widgets),
> + .fully_routed = true,
> +};
> +
> +static int tegra_rt5631_probe(struct platform_device *pdev)
> +{
> + struct snd_soc_card *card = &snd_soc_tegra_rt5631;
> + struct device_node *np_codec, *np_i2s;
> + struct tegra_rt5631 *machine;
> + struct gpio_desc *gpiod;
> + int ret;
> +
> + machine = devm_kzalloc(&pdev->dev, sizeof(*machine), GFP_KERNEL);
> + if (!machine)
> + return -ENOMEM;
> +
> + card->dev = &pdev->dev;
> + snd_soc_card_set_drvdata(card, machine);
> +
> + gpiod = devm_gpiod_get_optional(&pdev->dev, "nvidia,hp-mute",
> + GPIOD_OUT_HIGH);
> + if (IS_ERR(gpiod))
> + return PTR_ERR(gpiod);
> +
> + machine->gpiod_hp_mute = gpiod;
> +
> + gpiod = devm_gpiod_get_optional(&pdev->dev, "nvidia,hp-det",
> + GPIOD_IN);
> + if (IS_ERR(gpiod))
> + return PTR_ERR(gpiod);
> +
> + machine->gpiod_hp_det = gpiod;
> +
> + ret = snd_soc_of_parse_card_name(card, "nvidia,model");
> + if (ret)
> + return ret;
> +
> + ret = snd_soc_of_parse_audio_routing(card, "nvidia,audio-routing");
> + if (ret)
> + return ret;
> +
> + np_codec = of_parse_phandle(pdev->dev.of_node, "nvidia,audio-codec", 0);
> + if (!np_codec) {
> + dev_err(&pdev->dev,
> + "Property 'nvidia,audio-codec' missing or invalid\n");
> + return -EINVAL;
> + }
> +
> + np_i2s = of_parse_phandle(pdev->dev.of_node, "nvidia,i2s-controller", 0);
> + if (!np_i2s) {
> + dev_err(&pdev->dev,
> + "Property 'nvidia,i2s-controller' missing or invalid\n");
> + return -EINVAL;
> + }
> +
> + tegra_rt5631_dai.cpus->of_node = np_i2s;
> + tegra_rt5631_dai.codecs->of_node = np_codec;
> + tegra_rt5631_dai.platforms->of_node = np_i2s;
> +
> + ret = tegra_asoc_utils_init(&machine->util_data, &pdev->dev);
> + if (ret)
> + return ret;
> +
> + ret = devm_snd_soc_register_card(&pdev->dev, card);
> + if (ret)
> + return ret;
> +
> + return 0;
> +}
> +
> +static const struct of_device_id tegra_rt5631_of_match[] = {
> + { .compatible = "nvidia,tegra-audio-rt5631", },
> + {},
> +};
> +MODULE_DEVICE_TABLE(of, tegra_rt5631_of_match);
> +
> +static struct platform_driver tegra_rt5631_driver = {
> + .driver = {
> + .name = "tegra-snd-rt5631",
> + .pm = &snd_soc_pm_ops,
> + .of_match_table = tegra_rt5631_of_match,
> + },
> + .probe = tegra_rt5631_probe,
> +};
> +module_platform_driver(tegra_rt5631_driver);
> +
> +MODULE_DESCRIPTION("Tegra+RT5631 machine ASoC driver");
> +MODULE_AUTHOR("Stephen Warren <swarren@nvidia.com>");
> +MODULE_AUTHOR("Svyatoslav Ryhel <clamor95@gmail.com>");
> +MODULE_AUTHOR("Ion Agorria <ion@agorria.com>");
> +MODULE_LICENSE("GPL");


So this is very similar to tegra_rt5677, is it not possible to support
both codecs with the same machine driver?

Jon

--
nvpublic

\
 
 \ /
  Last update: 2021-02-02 14:25    [W:0.118 / U:0.480 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site