From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jean Delvare Subject: Re: [PATCH v2] i2c: convert drivers/i2c/* to use module_platform_driver() Date: Sun, 8 Jan 2012 11:36:56 +0100 Message-ID: <20120108113656.70c6fcd0@endymion.delvare> References: <1322439206.2709.2.camel@phoenix> <20111128092721.15ac7b7d@endymion.delvare> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20111128092721.15ac7b7d-R0o5gVi9kd7kN2dkZ6Wm7A@public.gmane.org> Sender: linux-i2c-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Axel Lin , Ben Dooks Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Jochen Friedrich , Peter Korsgaard , Wolfram Sang , 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 Mon, 28 Nov 2011 09:27:21 +0100, Jean Delvare wrote: > On Mon, 28 Nov 2011 08:13:26 +0800, Axel Lin wrote: > > This patch converts the drivers in drivers/i2c/* to use the > > Now drivers/i2c/busses/* even. > > > module_platform_driver() macro which makes the code smaller and a bit > > simpler. > > > > Cc: Jean Delvare > > Cc: Ben Dooks > > Cc: Jochen Friedrich > > Cc: Peter Korsgaard > > Cc: Wolfram Sang > > Cc: Guan Xuetao > > Cc: Manuel Lauss > > Cc: Barry Song <21cnbao-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> > > Cc: Linus Walleij > > Cc: Yong Zhang > > Cc: Lucas De Marchi > > Cc: Grant Likely > > Cc: Samuel Ortiz > > Acked-by: Peter Korsgaard > > Acked-by: Wolfram Sang > > Acked-by: Jochen Friedrich > > Signed-off-by: Axel Lin > > --- > > drivers/i2c/busses/i2c-at91.c | 17 ++--------------- > > drivers/i2c/busses/i2c-au1550.c | 13 +------------ > > drivers/i2c/busses/i2c-cpm.c | 13 +------------ > > drivers/i2c/busses/i2c-highlander.c | 13 +------------ > > drivers/i2c/busses/i2c-ibm_iic.c | 13 +------------ > > drivers/i2c/busses/i2c-iop3xx.c | 16 +--------------- > > drivers/i2c/busses/i2c-isch.c | 13 +------------ > > drivers/i2c/busses/i2c-ixp2000.c | 13 +------------ > > drivers/i2c/busses/i2c-mpc.c | 13 +------------ > > drivers/i2c/busses/i2c-mv64xxx.c | 15 +-------------- > > drivers/i2c/busses/i2c-ocores.c | 17 ++--------------- > > drivers/i2c/busses/i2c-octeon.c | 16 +--------------- > > drivers/i2c/busses/i2c-pca-platform.c | 14 +------------- > > drivers/i2c/busses/i2c-pmcmsp.c | 17 ++--------------- > > drivers/i2c/busses/i2c-powermac.c | 19 ++----------------- > > drivers/i2c/busses/i2c-puv3.c | 16 ++-------------- > > drivers/i2c/busses/i2c-sh7760.c | 13 +------------ > > drivers/i2c/busses/i2c-simtec.c | 18 ++---------------- > > drivers/i2c/busses/i2c-xiic.c | 18 ++---------------- > > 19 files changed, 26 insertions(+), 261 deletions(-) > > (...) > > Reviewed-by: Jean Delvare > > As said before, I'll let Ben pick the patch in his tree to minimize the > risks of merge conflicts. As Ben did not pick this patch after over a month, I'll include it in my own tree. As I plan to send it to Linus quickly, hopefully it won't have the time to cause merge conflicts. -- Jean Delvare