Skip to content

Commit

Permalink
ASoC: wm9705: add private structure
Browse files Browse the repository at this point in the history
Add a private data structure. This is a preparation for a codec which
would need an another data on top of snd_ac97, which will be the case
when an MFD wm97xx device will probe wm9705.

Signed-off-by: Robert Jarzmik <robert.jarzmik@free.fr>
Acked-by: Charles Keepax <ckeepax@opensource.wolfsonmicro.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
  • Loading branch information
Robert Jarzmik authored and Mark Brown committed Sep 19, 2017
1 parent 2ed1a8e commit c6e46e5
Showing 1 changed file with 24 additions and 12 deletions.
36 changes: 24 additions & 12 deletions sound/soc/codecs/wm9705.c
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,10 @@
#define WM9705_VENDOR_ID 0x574d4c05
#define WM9705_VENDOR_ID_MASK 0xffffffff

struct wm9705_priv {
struct snd_ac97 *ac97;
};

static const struct reg_default wm9705_reg_defaults[] = {
{ 0x02, 0x8000 },
{ 0x04, 0x8000 },
Expand Down Expand Up @@ -292,10 +296,10 @@ static int wm9705_soc_suspend(struct snd_soc_codec *codec)

static int wm9705_soc_resume(struct snd_soc_codec *codec)
{
struct snd_ac97 *ac97 = snd_soc_codec_get_drvdata(codec);
struct wm9705_priv *wm9705 = snd_soc_codec_get_drvdata(codec);
int ret;

ret = snd_ac97_reset(ac97, true, WM9705_VENDOR_ID,
ret = snd_ac97_reset(wm9705->ac97, true, WM9705_VENDOR_ID,
WM9705_VENDOR_ID_MASK);
if (ret < 0)
return ret;
Expand All @@ -311,38 +315,38 @@ static int wm9705_soc_resume(struct snd_soc_codec *codec)

static int wm9705_soc_probe(struct snd_soc_codec *codec)
{
struct snd_ac97 *ac97;
struct wm9705_priv *wm9705 = snd_soc_codec_get_drvdata(codec);
struct regmap *regmap;
int ret;

ac97 = snd_soc_new_ac97_codec(codec, WM9705_VENDOR_ID,
WM9705_VENDOR_ID_MASK);
if (IS_ERR(ac97)) {
wm9705->ac97 = snd_soc_new_ac97_codec(codec, WM9705_VENDOR_ID,
WM9705_VENDOR_ID_MASK);
if (IS_ERR(wm9705->ac97)) {
dev_err(codec->dev, "Failed to register AC97 codec\n");
return PTR_ERR(ac97);
return PTR_ERR(wm9705->ac97);
}

regmap = regmap_init_ac97(ac97, &wm9705_regmap_config);
regmap = regmap_init_ac97(wm9705->ac97, &wm9705_regmap_config);
if (IS_ERR(regmap)) {
ret = PTR_ERR(regmap);
goto err_free_ac97_codec;
}

snd_soc_codec_set_drvdata(codec, ac97);
snd_soc_codec_set_drvdata(codec, wm9705->ac97);
snd_soc_codec_init_regmap(codec, regmap);

return 0;
err_free_ac97_codec:
snd_soc_free_ac97_codec(ac97);
snd_soc_free_ac97_codec(wm9705->ac97);
return ret;
}

static int wm9705_soc_remove(struct snd_soc_codec *codec)
{
struct snd_ac97 *ac97 = snd_soc_codec_get_drvdata(codec);
struct wm9705_priv *wm9705 = snd_soc_codec_get_drvdata(codec);

snd_soc_codec_exit_regmap(codec);
snd_soc_free_ac97_codec(ac97);
snd_soc_free_ac97_codec(wm9705->ac97);
return 0;
}

Expand All @@ -364,6 +368,14 @@ static const struct snd_soc_codec_driver soc_codec_dev_wm9705 = {

static int wm9705_probe(struct platform_device *pdev)
{
struct wm9705_priv *wm9705;

wm9705 = devm_kzalloc(&pdev->dev, sizeof(*wm9705), GFP_KERNEL);
if (wm9705 == NULL)
return -ENOMEM;

platform_set_drvdata(pdev, wm9705);

return snd_soc_register_codec(&pdev->dev,
&soc_codec_dev_wm9705, wm9705_dai, ARRAY_SIZE(wm9705_dai));
}
Expand Down

0 comments on commit c6e46e5

Please sign in to comment.