[PATCH] ASoC: davinci-evm: drop un-necessary remove function

[PATCH] ASoC: davinci-evm: drop un-necessary remove function

[Jyri Sarha]

From: Manish Badarkhe <manishvb@ti.com>

As davinci card gets registered using 'devm_' api
there is no need to unregister the card in 'remove'
function.
Hence drop the 'remove' function.

Signed-off-by: Manish Badarkhe <manishvb@ti.com>
Signed-off-by: Jyri Sarha <jsarha@ti.com>
---
 sound/soc/davinci/davinci-evm.c | 10 ----------
 1 file changed, 10 deletions(-)

diff --git a/sound/soc/davinci/davinci-evm.c b/sound/soc/davinci/davinci-evm.c
index b6bb594..8c2b9be 100644
--- a/sound/soc/davinci/davinci-evm.c
+++ b/sound/soc/davinci/davinci-evm.c
@@ -425,18 +425,8 @@ static int davinci_evm_probe(struct platform_device *pdev)
 	return ret;
 }
 
-static int davinci_evm_remove(struct platform_device *pdev)
-{
-	struct snd_soc_card *card = platform_get_drvdata(pdev);
-
-	snd_soc_unregister_card(card);
-
-	return 0;
-}
-
 static struct platform_driver davinci_evm_driver = {
 	.probe		= davinci_evm_probe,
-	.remove		= davinci_evm_remove,
 	.driver		= {
 		.name	= "davinci_evm",
 		.pm	= &snd_soc_pm_ops,
-- 
1.9.1

Re: [PATCH] ASoC: davinci-evm: drop un-necessary remove function

[Mark Brown]

[-- Attachment #1: Type: text/plain, Size: 356 bytes --]

On Thu, Mar 26, 2015 at 03:38:25PM +0200, Jyri Sarha wrote:
> From: Manish Badarkhe <manishvb@ti.com>
> 
> As davinci card gets registered using 'devm_' api
> there is no need to unregister the card in 'remove'
> function.
> Hence drop the 'remove' function.

Not only that but the current remove function creates a double free.
Applied, thanks.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]