Skip to content

Commit

Permalink
net: phy: marvell: make some functions static
Browse files Browse the repository at this point in the history
functions m88e1510_get_temp_critical, m88e1510_set_temp_critical and
m88e1510_get_temp_alarm can be made static as they not need to be
in global scope.

Cleans up sparse warnings:
 "symbol 'm88e1510_get_temp_alarm' was not declared. Should it be static?"
 "symbol 'm88e1510_get_temp_critical' was not declared. Should it be
  static?"

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
Colin Ian King authored and David S. Miller committed Jun 4, 2017
1 parent a5eb62f commit f0a4581
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions drivers/net/phy/marvell.c
Original file line number Diff line number Diff line change
Expand Up @@ -1706,7 +1706,7 @@ static int m88e1510_get_temp(struct phy_device *phydev, long *temp)
return ret;
}

int m88e1510_get_temp_critical(struct phy_device *phydev, long *temp)
static int m88e1510_get_temp_critical(struct phy_device *phydev, long *temp)
{
int oldpage;
int ret;
Expand Down Expand Up @@ -1737,7 +1737,7 @@ int m88e1510_get_temp_critical(struct phy_device *phydev, long *temp)
return ret;
}

int m88e1510_set_temp_critical(struct phy_device *phydev, long temp)
static int m88e1510_set_temp_critical(struct phy_device *phydev, long temp)
{
int oldpage;
int ret;
Expand Down Expand Up @@ -1767,7 +1767,7 @@ int m88e1510_set_temp_critical(struct phy_device *phydev, long temp)
return ret;
}

int m88e1510_get_temp_alarm(struct phy_device *phydev, long *alarm)
static int m88e1510_get_temp_alarm(struct phy_device *phydev, long *alarm)
{
int oldpage;
int ret;
Expand Down

0 comments on commit f0a4581

Please sign in to comment.