[alsa-devel] [PATCH] ASoC: ab8500: Remove pointless cast

Lee Jones lee.jones at linaro.org
Thu Jul 26 17:07:04 CEST 2012


On 26/07/12 15:49, Mark Brown wrote:
> There's never any need to cast away from void.
>
> Signed-off-by: Mark Brown <broonie at opensource.wolfsonmicro.com>
> ---
>   sound/soc/codecs/ab8500-codec.c |    2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/sound/soc/codecs/ab8500-codec.c b/sound/soc/codecs/ab8500-codec.c
> index 3c79592..05d70934 100644
> --- a/sound/soc/codecs/ab8500-codec.c
> +++ b/sound/soc/codecs/ab8500-codec.c
> @@ -2405,7 +2405,7 @@ static int ab8500_codec_probe(struct snd_soc_codec *codec)
>   	dev_dbg(dev, "%s: Enter.\n", __func__);
>
>   	/* Setup AB8500 according to board-settings */
> -	pdata = (struct ab8500_platform_data *)dev_get_platdata(dev->parent);
> +	pdata = dev_get_platdata(dev->parent);
>   	status = ab8500_audio_setup_mics(codec, &pdata->codec->amics);
>   	if (status < 0) {
>   		pr_err("%s: Failed to setup mics (%d)!\n", __func__, status);
>

I think Ola needs to ACK this, but fwiw:

Acked-by: Lee Jones <lee.jones at linaro.org>

-- 
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog


More information about the Alsa-devel mailing list