From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from saturn.retrosnub.co.uk ([178.18.118.26]:45092 "EHLO saturn.retrosnub.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753350Ab3AZKk7 (ORCPT ); Sat, 26 Jan 2013 05:40:59 -0500 Message-ID: <5103B2B8.2050901@kernel.org> Date: Sat, 26 Jan 2013 10:40:56 +0000 From: Jonathan Cameron MIME-Version: 1.0 To: Peter Meerwald CC: linux-iio@vger.kernel.org, jbrenner@taosinc.com, amit.kucheria@verdurent.com, Tony Lindgren Subject: Re: [PATCH 5/5] iio:light:tsl2563 move out of staging References: <1357986924-11471-1-git-send-email-jic23@kernel.org> <1357986924-11471-6-git-send-email-jic23@kernel.org> <50FC6592.6030005@kernel.org> In-Reply-To: <50FC6592.6030005@kernel.org> Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-iio-owner@vger.kernel.org List-Id: linux-iio@vger.kernel.org On 01/20/2013 09:45 PM, Jonathan Cameron wrote: > On 01/20/2013 01:32 PM, Peter Meerwald wrote: >> Hello, >> >>> This driver is simple, uses the latest interfaces and contains few if >>> any controversial elements. All of its interfaces have been in place >>> for a long time now. Hence let's move it out of staging. >> >> two small nitpicks inline >> > Thanks! >> regards, p. > ... >>> obj-$(CONFIG_ADJD_S311) += adjd_s311.o >>> obj-$(CONFIG_SENSORS_LM3533) += lm3533-als.o >>> +obj-$(CONFIG_SENSORS_TSL2563) += tsl2563.o >> >> there is the SENSORS_ prefix which is an inconsistency; there were some >> arguments against it... > Hmm. I was holding off on this purely as it might break things. It's used in > one of the omap board files for starters. > > Tony, what are your views on cleaning this up? It's in > arch/arm/mach-omap2/board-rx51-peripherals.c. > There presumably isn't really a problem in a patch fixing this end > going via IIO and another to change the ifdef via your tree? At somepoint > the support won't be enabled but it won't cause any problems. > > There is still a bit of debate on whether an IIO prefix makes sense > or if such prefixes should be dropped entirely so we end up with > CONFIG_TSL2563... > I've left this whole issue be for now so as to not hold up the rest of the patch series. Anyhow all added to togreg branch of iio.git > >> >>> obj-$(CONFIG_VCNL4000) += vcnl4000.o >>> obj-$(CONFIG_HID_SENSOR_ALS) += hid-sensor-als.o >>> diff --git a/drivers/iio/light/tsl2563.c b/drivers/iio/light/tsl2563.c >>> new file mode 100644 >>> index 0000000..86bcadc >>> --- /dev/null >>> +++ b/drivers/iio/light/tsl2563.c >>> @@ -0,0 +1,887 @@ >>> +/* >>> + * drivers/i2c/chips/tsl2563.c >> >> this should be drivers/iio/light/tsl2563.c >> > Good spot, thanks. > -- > To unsubscribe from this list: send the line "unsubscribe linux-iio" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html >