From: Jingjing Wu Date: Sun, 6 Sep 2015 07:12:00 +0000 (+0800) Subject: i40e/base: remove useless assignments X-Git-Tag: spdx-start~8435 X-Git-Url: http://git.droids-corp.org/?a=commitdiff_plain;h=8af9e07ff73336ca3860cfeaaeac381bcdd15c93;p=dpdk.git i40e/base: remove useless assignments The assignments of addr_high and addr_low were not only wrong because they were assigning virtual addresses to a hardware accessed variable, but they were shortly thereafter overwritten by the send_asq_command call with a non-NULL third argument (buffer) with correctly formatted data. Signed-off-by: Jingjing Wu Acked-by: Helin Zhang Tested-by: Huilong Xu --- diff --git a/drivers/net/i40e/base/i40e_common.c b/drivers/net/i40e/base/i40e_common.c index 1dac22f7b9..754f77c5ae 100644 --- a/drivers/net/i40e/base/i40e_common.c +++ b/drivers/net/i40e/base/i40e_common.c @@ -5116,8 +5116,6 @@ enum i40e_status_code i40e_aq_alternate_write_indirect(struct i40e_hw *hw, cmd_resp->address = CPU_TO_LE32(addr); cmd_resp->length = CPU_TO_LE32(dw_count); - cmd_resp->addr_high = CPU_TO_LE32(I40E_HI_WORD((u64)buffer)); - cmd_resp->addr_low = CPU_TO_LE32(I40E_LO_DWORD((u64)buffer)); status = i40e_asq_send_command(hw, &desc, buffer, I40E_LO_DWORD(4*dw_count), NULL); @@ -5199,8 +5197,6 @@ enum i40e_status_code i40e_aq_alternate_read_indirect(struct i40e_hw *hw, cmd_resp->address = CPU_TO_LE32(addr); cmd_resp->length = CPU_TO_LE32(dw_count); - cmd_resp->addr_high = CPU_TO_LE32(I40E_HI_DWORD((u64)buffer)); - cmd_resp->addr_low = CPU_TO_LE32(I40E_LO_DWORD((u64)buffer)); status = i40e_asq_send_command(hw, &desc, buffer, I40E_LO_DWORD(4*dw_count), NULL);