From nobody Tue Apr 7 12:20:32 2026 Received: from mail-wm1-f45.google.com (mail-wm1-f45.google.com [209.85.128.45]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 22AC131E820 for ; Fri, 3 Apr 2026 14:07:00 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.45 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775225224; cv=none; b=aBcJiMGUpYMfolAE+9ws0nZaGqOvGdcZytsaEDrj6RrWRWVgG1OjdKcAT3oEA6ADlxrnTqkFhQTPDX2nLJ9rd71qEWQNBd6J0ITnJx0r8gNOgu7r3HvF+4NaDRgvNHA7ZK4593fx423NBH7nwOMwrE9Z2zfGCXg3fsuobSsHYCk= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775225224; c=relaxed/simple; bh=ZaQYLQ79Qi1e/GYAO700zaQe68OP/sNqETSfVAS4jbg=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=Oy1++MVUUdWLvm+FGu+HKyrKw6WnE3Wpv34qhA6ydDrTkxOTy5bXsnk3KAw+bDZvnbAbE9LHUyGwFU/BI7D8Mq3dHFUyDP0YbTMl4LSrcCoeLZAi+wjC0Vd0VYqv96NDZpHV1Pg6pm19UMn6CQ+63J7P7uNa8gT9wqCLb5Yld40= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=DLlddQU4; arc=none smtp.client-ip=209.85.128.45 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="DLlddQU4" Received: by mail-wm1-f45.google.com with SMTP id 5b1f17b1804b1-486507134e4so27953775e9.0 for ; Fri, 03 Apr 2026 07:07:00 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1775225219; x=1775830019; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=KTKG/4DEdSl73iOWcC16gcuOW+Fph4H/Ug8xetwoxhY=; b=DLlddQU4UGBGTZ6+VbxOKgUKJpGtJ7CHADzwp4SNqrWPBOmPwYvTWt6YbbZPVKfG/d Zz0cZUxuENPL5zM22Nuz29HIRRN6ong3nbJNbGuBY8PV1hJzuAN2RuQIzIVRCfLiU/pi vARHeTcEzk7Mr+cyt/POAtii9XuOFRdLxbm0l91OtDhkUqh72dvRboSTNpqhp3GvrGfR nkJp+t9SRyvT1DQ1owI9znbpBS35F0G7MbTFTSKA9ONvi7TDprwYrEHi+2dyxxJHjl1q 1QQVnB1B5aQ9THvcGzvseBOqHZvVAe4zufU+ykOoq5LymdggnKiEvxC7kHCuSRhjSxOi 4oaw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1775225219; x=1775830019; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=KTKG/4DEdSl73iOWcC16gcuOW+Fph4H/Ug8xetwoxhY=; b=jtCY/xdCsfiApQqlYkdrFk5qbrgdyRkCSDyF12EbuWzIzgqJkHJxHmq851N9Zf7FU7 NqfOLEPhZGnVzEsN4YP+tfzKaOyuG3n+tvs3Nc6pIMKxy7Wig9l+1YK7Tqb+Z0eCmQCW b1MY4OYdhVsJu4eADwQ4xWWDX67poCeFMR9yOECgXJbAdRkPUiEJMewbmBY3ButUleyf aliOFSrShxki+7E0Ru4CKy/WomgpiJwRFEcmB0bP8qqLUcJVJeQeHY03t9b2M3RxCNwy FiW7xZ0uP7z8R5fGzKgAd/nG4+mexDmLZqdjOVa4VLkQy5WjFpY+VkekiVdTFviPJk1O xRDw== X-Forwarded-Encrypted: i=1; AJvYcCWNDrRq5tYwCNS3TaDfEFnEXULXmoI9LvraS7QQ4ykgKhveOI2degjqmgTw3cl4uB5xdabiMjViXExPEDU=@vger.kernel.org X-Gm-Message-State: AOJu0YyPnulwmop71ArmzRmDu4px1kEHVx/HZNSiRxiBmvd613mKl7zD TRZdQgGE2eUikg07gCYG5mX6wBThU8LhYUTLptRYk60Ozbj4vOeRKcw5 X-Gm-Gg: ATEYQzzpJ7/qP6eD/Sj/3tnDWFc7BxV/Kf90pds8bz3r2xeTJqnnEjrkpenC5doLzVe SthjUyy5EWGlmFUbMg2aueTucr7ZxDUc3JBwZeBV6IO4zv341oLjQSPslFUM7bIKL28ImVe4yU9 ybMXsTZGIgAcbPhp0S4bwWOqg7OVzwr+4yMu1Z0yKLxdJoYoGnlqTyTJ+33ZUGO1K36uRNKqRIp ExSgTNsPBtvlTdnKTpSZ1sohKMUB713hLUzLl1t93con8x8E7YAAmXp0fx+83UoZCTOnIeQwrIi V2ar2niDEZsI0xxOObk3FVmep2bv+KHkbHR3rfD1B66ZB63UVYBceGN+2LmE6U5VuXl1K8w4mct nI9+b06BTSHut9Juv5iBlq8w60NnhgrP/PhefzDSjTow+2kSl0SHUirjVlTKDBJFm+OAYIWMPXQ JU55SBMD3YEhz4S0KLuJMDATUhKSN4e04cGbqKxwVFB1yMygpbEzWhHp2J4DAQ8E9g7l+CbmIcX og3JTMX5Vc1 X-Received: by 2002:a05:600c:4752:b0:488:7ff5:2c67 with SMTP id 5b1f17b1804b1-4889976ea80mr51847825e9.12.1775225218911; Fri, 03 Apr 2026 07:06:58 -0700 (PDT) Received: from flaviu-Aspire-E5-572G.. ([93.122.249.8]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-48899d3a4e9sm42069665e9.6.2026.04.03.07.06.57 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 03 Apr 2026 07:06:58 -0700 (PDT) From: Flaviu Nistor To: Guenter Roeck , Jonathan Corbet , Shuah Khan Cc: Flaviu Nistor , linux-hwmon@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH v3] hwmon: (tmp102) add support for update interval Date: Fri, 3 Apr 2026 17:06:54 +0300 Message-ID: <20260403140654.10368-1-flaviu.nistor@gmail.com> X-Mailer: git-send-email 2.43.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Since the sensor supports different sampling intervals via bits CR0 and CR1 from the CONFIG register, add support in order for the conversion rate to be changed from user space. Default is 4 conv/sec. Signed-off-by: Flaviu Nistor --- Changes in v3: - Implement all changes suggested inline by Guenter Roeck. - Fixed identified issues from link provided by Guenter Roeck: https://sashiko.dev/#/patchset/20260401164701.18456-1-flaviu.nistor%40gmail= .com - Link to v2: https://lore.kernel.org/all/20260401164701.18456-1-flaviu.nis= tor@gmail.com/ Changes in v2: - Implement all changes suggested inline by Guenter Roeck. - Fixed identified issues from link provided by Guenter Roeck: https://sashiko.dev/#/patchset/20260331175418.16145-1-flaviu.nistor%40gmail= .com - Link to v1: https://lore.kernel.org/all/20260331175418.16145-1-flaviu.nis= tor@gmail.com/ Documentation/hwmon/tmp102.rst | 19 ++++- drivers/hwmon/tmp102.c | 128 +++++++++++++++++++++++++++------ 2 files changed, 123 insertions(+), 24 deletions(-) diff --git a/Documentation/hwmon/tmp102.rst b/Documentation/hwmon/tmp102.rst index 3c2cb5bab1e9..425a09a3c9b3 100644 --- a/Documentation/hwmon/tmp102.rst +++ b/Documentation/hwmon/tmp102.rst @@ -41,12 +41,25 @@ degree from -40 to +125 C. Resolution of the sensor is = 0.0625 degree. The operating temperature has a minimum of -55 C and a maximum of +150 C. =20 The TMP102 has a programmable update rate that can select between 8, 4, 1,= and -0.5 Hz. (Currently the driver only supports the default of 4 Hz). +0.25 Hz. =20 The TMP110 and TMP113 are software compatible with TMP102, but have differ= ent accuracy (maximum error) specifications. The TMP110 has an accuracy (maxim= um error) of 1.0 degree, TMP113 has an accuracy (maximum error) of 0.3 degree, while= TMP102 has an accuracy (maximum error) of 2.0 degree. =20 -The driver provides the common sysfs-interface for temperatures (see -Documentation/hwmon/sysfs-interface.rst under Temperatures). +sysfs-Interface +--------------- + +The following list includes the sysfs attributes that the driver provides,= their +permissions and a short description: + +=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D =3D=3D=3D=3D=3D=3D=3D =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D +Name Perm Description +=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D =3D=3D=3D=3D=3D=3D=3D =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D +temp1_input: RO Temperature input +temp1_label: RO Descriptive name for the sensor +temp1_max: RW Maximum temperature +temp1_max_hyst: RW Maximum hysteresis temperature +update_interval RW Update conversions interval in mil= liseconds +=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D =3D=3D=3D=3D=3D=3D=3D =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D diff --git a/drivers/hwmon/tmp102.c b/drivers/hwmon/tmp102.c index 5b10c395a84d..3aa1a3fbeaa9 100644 --- a/drivers/hwmon/tmp102.c +++ b/drivers/hwmon/tmp102.c @@ -50,11 +50,16 @@ =20 #define CONVERSION_TIME_MS 35 /* in milli-seconds */ =20 +#define NUM_SAMPLE_TIMES 4 +#define DEFAULT_SAMPLE_TIME_MS 250 +static const unsigned int *sample_times =3D (const unsigned int []){ 125, = 250, 1000, 4000 }; + struct tmp102 { const char *label; struct regmap *regmap; u16 config_orig; unsigned long ready_time; + u16 sample_time; }; =20 /* convert left adjusted 13-bit TMP102 register value to milliCelsius */ @@ -79,8 +84,20 @@ static int tmp102_read_string(struct device *dev, enum h= wmon_sensor_types type, return 0; } =20 -static int tmp102_read(struct device *dev, enum hwmon_sensor_types type, - u32 attr, int channel, long *temp) +static int tmp102_read_chip(struct device *dev, u32 attr, long *val) +{ + struct tmp102 *tmp102 =3D dev_get_drvdata(dev); + + switch (attr) { + case hwmon_chip_update_interval: + *val =3D tmp102->sample_time; + return 0; + default: + return -EOPNOTSUPP; + } +} + +static int tmp102_read_temp(struct device *dev, u32 attr, long *val) { struct tmp102 *tmp102 =3D dev_get_drvdata(dev); unsigned int regval; @@ -108,13 +125,54 @@ static int tmp102_read(struct device *dev, enum hwmon= _sensor_types type, err =3D regmap_read(tmp102->regmap, reg, ®val); if (err < 0) return err; - *temp =3D tmp102_reg_to_mC(regval); + + *val =3D tmp102_reg_to_mC(regval); =20 return 0; } =20 -static int tmp102_write(struct device *dev, enum hwmon_sensor_types type, - u32 attr, int channel, long temp) +static int tmp102_read(struct device *dev, enum hwmon_sensor_types type, + u32 attr, int channel, long *val) +{ + switch (type) { + case hwmon_chip: + return tmp102_read_chip(dev, attr, val); + case hwmon_temp: + return tmp102_read_temp(dev, attr, val); + default: + return -EOPNOTSUPP; + } +} + +static int tmp102_update_interval(struct device *dev, long val) +{ + struct tmp102 *tmp102 =3D dev_get_drvdata(dev); + u8 index; + s32 err; + + index =3D find_closest(val, sample_times, NUM_SAMPLE_TIMES); + + err =3D regmap_update_bits(tmp102->regmap, TMP102_CONF_REG, + (TMP102_CONF_CR1 | TMP102_CONF_CR0), (3 - index) << 6); + if (err < 0) + return err; + tmp102->sample_time =3D sample_times[index]; + + return 0; +} + +static int tmp102_write_chip(struct device *dev, u32 attr, long val) +{ + switch (attr) { + case hwmon_chip_update_interval: + return tmp102_update_interval(dev, val); + default: + return -EOPNOTSUPP; + } + return 0; +} + +static int tmp102_write_temp(struct device *dev, u32 attr, long val) { struct tmp102 *tmp102 =3D dev_get_drvdata(dev); int reg; @@ -130,8 +188,22 @@ static int tmp102_write(struct device *dev, enum hwmon= _sensor_types type, return -EOPNOTSUPP; } =20 - temp =3D clamp_val(temp, -256000, 255000); - return regmap_write(tmp102->regmap, reg, tmp102_mC_to_reg(temp)); + val =3D clamp_val(val, -256000, 255000); + return regmap_write(tmp102->regmap, reg, tmp102_mC_to_reg(val)); +} + +static int tmp102_write(struct device *dev, enum hwmon_sensor_types type, + u32 attr, int channel, long val) +{ + switch (type) { + case hwmon_chip: + return tmp102_write_chip(dev, attr, val); + case hwmon_temp: + return tmp102_write_temp(dev, attr, val); + default: + return -EOPNOTSUPP; + } + return 0; } =20 static umode_t tmp102_is_visible(const void *data, enum hwmon_sensor_types= type, @@ -139,27 +211,39 @@ static umode_t tmp102_is_visible(const void *data, en= um hwmon_sensor_types type, { const struct tmp102 *tmp102 =3D data; =20 - if (type !=3D hwmon_temp) - return 0; - - switch (attr) { - case hwmon_temp_input: - return 0444; - case hwmon_temp_label: - if (tmp102->label) + switch (type) { + case hwmon_chip: + switch (attr) { + case hwmon_chip_update_interval: + return 0644; + default: + break; + } + break; + case hwmon_temp: + switch (attr) { + case hwmon_temp_input: return 0444; - return 0; - case hwmon_temp_max_hyst: - case hwmon_temp_max: - return 0644; + case hwmon_temp_label: + if (tmp102->label) + return 0444; + return 0; + case hwmon_temp_max_hyst: + case hwmon_temp_max: + return 0644; + default: + break; + } + break; default: - return 0; + break; } + return 0; } =20 static const struct hwmon_channel_info * const tmp102_info[] =3D { HWMON_CHANNEL_INFO(chip, - HWMON_C_REGISTER_TZ), + HWMON_C_REGISTER_TZ | HWMON_C_UPDATE_INTERVAL), HWMON_CHANNEL_INFO(temp, HWMON_T_INPUT | HWMON_T_LABEL | HWMON_T_MAX | HWMON_T_MAX_HYST), NULL @@ -237,6 +321,8 @@ static int tmp102_probe(struct i2c_client *client) if (IS_ERR(tmp102->regmap)) return PTR_ERR(tmp102->regmap); =20 + tmp102->sample_time =3D DEFAULT_SAMPLE_TIME_MS; + err =3D regmap_read(tmp102->regmap, TMP102_CONF_REG, ®val); if (err < 0) { dev_err(dev, "error reading config register\n"); --=20 2.34.1