Thread (17 messages) flat view 17 messages, 4 authors, 2017-09-05

[PATCH v6 09/12] ASoC: wm9712: add ac97 new bus support

From: ckeepax@opensource.cirrus.com (Charles Keepax)
Date: 2017-09-04 10:21:20
Also in: alsa-devel, linux-input, lkml

On Sat, Sep 02, 2017 at 09:54:11PM +0200, Robert Jarzmik wrote:
quoted hunk ↗ jump to hunk
Add support for the new ac97 bus model, where devices are automatically
discovered on AC-Links.

Signed-off-by: Robert Jarzmik <robert.jarzmik@free.fr>
---
Since v4:
 - added #ifdef spotted by Charles
---
 	snd_soc_codec_init_regmap(codec, regmap);
@@ -666,7 +677,8 @@ static int wm9712_soc_remove(struct snd_soc_codec *codec)
 	struct wm9712_priv *wm9712 = snd_soc_codec_get_drvdata(codec);
 
 	snd_soc_codec_exit_regmap(codec);
-	snd_soc_free_ac97_codec(wm9712->ac97);
+	if (!wm9712->mfd_pdata)
+		snd_soc_free_ac97_codec(wm9712->ac97);
This bit is still missing the ifdef it has in the preceeding
patch.

Otherwise looks good to me.

Thanks,
Charles
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help