From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jonathan Cameron Subject: Re: [RCF 1/3] hwmon: Add ads1118 driver Date: Sat, 23 Jul 2016 08:02:01 +0200 Message-ID: References: <0294fbcb211e8f14f4e68fed4d9375cf23c6a70e.1468626668.git.stillcompiling@gmail.com> <578990A1.6020300@roeck-us.net> <5792301A.6030206@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <5792301A.6030206@gmail.com> Sender: linux-kernel-owner@vger.kernel.org To: Joshua Clayton , Guenter Roeck , Rob Herring , Mark Rutland , Shawn Guo , Sascha Hauer , Fabio Estevam , Russell King , Jean Delvare , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-hwmon@vger.kernel.org, "linux-iio@vger.kernel.org" List-Id: devicetree@vger.kernel.org On 22/07/16 16:39, Joshua Clayton wrote: > Greetings Guenter, > > Thank you for reviewing my submission. > > On 07/15/2016 06:40 PM, Guenter Roeck wrote: >> On 07/15/2016 05:18 PM, Joshua Clayton wrote: >>> Add new driver for Texas Instruments ADS1118 and and ADS1018. >>> This driver works with ADS1018, because of code borrowed >>> from asd1015, which is similar, but I can only test ADS1118 >>> >> >> Browsing through the datasheet, I think this should probably be implemented >> as iio driver (and iio already has a driver for ads1015). >> >> Jonathan, what do you think ? >> >> Thanks, >> Guenter >> > No response from Jonathan as yet, but I am willing to rework the driver when I > have some time. It might be weeks before I can start, though. Sorry, made week and I thought I'd have time to get reviews done in evenings so didn't look at anything last weekend. Catching up on train home this morning. > > I am not very familiar with the iio subsystem, (actually had never heard of it before > receiving your response). Learning it might be beneficial, as it looks like a good fit > for an in house driver, which I'd like to eventually upstream. Cool > > If an hwmon driver is adequate, I have new patches that address kbuild and endianness > issues in the first version. I'd flip it over to IIO. If you keep to just the simple bits it'll end up pretty simple to do. > > > ~Joshua Clayton > -- > 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 >