diff options
| author | Peter Ujfalusi <peter.ujfalusi@ti.com> | 2016-06-02 09:55:24 (GMT) |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2016-06-02 09:58:19 (GMT) |
| commit | 272ee030ebc9da58a5796e6e26f78b77ecce7c21 (patch) | |
| tree | 09ce9ad550360aff537f1e30eec3de3c2473abbe | |
| parent | 9ac0013ce6744fb49d961e592e1339ab1453b914 (diff) | |
| download | linux-272ee030ebc9da58a5796e6e26f78b77ecce7c21.tar.xz | |
ASoC: davinci-mcasp: Use a copy of pdata per instance during DT boot
Instead of modifying the static pdata struct per McASP instance we need to
allocate pdata for each McASP.
This way we can avoid configuration leakage from prior McASP to McASP
drivers probed at later time.
Reported-by: Misael Lopez Cruz <misael.lopez@ti.com>
Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
| -rw-r--r-- | sound/soc/davinci/davinci-mcasp.c | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/sound/soc/davinci/davinci-mcasp.c b/sound/soc/davinci/davinci-mcasp.c index 237dc67..05c2d33 100644 --- a/sound/soc/davinci/davinci-mcasp.c +++ b/sound/soc/davinci/davinci-mcasp.c @@ -1599,7 +1599,14 @@ static struct davinci_mcasp_pdata *davinci_mcasp_set_pdata_from_of( pdata = pdev->dev.platform_data; return pdata; } else if (match) { - pdata = (struct davinci_mcasp_pdata*) match->data; + pdata = devm_kmemdup(&pdev->dev, match->data, sizeof(*pdata), + GFP_KERNEL); + if (!pdata) { + dev_err(&pdev->dev, + "Failed to allocate memory for pdata\n"); + ret = -ENOMEM; + return pdata; + } } else { /* control shouldn't reach here. something is wrong */ ret = -EINVAL; |