ether: Spatch Linux MII files
authorBarret Rhoden <brho@cs.berkeley.edu>
Wed, 1 Nov 2017 14:56:38 +0000 (10:56 -0400)
committerBarret Rhoden <brho@cs.berkeley.edu>
Thu, 16 Nov 2017 15:46:55 +0000 (10:46 -0500)
$ for i in scripts/spatch/linux/*.cocci; do echo $i; spatch -sp-file $i
-in-place kern/drivers/net/linux_mii.c; done

Looks like these had no spatch complaints either (unlike r8169).  We'll
see.

Signed-off-by: Barret Rhoden <brho@cs.berkeley.edu>
kern/drivers/net/linux_mii.c
kern/drivers/net/linux_mii.h
kern/drivers/net/linux_uapi_mii.h

index 4461212..a4abced 100644 (file)
@@ -33,7 +33,7 @@
 #include <linux/ethtool.h>
 #include <linux/mii.h>
 
-static u32 mii_get_an(struct mii_if_info *mii, u16 addr)
+static uint32_t mii_get_an(struct mii_if_info *mii, uint16_t addr)
 {
        int advert;
 
@@ -54,9 +54,9 @@ static u32 mii_get_an(struct mii_if_info *mii, u16 addr)
  */
 int mii_ethtool_gset(struct mii_if_info *mii, struct ethtool_cmd *ecmd)
 {
-       struct net_device *dev = mii->dev;
-       u16 bmcr, bmsr, ctrl1000 = 0, stat1000 = 0;
-       u32 nego;
+       struct ether *dev = mii->dev;
+       uint16_t bmcr, bmsr, ctrl1000 = 0, stat1000 = 0;
+       uint32_t nego;
 
        ecmd->supported =
            (SUPPORTED_10baseT_Half | SUPPORTED_10baseT_Full |
@@ -145,9 +145,9 @@ int mii_ethtool_gset(struct mii_if_info *mii, struct ethtool_cmd *ecmd)
 void mii_ethtool_get_link_ksettings(struct mii_if_info *mii,
                                    struct ethtool_link_ksettings *cmd)
 {
-       struct net_device *dev = mii->dev;
-       u16 bmcr, bmsr, ctrl1000 = 0, stat1000 = 0;
-       u32 nego, supported, advertising, lp_advertising;
+       struct ether *dev = mii->dev;
+       uint16_t bmcr, bmsr, ctrl1000 = 0, stat1000 = 0;
+       uint32_t nego, supported, advertising, lp_advertising;
 
        supported = (SUPPORTED_10baseT_Half | SUPPORTED_10baseT_Full |
                     SUPPORTED_100baseT_Half | SUPPORTED_100baseT_Full |
@@ -236,8 +236,8 @@ void mii_ethtool_get_link_ksettings(struct mii_if_info *mii,
  */
 int mii_ethtool_sset(struct mii_if_info *mii, struct ethtool_cmd *ecmd)
 {
-       struct net_device *dev = mii->dev;
-       u32 speed = ethtool_cmd_speed(ecmd);
+       struct ether *dev = mii->dev;
+       uint32_t speed = ethtool_cmd_speed(ecmd);
 
        if (speed != SPEED_10 &&
            speed != SPEED_100 &&
@@ -259,8 +259,8 @@ int mii_ethtool_sset(struct mii_if_info *mii, struct ethtool_cmd *ecmd)
        /* ignore supported, maxtxpkt, maxrxpkt */
 
        if (ecmd->autoneg == AUTONEG_ENABLE) {
-               u32 bmcr, advert, tmp;
-               u32 advert2 = 0, tmp2 = 0;
+               uint32_t bmcr, advert, tmp;
+               uint32_t advert2 = 0, tmp2 = 0;
 
                if ((ecmd->advertising & (ADVERTISED_10baseT_Half |
                                          ADVERTISED_10baseT_Full |
@@ -296,7 +296,7 @@ int mii_ethtool_sset(struct mii_if_info *mii, struct ethtool_cmd *ecmd)
 
                mii->force_media = 0;
        } else {
-               u32 bmcr, tmp;
+               uint32_t bmcr, tmp;
 
                /* turn off auto negotiation, set speed and duplexity */
                bmcr = mii->mdio_read(dev, mii->phy_id, MII_BMCR);
@@ -329,8 +329,8 @@ int mii_ethtool_sset(struct mii_if_info *mii, struct ethtool_cmd *ecmd)
 int mii_ethtool_set_link_ksettings(struct mii_if_info *mii,
                                   const struct ethtool_link_ksettings *cmd)
 {
-       struct net_device *dev = mii->dev;
-       u32 speed = cmd->base.speed;
+       struct ether *dev = mii->dev;
+       uint32_t speed = cmd->base.speed;
 
        if (speed != SPEED_10 &&
            speed != SPEED_100 &&
@@ -351,9 +351,9 @@ int mii_ethtool_set_link_ksettings(struct mii_if_info *mii,
        /* ignore supported, maxtxpkt, maxrxpkt */
 
        if (cmd->base.autoneg == AUTONEG_ENABLE) {
-               u32 bmcr, advert, tmp;
-               u32 advert2 = 0, tmp2 = 0;
-               u32 advertising;
+               uint32_t bmcr, advert, tmp;
+               uint32_t advert2 = 0, tmp2 = 0;
+               uint32_t advertising;
 
                ethtool_convert_link_mode_to_legacy_u32(
                        &advertising, cmd->link_modes.advertising);
@@ -393,7 +393,7 @@ int mii_ethtool_set_link_ksettings(struct mii_if_info *mii,
 
                mii->force_media = 0;
        } else {
-               u32 bmcr, tmp;
+               uint32_t bmcr, tmp;
 
                /* turn off auto negotiation, set speed and duplexity */
                bmcr = mii->mdio_read(dev, mii->phy_id, MII_BMCR);
@@ -606,7 +606,7 @@ int generic_mii_ioctl(struct mii_if_info *mii_if,
                break;
 
        case SIOCSMIIREG: {
-               u16 val = mii_data->val_in;
+               uint16_t val = mii_data->val_in;
 
                if (mii_data->phy_id == mii_if->phy_id) {
                        switch(mii_data->reg_num) {
index e870bfa..6ef6e8e 100644 (file)
@@ -23,9 +23,10 @@ struct mii_if_info {
        unsigned int force_media : 1;   /* is autoneg. disabled? */
        unsigned int supports_gmii : 1; /* are GMII registers supported? */
 
-       struct net_device *dev;
-       int (*mdio_read) (struct net_device *dev, int phy_id, int location);
-       void (*mdio_write) (struct net_device *dev, int phy_id, int location, int val);
+       struct ether *dev;
+       int (*mdio_read) (struct ether *dev, int phy_id, int location);
+       void (*mdio_write) (struct ether *dev, int phy_id, int location,
+                           int val);
 };
 
 extern int mii_link_ok (struct mii_if_info *mii);
@@ -110,9 +111,9 @@ static inline unsigned int mii_duplex (unsigned int duplex_lock,
  * settings to phy autonegotiation advertisements for the
  * MII_ADVERTISE register.
  */
-static inline u32 ethtool_adv_to_mii_adv_t(u32 ethadv)
+static inline uint32_t ethtool_adv_to_mii_adv_t(uint32_t ethadv)
 {
-       u32 result = 0;
+       uint32_t result = 0;
 
        if (ethadv & ADVERTISED_10baseT_Half)
                result |= ADVERTISE_10HALF;
@@ -137,9 +138,9 @@ static inline u32 ethtool_adv_to_mii_adv_t(u32 ethadv)
  * A small helper function that translates MII_ADVERTISE bits
  * to ethtool advertisement settings.
  */
-static inline u32 mii_adv_to_ethtool_adv_t(u32 adv)
+static inline uint32_t mii_adv_to_ethtool_adv_t(uint32_t adv)
 {
-       u32 result = 0;
+       uint32_t result = 0;
 
        if (adv & ADVERTISE_10HALF)
                result |= ADVERTISED_10baseT_Half;
@@ -165,9 +166,9 @@ static inline u32 mii_adv_to_ethtool_adv_t(u32 adv)
  * settings to phy autonegotiation advertisements for the
  * MII_CTRL1000 register when in 1000T mode.
  */
-static inline u32 ethtool_adv_to_mii_ctrl1000_t(u32 ethadv)
+static inline uint32_t ethtool_adv_to_mii_ctrl1000_t(uint32_t ethadv)
 {
-       u32 result = 0;
+       uint32_t result = 0;
 
        if (ethadv & ADVERTISED_1000baseT_Half)
                result |= ADVERTISE_1000HALF;
@@ -185,9 +186,9 @@ static inline u32 ethtool_adv_to_mii_ctrl1000_t(u32 ethadv)
  * bits, when in 1000Base-T mode, to ethtool
  * advertisement settings.
  */
-static inline u32 mii_ctrl1000_to_ethtool_adv_t(u32 adv)
+static inline uint32_t mii_ctrl1000_to_ethtool_adv_t(uint32_t adv)
 {
-       u32 result = 0;
+       uint32_t result = 0;
 
        if (adv & ADVERTISE_1000HALF)
                result |= ADVERTISED_1000baseT_Half;
@@ -205,9 +206,9 @@ static inline u32 mii_ctrl1000_to_ethtool_adv_t(u32 adv)
  * bits, when in 1000Base-T mode, to ethtool
  * LP advertisement settings.
  */
-static inline u32 mii_lpa_to_ethtool_lpa_t(u32 lpa)
+static inline uint32_t mii_lpa_to_ethtool_lpa_t(uint32_t lpa)
 {
-       u32 result = 0;
+       uint32_t result = 0;
 
        if (lpa & LPA_LPACK)
                result |= ADVERTISED_Autoneg;
@@ -223,9 +224,9 @@ static inline u32 mii_lpa_to_ethtool_lpa_t(u32 lpa)
  * bits, when in 1000Base-T mode, to ethtool
  * advertisement settings.
  */
-static inline u32 mii_stat1000_to_ethtool_lpa_t(u32 lpa)
+static inline uint32_t mii_stat1000_to_ethtool_lpa_t(uint32_t lpa)
 {
-       u32 result = 0;
+       uint32_t result = 0;
 
        if (lpa & LPA_1000HALF)
                result |= ADVERTISED_1000baseT_Half;
@@ -243,9 +244,9 @@ static inline u32 mii_stat1000_to_ethtool_lpa_t(u32 lpa)
  * settings to phy autonegotiation advertisements for the
  * MII_CTRL1000 register when in 1000Base-X mode.
  */
-static inline u32 ethtool_adv_to_mii_adv_x(u32 ethadv)
+static inline uint32_t ethtool_adv_to_mii_adv_x(uint32_t ethadv)
 {
-       u32 result = 0;
+       uint32_t result = 0;
 
        if (ethadv & ADVERTISED_1000baseT_Half)
                result |= ADVERTISE_1000XHALF;
@@ -267,9 +268,9 @@ static inline u32 ethtool_adv_to_mii_adv_x(u32 ethadv)
  * bits, when in 1000Base-X mode, to ethtool
  * advertisement settings.
  */
-static inline u32 mii_adv_to_ethtool_adv_x(u32 adv)
+static inline uint32_t mii_adv_to_ethtool_adv_x(uint32_t adv)
 {
-       u32 result = 0;
+       uint32_t result = 0;
 
        if (adv & ADVERTISE_1000XHALF)
                result |= ADVERTISED_1000baseT_Half;
@@ -291,9 +292,9 @@ static inline u32 mii_adv_to_ethtool_adv_x(u32 adv)
  * bits, when in 1000Base-X mode, to ethtool
  * LP advertisement settings.
  */
-static inline u32 mii_lpa_to_ethtool_lpa_x(u32 lpa)
+static inline uint32_t mii_lpa_to_ethtool_lpa_x(uint32_t lpa)
 {
-       u32 result = 0;
+       uint32_t result = 0;
 
        if (lpa & LPA_LPACK)
                result |= ADVERTISED_Autoneg;
@@ -305,9 +306,9 @@ static inline u32 mii_lpa_to_ethtool_lpa_x(u32 lpa)
  * mii_advertise_flowctrl - get flow control advertisement flags
  * @cap: Flow control capabilities (FLOW_CTRL_RX, FLOW_CTRL_TX or both)
  */
-static inline u16 mii_advertise_flowctrl(int cap)
+static inline uint16_t mii_advertise_flowctrl(int cap)
 {
-       u16 adv = 0;
+       uint16_t adv = 0;
 
        if (cap & FLOW_CTRL_RX)
                adv = ADVERTISE_PAUSE_CAP | ADVERTISE_PAUSE_ASYM;
@@ -324,9 +325,10 @@ static inline u16 mii_advertise_flowctrl(int cap)
  *
  * Resolve full duplex flow control as per IEEE 802.3-2005 table 28B-3
  */
-static inline u8 mii_resolve_flowctrl_fdx(u16 lcladv, u16 rmtadv)
+static inline uint8_t mii_resolve_flowctrl_fdx(uint16_t lcladv,
+                                              uint16_t rmtadv)
 {
-       u8 cap = 0;
+       uint8_t cap = 0;
 
        if (lcladv & rmtadv & ADVERTISE_PAUSE_CAP) {
                cap = FLOW_CTRL_TX | FLOW_CTRL_RX;
index 15d8510..beca9fe 100644 (file)
 
 /* This structure is used in all SIOCxMIIxxx ioctl calls */
 struct mii_ioctl_data {
-       __u16           phy_id;
-       __u16           reg_num;
-       __u16           val_in;
-       __u16           val_out;
+       uint16_t                phy_id;
+       uint16_t                reg_num;
+       uint16_t                val_in;
+       uint16_t                val_out;
 };
 
 #endif /* _UAPI__LINUX_MII_H__ */