From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jonathan Cameron Subject: Re: [PATCH 2/5] I2C: Add helper macro for i2c_driver boilerplate Date: Thu, 17 Nov 2011 06:47:39 +0000 Message-ID: <4EC4AE0B.1040500@kernel.org> References: <1321434819-23678-1-git-send-email-lars@metafoo.de> <1321434819-23678-3-git-send-email-lars@metafoo.de> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: Lars-Peter Clausen , Greg Kroah-Hartman , Jean Delvare , Michael Hennerich , linux-iio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org To: Grant Likely Return-path: In-Reply-To: Sender: linux-iio-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-spi.vger.kernel.org On 11/16/2011 05:12 PM, Grant Likely wrote: > On Wed, Nov 16, 2011 at 2:13 AM, Lars-Peter Clausen wrote: >> This patch introduces the module_i2c_driver macro which is a convenience macro >> for I2C driver modules similar to module_platform_driver. It is intended to be >> used by drivers which init/exit section does nothing but register/unregister >> the I2C driver. By using this macro it is possible to eliminate a few lines of >> boilerplate code per I2C driver. >> >> Signed-off-by: Lars-Peter Clausen > > Acked-by: Grant Likely Acked-by: Jonathan Cameron > >> --- >> include/linux/i2c.h | 13 +++++++++++++ >> 1 files changed, 13 insertions(+), 0 deletions(-) >> >> diff --git a/include/linux/i2c.h b/include/linux/i2c.h >> index a81bf6d..7e92854 100644 >> --- a/include/linux/i2c.h >> +++ b/include/linux/i2c.h >> @@ -485,6 +485,19 @@ static inline int i2c_adapter_id(struct i2c_adapter *adap) >> { >> return adap->nr; >> } >> + >> +/** >> + * module_i2c_driver() - Helper macro for registering a I2C driver >> + * @__i2c_driver: i2c_driver struct >> + * >> + * Helper macro for I2C drivers which do not do anything special in module >> + * init/exit. This eliminates a lot of boilerplate. Each module may only >> + * use this macro once, and calling it replaces module_init() and module_exit() >> + */ >> +#define module_i2c_driver(__i2c_driver) \ >> + module_driver(__i2c_driver, i2c_add_driver, \ >> + i2c_del_driver) >> + >> #endif /* I2C */ >> #endif /* __KERNEL__ */ >> >> -- >> 1.7.7.1 >> >> >> > > >