habanalabs: remove unused but set variable 'ctx_asid'
authorWei Yongjun <weiyongjun1@huawei.com>
Wed, 29 Jul 2020 15:59:02 +0000 (23:59 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 29 Jul 2020 16:02:21 +0000 (18:02 +0200)
Gcc report warning as follows:

drivers/misc/habanalabs/common/command_submission.c:373:6: warning:
 variable 'ctx_asid' set but not used [-Wunused-but-set-variable]
  373 |  int ctx_asid, rc;
      |      ^~~~~~~~

This variable is not used in function cs_timedout(), this commit
remove it to fix the warning.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
Link: https://lore.kernel.org/r/20200729155902.33976-1-weiyongjun1@huawei.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/misc/habanalabs/common/command_submission.c

index e096532c0e489b561afd771703389ce8f4286fd2..b9840e368eb576389cfb2747616cbce1a738ae19 100644 (file)
@@ -370,7 +370,7 @@ static void cs_do_release(struct kref *ref)
 static void cs_timedout(struct work_struct *work)
 {
        struct hl_device *hdev;
-       int ctx_asid, rc;
+       int rc;
        struct hl_cs *cs = container_of(work, struct hl_cs,
                                                 work_tdr.work);
        rc = cs_get_unless_zero(cs);
@@ -386,7 +386,6 @@ static void cs_timedout(struct work_struct *work)
        cs->timedout = true;
 
        hdev = cs->ctx->hdev;
-       ctx_asid = cs->ctx->asid;
 
        dev_err(hdev->dev,
                "Command submission %llu has not finished in time!\n",