ASoC: Intel: bytcr_rt5651: Use temporary variable for struct device
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Thu, 7 Oct 2021 17:02:48 +0000 (20:02 +0300)
committerMark Brown <broonie@kernel.org>
Mon, 11 Oct 2021 13:58:52 +0000 (14:58 +0100)
Use temporary variable for struct device to make code neater.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
Link: https://lore.kernel.org/r/20211007170250.27997-3-andriy.shevchenko@linux.intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/intel/boards/bytcr_rt5651.c

index 2335cca17cc87b65491cc181886261dde3d241af..36f63516f0cbf4f601ba251de60992eabc3a9a2a 100644 (file)
@@ -907,12 +907,12 @@ static int snd_byt_rt5651_mc_probe(struct platform_device *pdev)
        int dai_index = 0;
        int i;
 
-       priv = devm_kzalloc(&pdev->dev, sizeof(*priv), GFP_KERNEL);
+       priv = devm_kzalloc(dev, sizeof(*priv), GFP_KERNEL);
        if (!priv)
                return -ENOMEM;
 
        /* register the soc card */
-       byt_rt5651_card.dev = &pdev->dev;
+       byt_rt5651_card.dev = dev;
        snd_soc_card_set_drvdata(&byt_rt5651_card, priv);
 
        /* fix index of codec dai */
@@ -932,7 +932,7 @@ static int snd_byt_rt5651_mc_probe(struct platform_device *pdev)
                put_device(&adev->dev);
                byt_rt5651_dais[dai_index].codecs->name = byt_rt5651_codec_name;
        } else {
-               dev_err(&pdev->dev, "Error cannot find '%s' dev\n", mach->id);
+               dev_err(dev, "Error cannot find '%s' dev\n", mach->id);
                return -ENXIO;
        }
 
@@ -980,13 +980,13 @@ static int snd_byt_rt5651_mc_probe(struct platform_device *pdev)
                                                               &pkg_ctx);
                if (pkg_found) {
                        if (chan_package.aif_value == 1) {
-                               dev_info(&pdev->dev, "BIOS Routing: AIF1 connected\n");
+                               dev_info(dev, "BIOS Routing: AIF1 connected\n");
                                byt_rt5651_quirk |= BYT_RT5651_SSP0_AIF1;
                        } else  if (chan_package.aif_value == 2) {
-                               dev_info(&pdev->dev, "BIOS Routing: AIF2 connected\n");
+                               dev_info(dev, "BIOS Routing: AIF2 connected\n");
                                byt_rt5651_quirk |= BYT_RT5651_SSP0_AIF2;
                        } else {
-                               dev_info(&pdev->dev, "BIOS Routing isn't valid, ignored\n");
+                               dev_info(dev, "BIOS Routing isn't valid, ignored\n");
                                pkg_found = false;
                        }
                }
@@ -1001,7 +1001,7 @@ static int snd_byt_rt5651_mc_probe(struct platform_device *pdev)
        dmi_check_system(byt_rt5651_quirk_table);
 
        if (quirk_override != -1) {
-               dev_info(&pdev->dev, "Overriding quirk 0x%lx => 0x%x\n",
+               dev_info(dev, "Overriding quirk 0x%lx => 0x%x\n",
                         byt_rt5651_quirk, quirk_override);
                byt_rt5651_quirk = quirk_override;
        }
@@ -1017,8 +1017,7 @@ static int snd_byt_rt5651_mc_probe(struct platform_device *pdev)
 
        if (byt_rt5651_gpios) {
                devm_acpi_dev_add_driver_gpios(codec_dev, byt_rt5651_gpios);
-               priv->ext_amp_gpio = devm_fwnode_gpiod_get(&pdev->dev,
-                                                          codec_dev->fwnode,
+               priv->ext_amp_gpio = devm_fwnode_gpiod_get(dev, codec_dev->fwnode,
                                                           "ext-amp-enable",
                                                           GPIOD_OUT_LOW,
                                                           "speaker-amp");
@@ -1029,15 +1028,13 @@ static int snd_byt_rt5651_mc_probe(struct platform_device *pdev)
                                priv->ext_amp_gpio = NULL;
                                break;
                        default:
-                               dev_err(&pdev->dev, "Failed to get ext-amp-enable GPIO: %d\n",
-                                       ret_val);
+                               dev_err(dev, "Failed to get ext-amp-enable GPIO: %d\n", ret_val);
                                fallthrough;
                        case -EPROBE_DEFER:
                                goto err;
                        }
                }
-               priv->hp_detect = devm_fwnode_gpiod_get(&pdev->dev,
-                                                       codec_dev->fwnode,
+               priv->hp_detect = devm_fwnode_gpiod_get(dev, codec_dev->fwnode,
                                                        "hp-detect",
                                                        GPIOD_IN,
                                                        "hp-detect");
@@ -1048,8 +1045,7 @@ static int snd_byt_rt5651_mc_probe(struct platform_device *pdev)
                                priv->hp_detect = NULL;
                                break;
                        default:
-                               dev_err(&pdev->dev, "Failed to get hp-detect GPIO: %d\n",
-                                       ret_val);
+                               dev_err(dev, "Failed to get hp-detect GPIO: %d\n", ret_val);
                                fallthrough;
                        case -EPROBE_DEFER:
                                goto err;
@@ -1057,7 +1053,7 @@ static int snd_byt_rt5651_mc_probe(struct platform_device *pdev)
                }
        }
 
-       log_quirks(&pdev->dev);
+       log_quirks(dev);
 
        if ((byt_rt5651_quirk & BYT_RT5651_SSP2_AIF2) ||
            (byt_rt5651_quirk & BYT_RT5651_SSP0_AIF2))
@@ -1068,10 +1064,10 @@ static int snd_byt_rt5651_mc_probe(struct platform_device *pdev)
                byt_rt5651_dais[dai_index].cpus->dai_name = "ssp0-port";
 
        if (byt_rt5651_quirk & BYT_RT5651_MCLK_EN) {
-               priv->mclk = devm_clk_get(&pdev->dev, "pmc_plt_clk_3");
+               priv->mclk = devm_clk_get(dev, "pmc_plt_clk_3");
                if (IS_ERR(priv->mclk)) {
                        ret_val = PTR_ERR(priv->mclk);
-                       dev_err(&pdev->dev,
+                       dev_err(dev,
                                "Failed to get MCLK from pmc_plt_clk_3: %d\n",
                                ret_val);
                        /*
@@ -1111,7 +1107,7 @@ static int snd_byt_rt5651_mc_probe(struct platform_device *pdev)
        if (ret_val)
                goto err;
 
-       sof_parent = snd_soc_acpi_sof_parent(&pdev->dev);
+       sof_parent = snd_soc_acpi_sof_parent(dev);
 
        /* set card and driver name */
        if (sof_parent) {
@@ -1124,13 +1120,11 @@ static int snd_byt_rt5651_mc_probe(struct platform_device *pdev)
 
        /* set pm ops */
        if (sof_parent)
-               pdev->dev.driver->pm = &snd_soc_pm_ops;
-
-       ret_val = devm_snd_soc_register_card(&pdev->dev, &byt_rt5651_card);
+               dev->driver->pm = &snd_soc_pm_ops;
 
+       ret_val = devm_snd_soc_register_card(dev, &byt_rt5651_card);
        if (ret_val) {
-               dev_err(&pdev->dev, "devm_snd_soc_register_card failed %d\n",
-                       ret_val);
+               dev_err(dev, "devm_snd_soc_register_card failed %d\n", ret_val);
                goto err;
        }
        platform_set_drvdata(pdev, &byt_rt5651_card);