Re: [PATCH 1/2] iio: ina2xx: add support for TI INA2xx Power Monitors

From: Guenter Roeck
Date: Tue Dec 01 2015 - 10:05:17 EST


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 <mtitinger@xxxxxxxxxxxx>
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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/