Re: [PATCH 0/2] iio: temperature: add mcp98xx driver support

On Sun, Jul 19, 2015 at 10:24:53AM +0100, Jonathan Cameron wrote:

On 19/07/15 04:04, Matt Ranostay wrote:

quoted

This changeset adds driver support for the Microchip mcp98xx series of
temperature sensors.

MCP98xx is pretty a pretty far reaching claim. This could also be MCP9804
or MCP9843, which are JC42 compatible sensor chips and supported by the jc42
driver. Does the new driver claim to support those as well ?

Guenter

quoted

This includes temperature reading, and rising/falling threshold events.

Why an IIO driver?  These parts already look to be supported in hwmon by
the lm75 driver.  We need a pretty strong reason to contemplate having
support in both subsystems...

quoted

Matt Ranostay (2):
  iio: temperature: DT binding doc for mcp98xx
  iio: temperature: add support for mcp98xx sensors

 .../bindings/iio/temperature/mcp98xx.txt           |  22 +
 drivers/iio/temperature/Kconfig                    |  10 +
 drivers/iio/temperature/Makefile                   |   1 +
 drivers/iio/temperature/mcp98xx.c                  | 588 +++++++++++++++++++++
 4 files changed, 621 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/iio/temperature/mcp98xx.txt
 create mode 100644 drivers/iio/temperature/mcp98xx.c