net: hns3: modify an incorrect type in struct hclge_cfg_gro_status_cmd
authorHuazhong Tan <tanhuazhong@huawei.com>
Thu, 28 May 2020 13:48:12 +0000 (21:48 +0800)
committerDavid S. Miller <davem@davemloft.net>
Thu, 28 May 2020 23:39:03 +0000 (16:39 -0700)
Modify field .gro_en in struct hclge_cfg_gro_status_cmd to u8
according to the UM, otherwise, it will overwrite the reserved
byte which may be used for other purpose.

Signed-off-by: Huazhong Tan <tanhuazhong@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_cmd.h
drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c

index e3bab8f..463f291 100644 (file)
@@ -884,8 +884,8 @@ struct hclge_cfg_tso_status_cmd {
 
 #define HCLGE_GRO_EN_B         0
 struct hclge_cfg_gro_status_cmd {
-       __le16 gro_en;
-       u8 rsv[22];
+       u8 gro_en;
+       u8 rsv[23];
 };
 
 #define HCLGE_TSO_MSS_MIN      256
index e9b0e1c..1e4f285 100644 (file)
@@ -1456,7 +1456,7 @@ static int hclge_config_gro(struct hclge_dev *hdev, bool en)
        hclge_cmd_setup_basic_desc(&desc, HCLGE_OPC_GRO_GENERIC_CONFIG, false);
        req = (struct hclge_cfg_gro_status_cmd *)desc.data;
 
-       req->gro_en = cpu_to_le16(en ? 1 : 0);
+       req->gro_en = en ? 1 : 0;
 
        ret = hclge_cmd_send(&hdev->hw, &desc, 1);
        if (ret)