From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-13.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 4A9A9C43461 for ; Mon, 3 May 2021 13:52:06 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id F222C61006 for ; Mon, 3 May 2021 13:52:05 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234471AbhECNw5 (ORCPT ); Mon, 3 May 2021 09:52:57 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:40954 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233325AbhECNwz (ORCPT ); Mon, 3 May 2021 09:52:55 -0400 Received: from mail-oi1-x22d.google.com (mail-oi1-x22d.google.com [IPv6:2607:f8b0:4864:20::22d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 1BD9BC06174A; Mon, 3 May 2021 06:52:02 -0700 (PDT) Received: by mail-oi1-x22d.google.com with SMTP id l6so5472810oii.1; Mon, 03 May 2021 06:52:02 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=sender:date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=IE6EXMAjT6BgwhX5amZzy8AQhpUa/aIu5YV4iTEFXWw=; b=G4ThBeidhkulEc7TFedqjJXMuc1xDHm31N2fZQTZ3T64bntuv6hZlsZc/hsN6AWaKA Befp144kr6jtIj0FHK7BxIkf6rKMEUrM2IYS01d+9Vof71c2g5+p8I7kd1nVp+cElCAg Y3OWcHHzNyP4SRG5744eW2nzczmCEGxi3+/STUJfccHhSZlVQrxitWiXa+VpEAgF/DeH C+f2lrsF3Nb/7XjW/V16rUtv74ggLPLeF4vHwIN4+eKp2V8xW+S2nfdA8pWnaRoXwWoW KEqPJj3tFNCjG89MWhkcLhYSTL2gtrMQyjcqxN1yc74tkYXHM5M1uuJL1X6+Pl+kBbvp uGHQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:date:from:to:cc:subject:message-id :references:mime-version:content-disposition:in-reply-to; bh=IE6EXMAjT6BgwhX5amZzy8AQhpUa/aIu5YV4iTEFXWw=; b=M0SKBHWkHn5ZQJUQO8eBU6HeloOgwdUOOUl/hsl6FgEW04s3fhd0YWfXfmNccQGRaq TmGcW++c7DT1p8jKn3bjERLi7j8ajl1mh4p1CpeLaOkQtP0aG9k0fnWcqCzJeCJ5nQ24 27BkAVoDZGB80SC/TQChozi1RmyXHrF+gVSiozklamj06ydMh+Z/6hNa1AZfQUKg78GX XmBD1xzcNnzdMeMFsjPSVlsly4QuQBj+au+um7+P+DDapUMbbm9gT6MaRXKglL6sRH0x VgRLencCkgD/l3XFntuQy9Q6eYfTyIXrZxhv/0/hR17hFk4iJuJU/4k/v5aMHR9nkPgC yWdw== X-Gm-Message-State: AOAM530grpIC+8kiFKdZm+Li44ShBGa0vew3qvG+jPpukbQFpe5FOd3z 7me/tZxw88gNt11xmC/cW+k= X-Google-Smtp-Source: ABdhPJzvh55ywLaK3y9YWboMUmszO5SSzhiiK3/tvz1N+MSdoexU9BOlQnPUrMJNMLMqSH2PN68UPg== X-Received: by 2002:aca:6749:: with SMTP id b9mr21074246oiy.128.1620049920804; Mon, 03 May 2021 06:52:00 -0700 (PDT) Received: from localhost ([2600:1700:e321:62f0:329c:23ff:fee3:9d7c]) by smtp.gmail.com with ESMTPSA id w2sm2743362oov.23.2021.05.03.06.51.59 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 03 May 2021 06:52:00 -0700 (PDT) Sender: Guenter Roeck Date: Mon, 3 May 2021 06:51:58 -0700 From: Guenter Roeck To: Navin Sankar Velliangiri Cc: Jean Delvare , Jonathan Corbet , linux-hwmon@vger.kernel.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2] hwmon: Add sht4x Temperature and Humidity Sensor Driver Message-ID: <20210503135158.GA3040376@roeck-us.net> References: <20210503114503.59239-1-navin@linumiz.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210503114503.59239-1-navin@linumiz.com> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, May 03, 2021 at 05:15:00PM +0530, Navin Sankar Velliangiri wrote: > This patch adds a hwmon driver for the SHT4x Temperature and > Humidity sensor. > > Signed-off-by: Navin Sankar Velliangiri > --- Change log goes here. If it is missing, I won't know what changed. That means I will have to dig out older patch versions to compare. That costs time and would hold up both this patch as well as all other patches which I still have to review. For this reason, I will not review patches without change log. Thanks, Guenter > Documentation/hwmon/index.rst | 1 + > Documentation/hwmon/sht4x.rst | 45 +++++ > drivers/hwmon/Kconfig | 13 +- > drivers/hwmon/Makefile | 1 + > drivers/hwmon/sht4x.c | 301 ++++++++++++++++++++++++++++++++++ > 5 files changed, 360 insertions(+), 1 deletion(-) > create mode 100644 Documentation/hwmon/sht4x.rst > create mode 100644 drivers/hwmon/sht4x.c > > diff --git a/Documentation/hwmon/index.rst b/Documentation/hwmon/index.rst > index 8d5a2df1ecb6..2a20c6616e21 100644 > --- a/Documentation/hwmon/index.rst > +++ b/Documentation/hwmon/index.rst > @@ -160,6 +160,7 @@ Hardware Monitoring Kernel Drivers > sht15 > sht21 > sht3x > + sht4x > shtc1 > sis5595 > sl28cpld > diff --git a/Documentation/hwmon/sht4x.rst b/Documentation/hwmon/sht4x.rst > new file mode 100644 > index 000000000000..3b37abcd4a46 > --- /dev/null > +++ b/Documentation/hwmon/sht4x.rst > @@ -0,0 +1,45 @@ > +.. SPDX-License-Identifier: GPL-2.0 > + > +Kernel driver sht4x > +=================== > + > +Supported Chips: > + > + * Sensirion SHT4X > + > + Prefix: 'sht4x' > + > + Addresses scanned: None > + > + Datasheet: > + > + English: https://www.sensirion.com/fileadmin/user_upload/customers/sensirion/Dokumente/2_Humidity_Sensors/Datasheets/Sensirion_Humidity_Sensors_SHT4x_Datasheet.pdf > + > +Author: Navin Sankar Velliangiri > + > + > +Description > +----------- > + > +This driver implements support for the Sensirion SHT4x chip, a humidity > +and temperature sensor. Temperature is measured in degree celsius, relative > +humidity is expressed as a percentage. In sysfs interface, all values are > +scaled by 1000, i.e. the value for 31.5 degrees celsius is 31500. > + > +Usage Notes > +----------- > + > +The device communicates with the I2C protocol. Sensors can have the I2C > +address 0x44. See Documentation/i2c/instantiating-devices.rst for methods > +to instantiate the device. > + > +Sysfs entries > +------------- > + > +=============== ============================================ > +temp1_input Measured temperature in millidegrees Celcius > +humidity1_input Measured humidity in %H > +update_interval The minimum interval for polling the sensor, > + in milliseconds. Writable. Must be at least > + 2000. > +============== ============================================= > diff --git a/drivers/hwmon/Kconfig b/drivers/hwmon/Kconfig > index 54f04e61fb83..ad10469d7b13 100644 > --- a/drivers/hwmon/Kconfig > +++ b/drivers/hwmon/Kconfig > @@ -1583,6 +1583,17 @@ config SENSORS_SHT3x > This driver can also be built as a module. If so, the module > will be called sht3x. > > +config SENSORS_SHT4x > + tristate "Sensiron humidity and temperature sensors. SHT4x and compat." > + depends on I2C > + select CRC8 > + help > + If you say yes here you get support for thr Sensiron SHT40, SHT41 and > + SHT45 humidity and temperature sensors. > + > + This driver can also be built as a module. If so, the module > + will be called sht4x. > + > config SENSORS_SHTC1 > tristate "Sensiron humidity and temperature sensors. SHTC1 and compat." > depends on I2C > @@ -1798,7 +1809,7 @@ config SENSORS_ADS7871 > > config SENSORS_AMC6821 > tristate "Texas Instruments AMC6821" > - depends on I2C > + depends on I2C > help > If you say yes here you get support for the Texas Instruments > AMC6821 hardware monitoring chips. > diff --git a/drivers/hwmon/Makefile b/drivers/hwmon/Makefile > index fe38e8a5c979..62cee3e03c7f 100644 > --- a/drivers/hwmon/Makefile > +++ b/drivers/hwmon/Makefile > @@ -170,6 +170,7 @@ obj-$(CONFIG_SENSORS_SL28CPLD) += sl28cpld-hwmon.o > obj-$(CONFIG_SENSORS_SHT15) += sht15.o > obj-$(CONFIG_SENSORS_SHT21) += sht21.o > obj-$(CONFIG_SENSORS_SHT3x) += sht3x.o > +obj-$(CONFIG_SENSORS_SHT4x) += sht4x.o > obj-$(CONFIG_SENSORS_SHTC1) += shtc1.o > obj-$(CONFIG_SENSORS_SIS5595) += sis5595.o > obj-$(CONFIG_SENSORS_SMM665) += smm665.o > diff --git a/drivers/hwmon/sht4x.c b/drivers/hwmon/sht4x.c > new file mode 100644 > index 000000000000..fae18f8ab3d3 > --- /dev/null > +++ b/drivers/hwmon/sht4x.c > @@ -0,0 +1,301 @@ > +// SPDX-License-Identifier: GPL-2.0-only > + > +/* > + * Copyright (c) Linumiz 2021 > + * > + * sht4x.c - Linux hwmon driver for SHT4x Temperature and Humidity sensor > + * > + * Author: Navin Sankar Velliangiri > + */ > + > +#include > +#include > +#include > +#include > +#include > +#include > +#include > + > +/* > + * Poll intervals (in milliseconds) > + */ > +#define SHT4X_DEFAULT_MIN_POLL_INTERVAL 2000 > + > +/* > + * I2C command delays (in microseconds) > + */ > +#define SHT4X_MEAS_DELAY 1000 > +#define SHT4X_DELAY_EXTRA 10000 > + > +/* > + * Command Bytes > + */ > +#define SHT4X_CMD_MEASURE_HPM 0b11111101 > +#define SHT4X_CMD_RESET 0b10010100 > + > +#define SHT4X_CMD_LEN 1 > +#define SHT4X_CRC8_LEN 1 > +#define SHT4X_WORD_LEN 2 > +#define SHT4X_RESPONSE_LENGTH 6 > +#define SHT4X_CRC8_POLYNOMIAL 0x31 > +#define SHT4X_CRC8_INIT 0xff > +#define SHT4X_MIN_TEMPERATURE -45000 > +#define SHT4X_MAX_TEMPERATURE 125000 > +#define SHT4X_MIN_HUMIDITY 0 > +#define SHT4X_MAX_HUMIDITY 100000 > + > +DECLARE_CRC8_TABLE(sht4x_crc8_table); > + > +/** > + * struct sht4x_data - All the data required to operate an SHT4X chip > + * @client: the i2c client associated with the SHT4X > + * @lock: a mutex that is used to prevent parallel access to the i2c client > + * @update_interval: the minimum poll interval > + * @last_updated: the previous time that the SHT4X was polled > + * @temperature: the latest temperature value received from the SHT4X > + * @humidity: the latest humidity value received from the SHT4X > + */ > +struct sht4x_data { > + struct i2c_client *client; > + struct mutex lock; /* atomic read data updates */ > + bool valid; /* validity of fields below */ > + unsigned long update_interval; /* in milli-seconds */ > + unsigned long last_updated; /* in jiffies */ > + s32 temperature; > + s32 humidity; > +}; > + > +/** > + * sht4x_read_values() - read and parse the raw data from the SHT4X > + * @sht4x_data: the struct sht4x_data to use for the lock > + * Return: 0 if succesfull, 1 if not > + */ > +static int sht4x_read_values(struct sht4x_data *data) > +{ > + int ret; > + u16 t_ticks, rh_ticks; > + unsigned long next_update; > + struct i2c_client *client = data->client; > + u8 crc, raw_data[SHT4X_RESPONSE_LENGTH], > + cmd[] = {SHT4X_CMD_MEASURE_HPM}; > + > + mutex_lock(&data->lock); > + next_update = data->last_updated + > + msecs_to_jiffies(data->update_interval); > + if (!data->valid || time_after(jiffies, next_update)) { > + ret = i2c_master_send(client, cmd, SHT4X_CMD_LEN); > + if (ret < 0) > + goto unlock; > + > + usleep_range(SHT4X_MEAS_DELAY, > + SHT4X_MEAS_DELAY + SHT4X_DELAY_EXTRA); > + > + ret = i2c_master_recv(client, raw_data, SHT4X_RESPONSE_LENGTH); > + if (ret != SHT4X_RESPONSE_LENGTH) { > + if (ret >= 0) > + ret = -ENODATA; > + > + goto unlock; > + } > + > + t_ticks = raw_data[0] << 8 | raw_data[1]; > + rh_ticks = raw_data[3] << 8 | raw_data[4]; > + > + crc = crc8(sht4x_crc8_table, &raw_data[0], SHT4X_WORD_LEN, CRC8_INIT_VALUE); > + if (crc != raw_data[2]) { > + dev_err(&client->dev, "data integrity check failed\n"); > + ret = -EIO; > + goto unlock; > + } > + > + crc = crc8(sht4x_crc8_table, &raw_data[3], SHT4X_WORD_LEN, CRC8_INIT_VALUE); > + if (crc != raw_data[5]) { > + dev_err(&client->dev, "data integrity check failed\n"); > + ret = -EIO; > + goto unlock; > + } > + > + data->temperature = ((21875 * (int32_t)t_ticks) >> 13) - 45000; > + data->humidity = ((15625 * (int32_t)rh_ticks) >> 13) - 6000; > + data->last_updated = jiffies; > + data->valid = true; > + } > + > +unlock: > + mutex_unlock(&data->lock); > + return ret; > +} > + > +static ssize_t sht4x_interval_write(struct sht4x_data *data, > + long val) > +{ > + data->update_interval = val; > + return 0; > +} > + > +/** > + * sht4x_interval_read() - read the minimum poll interval > + * in milliseconds > + */ > +static size_t sht4x_interval_read(struct sht4x_data *data, > + long *val) > +{ > + *val = data->update_interval; > + return 0; > +} > + > +/** > + * sht4x_temperature1_read() - read the temperature in millidegrees > + */ > +static int sht4x_temperature1_read(struct sht4x_data *data, long *val) > +{ > + int ret; > + > + ret = sht4x_read_values(data); > + if (ret < 0) > + return ret; > + > + *val = data->temperature; > + > + return 0; > +} > + > +/** > + * sht4x_humidity1_read() - read a relative humidity in millipercent > + */ > +static int sht4x_humidity1_read(struct sht4x_data *data, long *val) > +{ > + int ret; > + > + ret = sht4x_read_values(data); > + if (ret < 0) > + return ret; > + > + *val = data->humidity; > + > + return 0; > +} > + > +static umode_t sht4x_hwmon_visible(const void *data, > + enum hwmon_sensor_types type, > + u32 attr, int channel) > +{ > + switch (type) { > + case hwmon_temp: > + case hwmon_humidity: > + return 0444; > + case hwmon_chip: > + return 0644; > + default: > + return 0; > + } > +} > + > +static int sht4x_hwmon_read(struct device *dev, enum hwmon_sensor_types type, > + u32 attr, int channel, long *val) > +{ > + struct sht4x_data *data = dev_get_drvdata(dev); > + > + switch (type) { > + case hwmon_temp: > + return sht4x_temperature1_read(data, val); > + case hwmon_humidity: > + return sht4x_humidity1_read(data, val); > + case hwmon_chip: > + return sht4x_interval_read(data, val); > + default: > + return -EOPNOTSUPP; > + } > +} > + > +static int sht4x_hwmon_write(struct device *dev, enum hwmon_sensor_types type, > + u32 attr, int channel, long val) > +{ > + struct sht4x_data *data = dev_get_drvdata(dev); > + > + switch (type) { > + case hwmon_chip: > + return sht4x_interval_write(data, val); > + default: > + return -EOPNOTSUPP; > + } > +} > + > +static const struct hwmon_channel_info *sht4x_info[] = { > + HWMON_CHANNEL_INFO(chip, HWMON_C_UPDATE_INTERVAL), > + HWMON_CHANNEL_INFO(temp, HWMON_T_INPUT), > + HWMON_CHANNEL_INFO(humidity, HWMON_H_INPUT), > + NULL, > +}; > + > +static const struct hwmon_ops sht4x_hwmon_ops = { > + .is_visible = sht4x_hwmon_visible, > + .read = sht4x_hwmon_read, > + .write = sht4x_hwmon_write, > +}; > + > +static const struct hwmon_chip_info sht4x_chip_info = { > + .ops = &sht4x_hwmon_ops, > + .info = sht4x_info, > +}; > + > +static int sht4x_probe(struct i2c_client *client, > + const struct i2c_device_id *sht4x_id) > +{ > + struct device *device = &client->dev; > + struct device *hwmon_dev; > + struct sht4x_data *data; > + u8 cmd[] = {SHT4X_CMD_RESET}; > + int ret; > + > + /* > + * we require full i2c support since the sht4x uses multi-byte read and > + * writes as well as multi-byte commands which are not supported by > + * the smbus protocol > + */ > + if (!i2c_check_functionality(client->adapter, I2C_FUNC_I2C)) > + return -EOPNOTSUPP; > + > + data = devm_kzalloc(device, sizeof(*data), GFP_KERNEL); > + if (!data) > + return -ENOMEM; > + > + data->update_interval = SHT4X_DEFAULT_MIN_POLL_INTERVAL; > + data->client = client; > + > + mutex_init(&data->lock); > + > + crc8_populate_msb(sht4x_crc8_table, SHT4X_CRC8_POLYNOMIAL); > + > + ret = i2c_master_send(client, cmd, SHT4X_CMD_LEN); > + if (ret != SHT4X_CMD_LEN) > + return -EIO; > + > + hwmon_dev = devm_hwmon_device_register_with_info(device, > + client->name, > + data, > + &sht4x_chip_info, > + NULL); > + > + return PTR_ERR_OR_ZERO(hwmon_dev); > +} > + > +static const struct i2c_device_id sht4x_id[] = { > + { "sht4x", 0 }, > + { }, > +}; > +MODULE_DEVICE_TABLE(i2c, sht4x_id); > + > +static struct i2c_driver sht4x_driver = { > + .driver = { > + .name = "sht4x", > + }, > + .probe = sht4x_probe, > + .id_table = sht4x_id, > +}; > + > +module_i2c_driver(sht4x_driver); > + > +MODULE_AUTHOR("Navin Sankar Velliangiri "); > +MODULE_DESCRIPTION("Sensirion SHT4x humidity and temperature sensor driver"); > +MODULE_LICENSE("GPL v2"); > -- > 2.31.1 >