cxgb3: Convert simple_strtoul to kstrtox
authorLABBE Corentin <clabbe.montjoie@gmail.com>
Mon, 7 Dec 2015 13:11:33 +0000 (14:11 +0100)
committerDavid S. Miller <davem@davemloft.net>
Mon, 7 Dec 2015 21:49:12 +0000 (16:49 -0500)
the simple_strtoul function is obsolete. This patch replace it by
kstrtox.

Signed-off-by: LABBE Corentin <clabbe.montjoie@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/chelsio/cxgb3/cxgb3_main.c
drivers/net/ethernet/chelsio/cxgb3/t3_hw.c

index 8f7aa53a4c4bec39040dc88fbf7640a3ea6a2389..60908eab3b3adf805d1b6cef462bd24ec5f64fe6 100644 (file)
@@ -701,15 +701,16 @@ static ssize_t attr_store(struct device *d,
                          ssize_t(*set) (struct net_device *, unsigned int),
                          unsigned int min_val, unsigned int max_val)
 {
-       char *endp;
        ssize_t ret;
        unsigned int val;
 
        if (!capable(CAP_NET_ADMIN))
                return -EPERM;
 
-       val = simple_strtoul(buf, &endp, 0);
-       if (endp == buf || val < min_val || val > max_val)
+       ret = kstrtouint(buf, 0, &val);
+       if (ret)
+               return ret;
+       if (val < min_val || val > max_val)
                return -EINVAL;
 
        rtnl_lock();
@@ -829,14 +830,15 @@ static ssize_t tm_attr_store(struct device *d,
        struct port_info *pi = netdev_priv(to_net_dev(d));
        struct adapter *adap = pi->adapter;
        unsigned int val;
-       char *endp;
        ssize_t ret;
 
        if (!capable(CAP_NET_ADMIN))
                return -EPERM;
 
-       val = simple_strtoul(buf, &endp, 0);
-       if (endp == buf || val > 10000000)
+       ret = kstrtouint(buf, 0, &val);
+       if (ret)
+               return ret;
+       if (val > 10000000)
                return -EINVAL;
 
        rtnl_lock();
index a22768c94200efe915016c85d18da2eaf806b5ff..ee04caa6c4d8522d0cc34ec15ec7575bfbdc4afa 100644 (file)
@@ -709,11 +709,21 @@ static int get_vpd_params(struct adapter *adapter, struct vpd_params *p)
                        return ret;
        }
 
-       p->cclk = simple_strtoul(vpd.cclk_data, NULL, 10);
-       p->mclk = simple_strtoul(vpd.mclk_data, NULL, 10);
-       p->uclk = simple_strtoul(vpd.uclk_data, NULL, 10);
-       p->mdc = simple_strtoul(vpd.mdc_data, NULL, 10);
-       p->mem_timing = simple_strtoul(vpd.mt_data, NULL, 10);
+       ret = kstrtouint(vpd.cclk_data, 10, &p->cclk);
+       if (ret)
+               return ret;
+       ret = kstrtouint(vpd.mclk_data, 10, &p->mclk);
+       if (ret)
+               return ret;
+       ret = kstrtouint(vpd.uclk_data, 10, &p->uclk);
+       if (ret)
+               return ret;
+       ret = kstrtouint(vpd.mdc_data, 10, &p->mdc);
+       if (ret)
+               return ret;
+       ret = kstrtouint(vpd.mt_data, 10, &p->mem_timing);
+       if (ret)
+               return ret;
        memcpy(p->sn, vpd.sn_data, SERNUM_LEN);
 
        /* Old eeproms didn't have port information */
@@ -723,8 +733,12 @@ static int get_vpd_params(struct adapter *adapter, struct vpd_params *p)
        } else {
                p->port_type[0] = hex_to_bin(vpd.port0_data[0]);
                p->port_type[1] = hex_to_bin(vpd.port1_data[0]);
-               p->xauicfg[0] = simple_strtoul(vpd.xaui0cfg_data, NULL, 16);
-               p->xauicfg[1] = simple_strtoul(vpd.xaui1cfg_data, NULL, 16);
+               ret = kstrtou16(vpd.xaui0cfg_data, 16, &p->xauicfg[0]);
+               if (ret)
+                       return ret;
+               ret = kstrtou16(vpd.xaui1cfg_data, 16, &p->xauicfg[1]);
+               if (ret)
+                       return ret;
        }
 
        ret = hex2bin(p->eth_base, vpd.na_data, 6);