ASoC: qcom: Remove useless function call
authorDonglin Peng <dolinux.peng@gmail.com>
Sun, 20 Aug 2017 05:43:57 +0000 (13:43 +0800)
committerMark Brown <broonie@kernel.org>
Mon, 21 Aug 2017 16:13:45 +0000 (17:13 +0100)
The function platform_set_drvdata(pdev, data) copies the value of
the variable data to pdev->dev.driver_data,but when calling
snd_soc_register_card,the function dev_set_drvdata(card->dev, card)
will override it, so i think that the former copy operation is
useless and can be removed.

Signed-off-by: Peng Donglin <dolinux.peng@gmail.com>
Acked-by: Banajit Goswami <bgoswami@codeaurora.org>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/qcom/apq8016_sbc.c

index f07aa1e1cdfe1627aeef875524347d49253cfaa8..96a079d9f6979cc87eacaa66c1d0176eefc1edf4 100644 (file)
@@ -206,7 +206,6 @@ static int apq8016_sbc_platform_probe(struct platform_device *pdev)
        if (IS_ERR(data->spkr_iomux))
                return PTR_ERR(data->spkr_iomux);
 
-       platform_set_drvdata(pdev, data);
        snd_soc_card_set_drvdata(card, data);
 
        return devm_snd_soc_register_card(&pdev->dev, card);