Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756639AbbLAPFL (ORCPT ); Tue, 1 Dec 2015 10:05:11 -0500 Received: from bh-25.webhostbox.net ([208.91.199.152]:44133 "EHLO bh-25.webhostbox.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756514AbbLAPFG (ORCPT ); Tue, 1 Dec 2015 10:05:06 -0500 Subject: Re: [PATCH 1/2] iio: ina2xx: add support for TI INA2xx Power Monitors To: Marc Titinger , Jonathan Cameron , knaack.h@gmx.de, lars@metafoo.de, pmeerw@pmeerw.net, jdelvare@suse.com References: <1448450896-24387-1-git-send-email-mtitinger@baylibre.com> <1448450896-24387-2-git-send-email-mtitinger@baylibre.com> <565B287B.6010008@kernel.org> <565B381F.7000208@roeck-us.net> <565D7014.4020806@baylibre.com> Cc: linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org, lm-sensors@lm-sensors.org From: Guenter Roeck Message-ID: <565DB69F.9000406@roeck-us.net> Date: Tue, 1 Dec 2015 07:02:55 -0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-Version: 1.0 In-Reply-To: <565D7014.4020806@baylibre.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit X-Authenticated_sender: linux@roeck-us.net X-OutGoing-Spam-Status: No, score=-1.0 X-AntiAbuse: This header was added to track abuse, please include it with any abuse report X-AntiAbuse: Primary Hostname - bh-25.webhostbox.net X-AntiAbuse: Original Domain - vger.kernel.org X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12] X-AntiAbuse: Sender Address Domain - roeck-us.net X-Get-Message-Sender-Via: bh-25.webhostbox.net: authenticated_id: linux@roeck-us.net X-Source: X-Source-Args: X-Source-Dir: Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2099 Lines: 47 On 12/01/2015 02:01 AM, Marc Titinger wrote: > On 29/11/2015 18:38, Guenter Roeck wrote: >> On 11/29/2015 08:31 AM, Jonathan Cameron wrote: >>> On 25/11/15 11:28, Marc Titinger wrote: >>>> in SOFTWARE buffer mode, a kthread will capture the active scan_elements >>>> into a kfifo, then compute the remaining time until the next capture >>>> tick >>>> and do an active wait (udelay). >>>> >>>> This will produce a stream of up to fours channels plus a 64bits >>>> timestamps (ns). >>>> >>>> Tested with ina226, on BeagleBoneBlack. >>>> >>>> Datasheet: http://www.ti.com/lit/gpn/ina226 >>>> >>>> Signed-off-by: Marc Titinger >>> A few more bits from me, but basically looking good. >>> >>> We do however, need to make the hwmon guys aware this is going on. >>> Please cc their list on the next version. >>> >>> Last thing we want is for this to turn up as a surprise! >>> >> >> I have seen the original patch, so no surprise here. Just not sure >> if we should remove the hwmon driver after the iio driver is accepted. >> Even though the stated goal is different, it seems to me that having >> two drivers really does not make sense. Any thoughts ? > > Hi Guenter, > > we for instance have two completely unrelated projects both using this chip, on one side we wish to measure power consumption on the host board and hwmon has the apis and tool ecosystem for it, while on the "ACME" system we needed IIO because it's buffer streaming scheme and remote features saves the day to create a lab instrument. But really both have their use, I'd recommend keeping the hwmon driver for now. I doubt those drivers will generate much maintenance work in the future either. > Marc, You can use the iio-hwmon bridge to get the hwmon ABI from the iio driver. There is no need to keep the hwmon driver as a separate entity for that. Guenter -- 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/