Re: [PATCH 0/2] Add support for MAX31722/MAX31723 temperature sensors

[Jonathan Cameron <jic23@kernel.org>]

On 09/03/16 13:30, Tiberiu Breana wrote:
> This patch set adds support for the MAX31722/MAX31723 temperature sensors /
> thermostats.
> Patch 1 adds basic support and power management.
> Patch 2 adds threshold interrupt support.
> 
> Tiberiu Breana (2):
>   iio: temperature: Add support for MAX31722/MAX31723 temperature
>     sensors
>   iio: temperature: Add threshold interrupt support for max31722
> 
>  drivers/iio/temperature/Kconfig    |  12 +
>  drivers/iio/temperature/Makefile   |   1 +
>  drivers/iio/temperature/max31722.c | 673 +++++++++++++++++++++++++++++++++++++
>  3 files changed, 686 insertions(+)
>  create mode 100644 drivers/iio/temperature/max31722.c
> 
Hi Tiberiu,

When proposing a temperature sensor driver, we need some additional justification
for why it is suited for IIO rather than the more focused (and in someways simpler)
hwmon.

So far we have had thermophiles that don't fit in as devices typically used for
hardware monitoring. Also in there are a few drivers which are for parts where
they are a cut down version of a more complex sensor (the measurement specialties
parts are either pressure sensors or humidty sensors at least in terms of what they
share interfaces with)

Also, please cc the hwmon maintainers (and probably list) as we want their agreement.

Jonathan