Re: [PATCH 13/17] ASoC: Tegra TrimSlice machine: Use devm_ APIs and module_platform_driver

[Thierry Reding <thierry.reding-RM9K5IK7kjKj5M59NBduVrNAH6kLmebB@public.gmane.org>]

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

* Stephen Warren wrote:
> module_platform_drive saves some boiler-plate code.

Some here.

> 
> The devm_ APIs remove the need to manually clean up allocations,
> thus removing some code.
> 
> Signed-off-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> ---
>  sound/soc/tegra/trimslice.c |   26 +++++++-------------------
>  1 files changed, 7 insertions(+), 19 deletions(-)
> 
> diff --git a/sound/soc/tegra/trimslice.c b/sound/soc/tegra/trimslice.c
> index 2699a6f..d564b40 100644
> --- a/sound/soc/tegra/trimslice.c
> +++ b/sound/soc/tegra/trimslice.c
> @@ -170,15 +170,17 @@ static __devinit int tegra_snd_trimslice_probe(struct platform_device *pdev)
>  	struct tegra_trimslice *trimslice;
>  	int ret;
>  
> -	trimslice = kzalloc(sizeof(struct tegra_trimslice), GFP_KERNEL);
> +	trimslice = devm_kzalloc(&pdev->dev, sizeof(struct tegra_trimslice),
> +				 GFP_KERNEL);
>  	if (!trimslice) {
>  		dev_err(&pdev->dev, "Can't allocate tegra_trimslice\n");
> -		return -ENOMEM;
> +		ret = -ENOMEM;
> +		goto err;
>  	}
>  
>  	ret = tegra_asoc_utils_init(&trimslice->util_data, &pdev->dev);
>  	if (ret)
> -		goto err_free_trimslice;
> +		goto err;
>  
>  	card->dev = &pdev->dev;
>  	platform_set_drvdata(pdev, card);
> @@ -195,8 +197,7 @@ static __devinit int tegra_snd_trimslice_probe(struct platform_device *pdev)
>  
>  err_fini_utils:
>  	tegra_asoc_utils_fini(&trimslice->util_data);
> -err_free_trimslice:
> -	kfree(trimslice);
> +err:
>  	return ret;
>  }
>  
> @@ -209,8 +210,6 @@ static int __devexit tegra_snd_trimslice_remove(struct platform_device *pdev)
>  
>  	tegra_asoc_utils_fini(&trimslice->util_data);
>  
> -	kfree(trimslice);
> -
>  	return 0;
>  }
>  
> @@ -222,18 +221,7 @@ static struct platform_driver tegra_snd_trimslice_driver = {
>  	.probe = tegra_snd_trimslice_probe,
>  	.remove = __devexit_p(tegra_snd_trimslice_remove),
>  };
> -
> -static int __init snd_tegra_trimslice_init(void)
> -{
> -	return platform_driver_register(&tegra_snd_trimslice_driver);
> -}
> -module_init(snd_tegra_trimslice_init);
> -
> -static void __exit snd_tegra_trimslice_exit(void)
> -{
> -	platform_driver_unregister(&tegra_snd_trimslice_driver);
> -}
> -module_exit(snd_tegra_trimslice_exit);
> +module_platform_driver(tegra_snd_trimslice_driver);
>  
>  MODULE_AUTHOR("Mike Rapoport <mike-UTxiZqZC01RS1MOuV/RT9w@public.gmane.org>");
>  MODULE_DESCRIPTION("Trimslice machine ASoC driver");
> -- 
> 1.7.0.4
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
> 

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