[alsa-devel] [PATCH] ASoC: fsl: add imx-wm8960 machine driver
This imx-wm8960 device-tree-only machine driver works with sai driver and have below feature. * support codec master and slave mode * support headphone jack detect * support headphone and micphone jack event * support asrc<->sai<->wm8960 mode
Signed-off-by: Zidan Wang zidan.wang@freescale.com --- .../devicetree/bindings/sound/imx-audio-wm8960.txt | 68 ++ sound/soc/fsl/Kconfig | 13 + sound/soc/fsl/Makefile | 2 + sound/soc/fsl/imx-wm8960.c | 711 +++++++++++++++++++++ 4 files changed, 794 insertions(+) create mode 100644 Documentation/devicetree/bindings/sound/imx-audio-wm8960.txt create mode 100644 sound/soc/fsl/imx-wm8960.c
diff --git a/Documentation/devicetree/bindings/sound/imx-audio-wm8960.txt b/Documentation/devicetree/bindings/sound/imx-audio-wm8960.txt new file mode 100644 index 0000000..300d027 --- /dev/null +++ b/Documentation/devicetree/bindings/sound/imx-audio-wm8960.txt @@ -0,0 +1,68 @@ +Freescale i.MX audio complex with WM8960 codec + +Required properties: + + - compatible : "fsl,imx-audio-wm8960" + + - model : The user-visible name of this sound complex + + - audio-codec : The phandle of the WM8960 audio codec + + - hp-det : ADCLRC/GPIO1, LINPUT3/JD2 and RINPUT3/JD3 pins can + be selected as headphone jack detect inputs to + automatically disable the speaker output and enable + the headphone output. + hp-det = <hp-det-pin hp-det-polarity>; + hp-det-pin = 1: ADCLRC/GPIO1 used as detect pin + hp-det-pin = 2: LINPUT3/JD2 used as detect pin + hp-det-pin = 3: RINPUT3/JD3 used as detect pin + hp-det-polarity = 0: hp detect high for headphone + hp-det-polarity = 1: hp detect high for speaker + + - codec-master : If "codec-master" present, codec works as master. + Otherwise, codec works as slave. + + - audio-routing : A list of the connections between audio components. + Each entry is a pair of strings, the first being the + connection's sink, the second being the connection's + source. Valid names could be power supplies, WM8962 + pins, and the jacks on the board: + + Power supplies: + * MICB + + Board connectors: + * Hp MIC + * Main MIC + * Headset Jack + * Ext Spk + +Example: + +sound { + compatible = "fsl,imx6ul-evk-wm8960", + "fsl,imx-audio-wm8960"; + model = "wm8960-audio"; + cpu-dai = <&sai2>; + audio-codec = <&codec>; + asrc-controller = <&asrc>; + codec-master; + hp-det = <3 0>; + audio-routing = + "Headset Jack", "HP_L", + "Headset Jack", "HP_R", + "Ext Spk", "SPK_LP", + "Ext Spk", "SPK_LN", + "Ext Spk", "SPK_RP", + "Ext Spk", "SPK_RN", + "LINPUT2", "Hp MIC", + "LINPUT3", "Hp MIC", + "RINPUT1", "Main MIC", + "RINPUT2", "Main MIC", + "Hp MIC", "MICB", + "Main MIC", "MICB", + "CPU-Playback", "ASRC-Playback", + "Playback", "CPU-Playback", + "ASRC-Capture", "CPU-Capture", + "CPU-Capture", "Capture"; +}; diff --git a/sound/soc/fsl/Kconfig b/sound/soc/fsl/Kconfig index 19c302b..0d9aa56 100644 --- a/sound/soc/fsl/Kconfig +++ b/sound/soc/fsl/Kconfig @@ -229,6 +229,19 @@ config SND_SOC_EUKREA_TLV320 Enable I2S based access to the TLV320AIC23B codec attached to the SSI interface
+config SND_SOC_IMX_WM8960 + tristate "SoC Audio support for i.MX boards with wm8960" + depends on OF && I2C + select SND_SOC_WM8960 + select SND_SOC_IMX_PCM_DMA + select SND_SOC_FSL_SAI + select SND_SOC_FSL_UTILS + select SND_KCTL_JACK + help + SoC Audio support for i.MX boards with WM8960 + Say Y if you want to add support for SoC audio on an i.MX board with + a wm8960 codec. + config SND_SOC_IMX_WM8962 tristate "SoC Audio support for i.MX boards with wm8962" depends on OF && I2C && INPUT diff --git a/sound/soc/fsl/Makefile b/sound/soc/fsl/Makefile index d28dc25..84a9a10 100644 --- a/sound/soc/fsl/Makefile +++ b/sound/soc/fsl/Makefile @@ -54,6 +54,7 @@ snd-soc-mx27vis-aic32x4-objs := mx27vis-aic32x4.o snd-soc-wm1133-ev1-objs := wm1133-ev1.o snd-soc-imx-es8328-objs := imx-es8328.o snd-soc-imx-sgtl5000-objs := imx-sgtl5000.o +snd-soc-imx-wm8960-objs := imx-wm8960.o snd-soc-imx-wm8962-objs := imx-wm8962.o snd-soc-imx-spdif-objs := imx-spdif.o snd-soc-imx-mc13783-objs := imx-mc13783.o @@ -64,6 +65,7 @@ obj-$(CONFIG_SND_SOC_MX27VIS_AIC32X4) += snd-soc-mx27vis-aic32x4.o obj-$(CONFIG_SND_MXC_SOC_WM1133_EV1) += snd-soc-wm1133-ev1.o obj-$(CONFIG_SND_SOC_IMX_ES8328) += snd-soc-imx-es8328.o obj-$(CONFIG_SND_SOC_IMX_SGTL5000) += snd-soc-imx-sgtl5000.o +obj-$(CONFIG_SND_SOC_IMX_WM8960) += snd-soc-imx-wm8960.o obj-$(CONFIG_SND_SOC_IMX_WM8962) += snd-soc-imx-wm8962.o obj-$(CONFIG_SND_SOC_IMX_SPDIF) += snd-soc-imx-spdif.o obj-$(CONFIG_SND_SOC_IMX_MC13783) += snd-soc-imx-mc13783.o diff --git a/sound/soc/fsl/imx-wm8960.c b/sound/soc/fsl/imx-wm8960.c new file mode 100644 index 0000000..ddd17cb --- /dev/null +++ b/sound/soc/fsl/imx-wm8960.c @@ -0,0 +1,711 @@ +/* + * Copyright (C) 2015 Freescale Semiconductor, Inc. + * + * The code contained herein is licensed under the GNU General Public + * License. You may obtain a copy of the GNU General Public License + * Version 2 or later at the following locations: + * + * http://www.opensource.org/licenses/gpl-license.html + * http://www.gnu.org/copyleft/gpl.html + */ + +#include <linux/module.h> +#include <linux/of_platform.h> +#include <linux/i2c.h> +#include <linux/of_gpio.h> +#include <linux/slab.h> +#include <linux/gpio.h> +#include <linux/clk.h> +#include <sound/soc.h> +#include <sound/jack.h> +#include <sound/control.h> +#include <sound/pcm_params.h> +#include <sound/soc-dapm.h> +#include <linux/pinctrl/consumer.h> +#include <linux/mfd/syscon.h> +#include "../codecs/wm8960.h" + +#define DAI_NAME_SIZE 32 + +struct imx_wm8960_data { + struct snd_soc_card card; + char codec_dai_name[DAI_NAME_SIZE]; + char platform_name[DAI_NAME_SIZE]; + struct clk *codec_clk; + unsigned int clk_frequency; + bool is_codec_master; + bool is_stream_in_use[2]; + struct regmap *gpr; + unsigned int hp_det[2]; + u32 asrc_rate; + u32 asrc_format; +}; + +struct imx_priv { + int hp_set_gpio; + int hp_active_low; + struct snd_kcontrol *headset_kctl; + struct snd_soc_codec *codec; + struct platform_device *pdev; + struct platform_device *asrc_pdev; + struct snd_card *snd_card; +}; + +static struct imx_priv card_priv; + +static struct snd_soc_jack imx_hp_set; +static struct snd_soc_jack_pin imx_hp_set_pins[] = { + { + .pin = "Headset Jack", + .mask = SND_JACK_HEADSET, + }, +}; +static struct snd_soc_jack_gpio imx_hp_set_gpio = { + .name = "headset detect", + .report = SND_JACK_HEADSET, + .debounce_time = 250, + .invert = 1, +}; + +static int hp_set_status_check(void *data) +{ + struct imx_priv *priv = &card_priv; + struct platform_device *pdev = priv->pdev; + char *envp[3], *buf; + int hp_status, ret; + + if (!gpio_is_valid(priv->hp_set_gpio)) + return 0; + + hp_status = gpio_get_value(priv->hp_set_gpio) ? 1 : 0; + buf = kmalloc(32, GFP_ATOMIC); + if (!buf) { + dev_err(&pdev->dev, "%s kmalloc failed\n", __func__); + return -ENOMEM; + } + + if (hp_status != priv->hp_active_low) { + snprintf(buf, 32, "STATE=%d", 2); + snd_soc_dapm_disable_pin(&priv->codec->dapm, "Ext Spk"); + snd_soc_dapm_disable_pin(&priv->codec->dapm, "Main MIC"); + ret = imx_hp_set_gpio.report; + + /* + * As the hp MIC only connect the input for left channel, we + * need to route it for right channel. + */ + snd_soc_update_bits(priv->codec, WM8960_ADDCTL1, 3<<2, 1<<2); + + snd_kctl_jack_report(priv->snd_card, priv->headset_kctl, 1); + } else { + snprintf(buf, 32, "STATE=%d", 0); + snd_soc_dapm_enable_pin(&priv->codec->dapm, "Ext Spk"); + snd_soc_dapm_enable_pin(&priv->codec->dapm, "Main MIC"); + ret = 0; + + /* + * As the Main MIC only connect the input for right channel, + * we need to route it for left channel. + */ + snd_soc_update_bits(priv->codec, WM8960_ADDCTL1, 3<<2, 2<<2); + + snd_kctl_jack_report(priv->snd_card, priv->headset_kctl, 0); + } + + envp[0] = "NAME=headset"; + envp[1] = buf; + envp[2] = NULL; + kobject_uevent_env(&pdev->dev.kobj, KOBJ_CHANGE, envp); + kfree(buf); + + return ret; +} + +static const struct snd_soc_dapm_widget imx_wm8960_dapm_widgets[] = { + SND_SOC_DAPM_HP("Headset Jack", NULL), + SND_SOC_DAPM_SPK("Ext Spk", NULL), + SND_SOC_DAPM_MIC("Hp MIC", NULL), + SND_SOC_DAPM_MIC("Main MIC", NULL), +}; + +static int imx_wm8960_gpio_init(struct snd_soc_card *card) +{ + struct snd_soc_dai *codec_dai = card->rtd[0].codec_dai; + struct snd_soc_codec *codec = codec_dai->codec; + struct imx_priv *priv = &card_priv; + int ret; + + priv->codec = codec; + + if (gpio_is_valid(priv->hp_set_gpio)) { + imx_hp_set_gpio.gpio = priv->hp_set_gpio; + imx_hp_set_gpio.jack_status_check = hp_set_status_check; + + ret = snd_soc_card_jack_new(card, "Headset Jack", + SND_JACK_HEADSET, &imx_hp_set, + imx_hp_set_pins, ARRAY_SIZE(imx_hp_set_pins)); + if (ret) + return ret; + ret = snd_soc_jack_add_gpios(&imx_hp_set, 1, + &imx_hp_set_gpio); + if (ret) + return ret; + } + + return 0; +} + +static ssize_t show_headphone(struct device_driver *dev, char *buf) +{ + struct imx_priv *priv = &card_priv; + int hp_status; + + if (!gpio_is_valid(priv->hp_set_gpio)) { + strcpy(buf, "no detect gpio connected\n"); + return strlen(buf); + } + + /* Check if headphone is plugged in */ + hp_status = gpio_get_value(priv->hp_set_gpio) ? 1 : 0; + + if (hp_status != priv->hp_active_low) + strcpy(buf, "Headphone\n"); + else + strcpy(buf, "Speaker\n"); + + return strlen(buf); +} + +static ssize_t show_micphone(struct device_driver *dev, char *buf) +{ + struct imx_priv *priv = &card_priv; + int hp_status; + + if (!gpio_is_valid(priv->hp_set_gpio)) { + strcpy(buf, "no detect gpio connected\n"); + return strlen(buf); + } + + /* Check if headphone is plugged in */ + hp_status = gpio_get_value(priv->hp_set_gpio) ? 1 : 0; + + if (hp_status != priv->hp_active_low) + strcpy(buf, "Hp MIC\n"); + else + strcpy(buf, "Main MIC\n"); + + return strlen(buf); +} +static DRIVER_ATTR(headphone, S_IRUGO | S_IWUSR, show_headphone, NULL); +static DRIVER_ATTR(micphone, S_IRUGO | S_IWUSR, show_micphone, NULL); + +static void wm8960_init(struct snd_soc_card *card) +{ + struct snd_soc_dai *codec_dai = card->rtd[0].codec_dai; + struct snd_soc_codec *codec = codec_dai->codec; + struct imx_wm8960_data *data = snd_soc_card_get_drvdata(card); + + /* + * codec ADCLRC pin configured as GPIO, DACLRC pin is used as + * a frame clock for ADCs and DACs + */ + snd_soc_update_bits(codec, WM8960_IFACE2, 1<<6, 1<<6); + + /* GPIO1 used as headphone detect output */ + snd_soc_update_bits(codec, WM8960_ADDCTL4, 7<<4, 3<<4); + + /* Enable headphone jack detect */ + snd_soc_update_bits(codec, WM8960_ADDCTL2, 1<<6, 1<<6); + snd_soc_update_bits(codec, WM8960_ADDCTL2, 1<<5, data->hp_det[1]<<5); + snd_soc_update_bits(codec, WM8960_ADDCTL4, 3<<2, data->hp_det[0]<<2); + snd_soc_update_bits(codec, WM8960_ADDCTL1, 3, 3); + + /* route ADC left channel to right channel in default */ + snd_soc_update_bits(codec, WM8960_ADDCTL1, 3<<2, 1<<2); +} + +/* -1 for reserved value */ +static const int sysclk_divs[] = { 1, -1, 2, -1 }; + +/* Multiply 256 for internal 256 div */ +static const int dac_divs[] = { 256, 384, 512, 768, 1024, 1408, 1536 }; + +/* Multiply 10 to eliminate decimials */ +static const int bclk_divs[] = { + 10, 15, 20, 30, 40, 55, 60, 80, 110, + 120, 160, 220, 240, 320, 320, 320 +}; + +static int imx_hifi_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 = rtd->codec_dai; + struct snd_soc_dai *cpu_dai = rtd->cpu_dai; + struct snd_soc_card *card = rtd->card; + struct imx_wm8960_data *data = snd_soc_card_get_drvdata(card); + bool tx = substream->stream == SNDRV_PCM_STREAM_PLAYBACK; + struct device *dev = card->dev; + unsigned int sample_rate = params_rate(params); + unsigned int sysclk, pll_in, pll_out; + int i, j, k, ret = 0; + int bclk = snd_soc_params_to_bclk(params); + unsigned int fmt, plln; + + if (params_channels(params) == 1) + bclk *= 2; + + data->is_stream_in_use[tx] = true; + + if (data->is_stream_in_use[!tx]) + return 0; + + if (data->is_codec_master) + fmt = SND_SOC_DAIFMT_I2S | SND_SOC_DAIFMT_NB_NF | + SND_SOC_DAIFMT_CBM_CFM; + else + fmt = SND_SOC_DAIFMT_I2S | SND_SOC_DAIFMT_NB_NF | + SND_SOC_DAIFMT_CBS_CFS; + + /* set cpu DAI configuration */ + ret = snd_soc_dai_set_fmt(cpu_dai, fmt); + if (ret) { + dev_err(dev, "failed to set cpu dai fmt: %d\n", ret); + return ret; + } + /* set codec DAI configuration */ + ret = snd_soc_dai_set_fmt(codec_dai, fmt); + if (ret) { + dev_err(dev, "failed to set codec dai fmt: %d\n", ret); + return ret; + } + + if (!data->is_codec_master) { + ret = snd_soc_dai_set_tdm_slot(cpu_dai, + 0, 0, 2, params_width(params)); + if (ret) { + dev_err(dev, + "failed to set cpu dai tdm slot: %d\n", ret); + return ret; + } + + ret = snd_soc_dai_set_sysclk(cpu_dai, 0, 0, SND_SOC_CLOCK_OUT); + if (ret) { + dev_err(dev, "failed to set cpu sysclk: %d\n", ret); + return ret; + } + return 0; + } else { + ret = snd_soc_dai_set_sysclk(cpu_dai, 0, 0, SND_SOC_CLOCK_IN); + if (ret) { + dev_err(dev, "failed to set cpu sysclk: %d\n", ret); + return ret; + } + } + + data->clk_frequency = clk_get_rate(data->codec_clk); + + /* Use MCLK to provide sysclk directly */ + sysclk = data->clk_frequency; + + for (i = 0; i < ARRAY_SIZE(sysclk_divs); ++i) { + if (sysclk_divs[i] == -1) + continue; + sysclk /= sysclk_divs[i]; + for (j = 0; j < ARRAY_SIZE(dac_divs); ++j) { + if (sysclk == sample_rate * dac_divs[j]) { + for (k = 0; k < ARRAY_SIZE(bclk_divs); ++k) + if (sysclk == bclk * bclk_divs[k] / 10) + break; + if (k != ARRAY_SIZE(bclk_divs)) + break; + } + } + if (j != ARRAY_SIZE(dac_divs)) + break; + } + if (i != ARRAY_SIZE(sysclk_divs)) { + /* Set codec sysclk */ + snd_soc_dai_set_sysclk(codec_dai, + WM8960_SYSCLK_MCLK, sysclk, 0); + snd_soc_dai_set_clkdiv(codec_dai, WM8960_SYSCLKDIV, i << 1); + return 0; + } + + /* Use PLL to provide sysclk */ + pll_in = data->clk_frequency; + + for (i = 0; i < ARRAY_SIZE(sysclk_divs); ++i) { + if (sysclk_divs[i] == -1) + continue; + for (j = 0; j < ARRAY_SIZE(dac_divs); ++j) { + sysclk = sample_rate * dac_divs[j]; + pll_out = sysclk * sysclk_divs[i]; + /* + * we should make sure 5 < plln < 13, or we can't get + * such pll_out. + */ + plln = pll_out * 4 / pll_in; + if (plln < 6) + plln *= 2; + if ((plln < 6) || (plln > 12)) + continue; + for (k = 0; k < ARRAY_SIZE(bclk_divs); ++k) { + if (sysclk == bclk * bclk_divs[k] / 10) { + /* Set codec pll */ + ret = snd_soc_dai_set_pll(codec_dai, 0, + 0, data->clk_frequency, + pll_out); + if (ret != 0) + continue; + else + break; + } + } + if (k != ARRAY_SIZE(bclk_divs)) + break; + } + if (j != ARRAY_SIZE(dac_divs)) + break; + } + if (i != ARRAY_SIZE(sysclk_divs)) { + /* Set codec sysclk */ + snd_soc_dai_set_sysclk(codec_dai, WM8960_SYSCLK_PLL, sysclk, 0); + snd_soc_dai_set_clkdiv(codec_dai, WM8960_SYSCLKDIV, i << 1); + return 0; + } + + dev_err(dev, "failed to configure system clock"); + return -EINVAL; +} + +static int imx_hifi_hw_free(struct snd_pcm_substream *substream) +{ + struct snd_soc_pcm_runtime *rtd = substream->private_data; + struct snd_soc_dai *codec_dai = rtd->codec_dai; + struct snd_soc_card *card = rtd->card; + struct imx_wm8960_data *data = snd_soc_card_get_drvdata(card); + bool tx = substream->stream == SNDRV_PCM_STREAM_PLAYBACK; + + data->is_stream_in_use[tx] = false; + + /* Power down PLL to save power*/ + if (data->is_codec_master && !data->is_stream_in_use[!tx]) + snd_soc_dai_set_pll(codec_dai, 0, 0, 0, 0); + + return 0; +} + +static u32 imx_wm8960_rates[] = { 8000, 16000, 32000, 48000 }; +static struct snd_pcm_hw_constraint_list imx_wm8960_rate_constraints = { + .count = ARRAY_SIZE(imx_wm8960_rates), + .list = imx_wm8960_rates, +}; + +static int imx_hifi_startup(struct snd_pcm_substream *substream) +{ + struct snd_soc_pcm_runtime *rtd = substream->private_data; + struct snd_soc_card *card = rtd->card; + struct imx_wm8960_data *data = snd_soc_card_get_drvdata(card); + int ret = 0; + + if (!data->is_codec_master) { + ret = snd_pcm_hw_constraint_list(substream->runtime, 0, + SNDRV_PCM_HW_PARAM_RATE, + &imx_wm8960_rate_constraints); + if (ret) + return ret; + } + + /* codec mclk should be enabled early to avoid jack detect error */ + ret = clk_prepare_enable(data->codec_clk); + if (ret) { + dev_err(card->dev, "Failed to enable MCLK: %d\n", ret); + return ret; + } + + return ret; +} + +static void imx_hifi_shutdown(struct snd_pcm_substream *substream) +{ + struct snd_soc_pcm_runtime *rtd = substream->private_data; + struct snd_soc_card *card = rtd->card; + struct imx_wm8960_data *data = snd_soc_card_get_drvdata(card); + + clk_disable_unprepare(data->codec_clk); +} + +static struct snd_soc_ops imx_hifi_ops = { + .hw_params = imx_hifi_hw_params, + .hw_free = imx_hifi_hw_free, + .startup = imx_hifi_startup, + .shutdown = imx_hifi_shutdown, +}; + +static int imx_wm8960_late_probe(struct snd_soc_card *card) +{ + struct imx_wm8960_data *data = snd_soc_card_get_drvdata(card); + + /* set SAI2_MCLK_DIR to enable codec MCLK */ + if (data->gpr) + regmap_update_bits(data->gpr, 4, 1<<20, 1<<20); + + wm8960_init(card); + return 0; +} + +static int be_hw_params_fixup(struct snd_soc_pcm_runtime *rtd, + struct snd_pcm_hw_params *params) +{ + struct snd_soc_card *card = rtd->card; + struct imx_wm8960_data *data = snd_soc_card_get_drvdata(card); + struct imx_priv *priv = &card_priv; + struct snd_interval *rate; + struct snd_mask *mask; + + if (!priv->asrc_pdev) + return -EINVAL; + + rate = hw_param_interval(params, SNDRV_PCM_HW_PARAM_RATE); + rate->max = rate->min = data->asrc_rate; + + mask = hw_param_mask(params, SNDRV_PCM_HW_PARAM_FORMAT); + snd_mask_none(mask); + snd_mask_set(mask, data->asrc_format); + + return 0; +} + +static struct snd_soc_dai_link imx_wm8960_dai[] = { + { + .name = "HiFi", + .stream_name = "HiFi", + .codec_dai_name = "wm8960-hifi", + .ops = &imx_hifi_ops, + }, + { + .name = "HiFi-ASRC-FE", + .stream_name = "HiFi-ASRC-FE", + .codec_name = "snd-soc-dummy", + .codec_dai_name = "snd-soc-dummy-dai", + .dynamic = 1, + .ignore_pmdown_time = 1, + .dpcm_playback = 1, + .dpcm_capture = 1, + }, + { + .name = "HiFi-ASRC-BE", + .stream_name = "HiFi-ASRC-BE", + .codec_dai_name = "wm8960-hifi", + .platform_name = "snd-soc-dummy", + .no_pcm = 1, + .ignore_pmdown_time = 1, + .dpcm_playback = 1, + .dpcm_capture = 1, + .ops = &imx_hifi_ops, + .be_hw_params_fixup = be_hw_params_fixup, + }, +}; + +static int imx_wm8960_probe(struct platform_device *pdev) +{ + struct device_node *cpu_np, *codec_np, *gpr_np; + struct platform_device *cpu_pdev; + struct imx_priv *priv = &card_priv; + struct i2c_client *codec_dev; + struct imx_wm8960_data *data; + struct platform_device *asrc_pdev = NULL; + struct device_node *asrc_np; + u32 width; + int ret; + + priv->pdev = pdev; + + cpu_np = of_parse_phandle(pdev->dev.of_node, "cpu-dai", 0); + if (!cpu_np) { + dev_err(&pdev->dev, "cpu dai phandle missing or invalid\n"); + ret = -EINVAL; + goto fail; + } + + codec_np = of_parse_phandle(pdev->dev.of_node, "audio-codec", 0); + if (!codec_np) { + dev_err(&pdev->dev, "phandle missing or invalid\n"); + ret = -EINVAL; + goto fail; + } + + cpu_pdev = of_find_device_by_node(cpu_np); + if (!cpu_pdev) { + dev_err(&pdev->dev, "failed to find SAI platform device\n"); + ret = -EINVAL; + goto fail; + } + + codec_dev = of_find_i2c_device_by_node(codec_np); + if (!codec_dev || !codec_dev->dev.driver) { + dev_err(&pdev->dev, "failed to find codec platform device\n"); + ret = -EINVAL; + goto fail; + } + + data = devm_kzalloc(&pdev->dev, sizeof(*data), GFP_KERNEL); + if (!data) { + ret = -ENOMEM; + goto fail; + } + + if (of_property_read_bool(pdev->dev.of_node, "codec-master")) + data->is_codec_master = true; + + data->codec_clk = devm_clk_get(&codec_dev->dev, "mclk"); + if (IS_ERR(data->codec_clk)) { + ret = PTR_ERR(data->codec_clk); + dev_err(&pdev->dev, "failed to get codec clk: %d\n", ret); + goto fail; + } + + gpr_np = of_parse_phandle(pdev->dev.of_node, "gpr", 0); + if (gpr_np) { + data->gpr = syscon_node_to_regmap(gpr_np); + if (IS_ERR(data->gpr)) { + ret = PTR_ERR(data->gpr); + dev_err(&pdev->dev, "failed to get gpr regmap\n"); + goto fail; + } + } + + /* + * hp-det = <hp-det-pin hp-det-polarity>; + * hp-det-pin: JD1 JD2 or JD3 + * hp-det-polarity = 0: hp detect high for headphone + * hp-det-polarity = 1: hp detect high for speaker + */ + of_property_read_u32_array(pdev->dev.of_node, + "hp-det", data->hp_det, 2); + + asrc_np = of_parse_phandle(pdev->dev.of_node, "asrc-controller", 0); + if (asrc_np) { + asrc_pdev = of_find_device_by_node(asrc_np); + priv->asrc_pdev = asrc_pdev; + } + + data->card.dai_link = imx_wm8960_dai; + + imx_wm8960_dai[0].codec_of_node = codec_np; + imx_wm8960_dai[0].cpu_dai_name = dev_name(&cpu_pdev->dev); + imx_wm8960_dai[0].platform_of_node = cpu_np; + + if (!asrc_pdev) { + data->card.num_links = 1; + } else { + imx_wm8960_dai[1].cpu_of_node = asrc_np; + imx_wm8960_dai[1].platform_of_node = asrc_np; + imx_wm8960_dai[2].codec_of_node = codec_np; + imx_wm8960_dai[2].cpu_dai_name = dev_name(&cpu_pdev->dev); + data->card.num_links = 3; + + ret = of_property_read_u32(asrc_np, "fsl,asrc-rate", + &data->asrc_rate); + if (ret) { + dev_err(&pdev->dev, "failed to get output rate\n"); + ret = -EINVAL; + goto fail; + } + + ret = of_property_read_u32(asrc_np, "fsl,asrc-width", &width); + if (ret) { + dev_err(&pdev->dev, "failed to get output rate\n"); + ret = -EINVAL; + goto fail; + } + + if (width == 24) + data->asrc_format = SNDRV_PCM_FORMAT_S24_LE; + else + data->asrc_format = SNDRV_PCM_FORMAT_S16_LE; + } + + data->card.dev = &pdev->dev; + ret = snd_soc_of_parse_card_name(&data->card, "model"); + if (ret) + goto fail; + data->card.dapm_widgets = imx_wm8960_dapm_widgets; + data->card.num_dapm_widgets = ARRAY_SIZE(imx_wm8960_dapm_widgets); + + ret = snd_soc_of_parse_audio_routing(&data->card, "audio-routing"); + if (ret) + goto fail; + + data->card.late_probe = imx_wm8960_late_probe; + + platform_set_drvdata(pdev, &data->card); + snd_soc_card_set_drvdata(&data->card, data); + ret = devm_snd_soc_register_card(&pdev->dev, &data->card); + if (ret) { + dev_err(&pdev->dev, "failed to register card (%d)\n", ret); + goto fail; + } + + priv->snd_card = data->card.snd_card; + + priv->hp_set_gpio = of_get_named_gpio_flags(pdev->dev.of_node, + "hp-det-gpios", 0, + (enum of_gpio_flags *)&priv->hp_active_low); + if (IS_ERR(ERR_PTR(priv->hp_set_gpio))) + goto fail; + + priv->headset_kctl = snd_kctl_jack_new("Headset", priv->snd_card); + ret = snd_ctl_add(data->card.snd_card, priv->headset_kctl); + if (ret) + goto fail; + + ret = imx_wm8960_gpio_init(&data->card); + + if (gpio_is_valid(priv->hp_set_gpio)) { + ret = driver_create_file(pdev->dev.driver, + &driver_attr_headphone); + if (ret) { + dev_err(&pdev->dev, + "create hp attr failed (%d)\n", ret); + goto fail; + } + ret = driver_create_file(pdev->dev.driver, + &driver_attr_micphone); + if (ret) { + dev_err(&pdev->dev, + "create mic attr failed (%d)\n", ret); + goto fail; + } + } +fail: + if (cpu_np) + of_node_put(cpu_np); + if (codec_np) + of_node_put(codec_np); + + return ret; +} + +static const struct of_device_id imx_wm8960_dt_ids[] = { + { .compatible = "fsl,imx-audio-wm8960", }, + { /* sentinel */ } +}; +MODULE_DEVICE_TABLE(of, imx_wm8960_dt_ids); + +static struct platform_driver imx_wm8960_driver = { + .driver = { + .name = "imx-wm8960", + .owner = THIS_MODULE, + .pm = &snd_soc_pm_ops, + .of_match_table = imx_wm8960_dt_ids, + }, + .probe = imx_wm8960_probe, +}; +module_platform_driver(imx_wm8960_driver); + +MODULE_AUTHOR("Freescale Semiconductor, Inc."); +MODULE_DESCRIPTION("Freescale i.MX WM8960 ASoC machine driver"); +MODULE_LICENSE("GPL v2"); +MODULE_ALIAS("platform:imx-wm8960");
On Wed, Jun 10, 2015 at 04:26:27PM +0800, Zidan Wang wrote:
- hp-det : ADCLRC/GPIO1, LINPUT3/JD2 and RINPUT3/JD3 pins can
be selected as headphone jack detect inputs to
automatically disable the speaker output and enable
the headphone output.
hp-det = <hp-det-pin hp-det-polarity>;
hp-det-pin = 1: ADCLRC/GPIO1 used as detect pin
hp-det-pin = 2: LINPUT3/JD2 used as detect pin
hp-det-pin = 3: RINPUT3/JD3 used as detect pin
hp-det-polarity = 0: hp detect high for headphone
hp-det-polarity = 1: hp detect high for speaker
This looks like something that should be in the DT binding for the CODEC, not the machine driver.
+static int hp_set_status_check(void *data) +{
Why is this in the driver?
- hp_status = gpio_get_value(priv->hp_set_gpio) ? 1 : 0;
gpio_get_value() already returns a boolean.
- if (hp_status != priv->hp_active_low) {
snprintf(buf, 32, "STATE=%d", 2);
snd_soc_dapm_disable_pin(&priv->codec->dapm, "Ext Spk");
snd_soc_dapm_disable_pin(&priv->codec->dapm, "Main MIC");
ret = imx_hp_set_gpio.report;
The generic jack code already has support for disabling and enabling pins, though disabling the speaker looks like a policy decision which probably doesn't belong here.
/*
* As the hp MIC only connect the input for left channel, we
* need to route it for right channel.
*/
snd_soc_update_bits(priv->codec, WM8960_ADDCTL1, 3<<2, 1<<2);
This looks like routing which we'd expect userspace to be doing.
snd_kctl_jack_report(priv->snd_card, priv->headset_kctl, 1);
Use the ASoC level helpers.
- envp[0] = "NAME=headset";
- envp[1] = buf;
- envp[2] = NULL;
- kobject_uevent_env(&pdev->dev.kobj, KOBJ_CHANGE, envp);
- kfree(buf);
Let the core deal with notifying userspace - it looks like you want to implement extcon integration there.
- /* set cpu DAI configuration */
- ret = snd_soc_dai_set_fmt(cpu_dai, fmt);
- if (ret) {
dev_err(dev, "failed to set cpu dai fmt: %d\n", ret);
return ret;
- }
- /* set codec DAI configuration */
- ret = snd_soc_dai_set_fmt(codec_dai, fmt);
- if (ret) {
dev_err(dev, "failed to set codec dai fmt: %d\n", ret);
return ret;
- }
Initialise these in the dai_link structure.
- for (i = 0; i < ARRAY_SIZE(sysclk_divs); ++i) {
if (sysclk_divs[i] == -1)
continue;
sysclk /= sysclk_divs[i];
for (j = 0; j < ARRAY_SIZE(dac_divs); ++j) {
if (sysclk == sample_rate * dac_divs[j]) {
for (k = 0; k < ARRAY_SIZE(bclk_divs); ++k)
if (sysclk == bclk * bclk_divs[k] / 10)
break;
if (k != ARRAY_SIZE(bclk_divs))
break;
}
}
if (j != ARRAY_SIZE(dac_divs))
break;
- }
- if (i != ARRAY_SIZE(sysclk_divs)) {
/* Set codec sysclk */
snd_soc_dai_set_sysclk(codec_dai,
WM8960_SYSCLK_MCLK, sysclk, 0);
snd_soc_dai_set_clkdiv(codec_dai, WM8960_SYSCLKDIV, i << 1);
return 0;
- }
Better, upgrade the CODEC driver to do this.
- /* codec mclk should be enabled early to avoid jack detect error */
- ret = clk_prepare_enable(data->codec_clk);
- if (ret) {
dev_err(card->dev, "Failed to enable MCLK: %d\n", ret);
return ret;
- }
Similarly integrating clock API support into the CODEC is better - it makes all the DT stuff work a lot more smoothly and avoids things having to get duplicated.
On Wed, Jun 10, 2015 at 04:26:27PM +0800, Zidan Wang wrote:
This imx-wm8960 device-tree-only machine driver works with sai driver and have below feature.
- support codec master and slave mode
- support headphone jack detect
- support headphone and micphone jack event
- support asrc<->sai<->wm8960 mode
Signed-off-by: Zidan Wang zidan.wang@freescale.com
Above all, could you merge this into fsl-asoc-card?
It'd be nicer to have jack detection over there. And we can put PLL settings to the codec driver as Mark suggested.
Thanks Nicolin
.../devicetree/bindings/sound/imx-audio-wm8960.txt | 68 ++ sound/soc/fsl/Kconfig | 13 + sound/soc/fsl/Makefile | 2 + sound/soc/fsl/imx-wm8960.c | 711 +++++++++++++++++++++ 4 files changed, 794 insertions(+) create mode 100644 Documentation/devicetree/bindings/sound/imx-audio-wm8960.txt create mode 100644 sound/soc/fsl/imx-wm8960.c
diff --git a/Documentation/devicetree/bindings/sound/imx-audio-wm8960.txt b/Documentation/devicetree/bindings/sound/imx-audio-wm8960.txt new file mode 100644 index 0000000..300d027 --- /dev/null +++ b/Documentation/devicetree/bindings/sound/imx-audio-wm8960.txt @@ -0,0 +1,68 @@ +Freescale i.MX audio complex with WM8960 codec
+Required properties:
- compatible : "fsl,imx-audio-wm8960"
- model : The user-visible name of this sound complex
- audio-codec : The phandle of the WM8960 audio codec
- hp-det : ADCLRC/GPIO1, LINPUT3/JD2 and RINPUT3/JD3 pins can
be selected as headphone jack detect inputs to
automatically disable the speaker output and enable
the headphone output.
hp-det = <hp-det-pin hp-det-polarity>;
hp-det-pin = 1: ADCLRC/GPIO1 used as detect pin
hp-det-pin = 2: LINPUT3/JD2 used as detect pin
hp-det-pin = 3: RINPUT3/JD3 used as detect pin
hp-det-polarity = 0: hp detect high for headphone
hp-det-polarity = 1: hp detect high for speaker
- codec-master : If "codec-master" present, codec works as master.
Otherwise, codec works as slave.
- audio-routing : A list of the connections between audio components.
Each entry is a pair of strings, the first being the
connection's sink, the second being the connection's
source. Valid names could be power supplies, WM8962
pins, and the jacks on the board:
Power supplies:
* MICB
Board connectors:
* Hp MIC
* Main MIC
* Headset Jack
* Ext Spk
+Example:
+sound {
compatible = "fsl,imx6ul-evk-wm8960",
"fsl,imx-audio-wm8960";
model = "wm8960-audio";
cpu-dai = <&sai2>;
audio-codec = <&codec>;
asrc-controller = <&asrc>;
codec-master;
hp-det = <3 0>;
audio-routing =
"Headset Jack", "HP_L",
"Headset Jack", "HP_R",
"Ext Spk", "SPK_LP",
"Ext Spk", "SPK_LN",
"Ext Spk", "SPK_RP",
"Ext Spk", "SPK_RN",
"LINPUT2", "Hp MIC",
"LINPUT3", "Hp MIC",
"RINPUT1", "Main MIC",
"RINPUT2", "Main MIC",
"Hp MIC", "MICB",
"Main MIC", "MICB",
"CPU-Playback", "ASRC-Playback",
"Playback", "CPU-Playback",
"ASRC-Capture", "CPU-Capture",
"CPU-Capture", "Capture";
+}; diff --git a/sound/soc/fsl/Kconfig b/sound/soc/fsl/Kconfig index 19c302b..0d9aa56 100644 --- a/sound/soc/fsl/Kconfig +++ b/sound/soc/fsl/Kconfig @@ -229,6 +229,19 @@ config SND_SOC_EUKREA_TLV320 Enable I2S based access to the TLV320AIC23B codec attached to the SSI interface
+config SND_SOC_IMX_WM8960
- tristate "SoC Audio support for i.MX boards with wm8960"
- depends on OF && I2C
- select SND_SOC_WM8960
- select SND_SOC_IMX_PCM_DMA
- select SND_SOC_FSL_SAI
- select SND_SOC_FSL_UTILS
- select SND_KCTL_JACK
- help
SoC Audio support for i.MX boards with WM8960
Say Y if you want to add support for SoC audio on an i.MX board with
a wm8960 codec.
config SND_SOC_IMX_WM8962 tristate "SoC Audio support for i.MX boards with wm8962" depends on OF && I2C && INPUT diff --git a/sound/soc/fsl/Makefile b/sound/soc/fsl/Makefile index d28dc25..84a9a10 100644 --- a/sound/soc/fsl/Makefile +++ b/sound/soc/fsl/Makefile @@ -54,6 +54,7 @@ snd-soc-mx27vis-aic32x4-objs := mx27vis-aic32x4.o snd-soc-wm1133-ev1-objs := wm1133-ev1.o snd-soc-imx-es8328-objs := imx-es8328.o snd-soc-imx-sgtl5000-objs := imx-sgtl5000.o +snd-soc-imx-wm8960-objs := imx-wm8960.o snd-soc-imx-wm8962-objs := imx-wm8962.o snd-soc-imx-spdif-objs := imx-spdif.o snd-soc-imx-mc13783-objs := imx-mc13783.o @@ -64,6 +65,7 @@ obj-$(CONFIG_SND_SOC_MX27VIS_AIC32X4) += snd-soc-mx27vis-aic32x4.o obj-$(CONFIG_SND_MXC_SOC_WM1133_EV1) += snd-soc-wm1133-ev1.o obj-$(CONFIG_SND_SOC_IMX_ES8328) += snd-soc-imx-es8328.o obj-$(CONFIG_SND_SOC_IMX_SGTL5000) += snd-soc-imx-sgtl5000.o +obj-$(CONFIG_SND_SOC_IMX_WM8960) += snd-soc-imx-wm8960.o obj-$(CONFIG_SND_SOC_IMX_WM8962) += snd-soc-imx-wm8962.o obj-$(CONFIG_SND_SOC_IMX_SPDIF) += snd-soc-imx-spdif.o obj-$(CONFIG_SND_SOC_IMX_MC13783) += snd-soc-imx-mc13783.o diff --git a/sound/soc/fsl/imx-wm8960.c b/sound/soc/fsl/imx-wm8960.c new file mode 100644 index 0000000..ddd17cb --- /dev/null +++ b/sound/soc/fsl/imx-wm8960.c @@ -0,0 +1,711 @@ +/*
- Copyright (C) 2015 Freescale Semiconductor, Inc.
- The code contained herein is licensed under the GNU General Public
- License. You may obtain a copy of the GNU General Public License
- Version 2 or later at the following locations:
- */
+#include <linux/module.h> +#include <linux/of_platform.h> +#include <linux/i2c.h> +#include <linux/of_gpio.h> +#include <linux/slab.h> +#include <linux/gpio.h> +#include <linux/clk.h> +#include <sound/soc.h> +#include <sound/jack.h> +#include <sound/control.h> +#include <sound/pcm_params.h> +#include <sound/soc-dapm.h> +#include <linux/pinctrl/consumer.h> +#include <linux/mfd/syscon.h> +#include "../codecs/wm8960.h"
+#define DAI_NAME_SIZE 32
+struct imx_wm8960_data {
- struct snd_soc_card card;
- char codec_dai_name[DAI_NAME_SIZE];
- char platform_name[DAI_NAME_SIZE];
- struct clk *codec_clk;
- unsigned int clk_frequency;
- bool is_codec_master;
- bool is_stream_in_use[2];
- struct regmap *gpr;
- unsigned int hp_det[2];
- u32 asrc_rate;
- u32 asrc_format;
+};
+struct imx_priv {
- int hp_set_gpio;
- int hp_active_low;
- struct snd_kcontrol *headset_kctl;
- struct snd_soc_codec *codec;
- struct platform_device *pdev;
- struct platform_device *asrc_pdev;
- struct snd_card *snd_card;
+};
+static struct imx_priv card_priv;
+static struct snd_soc_jack imx_hp_set; +static struct snd_soc_jack_pin imx_hp_set_pins[] = {
- {
.pin = "Headset Jack",
.mask = SND_JACK_HEADSET,
- },
+}; +static struct snd_soc_jack_gpio imx_hp_set_gpio = {
- .name = "headset detect",
- .report = SND_JACK_HEADSET,
- .debounce_time = 250,
- .invert = 1,
+};
+static int hp_set_status_check(void *data) +{
- struct imx_priv *priv = &card_priv;
- struct platform_device *pdev = priv->pdev;
- char *envp[3], *buf;
- int hp_status, ret;
- if (!gpio_is_valid(priv->hp_set_gpio))
return 0;
- hp_status = gpio_get_value(priv->hp_set_gpio) ? 1 : 0;
- buf = kmalloc(32, GFP_ATOMIC);
- if (!buf) {
dev_err(&pdev->dev, "%s kmalloc failed\n", __func__);
return -ENOMEM;
- }
- if (hp_status != priv->hp_active_low) {
snprintf(buf, 32, "STATE=%d", 2);
snd_soc_dapm_disable_pin(&priv->codec->dapm, "Ext Spk");
snd_soc_dapm_disable_pin(&priv->codec->dapm, "Main MIC");
ret = imx_hp_set_gpio.report;
/*
* As the hp MIC only connect the input for left channel, we
* need to route it for right channel.
*/
snd_soc_update_bits(priv->codec, WM8960_ADDCTL1, 3<<2, 1<<2);
snd_kctl_jack_report(priv->snd_card, priv->headset_kctl, 1);
- } else {
snprintf(buf, 32, "STATE=%d", 0);
snd_soc_dapm_enable_pin(&priv->codec->dapm, "Ext Spk");
snd_soc_dapm_enable_pin(&priv->codec->dapm, "Main MIC");
ret = 0;
/*
* As the Main MIC only connect the input for right channel,
* we need to route it for left channel.
*/
snd_soc_update_bits(priv->codec, WM8960_ADDCTL1, 3<<2, 2<<2);
snd_kctl_jack_report(priv->snd_card, priv->headset_kctl, 0);
- }
- envp[0] = "NAME=headset";
- envp[1] = buf;
- envp[2] = NULL;
- kobject_uevent_env(&pdev->dev.kobj, KOBJ_CHANGE, envp);
- kfree(buf);
- return ret;
+}
+static const struct snd_soc_dapm_widget imx_wm8960_dapm_widgets[] = {
- SND_SOC_DAPM_HP("Headset Jack", NULL),
- SND_SOC_DAPM_SPK("Ext Spk", NULL),
- SND_SOC_DAPM_MIC("Hp MIC", NULL),
- SND_SOC_DAPM_MIC("Main MIC", NULL),
+};
+static int imx_wm8960_gpio_init(struct snd_soc_card *card) +{
- struct snd_soc_dai *codec_dai = card->rtd[0].codec_dai;
- struct snd_soc_codec *codec = codec_dai->codec;
- struct imx_priv *priv = &card_priv;
- int ret;
- priv->codec = codec;
- if (gpio_is_valid(priv->hp_set_gpio)) {
imx_hp_set_gpio.gpio = priv->hp_set_gpio;
imx_hp_set_gpio.jack_status_check = hp_set_status_check;
ret = snd_soc_card_jack_new(card, "Headset Jack",
SND_JACK_HEADSET, &imx_hp_set,
imx_hp_set_pins, ARRAY_SIZE(imx_hp_set_pins));
if (ret)
return ret;
ret = snd_soc_jack_add_gpios(&imx_hp_set, 1,
&imx_hp_set_gpio);
if (ret)
return ret;
- }
- return 0;
+}
+static ssize_t show_headphone(struct device_driver *dev, char *buf) +{
- struct imx_priv *priv = &card_priv;
- int hp_status;
- if (!gpio_is_valid(priv->hp_set_gpio)) {
strcpy(buf, "no detect gpio connected\n");
return strlen(buf);
- }
- /* Check if headphone is plugged in */
- hp_status = gpio_get_value(priv->hp_set_gpio) ? 1 : 0;
- if (hp_status != priv->hp_active_low)
strcpy(buf, "Headphone\n");
- else
strcpy(buf, "Speaker\n");
- return strlen(buf);
+}
+static ssize_t show_micphone(struct device_driver *dev, char *buf) +{
- struct imx_priv *priv = &card_priv;
- int hp_status;
- if (!gpio_is_valid(priv->hp_set_gpio)) {
strcpy(buf, "no detect gpio connected\n");
return strlen(buf);
- }
- /* Check if headphone is plugged in */
- hp_status = gpio_get_value(priv->hp_set_gpio) ? 1 : 0;
- if (hp_status != priv->hp_active_low)
strcpy(buf, "Hp MIC\n");
- else
strcpy(buf, "Main MIC\n");
- return strlen(buf);
+} +static DRIVER_ATTR(headphone, S_IRUGO | S_IWUSR, show_headphone, NULL); +static DRIVER_ATTR(micphone, S_IRUGO | S_IWUSR, show_micphone, NULL);
+static void wm8960_init(struct snd_soc_card *card) +{
- struct snd_soc_dai *codec_dai = card->rtd[0].codec_dai;
- struct snd_soc_codec *codec = codec_dai->codec;
- struct imx_wm8960_data *data = snd_soc_card_get_drvdata(card);
- /*
* codec ADCLRC pin configured as GPIO, DACLRC pin is used as
* a frame clock for ADCs and DACs
*/
- snd_soc_update_bits(codec, WM8960_IFACE2, 1<<6, 1<<6);
- /* GPIO1 used as headphone detect output */
- snd_soc_update_bits(codec, WM8960_ADDCTL4, 7<<4, 3<<4);
- /* Enable headphone jack detect */
- snd_soc_update_bits(codec, WM8960_ADDCTL2, 1<<6, 1<<6);
- snd_soc_update_bits(codec, WM8960_ADDCTL2, 1<<5, data->hp_det[1]<<5);
- snd_soc_update_bits(codec, WM8960_ADDCTL4, 3<<2, data->hp_det[0]<<2);
- snd_soc_update_bits(codec, WM8960_ADDCTL1, 3, 3);
- /* route ADC left channel to right channel in default */
- snd_soc_update_bits(codec, WM8960_ADDCTL1, 3<<2, 1<<2);
+}
+/* -1 for reserved value */ +static const int sysclk_divs[] = { 1, -1, 2, -1 };
+/* Multiply 256 for internal 256 div */ +static const int dac_divs[] = { 256, 384, 512, 768, 1024, 1408, 1536 };
+/* Multiply 10 to eliminate decimials */ +static const int bclk_divs[] = {
- 10, 15, 20, 30, 40, 55, 60, 80, 110,
- 120, 160, 220, 240, 320, 320, 320
+};
+static int imx_hifi_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 = rtd->codec_dai;
- struct snd_soc_dai *cpu_dai = rtd->cpu_dai;
- struct snd_soc_card *card = rtd->card;
- struct imx_wm8960_data *data = snd_soc_card_get_drvdata(card);
- bool tx = substream->stream == SNDRV_PCM_STREAM_PLAYBACK;
- struct device *dev = card->dev;
- unsigned int sample_rate = params_rate(params);
- unsigned int sysclk, pll_in, pll_out;
- int i, j, k, ret = 0;
- int bclk = snd_soc_params_to_bclk(params);
- unsigned int fmt, plln;
- if (params_channels(params) == 1)
bclk *= 2;
- data->is_stream_in_use[tx] = true;
- if (data->is_stream_in_use[!tx])
return 0;
- if (data->is_codec_master)
fmt = SND_SOC_DAIFMT_I2S | SND_SOC_DAIFMT_NB_NF |
SND_SOC_DAIFMT_CBM_CFM;
- else
fmt = SND_SOC_DAIFMT_I2S | SND_SOC_DAIFMT_NB_NF |
SND_SOC_DAIFMT_CBS_CFS;
- /* set cpu DAI configuration */
- ret = snd_soc_dai_set_fmt(cpu_dai, fmt);
- if (ret) {
dev_err(dev, "failed to set cpu dai fmt: %d\n", ret);
return ret;
- }
- /* set codec DAI configuration */
- ret = snd_soc_dai_set_fmt(codec_dai, fmt);
- if (ret) {
dev_err(dev, "failed to set codec dai fmt: %d\n", ret);
return ret;
- }
- if (!data->is_codec_master) {
ret = snd_soc_dai_set_tdm_slot(cpu_dai,
0, 0, 2, params_width(params));
if (ret) {
dev_err(dev,
"failed to set cpu dai tdm slot: %d\n", ret);
return ret;
}
ret = snd_soc_dai_set_sysclk(cpu_dai, 0, 0, SND_SOC_CLOCK_OUT);
if (ret) {
dev_err(dev, "failed to set cpu sysclk: %d\n", ret);
return ret;
}
return 0;
- } else {
ret = snd_soc_dai_set_sysclk(cpu_dai, 0, 0, SND_SOC_CLOCK_IN);
if (ret) {
dev_err(dev, "failed to set cpu sysclk: %d\n", ret);
return ret;
}
- }
- data->clk_frequency = clk_get_rate(data->codec_clk);
- /* Use MCLK to provide sysclk directly */
- sysclk = data->clk_frequency;
- for (i = 0; i < ARRAY_SIZE(sysclk_divs); ++i) {
if (sysclk_divs[i] == -1)
continue;
sysclk /= sysclk_divs[i];
for (j = 0; j < ARRAY_SIZE(dac_divs); ++j) {
if (sysclk == sample_rate * dac_divs[j]) {
for (k = 0; k < ARRAY_SIZE(bclk_divs); ++k)
if (sysclk == bclk * bclk_divs[k] / 10)
break;
if (k != ARRAY_SIZE(bclk_divs))
break;
}
}
if (j != ARRAY_SIZE(dac_divs))
break;
- }
- if (i != ARRAY_SIZE(sysclk_divs)) {
/* Set codec sysclk */
snd_soc_dai_set_sysclk(codec_dai,
WM8960_SYSCLK_MCLK, sysclk, 0);
snd_soc_dai_set_clkdiv(codec_dai, WM8960_SYSCLKDIV, i << 1);
return 0;
- }
- /* Use PLL to provide sysclk */
- pll_in = data->clk_frequency;
- for (i = 0; i < ARRAY_SIZE(sysclk_divs); ++i) {
if (sysclk_divs[i] == -1)
continue;
for (j = 0; j < ARRAY_SIZE(dac_divs); ++j) {
sysclk = sample_rate * dac_divs[j];
pll_out = sysclk * sysclk_divs[i];
/*
* we should make sure 5 < plln < 13, or we can't get
* such pll_out.
*/
plln = pll_out * 4 / pll_in;
if (plln < 6)
plln *= 2;
if ((plln < 6) || (plln > 12))
continue;
for (k = 0; k < ARRAY_SIZE(bclk_divs); ++k) {
if (sysclk == bclk * bclk_divs[k] / 10) {
/* Set codec pll */
ret = snd_soc_dai_set_pll(codec_dai, 0,
0, data->clk_frequency,
pll_out);
if (ret != 0)
continue;
else
break;
}
}
if (k != ARRAY_SIZE(bclk_divs))
break;
}
if (j != ARRAY_SIZE(dac_divs))
break;
- }
- if (i != ARRAY_SIZE(sysclk_divs)) {
/* Set codec sysclk */
snd_soc_dai_set_sysclk(codec_dai, WM8960_SYSCLK_PLL, sysclk, 0);
snd_soc_dai_set_clkdiv(codec_dai, WM8960_SYSCLKDIV, i << 1);
return 0;
- }
- dev_err(dev, "failed to configure system clock");
- return -EINVAL;
+}
+static int imx_hifi_hw_free(struct snd_pcm_substream *substream) +{
- struct snd_soc_pcm_runtime *rtd = substream->private_data;
- struct snd_soc_dai *codec_dai = rtd->codec_dai;
- struct snd_soc_card *card = rtd->card;
- struct imx_wm8960_data *data = snd_soc_card_get_drvdata(card);
- bool tx = substream->stream == SNDRV_PCM_STREAM_PLAYBACK;
- data->is_stream_in_use[tx] = false;
- /* Power down PLL to save power*/
- if (data->is_codec_master && !data->is_stream_in_use[!tx])
snd_soc_dai_set_pll(codec_dai, 0, 0, 0, 0);
- return 0;
+}
+static u32 imx_wm8960_rates[] = { 8000, 16000, 32000, 48000 }; +static struct snd_pcm_hw_constraint_list imx_wm8960_rate_constraints = {
- .count = ARRAY_SIZE(imx_wm8960_rates),
- .list = imx_wm8960_rates,
+};
+static int imx_hifi_startup(struct snd_pcm_substream *substream) +{
- struct snd_soc_pcm_runtime *rtd = substream->private_data;
- struct snd_soc_card *card = rtd->card;
- struct imx_wm8960_data *data = snd_soc_card_get_drvdata(card);
- int ret = 0;
- if (!data->is_codec_master) {
ret = snd_pcm_hw_constraint_list(substream->runtime, 0,
SNDRV_PCM_HW_PARAM_RATE,
&imx_wm8960_rate_constraints);
if (ret)
return ret;
- }
- /* codec mclk should be enabled early to avoid jack detect error */
- ret = clk_prepare_enable(data->codec_clk);
- if (ret) {
dev_err(card->dev, "Failed to enable MCLK: %d\n", ret);
return ret;
- }
- return ret;
+}
+static void imx_hifi_shutdown(struct snd_pcm_substream *substream) +{
- struct snd_soc_pcm_runtime *rtd = substream->private_data;
- struct snd_soc_card *card = rtd->card;
- struct imx_wm8960_data *data = snd_soc_card_get_drvdata(card);
- clk_disable_unprepare(data->codec_clk);
+}
+static struct snd_soc_ops imx_hifi_ops = {
- .hw_params = imx_hifi_hw_params,
- .hw_free = imx_hifi_hw_free,
- .startup = imx_hifi_startup,
- .shutdown = imx_hifi_shutdown,
+};
+static int imx_wm8960_late_probe(struct snd_soc_card *card) +{
- struct imx_wm8960_data *data = snd_soc_card_get_drvdata(card);
- /* set SAI2_MCLK_DIR to enable codec MCLK */
- if (data->gpr)
regmap_update_bits(data->gpr, 4, 1<<20, 1<<20);
- wm8960_init(card);
- return 0;
+}
+static int be_hw_params_fixup(struct snd_soc_pcm_runtime *rtd,
struct snd_pcm_hw_params *params)
+{
- struct snd_soc_card *card = rtd->card;
- struct imx_wm8960_data *data = snd_soc_card_get_drvdata(card);
- struct imx_priv *priv = &card_priv;
- struct snd_interval *rate;
- struct snd_mask *mask;
- if (!priv->asrc_pdev)
return -EINVAL;
- rate = hw_param_interval(params, SNDRV_PCM_HW_PARAM_RATE);
- rate->max = rate->min = data->asrc_rate;
- mask = hw_param_mask(params, SNDRV_PCM_HW_PARAM_FORMAT);
- snd_mask_none(mask);
- snd_mask_set(mask, data->asrc_format);
- return 0;
+}
+static struct snd_soc_dai_link imx_wm8960_dai[] = {
- {
.name = "HiFi",
.stream_name = "HiFi",
.codec_dai_name = "wm8960-hifi",
.ops = &imx_hifi_ops,
- },
- {
.name = "HiFi-ASRC-FE",
.stream_name = "HiFi-ASRC-FE",
.codec_name = "snd-soc-dummy",
.codec_dai_name = "snd-soc-dummy-dai",
.dynamic = 1,
.ignore_pmdown_time = 1,
.dpcm_playback = 1,
.dpcm_capture = 1,
- },
- {
.name = "HiFi-ASRC-BE",
.stream_name = "HiFi-ASRC-BE",
.codec_dai_name = "wm8960-hifi",
.platform_name = "snd-soc-dummy",
.no_pcm = 1,
.ignore_pmdown_time = 1,
.dpcm_playback = 1,
.dpcm_capture = 1,
.ops = &imx_hifi_ops,
.be_hw_params_fixup = be_hw_params_fixup,
- },
+};
+static int imx_wm8960_probe(struct platform_device *pdev) +{
- struct device_node *cpu_np, *codec_np, *gpr_np;
- struct platform_device *cpu_pdev;
- struct imx_priv *priv = &card_priv;
- struct i2c_client *codec_dev;
- struct imx_wm8960_data *data;
- struct platform_device *asrc_pdev = NULL;
- struct device_node *asrc_np;
- u32 width;
- int ret;
- priv->pdev = pdev;
- cpu_np = of_parse_phandle(pdev->dev.of_node, "cpu-dai", 0);
- if (!cpu_np) {
dev_err(&pdev->dev, "cpu dai phandle missing or invalid\n");
ret = -EINVAL;
goto fail;
- }
- codec_np = of_parse_phandle(pdev->dev.of_node, "audio-codec", 0);
- if (!codec_np) {
dev_err(&pdev->dev, "phandle missing or invalid\n");
ret = -EINVAL;
goto fail;
- }
- cpu_pdev = of_find_device_by_node(cpu_np);
- if (!cpu_pdev) {
dev_err(&pdev->dev, "failed to find SAI platform device\n");
ret = -EINVAL;
goto fail;
- }
- codec_dev = of_find_i2c_device_by_node(codec_np);
- if (!codec_dev || !codec_dev->dev.driver) {
dev_err(&pdev->dev, "failed to find codec platform device\n");
ret = -EINVAL;
goto fail;
- }
- data = devm_kzalloc(&pdev->dev, sizeof(*data), GFP_KERNEL);
- if (!data) {
ret = -ENOMEM;
goto fail;
- }
- if (of_property_read_bool(pdev->dev.of_node, "codec-master"))
data->is_codec_master = true;
- data->codec_clk = devm_clk_get(&codec_dev->dev, "mclk");
- if (IS_ERR(data->codec_clk)) {
ret = PTR_ERR(data->codec_clk);
dev_err(&pdev->dev, "failed to get codec clk: %d\n", ret);
goto fail;
- }
- gpr_np = of_parse_phandle(pdev->dev.of_node, "gpr", 0);
- if (gpr_np) {
data->gpr = syscon_node_to_regmap(gpr_np);
if (IS_ERR(data->gpr)) {
ret = PTR_ERR(data->gpr);
dev_err(&pdev->dev, "failed to get gpr regmap\n");
goto fail;
}
- }
- /*
* hp-det = <hp-det-pin hp-det-polarity>;
* hp-det-pin: JD1 JD2 or JD3
* hp-det-polarity = 0: hp detect high for headphone
* hp-det-polarity = 1: hp detect high for speaker
*/
- of_property_read_u32_array(pdev->dev.of_node,
"hp-det", data->hp_det, 2);
- asrc_np = of_parse_phandle(pdev->dev.of_node, "asrc-controller", 0);
- if (asrc_np) {
asrc_pdev = of_find_device_by_node(asrc_np);
priv->asrc_pdev = asrc_pdev;
- }
- data->card.dai_link = imx_wm8960_dai;
- imx_wm8960_dai[0].codec_of_node = codec_np;
- imx_wm8960_dai[0].cpu_dai_name = dev_name(&cpu_pdev->dev);
- imx_wm8960_dai[0].platform_of_node = cpu_np;
- if (!asrc_pdev) {
data->card.num_links = 1;
- } else {
imx_wm8960_dai[1].cpu_of_node = asrc_np;
imx_wm8960_dai[1].platform_of_node = asrc_np;
imx_wm8960_dai[2].codec_of_node = codec_np;
imx_wm8960_dai[2].cpu_dai_name = dev_name(&cpu_pdev->dev);
data->card.num_links = 3;
ret = of_property_read_u32(asrc_np, "fsl,asrc-rate",
&data->asrc_rate);
if (ret) {
dev_err(&pdev->dev, "failed to get output rate\n");
ret = -EINVAL;
goto fail;
}
ret = of_property_read_u32(asrc_np, "fsl,asrc-width", &width);
if (ret) {
dev_err(&pdev->dev, "failed to get output rate\n");
ret = -EINVAL;
goto fail;
}
if (width == 24)
data->asrc_format = SNDRV_PCM_FORMAT_S24_LE;
else
data->asrc_format = SNDRV_PCM_FORMAT_S16_LE;
- }
- data->card.dev = &pdev->dev;
- ret = snd_soc_of_parse_card_name(&data->card, "model");
- if (ret)
goto fail;
- data->card.dapm_widgets = imx_wm8960_dapm_widgets;
- data->card.num_dapm_widgets = ARRAY_SIZE(imx_wm8960_dapm_widgets);
- ret = snd_soc_of_parse_audio_routing(&data->card, "audio-routing");
- if (ret)
goto fail;
- data->card.late_probe = imx_wm8960_late_probe;
- platform_set_drvdata(pdev, &data->card);
- snd_soc_card_set_drvdata(&data->card, data);
- ret = devm_snd_soc_register_card(&pdev->dev, &data->card);
- if (ret) {
dev_err(&pdev->dev, "failed to register card (%d)\n", ret);
goto fail;
- }
- priv->snd_card = data->card.snd_card;
- priv->hp_set_gpio = of_get_named_gpio_flags(pdev->dev.of_node,
"hp-det-gpios", 0,
(enum of_gpio_flags *)&priv->hp_active_low);
- if (IS_ERR(ERR_PTR(priv->hp_set_gpio)))
goto fail;
- priv->headset_kctl = snd_kctl_jack_new("Headset", priv->snd_card);
- ret = snd_ctl_add(data->card.snd_card, priv->headset_kctl);
- if (ret)
goto fail;
- ret = imx_wm8960_gpio_init(&data->card);
- if (gpio_is_valid(priv->hp_set_gpio)) {
ret = driver_create_file(pdev->dev.driver,
&driver_attr_headphone);
if (ret) {
dev_err(&pdev->dev,
"create hp attr failed (%d)\n", ret);
goto fail;
}
ret = driver_create_file(pdev->dev.driver,
&driver_attr_micphone);
if (ret) {
dev_err(&pdev->dev,
"create mic attr failed (%d)\n", ret);
goto fail;
}
- }
+fail:
- if (cpu_np)
of_node_put(cpu_np);
- if (codec_np)
of_node_put(codec_np);
- return ret;
+}
+static const struct of_device_id imx_wm8960_dt_ids[] = {
- { .compatible = "fsl,imx-audio-wm8960", },
- { /* sentinel */ }
+}; +MODULE_DEVICE_TABLE(of, imx_wm8960_dt_ids);
+static struct platform_driver imx_wm8960_driver = {
- .driver = {
.name = "imx-wm8960",
.owner = THIS_MODULE,
.pm = &snd_soc_pm_ops,
.of_match_table = imx_wm8960_dt_ids,
- },
- .probe = imx_wm8960_probe,
+}; +module_platform_driver(imx_wm8960_driver);
+MODULE_AUTHOR("Freescale Semiconductor, Inc."); +MODULE_DESCRIPTION("Freescale i.MX WM8960 ASoC machine driver"); +MODULE_LICENSE("GPL v2");
+MODULE_ALIAS("platform:imx-wm8960");
1.9.1
On Wed, Jun 10, 2015 at 11:36:21AM -0700, Nicolin Chen wrote:
On Wed, Jun 10, 2015 at 04:26:27PM +0800, Zidan Wang wrote:
This imx-wm8960 device-tree-only machine driver works with sai driver and have below feature.
- support codec master and slave mode
- support headphone jack detect
- support headphone and micphone jack event
- support asrc<->sai<->wm8960 mode
Signed-off-by: Zidan Wang zidan.wang@freescale.com
Above all, could you merge this into fsl-asoc-card?
It'd be nicer to have jack detection over there. And we can put PLL settings to the codec driver as Mark suggested.
Hi Nicolin,
I want to modify imx-wm8960 and upstream it because our release is using it now. After upstreaming imx-wm8960, i will try to merge it to fsl-asoc-card.
Best Regards, Zidan Wang
Thanks Nicolin
On Thu, Jun 11, 2015 at 07:10:00PM +0800, Zidan Wang wrote:
This imx-wm8960 device-tree-only machine driver works with sai driver and have below feature.
- support codec master and slave mode
- support headphone jack detect
- support headphone and micphone jack event
- support asrc<->sai<->wm8960 mode
Signed-off-by: Zidan Wang zidan.wang@freescale.com
Above all, could you merge this into fsl-asoc-card?
It'd be nicer to have jack detection over there. And we can put PLL settings to the codec driver as Mark suggested.
I want to modify imx-wm8960 and upstream it because our release is using it now. After upstreaming imx-wm8960, i will try to merge it to fsl-asoc-card.
I knew your intention. That's why I suggested you to merge it directly to fsl-asoc-card instead of making a duplicated copy of that. Remember that getting code upstream should be a chance for us to improve thing.
If you find painstaking to handle it in parallel, you can focus on the improvements of the WM8960 driver while I would help you on the dai- link part. How about this?
Nicolin
On Thu, Jun 11, 2015 at 08:51:07AM -0700, Nicolin Chen wrote:
On Thu, Jun 11, 2015 at 07:10:00PM +0800, Zidan Wang wrote:
This imx-wm8960 device-tree-only machine driver works with sai driver and have below feature.
- support codec master and slave mode
- support headphone jack detect
- support headphone and micphone jack event
- support asrc<->sai<->wm8960 mode
Signed-off-by: Zidan Wang zidan.wang@freescale.com
Above all, could you merge this into fsl-asoc-card?
It'd be nicer to have jack detection over there. And we can put PLL settings to the codec driver as Mark suggested.
I want to modify imx-wm8960 and upstream it because our release is using it now. After upstreaming imx-wm8960, i will try to merge it to fsl-asoc-card.
I knew your intention. That's why I suggested you to merge it directly to fsl-asoc-card instead of making a duplicated copy of that. Remember that getting code upstream should be a chance for us to improve thing.
If you find painstaking to handle it in parallel, you can focus on the improvements of the WM8960 driver while I would help you on the dai- link part. How about this?
Nicolin
The new imx7 release have two new codec wm8958 and wm8960. So we have two machine driver imx-wm8960 and imx-wm8958 which should be upstreamed. I am appreciate that you can help me merge the imx-wm8960 to fsl-asoc-card. And i will focus on wm8960 and wm8958 driver. And i will try to merge imx-wm8958 to fsl-asoc-card.
Best Regards, Zidan Wang
participants (3)
-
Mark Brown
-
Nicolin Chen
-
Zidan Wang