net: hns: change code style from a = a + x to a += x
authorDaode Huang <huangdaode@hisilicon.com>
Fri, 1 Jul 2016 09:34:07 +0000 (17:34 +0800)
committerDavid S. Miller <davem@davemloft.net>
Fri, 1 Jul 2016 20:56:52 +0000 (16:56 -0400)
This patch fixes the code style in hns driver. Change it from
"buff = buff + xxx" to "buff += xxx". The reveiw comments is
from andy.

Reviewed-by: Andriy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Daode Huang <huangdaode@hisilicon.com>
Signed-off-by: Yisen Zhuang <Yisen.Zhuang@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/hisilicon/hns/hns_dsaf_main.c

index 67e8e13..b9d01ea 100644 (file)
@@ -2540,45 +2540,45 @@ static char *hns_dsaf_get_node_stats_strings(char *data, int node,
        bool is_ver1 = AE_IS_VER1(dsaf_dev->dsaf_ver);
 
        snprintf(buff, ETH_GSTRING_LEN, "innod%d_pad_drop_pkts", node);
-       buff = buff + ETH_GSTRING_LEN;
+       buff += ETH_GSTRING_LEN;
        snprintf(buff, ETH_GSTRING_LEN, "innod%d_manage_pkts", node);
-       buff = buff + ETH_GSTRING_LEN;
+       buff += ETH_GSTRING_LEN;
        snprintf(buff, ETH_GSTRING_LEN, "innod%d_rx_pkts", node);
-       buff = buff + ETH_GSTRING_LEN;
+       buff += ETH_GSTRING_LEN;
        snprintf(buff, ETH_GSTRING_LEN, "innod%d_rx_pkt_id", node);
-       buff = buff + ETH_GSTRING_LEN;
+       buff += ETH_GSTRING_LEN;
        snprintf(buff, ETH_GSTRING_LEN, "innod%d_rx_pause_frame", node);
-       buff = buff + ETH_GSTRING_LEN;
+       buff += ETH_GSTRING_LEN;
        snprintf(buff, ETH_GSTRING_LEN, "innod%d_release_buf_num", node);
-       buff = buff + ETH_GSTRING_LEN;
+       buff += ETH_GSTRING_LEN;
        snprintf(buff, ETH_GSTRING_LEN, "innod%d_sbm_drop_pkts", node);
-       buff = buff + ETH_GSTRING_LEN;
+       buff += ETH_GSTRING_LEN;
        snprintf(buff, ETH_GSTRING_LEN, "innod%d_crc_false_pkts", node);
-       buff = buff + ETH_GSTRING_LEN;
+       buff += ETH_GSTRING_LEN;
        snprintf(buff, ETH_GSTRING_LEN, "innod%d_bp_drop_pkts", node);
-       buff = buff + ETH_GSTRING_LEN;
+       buff += ETH_GSTRING_LEN;
        snprintf(buff, ETH_GSTRING_LEN, "innod%d_lookup_rslt_drop_pkts", node);
-       buff = buff + ETH_GSTRING_LEN;
+       buff += ETH_GSTRING_LEN;
        snprintf(buff, ETH_GSTRING_LEN, "innod%d_local_rslt_fail_pkts", node);
-       buff = buff + ETH_GSTRING_LEN;
+       buff += ETH_GSTRING_LEN;
        snprintf(buff, ETH_GSTRING_LEN, "innod%d_vlan_drop_pkts", node);
-       buff = buff + ETH_GSTRING_LEN;
+       buff += ETH_GSTRING_LEN;
        snprintf(buff, ETH_GSTRING_LEN, "innod%d_stp_drop_pkts", node);
-       buff = buff + ETH_GSTRING_LEN;
+       buff += ETH_GSTRING_LEN;
        if ((node < DSAF_SERVICE_NW_NUM) && (!is_ver1)) {
                for (i = 0; i < DSAF_PRIO_NR; i++) {
                        snprintf(buff, ETH_GSTRING_LEN,
                                 "inod%d_pfc_prio%d_pkts", node, i);
-                       buff = buff + ETH_GSTRING_LEN;
+                       buff += ETH_GSTRING_LEN;
                }
                for (i = 0; i < DSAF_PRIO_NR; i++) {
                        snprintf(buff, ETH_GSTRING_LEN,
                                 "onod%d_pfc_prio%d_pkts", node, i);
-                       buff = buff + ETH_GSTRING_LEN;
+                       buff += ETH_GSTRING_LEN;
                }
        }
        snprintf(buff, ETH_GSTRING_LEN, "onnod%d_tx_pkts", node);
-       buff = buff + ETH_GSTRING_LEN;
+       buff += ETH_GSTRING_LEN;
 
        return buff;
 }