Commit 9540cdda authored by Luo Jie's avatar Luo Jie Committed by David S. Miller
Browse files

net: phy: at803x: use GENMASK() for speed status 



Use GENMASK() for the current speed value.

Signed-off-by: default avatarLuo Jie <luoj@codeaurora.org>
Reviewed-by: default avatarAndrew Lunn <andrew@lunn.ch>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 7beecaf7

drivers/net/phy/at803x.c

+5 −5
Original line number Diff line number Diff line
@@ -33,10 +33,10 @@ 
#define AT803X_SFC_DISABLE_JABBER		BIT(0)



#define AT803X_SPECIFIC_STATUS			0x11

#define AT803X_SS_SPEED_MASK			(3 << 14)

#define AT803X_SS_SPEED_1000			(2 << 14)

#define AT803X_SS_SPEED_100			(1 << 14)

#define AT803X_SS_SPEED_10			(0 << 14)

#define AT803X_SS_SPEED_MASK			GENMASK(15, 14)

#define AT803X_SS_SPEED_1000			2

#define AT803X_SS_SPEED_100			1

#define AT803X_SS_SPEED_10			0

#define AT803X_SS_DUPLEX			BIT(13)

#define AT803X_SS_SPEED_DUPLEX_RESOLVED		BIT(11)

#define AT803X_SS_MDIX				BIT(6)
@@ -994,7 +994,7 @@ static int at803x_read_status(struct phy_device *phydev) 
		if (sfc < 0)

			return sfc;



		switch (ss & AT803X_SS_SPEED_MASK) {

		switch (FIELD_GET(AT803X_SS_SPEED_MASK, ss)) {

		case AT803X_SS_SPEED_10:

			phydev->speed = SPEED_10;

			break;