diff options
| author | Vladimir Oltean <vladimir.oltean@nxp.com> | 2025-11-25 14:48:46 +0300 |
|---|---|---|
| committer | Vinod Koul <vkoul@kernel.org> | 2025-12-23 20:41:06 +0300 |
| commit | 04dceaa3c97d3cdc51e1d78dce32ed7388872d07 (patch) | |
| tree | 49869be40b2dada22c36dbd7ccaa4529aaac6922 | |
| parent | 055d08beea2c1a1d0f4eccabbcf570009969e3ce (diff) | |
| download | linux-04dceaa3c97d3cdc51e1d78dce32ed7388872d07.tar.xz | |
phy: lynx-28g: use "dev" argument more in lynx_28g_probe()
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>
Link: https://patch.msgid.link/20251125114847.804961-15-vladimir.oltean@nxp.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
| -rw-r--r-- | drivers/phy/freescale/phy-fsl-lynx-28g.c | 8 |
1 files 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 be804f9b7d5e..423223371dd0 100644 --- a/drivers/phy/freescale/phy-fsl-lynx-28g.c +++ b/drivers/phy/freescale/phy-fsl-lynx-28g.c @@ -1160,10 +1160,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)) @@ -1216,8 +1216,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); } |