summaryrefslogtreecommitdiff
path: root/drivers/edac
diff options
context:
space:
mode:
authorChris Packham <chris.packham@alliedtelesis.co.nz>2017-05-30 09:21:40 +1200
committerBorislav Petkov <bp@suse.de>2017-05-30 09:55:51 +0200
commit7d2fdaa694c596671c0d1d8d625966f7eb8cdea8 (patch)
treec1c86137721e9fa04260c97f38153de860e772e7 /drivers/edac
parent7103de0e5872605d34e951fd03c30fe7d87ea703 (diff)
EDAC, mv64x60: Check driver registration success
Check the return status of platform_driver_register() in mv64x60_edac_init(). Only output messages and initialise the edac_op_state if the registration is successful. Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz> Cc: linux-edac <linux-edac@vger.kernel.org> Cc: linuxppc-dev@lists.ozlabs.org Link: http://lkml.kernel.org/r/20170529212142.25572-2-chris.packham@alliedtelesis.co.nz Signed-off-by: Borislav Petkov <bp@suse.de>
Diffstat (limited to 'drivers/edac')
-rw-r--r--drivers/edac/mv64x60_edac.c9
1 files changed, 7 insertions, 2 deletions
diff --git a/drivers/edac/mv64x60_edac.c b/drivers/edac/mv64x60_edac.c
index 6859bba1a133..77bfc3d882a8 100644
--- a/drivers/edac/mv64x60_edac.c
+++ b/drivers/edac/mv64x60_edac.c
@@ -853,10 +853,15 @@ static struct platform_driver * const drivers[] = {
static int __init mv64x60_edac_init(void)
{
- int ret = 0;
+ int ret;
+
+ ret = platform_register_drivers(drivers, ARRAY_SIZE(drivers));
+ if (ret)
+ return ret;
printk(KERN_INFO "Marvell MV64x60 EDAC driver " MV64x60_REVISION "\n");
printk(KERN_INFO "\t(C) 2006-2007 MontaVista Software\n");
+
/* make sure error reporting method is sane */
switch (edac_op_state) {
case EDAC_OPSTATE_POLL:
@@ -867,7 +872,7 @@ static int __init mv64x60_edac_init(void)
break;
}
- return platform_register_drivers(drivers, ARRAY_SIZE(drivers));
+ return 0;
}
module_init(mv64x60_edac_init);