Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752142Ab3J2XWv (ORCPT ); Tue, 29 Oct 2013 19:22:51 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:39829 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751300Ab3J2XWu (ORCPT ); Tue, 29 Oct 2013 19:22:50 -0400 Date: Tue, 29 Oct 2013 16:22:49 -0700 From: Greg Kroah-Hartman To: Johan Hovold Cc: Arnd Bergmann , linux-kernel@vger.kernel.org, Nicolas Ferre , linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH 2/2] misc: atmel_pwm: set initcall level to subsys Message-ID: <20131029232249.GA20476@kroah.com> References: <1382459560-26596-1-git-send-email-jhovold@gmail.com> <1382459560-26596-3-git-send-email-jhovold@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1382459560-26596-3-git-send-email-jhovold@gmail.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1556 Lines: 41 On Tue, Oct 22, 2013 at 06:32:40PM +0200, Johan Hovold wrote: > Even with the atmel_pwm driver and the atmel-pwm-bl backlight driver > supporting deferred probing, we still want to make sure that any > pwm-device is available when the backlight devices are probed to avoid > any unnecessary delays before enabling the backlight. > > Signed-off-by: Johan Hovold > --- > drivers/misc/atmel_pwm.c | 12 +++++++++++- > 1 file changed, 11 insertions(+), 1 deletion(-) > > diff --git a/drivers/misc/atmel_pwm.c b/drivers/misc/atmel_pwm.c > index a6dc56e..0d0f599 100644 > --- a/drivers/misc/atmel_pwm.c > +++ b/drivers/misc/atmel_pwm.c > @@ -395,7 +395,17 @@ static struct platform_driver atmel_pwm_driver = { > */ > }; > > -module_platform_driver_probe(atmel_pwm_driver, pwm_probe); > +static int __init pwm_init(void) > +{ > + return platform_driver_probe(&atmel_pwm_driver, pwm_probe); > +} > +subsys_initcall(pwm_init); I really hate this type of patch, as it's papering over the real problem. What happens when someone else moves their driver to this level? Then you are back to the original problem. This is what deferred probing was supposed to fix. If it doesn't, then something else needs to be done, or fix the deferred probing mess... Sorry, I can't take this. greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/