Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 914834a7 authored by Arnd Bergmann's avatar Arnd Bergmann Committed by Greg Kroah-Hartman
Browse files

net: phy: mdio-bcm-unimac: mark PM functions as __maybe_unused 



[ Upstream commit 9b97123a584f60a5bca5a2663485768a1f6cd0a4 ]

The newly added runtime-pm support causes a harmless warning
when CONFIG_PM is disabled:

drivers/net/phy/mdio-bcm-unimac.c:330:12: error: 'unimac_mdio_resume' defined but not used [-Werror=unused-function]
 static int unimac_mdio_resume(struct device *d)
drivers/net/phy/mdio-bcm-unimac.c:321:12: error: 'unimac_mdio_suspend' defined but not used [-Werror=unused-function]
 static int unimac_mdio_suspend(struct device *d)

Marking the functions as __maybe_unused is the easiest workaround
and avoids adding #ifdef checks.

Fixes: b78ac6ecd1b6 ("net: phy: mdio-bcm-unimac: Allow configuring MDIO clock divider")
Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
Signed-off-by: default avatarSasha Levin <sashal@kernel.org>
parent 2ed924f2

drivers/net/phy/mdio-bcm-unimac.c

+2 −2
Original line number Diff line number Diff line
@@ -318,7 +318,7 @@ static int unimac_mdio_remove(struct platform_device *pdev) 
	return 0;

}



static int unimac_mdio_suspend(struct device *d)

static int __maybe_unused unimac_mdio_suspend(struct device *d)

{

	struct unimac_mdio_priv *priv = dev_get_drvdata(d);
@@ -327,7 +327,7 @@ static int unimac_mdio_suspend(struct device *d) 
	return 0;

}



static int unimac_mdio_resume(struct device *d)

static int __maybe_unused unimac_mdio_resume(struct device *d)

{

	struct unimac_mdio_priv *priv = dev_get_drvdata(d);

	int ret;