power: supply: rt5033_charger: Replace "&pdev->dev" by "charger->dev" in probe
authorJakob Hauser <jahau@rocketmail.com>
Sun, 1 Oct 2023 13:17:44 +0000 (15:17 +0200)
committerSebastian Reichel <sebastian.reichel@collabora.com>
Wed, 4 Oct 2023 20:27:34 +0000 (22:27 +0200)
At the beginning of the probe function, "charger->dev" is set equal to
"&pdev->dev". Therefore it's more clear to subsequently use "charger->dev"
instead of "&pdev->dev".

Signed-off-by: Jakob Hauser <jahau@rocketmail.com>
Link: https://lore.kernel.org/r/39b4f00e010446e453905c8dc3ca1e47e9f2c5f8.1696165240.git.jahau@rocketmail.com
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
drivers/power/supply/rt5033_charger.c

index b34ef0ea6f8a9d5448c8fee85d3f76f259b983b0..d19c7e80a92aa8f1f2f018e8e8d59ed0c58cc8a9 100644 (file)
@@ -678,11 +678,11 @@ static int rt5033_charger_probe(struct platform_device *pdev)
        psy_cfg.of_node = pdev->dev.of_node;
        psy_cfg.drv_data = charger;
 
-       charger->psy = devm_power_supply_register(&pdev->dev,
+       charger->psy = devm_power_supply_register(charger->dev,
                                                  &rt5033_charger_desc,
                                                  &psy_cfg);
        if (IS_ERR(charger->psy))
-               return dev_err_probe(&pdev->dev, PTR_ERR(charger->psy),
+               return dev_err_probe(charger->dev, PTR_ERR(charger->psy),
                                     "Failed to register power supply\n");
 
        ret = rt5033_charger_dt_init(charger);
@@ -701,22 +701,22 @@ static int rt5033_charger_probe(struct platform_device *pdev)
        np_edev = of_get_parent(np_conn);
        charger->edev = extcon_find_edev_by_node(np_edev);
        if (IS_ERR(charger->edev)) {
-               dev_warn(&pdev->dev, "no extcon device found in device-tree\n");
+               dev_warn(charger->dev, "no extcon device found in device-tree\n");
                goto out;
        }
 
-       ret = devm_work_autocancel(&pdev->dev, &charger->extcon_work,
+       ret = devm_work_autocancel(charger->dev, &charger->extcon_work,
                                   rt5033_charger_extcon_work);
        if (ret) {
-               dev_err(&pdev->dev, "failed to initialize extcon work\n");
+               dev_err(charger->dev, "failed to initialize extcon work\n");
                return ret;
        }
 
        charger->extcon_nb.notifier_call = rt5033_charger_extcon_notifier;
-       ret = devm_extcon_register_notifier_all(&pdev->dev, charger->edev,
+       ret = devm_extcon_register_notifier_all(charger->dev, charger->edev,
                                                &charger->extcon_nb);
        if (ret) {
-               dev_err(&pdev->dev, "failed to register extcon notifier\n");
+               dev_err(charger->dev, "failed to register extcon notifier\n");
                return ret;
        }
 out: