From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mika Westerberg Subject: Re: i2c-core.ko renamed to i2ccore.ko Date: Thu, 18 Sep 2014 15:58:57 +0300 Message-ID: <20140918125825.GR10854@lahna.fi.intel.com> References: <1411042678.7338.20.camel@chaos.site> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1411042678.7338.20.camel-H7Kp9ZFCxt/N0uC3ymp8PA@public.gmane.org> Sender: linux-i2c-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Jean Delvare Cc: Lan Tianyu , Wolfram Sang , Linux I2C List-Id: linux-i2c@vger.kernel.org On Thu, Sep 18, 2014 at 02:17:58PM +0200, Jean Delvare wrote: > Hi guys, > > Sorry for jumping in a bit late, but given that I did not get involved > in the discussions that led to this decision, better late than never. > > I almost had an heart attack when noticing that module i2c-core.ko has > suddenly been renamed to i2ccore.ko without an hyphen. > > Renaming kernel modules is almost always wrong, in particular subsystem > core modules. Such renames have an impact at various levels. We just had > to change a configuration file in the openSUSE Factory kernel to adjust > that change, and this is how I noticed it. But there can be several > other issues, including mkinitrd or equivalent tools, as well as > documentation. The extra work and confusion is a high price to pay, and > it can easily be avoided. I understand completely your point and I guess this was rather inadvertent change. > So I am kindly asking that you revert this name change before kernel > v3.17 final. Rename source files if you really have to, git makes it > easy. Or merge the i2c-acpi.c code into i2c-core.c with ifdefs. Or > whatever. Just don't rename i2c-core.ko, please. My vote goes for merging i2c-acpi.c back to i2c-core.c with #ifdefs. Tianyu, Wolfram, comments?