ASoC: jz4740-i2s: Add local dev variable in probe function
authorPaul Cercueil <paul@crapouillou.net>
Fri, 6 Mar 2020 22:29:28 +0000 (23:29 +0100)
committerMark Brown <broonie@kernel.org>
Mon, 9 Mar 2020 14:25:05 +0000 (14:25 +0000)
Make the code cleaner by using a "struct device *dev" variable instead
of dereferencing it everytime from within the struct platform_device.

Signed-off-by: Paul Cercueil <paul@crapouillou.net>
Link: https://lore.kernel.org/r/20200306222931.39664-3-paul@crapouillou.net
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/jz4740/jz4740-i2s.c

index 9d540588120918b70cff08b31ae5100b29783bb0..2572aba843e3b2c0011dc6329460810e51245dfd 100644 (file)
@@ -492,45 +492,45 @@ MODULE_DEVICE_TABLE(of, jz4740_of_matches);
 
 static int jz4740_i2s_dev_probe(struct platform_device *pdev)
 {
+       struct device *dev = &pdev->dev;
        struct jz4740_i2s *i2s;
        struct resource *mem;
        int ret;
 
-       i2s = devm_kzalloc(&pdev->dev, sizeof(*i2s), GFP_KERNEL);
+       i2s = devm_kzalloc(dev, sizeof(*i2s), GFP_KERNEL);
        if (!i2s)
                return -ENOMEM;
 
-       i2s->version =
-               (enum jz47xx_i2s_version)of_device_get_match_data(&pdev->dev);
+       i2s->version = (enum jz47xx_i2s_version)of_device_get_match_data(dev);
 
        mem = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-       i2s->base = devm_ioremap_resource(&pdev->dev, mem);
+       i2s->base = devm_ioremap_resource(dev, mem);
        if (IS_ERR(i2s->base))
                return PTR_ERR(i2s->base);
 
        i2s->phys_base = mem->start;
 
-       i2s->clk_aic = devm_clk_get(&pdev->dev, "aic");
+       i2s->clk_aic = devm_clk_get(dev, "aic");
        if (IS_ERR(i2s->clk_aic))
                return PTR_ERR(i2s->clk_aic);
 
-       i2s->clk_i2s = devm_clk_get(&pdev->dev, "i2s");
+       i2s->clk_i2s = devm_clk_get(dev, "i2s");
        if (IS_ERR(i2s->clk_i2s))
                return PTR_ERR(i2s->clk_i2s);
 
        platform_set_drvdata(pdev, i2s);
 
        if (i2s->version == JZ_I2S_JZ4780)
-               ret = devm_snd_soc_register_component(&pdev->dev,
+               ret = devm_snd_soc_register_component(dev,
                        &jz4740_i2s_component, &jz4780_i2s_dai, 1);
        else
-               ret = devm_snd_soc_register_component(&pdev->dev,
+               ret = devm_snd_soc_register_component(dev,
                        &jz4740_i2s_component, &jz4740_i2s_dai, 1);
 
        if (ret)
                return ret;
 
-       return devm_snd_dmaengine_pcm_register(&pdev->dev, NULL,
+       return devm_snd_dmaengine_pcm_register(dev, NULL,
                SND_DMAENGINE_PCM_FLAG_COMPAT);
 }