Commit 8483c067 authored by Nicolin Chen's avatar Nicolin Chen Committed by Mark Brown

ASoC: fsl_ssi: Cache pdev->dev pointer

There should be no trouble to understand dev = pdev->dev.
This can save some space to have more print info or save
some wrapped lines.
Signed-off-by: default avatarNicolin Chen <nicoleotsuka@gmail.com>
Tested-by: default avatarMaciej S. Szmigiero <mail@maciej.szmigiero.name>
Reviewed-by: default avatarMaciej S. Szmigiero <mail@maciej.szmigiero.name>
Acked-by: default avatarTimur Tabi <timur@tabi.org>
Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent f3176834
...@@ -1379,23 +1379,24 @@ static int fsl_ssi_imx_probe(struct platform_device *pdev, ...@@ -1379,23 +1379,24 @@ static int fsl_ssi_imx_probe(struct platform_device *pdev,
struct fsl_ssi *ssi, void __iomem *iomem) struct fsl_ssi *ssi, void __iomem *iomem)
{ {
struct device_node *np = pdev->dev.of_node; struct device_node *np = pdev->dev.of_node;
struct device *dev = &pdev->dev;
u32 dmas[4]; u32 dmas[4];
int ret; int ret;
if (ssi->has_ipg_clk_name) if (ssi->has_ipg_clk_name)
ssi->clk = devm_clk_get(&pdev->dev, "ipg"); ssi->clk = devm_clk_get(dev, "ipg");
else else
ssi->clk = devm_clk_get(&pdev->dev, NULL); ssi->clk = devm_clk_get(dev, NULL);
if (IS_ERR(ssi->clk)) { if (IS_ERR(ssi->clk)) {
ret = PTR_ERR(ssi->clk); ret = PTR_ERR(ssi->clk);
dev_err(&pdev->dev, "could not get clock: %d\n", ret); dev_err(dev, "could not get clock: %d\n", ret);
return ret; return ret;
} }
if (!ssi->has_ipg_clk_name) { if (!ssi->has_ipg_clk_name) {
ret = clk_prepare_enable(ssi->clk); ret = clk_prepare_enable(ssi->clk);
if (ret) { if (ret) {
dev_err(&pdev->dev, "clk_prepare_enable failed: %d\n", ret); dev_err(dev, "clk_prepare_enable failed: %d\n", ret);
return ret; return ret;
} }
} }
...@@ -1403,9 +1404,9 @@ static int fsl_ssi_imx_probe(struct platform_device *pdev, ...@@ -1403,9 +1404,9 @@ static int fsl_ssi_imx_probe(struct platform_device *pdev,
/* For those SLAVE implementations, we ignore non-baudclk cases /* For those SLAVE implementations, we ignore non-baudclk cases
* and, instead, abandon MASTER mode that needs baud clock. * and, instead, abandon MASTER mode that needs baud clock.
*/ */
ssi->baudclk = devm_clk_get(&pdev->dev, "baud"); ssi->baudclk = devm_clk_get(dev, "baud");
if (IS_ERR(ssi->baudclk)) if (IS_ERR(ssi->baudclk))
dev_dbg(&pdev->dev, "could not get baud clock: %ld\n", dev_dbg(dev, "could not get baud clock: %ld\n",
PTR_ERR(ssi->baudclk)); PTR_ERR(ssi->baudclk));
ssi->dma_params_tx.maxburst = ssi->dma_maxburst; ssi->dma_params_tx.maxburst = ssi->dma_maxburst;
...@@ -1469,6 +1470,7 @@ static int fsl_ssi_probe(struct platform_device *pdev) ...@@ -1469,6 +1470,7 @@ static int fsl_ssi_probe(struct platform_device *pdev)
struct fsl_ssi *ssi; struct fsl_ssi *ssi;
int ret = 0; int ret = 0;
struct device_node *np = pdev->dev.of_node; struct device_node *np = pdev->dev.of_node;
struct device *dev = &pdev->dev;
const struct of_device_id *of_id; const struct of_device_id *of_id;
const char *p, *sprop; const char *p, *sprop;
const uint32_t *iprop; const uint32_t *iprop;
...@@ -1477,17 +1479,16 @@ static int fsl_ssi_probe(struct platform_device *pdev) ...@@ -1477,17 +1479,16 @@ static int fsl_ssi_probe(struct platform_device *pdev)
char name[64]; char name[64];
struct regmap_config regconfig = fsl_ssi_regconfig; struct regmap_config regconfig = fsl_ssi_regconfig;
of_id = of_match_device(fsl_ssi_ids, &pdev->dev); of_id = of_match_device(fsl_ssi_ids, dev);
if (!of_id || !of_id->data) if (!of_id || !of_id->data)
return -EINVAL; return -EINVAL;
ssi = devm_kzalloc(&pdev->dev, sizeof(*ssi), ssi = devm_kzalloc(dev, sizeof(*ssi), GFP_KERNEL);
GFP_KERNEL);
if (!ssi) if (!ssi)
return -ENOMEM; return -ENOMEM;
ssi->soc = of_id->data; ssi->soc = of_id->data;
ssi->dev = &pdev->dev; ssi->dev = dev;
sprop = of_get_property(np, "fsl,mode", NULL); sprop = of_get_property(np, "fsl,mode", NULL);
if (sprop) { if (sprop) {
...@@ -1507,10 +1508,10 @@ static int fsl_ssi_probe(struct platform_device *pdev) ...@@ -1507,10 +1508,10 @@ static int fsl_ssi_probe(struct platform_device *pdev)
memcpy(&ssi->cpu_dai_drv, &fsl_ssi_dai_template, memcpy(&ssi->cpu_dai_drv, &fsl_ssi_dai_template,
sizeof(fsl_ssi_dai_template)); sizeof(fsl_ssi_dai_template));
} }
ssi->cpu_dai_drv.name = dev_name(&pdev->dev); ssi->cpu_dai_drv.name = dev_name(dev);
res = platform_get_resource(pdev, IORESOURCE_MEM, 0); res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
iomem = devm_ioremap_resource(&pdev->dev, res); iomem = devm_ioremap_resource(dev, res);
if (IS_ERR(iomem)) if (IS_ERR(iomem))
return PTR_ERR(iomem); return PTR_ERR(iomem);
ssi->ssi_phys = res->start; ssi->ssi_phys = res->start;
...@@ -1528,21 +1529,20 @@ static int fsl_ssi_probe(struct platform_device *pdev) ...@@ -1528,21 +1529,20 @@ static int fsl_ssi_probe(struct platform_device *pdev)
ret = of_property_match_string(np, "clock-names", "ipg"); ret = of_property_match_string(np, "clock-names", "ipg");
if (ret < 0) { if (ret < 0) {
ssi->has_ipg_clk_name = false; ssi->has_ipg_clk_name = false;
ssi->regs = devm_regmap_init_mmio(&pdev->dev, iomem, ssi->regs = devm_regmap_init_mmio(dev, iomem, &regconfig);
&regconfig);
} else { } else {
ssi->has_ipg_clk_name = true; ssi->has_ipg_clk_name = true;
ssi->regs = devm_regmap_init_mmio_clk(&pdev->dev, ssi->regs = devm_regmap_init_mmio_clk(dev, "ipg", iomem,
"ipg", iomem, &regconfig); &regconfig);
} }
if (IS_ERR(ssi->regs)) { if (IS_ERR(ssi->regs)) {
dev_err(&pdev->dev, "Failed to init register map\n"); dev_err(dev, "Failed to init register map\n");
return PTR_ERR(ssi->regs); return PTR_ERR(ssi->regs);
} }
ssi->irq = platform_get_irq(pdev, 0); ssi->irq = platform_get_irq(pdev, 0);
if (ssi->irq < 0) { if (ssi->irq < 0) {
dev_err(&pdev->dev, "no irq for node %s\n", pdev->name); dev_err(dev, "no irq for node %s\n", pdev->name);
return ssi->irq; return ssi->irq;
} }
...@@ -1605,7 +1605,7 @@ static int fsl_ssi_probe(struct platform_device *pdev) ...@@ -1605,7 +1605,7 @@ static int fsl_ssi_probe(struct platform_device *pdev)
break; break;
} }
dev_set_drvdata(&pdev->dev, ssi); dev_set_drvdata(dev, ssi);
if (ssi->soc->imx) { if (ssi->soc->imx) {
ret = fsl_ssi_imx_probe(pdev, ssi, iomem); ret = fsl_ssi_imx_probe(pdev, ssi, iomem);
...@@ -1617,30 +1617,28 @@ static int fsl_ssi_probe(struct platform_device *pdev) ...@@ -1617,30 +1617,28 @@ static int fsl_ssi_probe(struct platform_device *pdev)
mutex_init(&ssi->ac97_reg_lock); mutex_init(&ssi->ac97_reg_lock);
ret = snd_soc_set_ac97_ops_of_reset(&fsl_ssi_ac97_ops, pdev); ret = snd_soc_set_ac97_ops_of_reset(&fsl_ssi_ac97_ops, pdev);
if (ret) { if (ret) {
dev_err(&pdev->dev, "could not set AC'97 ops\n"); dev_err(dev, "could not set AC'97 ops\n");
goto error_ac97_ops; goto error_ac97_ops;
} }
} }
ret = devm_snd_soc_register_component(&pdev->dev, &fsl_ssi_component, ret = devm_snd_soc_register_component(dev, &fsl_ssi_component,
&ssi->cpu_dai_drv, 1); &ssi->cpu_dai_drv, 1);
if (ret) { if (ret) {
dev_err(&pdev->dev, "failed to register DAI: %d\n", ret); dev_err(dev, "failed to register DAI: %d\n", ret);
goto error_asoc_register; goto error_asoc_register;
} }
if (ssi->use_dma) { if (ssi->use_dma) {
ret = devm_request_irq(&pdev->dev, ssi->irq, ret = devm_request_irq(dev, ssi->irq, fsl_ssi_isr, 0,
fsl_ssi_isr, 0, dev_name(&pdev->dev), dev_name(dev), ssi);
ssi);
if (ret < 0) { if (ret < 0) {
dev_err(&pdev->dev, "could not claim irq %u\n", dev_err(dev, "could not claim irq %u\n", ssi->irq);
ssi->irq);
goto error_asoc_register; goto error_asoc_register;
} }
} }
ret = fsl_ssi_debugfs_create(&ssi->dbg_stats, &pdev->dev); ret = fsl_ssi_debugfs_create(&ssi->dbg_stats, dev);
if (ret) if (ret)
goto error_asoc_register; goto error_asoc_register;
...@@ -1665,23 +1663,23 @@ static int fsl_ssi_probe(struct platform_device *pdev) ...@@ -1665,23 +1663,23 @@ static int fsl_ssi_probe(struct platform_device *pdev)
snprintf(name, sizeof(name), "snd-soc-%s", sprop); snprintf(name, sizeof(name), "snd-soc-%s", sprop);
make_lowercase(name); make_lowercase(name);
ssi->pdev = platform_device_register_data(&pdev->dev, name, 0, NULL, 0); ssi->pdev = platform_device_register_data(dev, name, 0, NULL, 0);
if (IS_ERR(ssi->pdev)) { if (IS_ERR(ssi->pdev)) {
ret = PTR_ERR(ssi->pdev); ret = PTR_ERR(ssi->pdev);
dev_err(&pdev->dev, "failed to register platform: %d\n", ret); dev_err(dev, "failed to register platform: %d\n", ret);
goto error_sound_card; goto error_sound_card;
} }
done: done:
if (ssi->dai_fmt) if (ssi->dai_fmt)
_fsl_ssi_set_dai_fmt(&pdev->dev, ssi, ssi->dai_fmt); _fsl_ssi_set_dai_fmt(dev, ssi, ssi->dai_fmt);
if (fsl_ssi_is_ac97(ssi)) { if (fsl_ssi_is_ac97(ssi)) {
u32 ssi_idx; u32 ssi_idx;
ret = of_property_read_u32(np, "cell-index", &ssi_idx); ret = of_property_read_u32(np, "cell-index", &ssi_idx);
if (ret) { if (ret) {
dev_err(&pdev->dev, "cannot get SSI index property\n"); dev_err(dev, "cannot get SSI index property\n");
goto error_sound_card; goto error_sound_card;
} }
...@@ -1689,7 +1687,7 @@ static int fsl_ssi_probe(struct platform_device *pdev) ...@@ -1689,7 +1687,7 @@ static int fsl_ssi_probe(struct platform_device *pdev)
"ac97-codec", ssi_idx, NULL, 0); "ac97-codec", ssi_idx, NULL, 0);
if (IS_ERR(ssi->pdev)) { if (IS_ERR(ssi->pdev)) {
ret = PTR_ERR(ssi->pdev); ret = PTR_ERR(ssi->pdev);
dev_err(&pdev->dev, dev_err(dev,
"failed to register AC97 codec platform: %d\n", "failed to register AC97 codec platform: %d\n",
ret); ret);
goto error_sound_card; goto error_sound_card;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment