[PATCH v4 phy 13/16] phy: lynx-28g: use "dev" argument more in lynx_28g_probe()

Vladimir Oltean
2025-11-10T09:23:19Z

We have "dev" which holds &pdev->dev, but we still dereference this
pointer 4 more times, instead of using the local variable.

Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
---
v3->v4: adapt to the new patch order where "[PATCH v4 phy 03/16] phy:
lynx-28g: support individual lanes as OF PHY providers" has
moved some priv->dev dereferences to lynx_28g_probe_lane()
v2->v3: none
v1->v2: patch is new

drivers/phy/freescale/phy-fsl-lynx-28g.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/phy/freescale/phy-fsl-lynx-28g.c b/drivers/phy/freescale/phy-fsl-lynx-28g.c
index 0010cfa61e07..5d5c94295726 100644
--- a/drivers/phy/freescale/phy-fsl-lynx-28g.c
+++ b/drivers/phy/freescale/phy-fsl-lynx-28g.c
@@ -1162,10 +1162,10 @@ static int lynx_28g_probe(struct platform_device *pdev)
struct device_node *dn;
int err;

- priv = devm_kzalloc(&pdev->dev, sizeof(*priv), GFP_KERNEL);
+ priv = devm_kzalloc(dev, sizeof(*priv), GFP_KERNEL);
if (!priv)
return -ENOMEM;
- priv->dev = &pdev->dev;
+ priv->dev = dev;

priv->base = devm_platform_ioremap_resource(pdev, 0);
if (IS_ERR(priv->base))
@@ -1218,8 +1218,8 @@ static int lynx_28g_probe(struct platform_device *pdev)
queue_delayed_work(system_power_efficient_wq, &priv->cdr_check,
msecs_to_jiffies(1000));

- dev_set_drvdata(&pdev->dev, priv);
- provider = devm_of_phy_provider_register(&pdev->dev, lynx_28g_xlate);
+ dev_set_drvdata(dev, priv);
+ provider = devm_of_phy_provider_register(dev, lynx_28g_xlate);

return PTR_ERR_OR_ZERO(provider);