Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932291AbaAWPFa (ORCPT ); Thu, 23 Jan 2014 10:05:30 -0500 Received: from ext3.cumulusnetworks.com ([198.211.106.187]:56744 "EHLO ext3.cumulusnetworks.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932215AbaAWPF1 (ORCPT ); Thu, 23 Jan 2014 10:05:27 -0500 Date: Thu, 23 Jan 2014 07:05:24 -0800 From: Curt Brune To: Laszlo Papp Cc: Thomas De Schampheleire , gregkh@linuxfoundation.org, Shrijeet Mukherjee , wolfram@the-dreams.de, linux-i2c@vger.kernel.org, LKML Subject: Re: [PATCH 2/2] Add at24 based EEPROMs to the eeprom_dev hardware class Message-ID: <20140123150524.GH29955@cumulusnetworks.com> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu Jan 23 07:44, Laszlo Papp wrote: > On Wed, Jan 22, 2014 at 5:23 PM, Curt Brune wrote: > > During device instantiation have the at24 driver add the new device to > > the eeprom_dev hardware class. The functionality is enabled by > > CONFIG_EEPROM_CLASS. [snip] > > static void __exit at24_exit(void) > > { > > i2c_del_driver(&at24_driver); > > } > > module_exit(at24_exit); > > Couldn't you use module_i2c_driver() instead of this? I'm not sure what you mean. Do you mean should the class registration/unregistration be put in module_i2c_driver()? That would not work as not all i2c devices are eeproms. > > > MODULE_DESCRIPTION("Driver for most I2C EEPROMs"); > > MODULE_AUTHOR("David Brownell and Wolfram Sang"); > > I would personally put your name in here if I were you, otherwise > David and Wolfram might get contacted by some people instead of you > (at least based on this). Hmmm. I don't know. I didn't change the driver very much, just added about 10 lines to a perfectly fine driver. git-blame would point right at me for anything to do with this patch. > > PS.: Fixing the broken i2c mailing list typo, and updating Wolfram's > address from the broken (obsolete?) version. Thanks for fixing that. Cheers, Curt -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/