Commit b6e643ad authored by Bjorn Andersson's avatar Bjorn Andersson Committed by Mark Brown

ASoC: qcom: Drop __func__ usage from log prints

The combination of dev_err() and __func__ make most of these log prints
over 100 chars long. Remove the usage of __func__ to clean the kernel
log and as the usage is not necessary to identify the individual log
prints.
Signed-off-by: default avatarBjorn Andersson <bjorn.andersson@linaro.org>
Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent 46dccc35
...@@ -175,29 +175,28 @@ static int apq8016_lpass_init(struct platform_device *pdev) ...@@ -175,29 +175,28 @@ static int apq8016_lpass_init(struct platform_device *pdev)
drvdata->pcnoc_mport_clk = devm_clk_get(dev, "pcnoc-mport-clk"); drvdata->pcnoc_mport_clk = devm_clk_get(dev, "pcnoc-mport-clk");
if (IS_ERR(drvdata->pcnoc_mport_clk)) { if (IS_ERR(drvdata->pcnoc_mport_clk)) {
dev_err(&pdev->dev, "%s() error getting pcnoc-mport-clk: %ld\n", dev_err(&pdev->dev, "error getting pcnoc-mport-clk: %ld\n",
__func__, PTR_ERR(drvdata->pcnoc_mport_clk)); PTR_ERR(drvdata->pcnoc_mport_clk));
return PTR_ERR(drvdata->pcnoc_mport_clk); return PTR_ERR(drvdata->pcnoc_mport_clk);
} }
ret = clk_prepare_enable(drvdata->pcnoc_mport_clk); ret = clk_prepare_enable(drvdata->pcnoc_mport_clk);
if (ret) { if (ret) {
dev_err(&pdev->dev, "%s() Error enabling pcnoc-mport-clk: %d\n", dev_err(&pdev->dev, "Error enabling pcnoc-mport-clk: %d\n",
__func__, ret); ret);
return ret; return ret;
} }
drvdata->pcnoc_sway_clk = devm_clk_get(dev, "pcnoc-sway-clk"); drvdata->pcnoc_sway_clk = devm_clk_get(dev, "pcnoc-sway-clk");
if (IS_ERR(drvdata->pcnoc_sway_clk)) { if (IS_ERR(drvdata->pcnoc_sway_clk)) {
dev_err(&pdev->dev, "%s() error getting pcnoc-sway-clk: %ld\n", dev_err(&pdev->dev, "error getting pcnoc-sway-clk: %ld\n",
__func__, PTR_ERR(drvdata->pcnoc_sway_clk)); PTR_ERR(drvdata->pcnoc_sway_clk));
return PTR_ERR(drvdata->pcnoc_sway_clk); return PTR_ERR(drvdata->pcnoc_sway_clk);
} }
ret = clk_prepare_enable(drvdata->pcnoc_sway_clk); ret = clk_prepare_enable(drvdata->pcnoc_sway_clk);
if (ret) { if (ret) {
dev_err(&pdev->dev, "%s() Error enabling pcnoc_sway_clk: %d\n", dev_err(&pdev->dev, "Error enabling pcnoc_sway_clk: %d\n", ret);
__func__, ret);
return ret; return ret;
} }
......
...@@ -35,8 +35,8 @@ static int lpass_cpu_daiops_set_sysclk(struct snd_soc_dai *dai, int clk_id, ...@@ -35,8 +35,8 @@ static int lpass_cpu_daiops_set_sysclk(struct snd_soc_dai *dai, int clk_id,
ret = clk_set_rate(drvdata->mi2s_osr_clk[dai->driver->id], freq); ret = clk_set_rate(drvdata->mi2s_osr_clk[dai->driver->id], freq);
if (ret) if (ret)
dev_err(dai->dev, "%s() error setting mi2s osrclk to %u: %d\n", dev_err(dai->dev, "error setting mi2s osrclk to %u: %d\n",
__func__, freq, ret); freq, ret);
return ret; return ret;
} }
...@@ -49,15 +49,13 @@ static int lpass_cpu_daiops_startup(struct snd_pcm_substream *substream, ...@@ -49,15 +49,13 @@ static int lpass_cpu_daiops_startup(struct snd_pcm_substream *substream,
ret = clk_prepare_enable(drvdata->mi2s_osr_clk[dai->driver->id]); ret = clk_prepare_enable(drvdata->mi2s_osr_clk[dai->driver->id]);
if (ret) { if (ret) {
dev_err(dai->dev, "%s() error in enabling mi2s osr clk: %d\n", dev_err(dai->dev, "error in enabling mi2s osr clk: %d\n", ret);
__func__, ret);
return ret; return ret;
} }
ret = clk_prepare_enable(drvdata->mi2s_bit_clk[dai->driver->id]); ret = clk_prepare_enable(drvdata->mi2s_bit_clk[dai->driver->id]);
if (ret) { if (ret) {
dev_err(dai->dev, "%s() error in enabling mi2s bit clk: %d\n", dev_err(dai->dev, "error in enabling mi2s bit clk: %d\n", ret);
__func__, ret);
clk_disable_unprepare(drvdata->mi2s_osr_clk[dai->driver->id]); clk_disable_unprepare(drvdata->mi2s_osr_clk[dai->driver->id]);
return ret; return ret;
} }
...@@ -87,8 +85,7 @@ static int lpass_cpu_daiops_hw_params(struct snd_pcm_substream *substream, ...@@ -87,8 +85,7 @@ static int lpass_cpu_daiops_hw_params(struct snd_pcm_substream *substream,
bitwidth = snd_pcm_format_width(format); bitwidth = snd_pcm_format_width(format);
if (bitwidth < 0) { if (bitwidth < 0) {
dev_err(dai->dev, "%s() invalid bit width given: %d\n", dev_err(dai->dev, "invalid bit width given: %d\n", bitwidth);
__func__, bitwidth);
return bitwidth; return bitwidth;
} }
...@@ -106,8 +103,7 @@ static int lpass_cpu_daiops_hw_params(struct snd_pcm_substream *substream, ...@@ -106,8 +103,7 @@ static int lpass_cpu_daiops_hw_params(struct snd_pcm_substream *substream,
regval |= LPAIF_I2SCTL_BITWIDTH_32; regval |= LPAIF_I2SCTL_BITWIDTH_32;
break; break;
default: default:
dev_err(dai->dev, "%s() invalid bitwidth given: %d\n", dev_err(dai->dev, "invalid bitwidth given: %d\n", bitwidth);
__func__, bitwidth);
return -EINVAL; return -EINVAL;
} }
...@@ -134,8 +130,8 @@ static int lpass_cpu_daiops_hw_params(struct snd_pcm_substream *substream, ...@@ -134,8 +130,8 @@ static int lpass_cpu_daiops_hw_params(struct snd_pcm_substream *substream,
regval |= LPAIF_I2SCTL_SPKMONO_STEREO; regval |= LPAIF_I2SCTL_SPKMONO_STEREO;
break; break;
default: default:
dev_err(dai->dev, "%s() invalid channels given: %u\n", dev_err(dai->dev, "invalid channels given: %u\n",
__func__, channels); channels);
return -EINVAL; return -EINVAL;
} }
} else { } else {
...@@ -161,8 +157,8 @@ static int lpass_cpu_daiops_hw_params(struct snd_pcm_substream *substream, ...@@ -161,8 +157,8 @@ static int lpass_cpu_daiops_hw_params(struct snd_pcm_substream *substream,
regval |= LPAIF_I2SCTL_MICMONO_STEREO; regval |= LPAIF_I2SCTL_MICMONO_STEREO;
break; break;
default: default:
dev_err(dai->dev, "%s() invalid channels given: %u\n", dev_err(dai->dev, "invalid channels given: %u\n",
__func__, channels); channels);
return -EINVAL; return -EINVAL;
} }
} }
...@@ -171,16 +167,15 @@ static int lpass_cpu_daiops_hw_params(struct snd_pcm_substream *substream, ...@@ -171,16 +167,15 @@ static int lpass_cpu_daiops_hw_params(struct snd_pcm_substream *substream,
LPAIF_I2SCTL_REG(drvdata->variant, dai->driver->id), LPAIF_I2SCTL_REG(drvdata->variant, dai->driver->id),
regval); regval);
if (ret) { if (ret) {
dev_err(dai->dev, "%s() error writing to i2sctl reg: %d\n", dev_err(dai->dev, "error writing to i2sctl reg: %d\n", ret);
__func__, ret);
return ret; return ret;
} }
ret = clk_set_rate(drvdata->mi2s_bit_clk[dai->driver->id], ret = clk_set_rate(drvdata->mi2s_bit_clk[dai->driver->id],
rate * bitwidth * 2); rate * bitwidth * 2);
if (ret) { if (ret) {
dev_err(dai->dev, "%s() error setting mi2s bitclk to %u: %d\n", dev_err(dai->dev, "error setting mi2s bitclk to %u: %d\n",
__func__, rate * bitwidth * 2, ret); rate * bitwidth * 2, ret);
return ret; return ret;
} }
...@@ -197,8 +192,7 @@ static int lpass_cpu_daiops_hw_free(struct snd_pcm_substream *substream, ...@@ -197,8 +192,7 @@ static int lpass_cpu_daiops_hw_free(struct snd_pcm_substream *substream,
LPAIF_I2SCTL_REG(drvdata->variant, dai->driver->id), LPAIF_I2SCTL_REG(drvdata->variant, dai->driver->id),
0); 0);
if (ret) if (ret)
dev_err(dai->dev, "%s() error writing to i2sctl reg: %d\n", dev_err(dai->dev, "error writing to i2sctl reg: %d\n", ret);
__func__, ret);
return ret; return ret;
} }
...@@ -222,8 +216,7 @@ static int lpass_cpu_daiops_prepare(struct snd_pcm_substream *substream, ...@@ -222,8 +216,7 @@ static int lpass_cpu_daiops_prepare(struct snd_pcm_substream *substream,
LPAIF_I2SCTL_REG(drvdata->variant, dai->driver->id), LPAIF_I2SCTL_REG(drvdata->variant, dai->driver->id),
mask, val); mask, val);
if (ret) if (ret)
dev_err(dai->dev, "%s() error writing to i2sctl reg: %d\n", dev_err(dai->dev, "error writing to i2sctl reg: %d\n", ret);
__func__, ret);
return ret; return ret;
} }
...@@ -252,8 +245,8 @@ static int lpass_cpu_daiops_trigger(struct snd_pcm_substream *substream, ...@@ -252,8 +245,8 @@ static int lpass_cpu_daiops_trigger(struct snd_pcm_substream *substream,
dai->driver->id), dai->driver->id),
mask, val); mask, val);
if (ret) if (ret)
dev_err(dai->dev, "%s() error writing to i2sctl reg: %d\n", dev_err(dai->dev, "error writing to i2sctl reg: %d\n",
__func__, ret); ret);
break; break;
case SNDRV_PCM_TRIGGER_STOP: case SNDRV_PCM_TRIGGER_STOP:
case SNDRV_PCM_TRIGGER_SUSPEND: case SNDRV_PCM_TRIGGER_SUSPEND:
...@@ -271,8 +264,8 @@ static int lpass_cpu_daiops_trigger(struct snd_pcm_substream *substream, ...@@ -271,8 +264,8 @@ static int lpass_cpu_daiops_trigger(struct snd_pcm_substream *substream,
dai->driver->id), dai->driver->id),
mask, val); mask, val);
if (ret) if (ret)
dev_err(dai->dev, "%s() error writing to i2sctl reg: %d\n", dev_err(dai->dev, "error writing to i2sctl reg: %d\n",
__func__, ret); ret);
break; break;
} }
...@@ -299,8 +292,7 @@ int asoc_qcom_lpass_cpu_dai_probe(struct snd_soc_dai *dai) ...@@ -299,8 +292,7 @@ int asoc_qcom_lpass_cpu_dai_probe(struct snd_soc_dai *dai)
ret = regmap_write(drvdata->lpaif_map, ret = regmap_write(drvdata->lpaif_map,
LPAIF_I2SCTL_REG(drvdata->variant, dai->driver->id), 0); LPAIF_I2SCTL_REG(drvdata->variant, dai->driver->id), 0);
if (ret) if (ret)
dev_err(dai->dev, "%s() error writing to i2sctl reg: %d\n", dev_err(dai->dev, "error writing to i2sctl reg: %d\n", ret);
__func__, ret);
return ret; return ret;
} }
...@@ -442,8 +434,7 @@ int asoc_qcom_lpass_cpu_platform_probe(struct platform_device *pdev) ...@@ -442,8 +434,7 @@ int asoc_qcom_lpass_cpu_platform_probe(struct platform_device *pdev)
dsp_of_node = of_parse_phandle(pdev->dev.of_node, "qcom,adsp", 0); dsp_of_node = of_parse_phandle(pdev->dev.of_node, "qcom,adsp", 0);
if (dsp_of_node) { if (dsp_of_node) {
dev_err(&pdev->dev, "%s() DSP exists and holds audio resources\n", dev_err(&pdev->dev, "DSP exists and holds audio resources\n");
__func__);
return -EBUSY; return -EBUSY;
} }
...@@ -464,8 +455,7 @@ int asoc_qcom_lpass_cpu_platform_probe(struct platform_device *pdev) ...@@ -464,8 +455,7 @@ int asoc_qcom_lpass_cpu_platform_probe(struct platform_device *pdev)
drvdata->lpaif = devm_ioremap_resource(&pdev->dev, res); drvdata->lpaif = devm_ioremap_resource(&pdev->dev, res);
if (IS_ERR((void const __force *)drvdata->lpaif)) { if (IS_ERR((void const __force *)drvdata->lpaif)) {
dev_err(&pdev->dev, "%s() error mapping reg resource: %ld\n", dev_err(&pdev->dev, "error mapping reg resource: %ld\n",
__func__,
PTR_ERR((void const __force *)drvdata->lpaif)); PTR_ERR((void const __force *)drvdata->lpaif));
return PTR_ERR((void const __force *)drvdata->lpaif); return PTR_ERR((void const __force *)drvdata->lpaif);
} }
...@@ -477,8 +467,8 @@ int asoc_qcom_lpass_cpu_platform_probe(struct platform_device *pdev) ...@@ -477,8 +467,8 @@ int asoc_qcom_lpass_cpu_platform_probe(struct platform_device *pdev)
drvdata->lpaif_map = devm_regmap_init_mmio(&pdev->dev, drvdata->lpaif, drvdata->lpaif_map = devm_regmap_init_mmio(&pdev->dev, drvdata->lpaif,
&lpass_cpu_regmap_config); &lpass_cpu_regmap_config);
if (IS_ERR(drvdata->lpaif_map)) { if (IS_ERR(drvdata->lpaif_map)) {
dev_err(&pdev->dev, "%s() error initializing regmap: %ld\n", dev_err(&pdev->dev, "error initializing regmap: %ld\n",
__func__, PTR_ERR(drvdata->lpaif_map)); PTR_ERR(drvdata->lpaif_map));
return PTR_ERR(drvdata->lpaif_map); return PTR_ERR(drvdata->lpaif_map);
} }
...@@ -496,8 +486,7 @@ int asoc_qcom_lpass_cpu_platform_probe(struct platform_device *pdev) ...@@ -496,8 +486,7 @@ int asoc_qcom_lpass_cpu_platform_probe(struct platform_device *pdev)
clk_name); clk_name);
if (IS_ERR(drvdata->mi2s_osr_clk[dai_id])) { if (IS_ERR(drvdata->mi2s_osr_clk[dai_id])) {
dev_warn(&pdev->dev, dev_warn(&pdev->dev,
"%s() error getting optional mi2s-osr-clk: %ld\n", "error getting optional mi2s-osr-clk: %ld\n",
__func__,
PTR_ERR(drvdata->mi2s_osr_clk[dai_id])); PTR_ERR(drvdata->mi2s_osr_clk[dai_id]));
drvdata->mi2s_osr_clk[dai_id] = NULL; drvdata->mi2s_osr_clk[dai_id] = NULL;
...@@ -512,8 +501,7 @@ int asoc_qcom_lpass_cpu_platform_probe(struct platform_device *pdev) ...@@ -512,8 +501,7 @@ int asoc_qcom_lpass_cpu_platform_probe(struct platform_device *pdev)
clk_name); clk_name);
if (IS_ERR(drvdata->mi2s_bit_clk[dai_id])) { if (IS_ERR(drvdata->mi2s_bit_clk[dai_id])) {
dev_err(&pdev->dev, dev_err(&pdev->dev,
"%s() error getting mi2s-bit-clk: %ld\n", "error getting mi2s-bit-clk: %ld\n",
__func__,
PTR_ERR(drvdata->mi2s_bit_clk[dai_id])); PTR_ERR(drvdata->mi2s_bit_clk[dai_id]));
return PTR_ERR(drvdata->mi2s_bit_clk[dai_id]); return PTR_ERR(drvdata->mi2s_bit_clk[dai_id]);
} }
...@@ -521,24 +509,23 @@ int asoc_qcom_lpass_cpu_platform_probe(struct platform_device *pdev) ...@@ -521,24 +509,23 @@ int asoc_qcom_lpass_cpu_platform_probe(struct platform_device *pdev)
drvdata->ahbix_clk = devm_clk_get(&pdev->dev, "ahbix-clk"); drvdata->ahbix_clk = devm_clk_get(&pdev->dev, "ahbix-clk");
if (IS_ERR(drvdata->ahbix_clk)) { if (IS_ERR(drvdata->ahbix_clk)) {
dev_err(&pdev->dev, "%s() error getting ahbix-clk: %ld\n", dev_err(&pdev->dev, "error getting ahbix-clk: %ld\n",
__func__, PTR_ERR(drvdata->ahbix_clk)); PTR_ERR(drvdata->ahbix_clk));
return PTR_ERR(drvdata->ahbix_clk); return PTR_ERR(drvdata->ahbix_clk);
} }
ret = clk_set_rate(drvdata->ahbix_clk, LPASS_AHBIX_CLOCK_FREQUENCY); ret = clk_set_rate(drvdata->ahbix_clk, LPASS_AHBIX_CLOCK_FREQUENCY);
if (ret) { if (ret) {
dev_err(&pdev->dev, "%s() error setting rate on ahbix_clk: %d\n", dev_err(&pdev->dev, "error setting rate on ahbix_clk: %d\n",
__func__, ret); ret);
return ret; return ret;
} }
dev_dbg(&pdev->dev, "%s() set ahbix_clk rate to %lu\n", __func__, dev_dbg(&pdev->dev, "set ahbix_clk rate to %lu\n",
clk_get_rate(drvdata->ahbix_clk)); clk_get_rate(drvdata->ahbix_clk));
ret = clk_prepare_enable(drvdata->ahbix_clk); ret = clk_prepare_enable(drvdata->ahbix_clk);
if (ret) { if (ret) {
dev_err(&pdev->dev, "%s() error enabling ahbix_clk: %d\n", dev_err(&pdev->dev, "error enabling ahbix_clk: %d\n", ret);
__func__, ret);
return ret; return ret;
} }
...@@ -547,15 +534,14 @@ int asoc_qcom_lpass_cpu_platform_probe(struct platform_device *pdev) ...@@ -547,15 +534,14 @@ int asoc_qcom_lpass_cpu_platform_probe(struct platform_device *pdev)
variant->dai_driver, variant->dai_driver,
variant->num_dai); variant->num_dai);
if (ret) { if (ret) {
dev_err(&pdev->dev, "%s() error registering cpu driver: %d\n", dev_err(&pdev->dev, "error registering cpu driver: %d\n", ret);
__func__, ret);
goto err_clk; goto err_clk;
} }
ret = asoc_qcom_lpass_platform_register(pdev); ret = asoc_qcom_lpass_platform_register(pdev);
if (ret) { if (ret) {
dev_err(&pdev->dev, "%s() error registering platform driver: %d\n", dev_err(&pdev->dev, "error registering platform driver: %d\n",
__func__, ret); ret);
goto err_clk; goto err_clk;
} }
......
...@@ -89,8 +89,7 @@ static int lpass_platform_pcmops_open(struct snd_pcm_substream *substream) ...@@ -89,8 +89,7 @@ static int lpass_platform_pcmops_open(struct snd_pcm_substream *substream)
LPAIF_DMACTL_REG(v, dma_ch, dir), 0); LPAIF_DMACTL_REG(v, dma_ch, dir), 0);
if (ret) { if (ret) {
dev_err(soc_runtime->dev, dev_err(soc_runtime->dev,
"%s() error writing to rdmactl reg: %d\n", "error writing to rdmactl reg: %d\n", ret);
__func__, ret);
return ret; return ret;
} }
...@@ -103,8 +102,8 @@ static int lpass_platform_pcmops_open(struct snd_pcm_substream *substream) ...@@ -103,8 +102,8 @@ static int lpass_platform_pcmops_open(struct snd_pcm_substream *substream)
ret = snd_pcm_hw_constraint_integer(runtime, ret = snd_pcm_hw_constraint_integer(runtime,
SNDRV_PCM_HW_PARAM_PERIODS); SNDRV_PCM_HW_PARAM_PERIODS);
if (ret < 0) { if (ret < 0) {
dev_err(soc_runtime->dev, "%s() setting constraints failed: %d\n", dev_err(soc_runtime->dev, "setting constraints failed: %d\n",
__func__, ret); ret);
return -EINVAL; return -EINVAL;
} }
...@@ -151,8 +150,8 @@ static int lpass_platform_pcmops_hw_params(struct snd_pcm_substream *substream, ...@@ -151,8 +150,8 @@ static int lpass_platform_pcmops_hw_params(struct snd_pcm_substream *substream,
bitwidth = snd_pcm_format_width(format); bitwidth = snd_pcm_format_width(format);
if (bitwidth < 0) { if (bitwidth < 0) {
dev_err(soc_runtime->dev, "%s() invalid bit width given: %d\n", dev_err(soc_runtime->dev, "invalid bit width given: %d\n",
__func__, bitwidth); bitwidth);
return bitwidth; return bitwidth;
} }
...@@ -177,8 +176,9 @@ static int lpass_platform_pcmops_hw_params(struct snd_pcm_substream *substream, ...@@ -177,8 +176,9 @@ static int lpass_platform_pcmops_hw_params(struct snd_pcm_substream *substream,
regval |= LPAIF_DMACTL_WPSCNT_FOUR; regval |= LPAIF_DMACTL_WPSCNT_FOUR;
break; break;
default: default:
dev_err(soc_runtime->dev, "%s() invalid PCM config given: bw=%d, ch=%u\n", dev_err(soc_runtime->dev,
__func__, bitwidth, channels); "invalid PCM config given: bw=%d, ch=%u\n",
bitwidth, channels);
return -EINVAL; return -EINVAL;
} }
break; break;
...@@ -201,22 +201,23 @@ static int lpass_platform_pcmops_hw_params(struct snd_pcm_substream *substream, ...@@ -201,22 +201,23 @@ static int lpass_platform_pcmops_hw_params(struct snd_pcm_substream *substream,
regval |= LPAIF_DMACTL_WPSCNT_EIGHT; regval |= LPAIF_DMACTL_WPSCNT_EIGHT;
break; break;
default: default:
dev_err(soc_runtime->dev, "%s() invalid PCM config given: bw=%d, ch=%u\n", dev_err(soc_runtime->dev,
__func__, bitwidth, channels); "invalid PCM config given: bw=%d, ch=%u\n",
bitwidth, channels);
return -EINVAL; return -EINVAL;
} }
break; break;
default: default:
dev_err(soc_runtime->dev, "%s() invalid PCM config given: bw=%d, ch=%u\n", dev_err(soc_runtime->dev, "invalid PCM config given: bw=%d, ch=%u\n",
__func__, bitwidth, channels); bitwidth, channels);
return -EINVAL; return -EINVAL;
} }
ret = regmap_write(drvdata->lpaif_map, ret = regmap_write(drvdata->lpaif_map,
LPAIF_DMACTL_REG(v, ch, dir), regval); LPAIF_DMACTL_REG(v, ch, dir), regval);
if (ret) { if (ret) {
dev_err(soc_runtime->dev, "%s() error writing to rdmactl reg: %d\n", dev_err(soc_runtime->dev, "error writing to rdmactl reg: %d\n",
__func__, ret); ret);
return ret; return ret;
} }
...@@ -237,8 +238,8 @@ static int lpass_platform_pcmops_hw_free(struct snd_pcm_substream *substream) ...@@ -237,8 +238,8 @@ static int lpass_platform_pcmops_hw_free(struct snd_pcm_substream *substream)
reg = LPAIF_DMACTL_REG(v, pcm_data->dma_ch, substream->stream); reg = LPAIF_DMACTL_REG(v, pcm_data->dma_ch, substream->stream);
ret = regmap_write(drvdata->lpaif_map, reg, 0); ret = regmap_write(drvdata->lpaif_map, reg, 0);
if (ret) if (ret)
dev_err(soc_runtime->dev, "%s() error writing to rdmactl reg: %d\n", dev_err(soc_runtime->dev, "error writing to rdmactl reg: %d\n",
__func__, ret); ret);
return ret; return ret;
} }
...@@ -260,8 +261,8 @@ static int lpass_platform_pcmops_prepare(struct snd_pcm_substream *substream) ...@@ -260,8 +261,8 @@ static int lpass_platform_pcmops_prepare(struct snd_pcm_substream *substream)
LPAIF_DMABASE_REG(v, ch, dir), LPAIF_DMABASE_REG(v, ch, dir),
runtime->dma_addr); runtime->dma_addr);
if (ret) { if (ret) {
dev_err(soc_runtime->dev, "%s() error writing to rdmabase reg: %d\n", dev_err(soc_runtime->dev, "error writing to rdmabase reg: %d\n",
__func__, ret); ret);
return ret; return ret;
} }
...@@ -269,8 +270,8 @@ static int lpass_platform_pcmops_prepare(struct snd_pcm_substream *substream) ...@@ -269,8 +270,8 @@ static int lpass_platform_pcmops_prepare(struct snd_pcm_substream *substream)
LPAIF_DMABUFF_REG(v, ch, dir), LPAIF_DMABUFF_REG(v, ch, dir),
(snd_pcm_lib_buffer_bytes(substream) >> 2) - 1); (snd_pcm_lib_buffer_bytes(substream) >> 2) - 1);
if (ret) { if (ret) {
dev_err(soc_runtime->dev, "%s() error writing to rdmabuff reg: %d\n", dev_err(soc_runtime->dev, "error writing to rdmabuff reg: %d\n",
__func__, ret); ret);
return ret; return ret;
} }
...@@ -278,8 +279,8 @@ static int lpass_platform_pcmops_prepare(struct snd_pcm_substream *substream) ...@@ -278,8 +279,8 @@ static int lpass_platform_pcmops_prepare(struct snd_pcm_substream *substream)
LPAIF_DMAPER_REG(v, ch, dir), LPAIF_DMAPER_REG(v, ch, dir),
(snd_pcm_lib_period_bytes(substream) >> 2) - 1); (snd_pcm_lib_period_bytes(substream) >> 2) - 1);
if (ret) { if (ret) {
dev_err(soc_runtime->dev, "%s() error writing to rdmaper reg: %d\n", dev_err(soc_runtime->dev, "error writing to rdmaper reg: %d\n",
__func__, ret); ret);
return ret; return ret;
} }
...@@ -287,8 +288,8 @@ static int lpass_platform_pcmops_prepare(struct snd_pcm_substream *substream) ...@@ -287,8 +288,8 @@ static int lpass_platform_pcmops_prepare(struct snd_pcm_substream *substream)
LPAIF_DMACTL_REG(v, ch, dir), LPAIF_DMACTL_REG(v, ch, dir),
LPAIF_DMACTL_ENABLE_MASK, LPAIF_DMACTL_ENABLE_ON); LPAIF_DMACTL_ENABLE_MASK, LPAIF_DMACTL_ENABLE_ON);
if (ret) { if (ret) {
dev_err(soc_runtime->dev, "%s() error writing to rdmactl reg: %d\n", dev_err(soc_runtime->dev, "error writing to rdmactl reg: %d\n",
__func__, ret); ret);
return ret; return ret;
} }
...@@ -317,8 +318,8 @@ static int lpass_platform_pcmops_trigger(struct snd_pcm_substream *substream, ...@@ -317,8 +318,8 @@ static int lpass_platform_pcmops_trigger(struct snd_pcm_substream *substream,
LPAIF_IRQCLEAR_REG(v, LPAIF_IRQ_PORT_HOST), LPAIF_IRQCLEAR_REG(v, LPAIF_IRQ_PORT_HOST),
LPAIF_IRQ_ALL(ch)); LPAIF_IRQ_ALL(ch));
if (ret) { if (ret) {
dev_err(soc_runtime->dev, "%s() error writing to irqclear reg: %d\n", dev_err(soc_runtime->dev,
__func__, ret); "error writing to irqclear reg: %d\n", ret);
return ret; return ret;
} }
...@@ -327,8 +328,8 @@ static int lpass_platform_pcmops_trigger(struct snd_pcm_substream *substream, ...@@ -327,8 +328,8 @@ static int lpass_platform_pcmops_trigger(struct snd_pcm_substream *substream,
LPAIF_IRQ_ALL(ch), LPAIF_IRQ_ALL(ch),
LPAIF_IRQ_ALL(ch)); LPAIF_IRQ_ALL(ch));
if (ret) { if (ret) {
dev_err(soc_runtime->dev, "%s() error writing to irqen reg: %d\n", dev_err(soc_runtime->dev,
__func__, ret); "error writing to irqen reg: %d\n", ret);
return ret; return ret;
} }
...@@ -337,8 +338,8 @@ static int lpass_platform_pcmops_trigger(struct snd_pcm_substream *substream, ...@@ -337,8 +338,8 @@ static int lpass_platform_pcmops_trigger(struct snd_pcm_substream *substream,
LPAIF_DMACTL_ENABLE_MASK, LPAIF_DMACTL_ENABLE_MASK,
LPAIF_DMACTL_ENABLE_ON); LPAIF_DMACTL_ENABLE_ON);
if (ret) { if (ret) {
dev_err(soc_runtime->dev, "%s() error writing to rdmactl reg: %d\n", dev_err(soc_runtime->dev,
__func__, ret); "error writing to rdmactl reg: %d\n", ret);
return ret; return ret;
} }
break; break;
...@@ -350,8 +351,8 @@ static int lpass_platform_pcmops_trigger(struct snd_pcm_substream *substream, ...@@ -350,8 +351,8 @@ static int lpass_platform_pcmops_trigger(struct snd_pcm_substream *substream,
LPAIF_DMACTL_ENABLE_MASK, LPAIF_DMACTL_ENABLE_MASK,
LPAIF_DMACTL_ENABLE_OFF); LPAIF_DMACTL_ENABLE_OFF);
if (ret) { if (ret) {
dev_err(soc_runtime->dev, "%s() error writing to rdmactl reg: %d\n", dev_err(soc_runtime->dev,
__func__, ret); "error writing to rdmactl reg: %d\n", ret);
return ret; return ret;
} }
...@@ -359,8 +360,8 @@ static int lpass_platform_pcmops_trigger(struct snd_pcm_substream *substream, ...@@ -359,8 +360,8 @@ static int lpass_platform_pcmops_trigger(struct snd_pcm_substream *substream,
LPAIF_IRQEN_REG(v, LPAIF_IRQ_PORT_HOST), LPAIF_IRQEN_REG(v, LPAIF_IRQ_PORT_HOST),
LPAIF_IRQ_ALL(ch), 0); LPAIF_IRQ_ALL(ch), 0);
if (ret) { if (ret) {
dev_err(soc_runtime->dev, "%s() error writing to irqen reg: %d\n", dev_err(soc_runtime->dev,
__func__, ret); "error writing to irqen reg: %d\n", ret);
return ret; return ret;
} }
break; break;
...@@ -386,16 +387,16 @@ static snd_pcm_uframes_t lpass_platform_pcmops_pointer( ...@@ -386,16 +387,16 @@ static snd_pcm_uframes_t lpass_platform_pcmops_pointer(
ret = regmap_read(drvdata->lpaif_map, ret = regmap_read(drvdata->lpaif_map,
LPAIF_DMABASE_REG(v, ch, dir), &base_addr); LPAIF_DMABASE_REG(v, ch, dir), &base_addr);
if (ret) { if (ret) {
dev_err(soc_runtime->dev, "%s() error reading from rdmabase reg: %d\n", dev_err(soc_runtime->dev,
__func__, ret); "error reading from rdmabase reg: %d\n", ret);
return ret; return ret;
} }
ret = regmap_read(drvdata->lpaif_map, ret = regmap_read(drvdata->lpaif_map,
LPAIF_DMACURR_REG(v, ch, dir), &curr_addr); LPAIF_DMACURR_REG(v, ch, dir), &curr_addr);
if (ret) { if (ret) {
dev_err(soc_runtime->dev, "%s() error reading from rdmacurr reg: %d\n", dev_err(soc_runtime->dev,
__func__, ret); "error reading from rdmacurr reg: %d\n", ret);
return ret; return ret;
} }
...@@ -439,8 +440,8 @@ static irqreturn_t lpass_dma_interrupt_handler( ...@@ -439,8 +440,8 @@ static irqreturn_t lpass_dma_interrupt_handler(
LPAIF_IRQCLEAR_REG(v, LPAIF_IRQ_PORT_HOST), LPAIF_IRQCLEAR_REG(v, LPAIF_IRQ_PORT_HOST),
LPAIF_IRQ_PER(chan)); LPAIF_IRQ_PER(chan));
if (rv) { if (rv) {
dev_err(soc_runtime->dev, "%s() error writing to irqclear reg: %d\n", dev_err(soc_runtime->dev,
__func__, rv); "error writing to irqclear reg: %d\n", rv);
return IRQ_NONE; return IRQ_NONE;
} }
snd_pcm_period_elapsed(substream); snd_pcm_period_elapsed(substream);
...@@ -452,11 +453,11 @@ static irqreturn_t lpass_dma_interrupt_handler( ...@@ -452,11 +453,11 @@ static irqreturn_t lpass_dma_interrupt_handler(
LPAIF_IRQCLEAR_REG(v, LPAIF_IRQ_PORT_HOST), LPAIF_IRQCLEAR_REG(v, LPAIF_IRQ_PORT_HOST),
LPAIF_IRQ_XRUN(chan)); LPAIF_IRQ_XRUN(chan));
if (rv) { if (rv) {
dev_err(soc_runtime->dev, "%s() error writing to irqclear reg: %d\n", dev_err(soc_runtime->dev,
__func__, rv); "error writing to irqclear reg: %d\n", rv);
return IRQ_NONE; return IRQ_NONE;
} }
dev_warn(soc_runtime->dev, "%s() xrun warning\n", __func__); dev_warn(soc_runtime->dev, "xrun warning\n");
snd_pcm_stop(substream, SNDRV_PCM_STATE_XRUN); snd_pcm_stop(substream, SNDRV_PCM_STATE_XRUN);
ret = IRQ_HANDLED; ret = IRQ_HANDLED;
} }
...@@ -466,11 +467,11 @@ static irqreturn_t lpass_dma_interrupt_handler( ...@@ -466,11 +467,11 @@ static irqreturn_t lpass_dma_interrupt_handler(
LPAIF_IRQCLEAR_REG(v, LPAIF_IRQ_PORT_HOST), LPAIF_IRQCLEAR_REG(v, LPAIF_IRQ_PORT_HOST),
LPAIF_IRQ_ERR(chan)); LPAIF_IRQ_ERR(chan));
if (rv) { if (rv) {
dev_err(soc_runtime->dev, "%s() error writing to irqclear reg: %d\n", dev_err(soc_runtime->dev,
__func__, rv); "error writing to irqclear reg: %d\n", rv);
return IRQ_NONE; return IRQ_NONE;
} }
dev_err(soc_runtime->dev, "%s() bus access error\n", __func__); dev_err(soc_runtime->dev, "bus access error\n");
snd_pcm_stop(substream, SNDRV_PCM_STATE_DISCONNECTED); snd_pcm_stop(substream, SNDRV_PCM_STATE_DISCONNECTED);
ret = IRQ_HANDLED; ret = IRQ_HANDLED;
} }
...@@ -488,8 +489,7 @@ static irqreturn_t lpass_platform_lpaif_irq(int irq, void *data) ...@@ -488,8 +489,7 @@ static irqreturn_t lpass_platform_lpaif_irq(int irq, void *data)
rv = regmap_read(drvdata->lpaif_map, rv = regmap_read(drvdata->lpaif_map,
LPAIF_IRQSTAT_REG(v, LPAIF_IRQ_PORT_HOST), &irqs); LPAIF_IRQSTAT_REG(v, LPAIF_IRQ_PORT_HOST), &irqs);
if (rv) { if (rv) {
pr_err("%s() error reading from irqstat reg: %d\n", pr_err("error reading from irqstat reg: %d\n", rv);
__func__, rv);
return IRQ_NONE; return IRQ_NONE;
} }
...@@ -571,8 +571,8 @@ int asoc_qcom_lpass_platform_register(struct platform_device *pdev) ...@@ -571,8 +571,8 @@ int asoc_qcom_lpass_platform_register(struct platform_device *pdev)
drvdata->lpaif_irq = platform_get_irq_byname(pdev, "lpass-irq-lpaif"); drvdata->lpaif_irq = platform_get_irq_byname(pdev, "lpass-irq-lpaif");
if (drvdata->lpaif_irq < 0) { if (drvdata->lpaif_irq < 0) {
dev_err(&pdev->dev, "%s() error getting irq handle: %d\n", dev_err(&pdev->dev, "error getting irq handle: %d\n",
__func__, drvdata->lpaif_irq); drvdata->lpaif_irq);
return -ENODEV; return -ENODEV;
} }
...@@ -580,8 +580,7 @@ int asoc_qcom_lpass_platform_register(struct platform_device *pdev) ...@@ -580,8 +580,7 @@ int asoc_qcom_lpass_platform_register(struct platform_device *pdev)
ret = regmap_write(drvdata->lpaif_map, ret = regmap_write(drvdata->lpaif_map,
LPAIF_IRQEN_REG(v, LPAIF_IRQ_PORT_HOST), 0); LPAIF_IRQEN_REG(v, LPAIF_IRQ_PORT_HOST), 0);
if (ret) { if (ret) {
dev_err(&pdev->dev, "%s() error writing to irqen reg: %d\n", dev_err(&pdev->dev, "error writing to irqen reg: %d\n", ret);
__func__, ret);
return ret; return ret;
} }
...@@ -589,8 +588,7 @@ int asoc_qcom_lpass_platform_register(struct platform_device *pdev) ...@@ -589,8 +588,7 @@ int asoc_qcom_lpass_platform_register(struct platform_device *pdev)
lpass_platform_lpaif_irq, IRQF_TRIGGER_RISING, lpass_platform_lpaif_irq, IRQF_TRIGGER_RISING,
"lpass-irq-lpaif", drvdata); "lpass-irq-lpaif", drvdata);
if (ret) { if (ret) {
dev_err(&pdev->dev, "%s() irq request failed: %d\n", dev_err(&pdev->dev, "irq request failed: %d\n", ret);
__func__, ret);
return ret; return ret;
} }
......
...@@ -36,8 +36,7 @@ static int storm_ops_hw_params(struct snd_pcm_substream *substream, ...@@ -36,8 +36,7 @@ static int storm_ops_hw_params(struct snd_pcm_substream *substream,
bitwidth = snd_pcm_format_width(format); bitwidth = snd_pcm_format_width(format);
if (bitwidth < 0) { if (bitwidth < 0) {
dev_err(card->dev, "%s() invalid bit width given: %d\n", dev_err(card->dev, "invalid bit width given: %d\n", bitwidth);
__func__, bitwidth);
return bitwidth; return bitwidth;
} }
...@@ -50,8 +49,8 @@ static int storm_ops_hw_params(struct snd_pcm_substream *substream, ...@@ -50,8 +49,8 @@ static int storm_ops_hw_params(struct snd_pcm_substream *substream,
ret = snd_soc_dai_set_sysclk(soc_runtime->cpu_dai, 0, sysclk_freq, 0); ret = snd_soc_dai_set_sysclk(soc_runtime->cpu_dai, 0, sysclk_freq, 0);
if (ret) { if (ret) {
dev_err(card->dev, "%s() error setting sysclk to %u: %d\n", dev_err(card->dev, "error setting sysclk to %u: %d\n",
__func__, sysclk_freq, ret); sysclk_freq, ret);
return ret; return ret;
} }
...@@ -76,16 +75,14 @@ static int storm_parse_of(struct snd_soc_card *card) ...@@ -76,16 +75,14 @@ static int storm_parse_of(struct snd_soc_card *card)
dai_link->cpu_of_node = of_parse_phandle(np, "cpu", 0); dai_link->cpu_of_node = of_parse_phandle(np, "cpu", 0);
if (!dai_link->cpu_of_node) { if (!dai_link->cpu_of_node) {
dev_err(card->dev, "%s() error getting cpu phandle\n", dev_err(card->dev, "error getting cpu phandle\n");
__func__);
return -EINVAL; return -EINVAL;
} }
dai_link->platform_of_node = dai_link->cpu_of_node; dai_link->platform_of_node = dai_link->cpu_of_node;
dai_link->codec_of_node = of_parse_phandle(np, "codec", 0); dai_link->codec_of_node = of_parse_phandle(np, "codec", 0);
if (!dai_link->codec_of_node) { if (!dai_link->codec_of_node) {
dev_err(card->dev, "%s() error getting codec phandle\n", dev_err(card->dev, "error getting codec phandle\n");
__func__);
return -EINVAL; return -EINVAL;
} }
...@@ -106,8 +103,7 @@ static int storm_platform_probe(struct platform_device *pdev) ...@@ -106,8 +103,7 @@ static int storm_platform_probe(struct platform_device *pdev)
ret = snd_soc_of_parse_card_name(card, "qcom,model"); ret = snd_soc_of_parse_card_name(card, "qcom,model");
if (ret) { if (ret) {
dev_err(&pdev->dev, "%s() error parsing card name: %d\n", dev_err(&pdev->dev, "error parsing card name: %d\n", ret);
__func__, ret);
return ret; return ret;
} }
...@@ -116,15 +112,13 @@ static int storm_platform_probe(struct platform_device *pdev) ...@@ -116,15 +112,13 @@ static int storm_platform_probe(struct platform_device *pdev)
ret = storm_parse_of(card); ret = storm_parse_of(card);
if (ret) { if (ret) {
dev_err(&pdev->dev, "%s() error resolving dai links: %d\n", dev_err(&pdev->dev, "error resolving dai links: %d\n", ret);
__func__, ret);
return ret; return ret;
} }
ret = devm_snd_soc_register_card(&pdev->dev, card); ret = devm_snd_soc_register_card(&pdev->dev, card);
if (ret) if (ret)
dev_err(&pdev->dev, "%s() error registering soundcard: %d\n", dev_err(&pdev->dev, "error registering soundcard: %d\n", ret);
__func__, ret);
return ret; return ret;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment