From mboxrd@z Thu Jan 1 00:00:00 1970 From: wsa@the-dreams.de (Wolfram Sang) Date: Mon, 10 Jun 2013 15:45:36 +0200 Subject: [PATCH] i2c: sirf: move driver init from module_init to subsys_initcall In-Reply-To: References: <1368672607-5536-1-git-send-email-Baohua.Song@csr.com> <20130516093846.GA2927@katana> <20130525201032.GD32043@sirena.org.uk> <20130527121658.GA3660@sirena.org.uk> Message-ID: <20130610134535.GB2987@katana> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Mon, May 27, 2013 at 11:36:14PM +0800, Barry Song wrote: > 2013/5/27 Mark Brown : > > On Mon, May 27, 2013 at 09:54:56AM +0800, Barry Song wrote: > > > >> Mark, the case is not that deferred probing is slow or not. deferred > >> probing is pretty good. > >> the case is that we want to i2c and media connected with i2c probed > >> earlier than other devices. > >> in auto infotainment devices, we actually do some hacking in kernel > >> that makes rear view work earlier than other device driver > >> initialization with a kernel thread which take care of backing-car > >> policy not only mechanism. that means, we make camera work to see > >> backview image even earlier than other drivers' initialization. > >> we don't want media deferred to wait for i2c. we want make some early > >> jobs ready earlier. > > > > So this change makes no practical difference in mainline and exists to > > support out of tree hacks for performance? It doesn't seem like that > > big a patch to carry along with the out of tree stuff... > > yes. but i don't think we are easy to make those out-of-mainline hacks > be in mainline. but this patch is both ok to mainline and local tree. > making local tree and mainline same as many as possible decreases our > maintaince efforts totally. I understand that, yet I agree with Mark. The mainline idea is to use deferred probing to make sure all components come up correctly. In fact, I wish for patches removing subsys_initcall with deferred probing. -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: