[dpdk-dev,1/2] net/ena: remove redundant variable

Message ID 1487075863-3056-1-git-send-email-wang.yong19@zte.com.cn (mailing list archive)
State Accepted, archived
Delegated to: Ferruh Yigit
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

Yong Wang Feb. 14, 2017, 12:37 p.m. UTC
  Signed-off-by: Yong Wang <wang.yong19@zte.com.cn>
---
 drivers/net/ena/base/ena_com.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
  

Comments

Jan Medala March 14, 2017, 5:59 p.m. UTC | #1
Acked-by: Jan Medala <jan@semihalf.com>

  Jan

2017-02-14 13:37 GMT+01:00 Yong Wang <wang.yong19@zte.com.cn>:

> Signed-off-by: Yong Wang <wang.yong19@zte.com.cn>
> ---
>  drivers/net/ena/base/ena_com.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/net/ena/base/ena_com.c b/drivers/net/ena/base/ena_
> com.c
> index bd6f3c6..39356d2 100644
> --- a/drivers/net/ena/base/ena_com.c
> +++ b/drivers/net/ena/base/ena_com.c
> @@ -2242,7 +2242,6 @@ int ena_com_set_hash_ctrl(struct ena_com_dev
> *ena_dev)
>  {
>         struct ena_com_admin_queue *admin_queue = &ena_dev->admin_queue;
>         struct ena_rss *rss = &ena_dev->rss;
> -       struct ena_admin_feature_rss_hash_control *hash_ctrl =
> rss->hash_ctrl;
>         struct ena_admin_set_feat_cmd cmd;
>         struct ena_admin_set_feat_resp resp;
>         int ret;
> @@ -2269,7 +2268,8 @@ int ena_com_set_hash_ctrl(struct ena_com_dev
> *ena_dev)
>                 ena_trc_err("memory address set failed\n");
>                 return ret;
>         }
> -       cmd.control_buffer.length = sizeof(*hash_ctrl);
> +       cmd.control_buffer.length =
> +               sizeof(struct ena_admin_feature_rss_hash_control);
>
>         ret = ena_com_execute_admin_command(admin_queue,
>                                             (struct ena_admin_aq_entry
> *)&cmd,
> --
> 1.8.3.1
>
>
>
  

Patch

diff --git a/drivers/net/ena/base/ena_com.c b/drivers/net/ena/base/ena_com.c
index bd6f3c6..39356d2 100644
--- a/drivers/net/ena/base/ena_com.c
+++ b/drivers/net/ena/base/ena_com.c
@@ -2242,7 +2242,6 @@  int ena_com_set_hash_ctrl(struct ena_com_dev *ena_dev)
 {
 	struct ena_com_admin_queue *admin_queue = &ena_dev->admin_queue;
 	struct ena_rss *rss = &ena_dev->rss;
-	struct ena_admin_feature_rss_hash_control *hash_ctrl = rss->hash_ctrl;
 	struct ena_admin_set_feat_cmd cmd;
 	struct ena_admin_set_feat_resp resp;
 	int ret;
@@ -2269,7 +2268,8 @@  int ena_com_set_hash_ctrl(struct ena_com_dev *ena_dev)
 		ena_trc_err("memory address set failed\n");
 		return ret;
 	}
-	cmd.control_buffer.length = sizeof(*hash_ctrl);
+	cmd.control_buffer.length =
+		sizeof(struct ena_admin_feature_rss_hash_control);
 
 	ret = ena_com_execute_admin_command(admin_queue,
 					    (struct ena_admin_aq_entry *)&cmd,