Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754514AbbFJSgb (ORCPT ); Wed, 10 Jun 2015 14:36:31 -0400 Received: from mail-qg0-f66.google.com ([209.85.192.66]:34784 "EHLO mail-qg0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752417AbbFJSg3 (ORCPT ); Wed, 10 Jun 2015 14:36:29 -0400 Date: Wed, 10 Jun 2015 11:36:21 -0700 From: Nicolin Chen To: Zidan Wang Cc: broonie@kernel.org, perex@perex.cz, tiwai@suse.de, timur@tabi.org, Xiubo.Lee@gmail.com, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, alsa-devel@alsa-project.org, linuxppc-dev@lists.ozlabs.org Subject: Re: [alsa-devel][PATCH] ASoC: fsl: add imx-wm8960 machine driver Message-ID: <20150610183619.GA3103@Asurada-CZ80> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 26537 Lines: 870 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 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 = 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 > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#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: 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 > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/