Re: [PATCH 0/2] iio: humidity: sht31: add Sensirion SHT31 support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 14.06.2016 00:23, Matt Ranostay wrote:
Didn't notice that will till now, and looking at the timestamp I see why :).
However this is more for the sw triggered buffer support which hwmon
lacks... there is a few examples of this already in the tree.

Maybe Jonathan and Peter will have some input on this.

Thanks for highlighting this Alison.  Not the first time we have had
simultaneous driver postings under review for both IIO and
HWMON unfortunately.

The hwmon guys are fairly flexible iff there is a good reason to want stuff that is in IIO. The original hwmon humidity driver was actually my fault as it slipped in when hwmon was effectively not maintained (Andrew Morton was babysitting). People on their side have always been a bit split on humidity
drivers and whether they are within their scope.

Anyhow, best bet is to email their list (perhaps reply to the driver thread)
to make sure everyone knows this is going on (cc linux-iio as well).
Then we can work out how to move forward.

Two drivers, one in each place is a non starter for new parts.  We have
ended up with a few instances, but it's mostly been a case of a driver
with much wider scope ending up covering parts that were already in hwmon.
Also a very small number of devices have moved across from hwmon to IIO.

In this particular case the sensor is reasonably fast so there is at
least some argument for IIO support. Note I tend to stay out of these
and let the driver author work on convincing Guenter / Jean
(they are very reasonable and responsive!) Only thing I'll say is that
I'm more than happy to have this in IIO if the hwmon guys don't mind.

Jonathan
On Mon, Jun 13, 2016 at 3:25 PM, Alison Schofield <amsfield22@xxxxxxxxx> wrote:
On Sat, Jun 11, 2016 at 10:13:15PM -0700, Matt Ranostay wrote:
Add support for the humidity and temperature functionally for the SHT31
chipset. In addition add support for using using software triggers.

Now that I'm hwmon-aware ;) I see this same driver under review in hwmon.
Can support exist in 2 places?

alisons


Matt Ranostay (2):
  devicetree: add Sensirion AG vendor id
  iio: humidity: sht31: add Sensirion SHT31 support

 .../devicetree/bindings/i2c/trivial-devices.txt    |   1 +
 .../devicetree/bindings/vendor-prefixes.txt        |   1 +
 drivers/iio/humidity/Kconfig                       |  11 +
 drivers/iio/humidity/Makefile                      |   1 +
drivers/iio/humidity/sht31.c | 416 +++++++++++++++++++++
 5 files changed, 430 insertions(+)
 create mode 100644 drivers/iio/humidity/sht31.c

--
2.7.4

--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux