Message ID | 20171003104633.27151-1-colin.king@canonical.com |
---|---|
State | Accepted, archived |
Delegated to: | David Miller |
Headers | show |
Series | net: dsa: mt7530: make functions mt7530_phy_write static | expand |
On Tue, Oct 03, 2017 at 11:46:33AM +0100, Colin King wrote: > From: Colin Ian King <colin.king@canonical.com> > > The function mt7530_phy_write is local to the source and does not need to > be in global scope, so make it static. > > Cleans up sparse warnings: > symbol 'mt7530_phy_write' was not declared. Should it be static? > > Signed-off-by: Colin Ian King <colin.king@canonical.com> Reviewed-by: Andrew Lunn <andrew@lunn.ch> Andrew
From: Colin King <colin.king@canonical.com> Date: Tue, 3 Oct 2017 11:46:33 +0100 > From: Colin Ian King <colin.king@canonical.com> > > The function mt7530_phy_write is local to the source and does not need to > be in global scope, so make it static. > > Cleans up sparse warnings: > symbol 'mt7530_phy_write' was not declared. Should it be static? > > Signed-off-by: Colin Ian King <colin.king@canonical.com> Applied.
diff --git a/drivers/net/dsa/mt7530.c b/drivers/net/dsa/mt7530.c index faa3b88d2206..034241696ce2 100644 --- a/drivers/net/dsa/mt7530.c +++ b/drivers/net/dsa/mt7530.c @@ -564,7 +564,8 @@ static int mt7530_phy_read(struct dsa_switch *ds, int port, int regnum) return mdiobus_read_nested(priv->bus, port, regnum); } -int mt7530_phy_write(struct dsa_switch *ds, int port, int regnum, u16 val) +static int mt7530_phy_write(struct dsa_switch *ds, int port, int regnum, + u16 val) { struct mt7530_priv *priv = ds->priv;