diff --git a/drivers/net/ethernet/huawei/hinic/hinic_cfg.c b/drivers/net/ethernet/huawei/hinic/hinic_cfg.c
index b2599b5c20cd4497cabf8b899142ed79f940fb36..85e7be285132087bb3f5abcfdc15b3a805c589a1 100644
--- a/drivers/net/ethernet/huawei/hinic/hinic_cfg.c
+++ b/drivers/net/ethernet/huawei/hinic/hinic_cfg.c
@@ -104,6 +104,7 @@ int hinic_sync_time(void *hwdev, u64 time)
 		sdk_err(((struct hinic_hwdev *)hwdev)->dev_hdl,
 			"Failed to sync time to mgmt, err: %d, status: 0x%x, out size: 0x%x\n",
 			err, time_info.status, out_size);
+		return -EFAULT;
 	}
 
 	return err;
diff --git a/drivers/net/ethernet/huawei/hinic/hinic_nictool.c b/drivers/net/ethernet/huawei/hinic/hinic_nictool.c
index f281c19a275b20222012a9f494aea28f772c4f45..377ac678b4e8659ceab42a86ed43bf55428ec908 100644
--- a/drivers/net/ethernet/huawei/hinic/hinic_nictool.c
+++ b/drivers/net/ethernet/huawei/hinic/hinic_nictool.c
@@ -1545,10 +1545,8 @@ static int send_to_nic_driver(struct hinic_nic_dev *nic_dev,
 	}
 	mutex_unlock(&nic_dev->nic_mutex);
 
-	if (index == num_cmds) {
-		pr_err("Can't find callback for %d\n", cmd_type);
+	if (index == num_cmds)
 		return -EINVAL;
-	}
 	return err;
 }
 
@@ -1571,7 +1569,8 @@ static int send_to_hw_driver(void *hwdev, struct msg_module *nt_msg,
 			break;
 		}
 	}
-
+	if (index == num_cmds)
+		return -EINVAL;
 	return err;
 }