From: Felipe Balbi <felipe.balbi@xxxxxxxxx> it's enough to have it as: return platform_driver_register(); in case of failure, platform code should print any necessary info. Signed-off-by: Felipe Balbi <felipe.balbi@xxxxxxxxx> --- drivers/cbus/retu-headset.c | 10 +--------- 1 files changed, 1 insertions(+), 9 deletions(-) diff --git a/drivers/cbus/retu-headset.c b/drivers/cbus/retu-headset.c index e798bc2..1def743 100644 --- a/drivers/cbus/retu-headset.c +++ b/drivers/cbus/retu-headset.c @@ -331,15 +331,7 @@ static struct platform_driver retu_headset_driver = { static int __init retu_headset_init(void) { - int r; - - printk(KERN_INFO "Retu/Vilma headset driver initializing\n"); - - r = platform_driver_register(&retu_headset_driver); - if (r < 0) - return r; - - return 0; + return platform_driver_register(&retu_headset_driver); } static void __exit retu_headset_exit(void) -- 1.7.0.rc0.33.g7c3932 -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html