ASoC: imx-wm8962: Let codec driver enable/disable its MCLK
WM8962 needs its MCLK when powerup in wm8962_resume(). Thus it's better to control the MCLK in codec driver. Thus remove the clock enable in machine driver accordingly. While at it, get rid of imx_wm8962_remove function since it is now empty. Signed-off-by: Daniel Baluta <daniel.baluta@nxp.com> Signed-off-by: Mark Brown <broonie@kernel.org>
This commit is contained in:
parent
70d435ba1c
commit
fd8ba1e309
|
@ -33,7 +33,6 @@ struct imx_wm8962_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;
|
||||
};
|
||||
|
||||
|
@ -163,6 +162,7 @@ static int imx_wm8962_probe(struct platform_device *pdev)
|
|||
struct imx_priv *priv = &card_priv;
|
||||
struct i2c_client *codec_dev;
|
||||
struct imx_wm8962_data *data;
|
||||
struct clk *codec_clk;
|
||||
int int_port, ext_port;
|
||||
int ret;
|
||||
|
||||
|
@ -231,19 +231,14 @@ static int imx_wm8962_probe(struct platform_device *pdev)
|
|||
goto fail;
|
||||
}
|
||||
|
||||
data->codec_clk = devm_clk_get(&codec_dev->dev, NULL);
|
||||
if (IS_ERR(data->codec_clk)) {
|
||||
ret = PTR_ERR(data->codec_clk);
|
||||
codec_clk = devm_clk_get(&codec_dev->dev, NULL);
|
||||
if (IS_ERR(codec_clk)) {
|
||||
ret = PTR_ERR(codec_clk);
|
||||
dev_err(&codec_dev->dev, "failed to get codec clk: %d\n", ret);
|
||||
goto fail;
|
||||
}
|
||||
|
||||
data->clk_frequency = clk_get_rate(data->codec_clk);
|
||||
ret = clk_prepare_enable(data->codec_clk);
|
||||
if (ret) {
|
||||
dev_err(&codec_dev->dev, "failed to enable codec clk: %d\n", ret);
|
||||
goto fail;
|
||||
}
|
||||
data->clk_frequency = clk_get_rate(codec_clk);
|
||||
|
||||
data->dai.name = "HiFi";
|
||||
data->dai.stream_name = "HiFi";
|
||||
|
@ -258,10 +253,10 @@ static int imx_wm8962_probe(struct platform_device *pdev)
|
|||
data->card.dev = &pdev->dev;
|
||||
ret = snd_soc_of_parse_card_name(&data->card, "model");
|
||||
if (ret)
|
||||
goto clk_fail;
|
||||
goto fail;
|
||||
ret = snd_soc_of_parse_audio_routing(&data->card, "audio-routing");
|
||||
if (ret)
|
||||
goto clk_fail;
|
||||
goto fail;
|
||||
data->card.num_links = 1;
|
||||
data->card.owner = THIS_MODULE;
|
||||
data->card.dai_link = &data->dai;
|
||||
|
@ -277,16 +272,9 @@ static int imx_wm8962_probe(struct platform_device *pdev)
|
|||
ret = devm_snd_soc_register_card(&pdev->dev, &data->card);
|
||||
if (ret) {
|
||||
dev_err(&pdev->dev, "snd_soc_register_card failed (%d)\n", ret);
|
||||
goto clk_fail;
|
||||
goto fail;
|
||||
}
|
||||
|
||||
of_node_put(ssi_np);
|
||||
of_node_put(codec_np);
|
||||
|
||||
return 0;
|
||||
|
||||
clk_fail:
|
||||
clk_disable_unprepare(data->codec_clk);
|
||||
fail:
|
||||
of_node_put(ssi_np);
|
||||
of_node_put(codec_np);
|
||||
|
@ -294,17 +282,6 @@ fail:
|
|||
return ret;
|
||||
}
|
||||
|
||||
static int imx_wm8962_remove(struct platform_device *pdev)
|
||||
{
|
||||
struct snd_soc_card *card = platform_get_drvdata(pdev);
|
||||
struct imx_wm8962_data *data = snd_soc_card_get_drvdata(card);
|
||||
|
||||
if (!IS_ERR(data->codec_clk))
|
||||
clk_disable_unprepare(data->codec_clk);
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
||||
static const struct of_device_id imx_wm8962_dt_ids[] = {
|
||||
{ .compatible = "fsl,imx-audio-wm8962", },
|
||||
{ /* sentinel */ }
|
||||
|
@ -318,7 +295,6 @@ static struct platform_driver imx_wm8962_driver = {
|
|||
.of_match_table = imx_wm8962_dt_ids,
|
||||
},
|
||||
.probe = imx_wm8962_probe,
|
||||
.remove = imx_wm8962_remove,
|
||||
};
|
||||
module_platform_driver(imx_wm8962_driver);
|
||||
|
||||
|
|
Loading…
Reference in New Issue