[PATCH] ASoC: ti: delete some dead code in omap_abe_probe()

Péter Ujfalusi peter.ujfalusi at gmail.com
Wed Jun 30 16:04:15 CEST 2021



On 30/06/2021 14:18, Dan Carpenter wrote:
> This code checks "priv->mclk_freq" twice and the second check is not
> required.  The code is left over from when removed support for legacy
> boot.

Good find, thank you!

Acked-by: Peter Ujfalusi <peter.ujfalusi at gmail.com>

> Fixes: 8fe120b5a665 ("ASoC: omap-abe-twl6040: Remove support for pdata (legacy boot)")
> Signed-off-by: Dan Carpenter <dan.carpenter at oracle.com>
> ---
>  sound/soc/ti/omap-abe-twl6040.c | 5 -----
>  1 file changed, 5 deletions(-)
> 
> diff --git a/sound/soc/ti/omap-abe-twl6040.c b/sound/soc/ti/omap-abe-twl6040.c
> index 91cc9a4f44d7..2e3d1eea77c1 100644
> --- a/sound/soc/ti/omap-abe-twl6040.c
> +++ b/sound/soc/ti/omap-abe-twl6040.c
> @@ -292,11 +292,6 @@ static int omap_abe_probe(struct platform_device *pdev)
>  
>  	card->fully_routed = 1;
>  
> -	if (!priv->mclk_freq) {
> -		dev_err(&pdev->dev, "MCLK frequency missing\n");
> -		return -ENODEV;
> -	}
> -
>  	card->dai_link = priv->dai_links;
>  	card->num_links = num_links;
>  
> 

-- 
Péter


More information about the Alsa-devel mailing list