powerpc: Drop useless warning in eeh_init()
authorGreg Kurz <gkurz@linux.vnet.ibm.com>
Tue, 25 Nov 2014 16:10:06 +0000 (17:10 +0100)
committerBenjamin Herrenschmidt <benh@kernel.crashing.org>
Tue, 2 Dec 2014 00:03:45 +0000 (11:03 +1100)
This is what we get in dmesg when booting a pseries guest and
the hypervisor doesn't provide EEH support.

[    0.166655] EEH functionality not supported
[    0.166778] eeh_init: Failed to call platform init function (-22)

Since both powernv_eeh_init() and pseries_eeh_init() already complain when
hitting an error, it is not needed to print more (especially such an
uninformative message).

Signed-off-by: Greg Kurz <gkurz@linux.vnet.ibm.com>
Acked-by: Gavin Shan <gwshan@linux.vnet.ibm.com>
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
arch/powerpc/kernel/eeh.c

index 05be77d9ea0ea874ba0c20f9854ed4e393b55d82..e1b6d8e172893c3fba28371eaf0b641b59f3f955 100644 (file)
@@ -933,11 +933,8 @@ int eeh_init(void)
                pr_warn("%s: Platform EEH operation not found\n",
                        __func__);
                return -EEXIST;
-       } else if ((ret = eeh_ops->init())) {
-               pr_warn("%s: Failed to call platform init function (%d)\n",
-                       __func__, ret);
+       } else if ((ret = eeh_ops->init()))
                return ret;
-       }
 
        /* Initialize EEH event */
        ret = eeh_event_init();