summaryrefslogtreecommitdiff
path: root/arch/powerpc/platforms/powernv
diff options
context:
space:
mode:
authorRussell Currey <ruscur@russell.cc>2023-02-10 11:03:40 +0300
committerMichael Ellerman <mpe@ellerman.id.au>2023-02-12 14:12:36 +0300
commit26149b02021158248b13e323f06372d87f076883 (patch)
treea51bb3bd4339839314d256dba48f78561fc4a5dd /arch/powerpc/platforms/powernv
parent53cea34b0a0a03568e189f8dfe2eb06f938986c8 (diff)
downloadlinux-26149b02021158248b13e323f06372d87f076883.tar.xz
powerpc/secvar: Warn and error if multiple secvar ops are set
The secvar code only supports one consumer at a time. Multiple consumers aren't possible at this point in time, but we'd want it to be obvious if it ever could happen. Signed-off-by: Russell Currey <ruscur@russell.cc> Co-developed-by: Andrew Donnellan <ajd@linux.ibm.com> Signed-off-by: Andrew Donnellan <ajd@linux.ibm.com> Signed-off-by: Michael Ellerman <mpe@ellerman.id.au> Link: https://lore.kernel.org/r/20230210080401.345462-6-ajd@linux.ibm.com
Diffstat (limited to 'arch/powerpc/platforms/powernv')
-rw-r--r--arch/powerpc/platforms/powernv/opal-secvar.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/arch/powerpc/platforms/powernv/opal-secvar.c b/arch/powerpc/platforms/powernv/opal-secvar.c
index ef89861569e0..4c0a3b030fe0 100644
--- a/arch/powerpc/platforms/powernv/opal-secvar.c
+++ b/arch/powerpc/platforms/powernv/opal-secvar.c
@@ -113,9 +113,7 @@ static int opal_secvar_probe(struct platform_device *pdev)
return -ENODEV;
}
- set_secvar_ops(&opal_secvar_ops);
-
- return 0;
+ return set_secvar_ops(&opal_secvar_ops);
}
static const struct of_device_id opal_secvar_match[] = {