Commit 7d2fdaa6 authored by Chris Packham's avatar Chris Packham Committed by Borislav Petkov

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: default avatarChris 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.nzSigned-off-by: default avatarBorislav Petkov <bp@suse.de>
parent 7103de0e
...@@ -853,10 +853,15 @@ static struct platform_driver * const drivers[] = { ...@@ -853,10 +853,15 @@ static struct platform_driver * const drivers[] = {
static int __init mv64x60_edac_init(void) 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 "Marvell MV64x60 EDAC driver " MV64x60_REVISION "\n");
printk(KERN_INFO "\t(C) 2006-2007 MontaVista Software\n"); printk(KERN_INFO "\t(C) 2006-2007 MontaVista Software\n");
/* make sure error reporting method is sane */ /* make sure error reporting method is sane */
switch (edac_op_state) { switch (edac_op_state) {
case EDAC_OPSTATE_POLL: case EDAC_OPSTATE_POLL:
...@@ -867,7 +872,7 @@ static int __init mv64x60_edac_init(void) ...@@ -867,7 +872,7 @@ static int __init mv64x60_edac_init(void)
break; break;
} }
return platform_register_drivers(drivers, ARRAY_SIZE(drivers)); return 0;
} }
module_init(mv64x60_edac_init); module_init(mv64x60_edac_init);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment