From mboxrd@z Thu Jan 1 00:00:00 1970 From: lethal@linux-sh.org (Paul Mundt) Date: Tue, 22 Mar 2011 23:33:07 +0900 Subject: [PATCH] Input: tca6416-keypad: Change to module_init() In-Reply-To: <20110322142855.GB2202@sirena.org.uk> References: <20110322142619.1109.89105.sendpatchset@t400s> <20110322142855.GB2202@sirena.org.uk> Message-ID: <20110322143307.GB24004@linux-sh.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tue, Mar 22, 2011 at 02:28:55PM +0000, Mark Brown wrote: > On Tue, Mar 22, 2011 at 11:26:19PM +0900, Magnus Damm wrote: > > > The tca6416 driver makes use of the I2C bus for chatting > > with the actual hardware device. Without this patch both > > the I2C bus driver and the tca6416 driver are initialized > > at the subsys_initcall() level. This may lead to problems > > with the tca6416 driver being initialized before the I2C > > bus driver. > > While this change seems reasonable I'm curious what the problems caused > by out of order registration are? I'm also curious as to why link order isn't a sufficient gaurantee like it is for everyone else?