From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757693Ab0JHOUV (ORCPT ); Fri, 8 Oct 2010 10:20:21 -0400 Received: from ppsw-50.csi.cam.ac.uk ([131.111.8.150]:57340 "EHLO ppsw-50.csi.cam.ac.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753966Ab0JHOUU (ORCPT ); Fri, 8 Oct 2010 10:20:20 -0400 X-Cam-AntiVirus: no malware found X-Cam-SpamDetails: not scanned X-Cam-ScannerInfo: http://www.cam.ac.uk/cs/email/scanner/ Message-ID: <4CAF29F4.9010709@cam.ac.uk> Date: Fri, 08 Oct 2010 15:25:56 +0100 From: Jonathan Cameron User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.9) Gecko/20100928 Lightning/1.0b3pre Thunderbird/3.1.4 MIME-Version: 1.0 To: Samu Onkalo CC: linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org, alan@lxorguk.ukuu.org.uk Subject: Re: [PATCHv2 2/5] misc: update bhsfh driver to Kconfig and Makefile References: <1286545324-18506-1-git-send-email-samu.p.onkalo@nokia.com> <1286545324-18506-3-git-send-email-samu.p.onkalo@nokia.com> In-Reply-To: <1286545324-18506-3-git-send-email-samu.p.onkalo@nokia.com> X-Enigmail-Version: 1.1.2 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 10/08/10 14:42, Samu Onkalo wrote: > Compilation support for bhsfh driver > > Signed-off-by: Samu Onkalo > --- > drivers/misc/Kconfig | 10 ++++++++++ > drivers/misc/Makefile | 1 + > 2 files changed, 11 insertions(+), 0 deletions(-) > > diff --git a/drivers/misc/Kconfig b/drivers/misc/Kconfig > index b743312..f2e8065 100644 > --- a/drivers/misc/Kconfig > +++ b/drivers/misc/Kconfig > @@ -314,6 +314,16 @@ config SENSORS_BH1780 > This driver can also be built as a module. If so, the module > will be called bh1780gli. > > +config SENSORS_BHSFH > + tristate "BH1770GLC / SFH7770 combined ALS / Proximity sensor"; > + depends on I2C > + ---help--- > + Say Y here if you want to build a driver for BH1770GLC / SFH7770 > + combined ambient light and proximity sensor chip > + > + To compile this driver as a module, choose M here: the > + module will be called bhsfh. If unsure, say N here. > + Really not keen on the naming. The above comment made me think this was a single chip integrating both of the parts. Reading the code makes it clear that they just share an interface. So please name it after one of them. Lots of reasons not to invent combined names like this. * non obvious thing to grep for. * 3rd part comes along with a different name. Do you change the driver name? So best bet is always just to pick a supported part and name the driver after that. The device table stuff and clear descriptions in kconfig will make it obvious what the part supports. > config HMC6352 > tristate "Honeywell HMC6352 compass" > depends on I2C > diff --git a/drivers/misc/Makefile b/drivers/misc/Makefile > index 42eab95..fd5a4b7 100644 > --- a/drivers/misc/Makefile > +++ b/drivers/misc/Makefile > @@ -16,6 +16,7 @@ obj-$(CONFIG_TIFM_CORE) += tifm_core.o > obj-$(CONFIG_TIFM_7XX1) += tifm_7xx1.o > obj-$(CONFIG_PHANTOM) += phantom.o > obj-$(CONFIG_SENSORS_BH1780) += bh1780gli.o > +obj-$(CONFIG_SENSORS_BHSFH) += bhsfh.o > obj-$(CONFIG_SGI_IOC4) += ioc4.o > obj-$(CONFIG_ENCLOSURE_SERVICES) += enclosure.o > obj-$(CONFIG_KGDB_TESTS) += kgdbts.o