From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jean Delvare Subject: Re: [PATCH] i2c: convert drivers/i2c/* to use module_platform_driver() Date: Sun, 27 Nov 2011 20:50:28 +0100 Message-ID: <20111127205028.708ea59f@endymion.delvare> References: <1322374332.2226.1.camel@phoenix> <20111127202212.44ebf4c4@endymion.delvare> <20111127194648.GA26533@pengutronix.de> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20111127194648.GA26533-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org> Sender: linux-i2c-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Wolfram Sang Cc: Axel Lin , linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Ben Dooks , Jochen Friedrich , Peter Korsgaard , Guan Xuetao , Manuel Lauss , Barry Song <21cnbao-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>, Linus Walleij , Yong Zhang , Lucas De Marchi , Grant Likely , Samuel Ortiz , linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-i2c@vger.kernel.org On Sun, 27 Nov 2011 20:46:48 +0100, Wolfram Sang wrote: > > > > -module_init(gpiomux_init); > > > -module_exit(gpiomux_exit); > > > +module_platform_driver(gpiomux_driver); > > > > This one would be better as a separate patch, as it touches a different > > subdirectory, which I am responsible for. > > ... which has a patch pending[1], hint ;) > > [1] http://comments.gmane.org/gmane.linux.drivers.i2c/9432 Yes, that's one of the reasons why I think a separate patch would be more appropriate. It might be difficult to avoid patch collisions otherwise. -- Jean Delvare