summaryrefslogtreecommitdiffstats
path: root/drivers/misc/ocxl
diff options
context:
space:
mode:
authorMinghao Chi <chi.minghao@zte.com.cn>2021-12-15 07:04:38 +0100
committerMichael Ellerman <mpe@ellerman.id.au>2021-12-20 02:15:33 +0100
commit30e120e6a9d247cec7effd55fd6783d5c619ed4c (patch)
tree56cd5517760e217930701a27d48e8d1537b8e54d /drivers/misc/ocxl
parentpowerpc/64s/radix: Fix huge vmap false positive (diff)
downloadlinux-30e120e6a9d247cec7effd55fd6783d5c619ed4c.tar.xz
linux-30e120e6a9d247cec7effd55fd6783d5c619ed4c.zip
ocxl: remove redundant rc variable
Return value from ocxl_context_attach() directly instead of taking this in another redundant variable. Reported-by: Zeal Robot <zealci@zte.com.cn> Signed-off-by: Minghao Chi <chi.minghao@zte.com.cn> Acked-by: Andrew Donnellan <ajd@linux.ibm.com> Signed-off-by: Michael Ellerman <mpe@ellerman.id.au> Link: https://lore.kernel.org/r/20211215060438.441918-1-chi.minghao@zte.com.cn
Diffstat (limited to 'drivers/misc/ocxl')
-rw-r--r--drivers/misc/ocxl/file.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/misc/ocxl/file.c b/drivers/misc/ocxl/file.c
index e70525eedaae..d881f5e40ad9 100644
--- a/drivers/misc/ocxl/file.c
+++ b/drivers/misc/ocxl/file.c
@@ -74,7 +74,6 @@ static long afu_ioctl_attach(struct ocxl_context *ctx,
{
struct ocxl_ioctl_attach arg;
u64 amr = 0;
- int rc;
pr_debug("%s for context %d\n", __func__, ctx->pasid);
@@ -86,8 +85,7 @@ static long afu_ioctl_attach(struct ocxl_context *ctx,
return -EINVAL;
amr = arg.amr & mfspr(SPRN_UAMOR);
- rc = ocxl_context_attach(ctx, amr, current->mm);
- return rc;
+ return ocxl_context_attach(ctx, amr, current->mm);
}
static long afu_ioctl_get_metadata(struct ocxl_context *ctx,