From: "Antonino A. Daplas" Check the return value of pci_register_driver() Signed-off-by: Antonino Daplas Signed-off-by: Andrew Morton --- drivers/video/matrox/matroxfb_base.c | 12 ++++++++---- 1 files changed, 8 insertions(+), 4 deletions(-) diff -puN drivers/video/matrox/matroxfb_base.c~matroxfb-honor-the-return-value-of-pci_register_driver drivers/video/matrox/matroxfb_base.c --- a/drivers/video/matrox/matroxfb_base.c~matroxfb-honor-the-return-value-of-pci_register_driver +++ a/drivers/video/matrox/matroxfb_base.c @@ -2277,10 +2277,13 @@ static void __init matroxfb_init_params( } } -static void __init matrox_init(void) { +static int __init matrox_init(void) { + int err; + matroxfb_init_params(); - pci_register_driver(&matroxfb_driver); + err = pci_register_driver(&matroxfb_driver); dev = -1; /* accept all new devices... */ + return err; } /* **************************** exit-time only **************************** */ @@ -2437,6 +2440,7 @@ static int __initdata initialized = 0; static int __init matroxfb_init(void) { char *option = NULL; + int err = 0; DBG(__FUNCTION__) @@ -2448,11 +2452,11 @@ static int __init matroxfb_init(void) return -ENXIO; if (!initialized) { initialized = 1; - matrox_init(); + err = matrox_init(); } hotplug = 1; /* never return failure, user can hotplug matrox later... */ - return 0; + return err; } module_init(matroxfb_init); _