From nobody Fri Apr 3 03:48:29 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 9DBAFC54EE9 for ; Fri, 16 Sep 2022 11:00:10 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231758AbiIPLAG (ORCPT ); Fri, 16 Sep 2022 07:00:06 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:34638 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231636AbiIPK6c (ORCPT ); Fri, 16 Sep 2022 06:58:32 -0400 Received: from mail-ej1-x634.google.com (mail-ej1-x634.google.com [IPv6:2a00:1450:4864:20::634]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D2040B7284 for ; Fri, 16 Sep 2022 03:46:17 -0700 (PDT) Received: by mail-ej1-x634.google.com with SMTP id go34so48523360ejc.2 for ; Fri, 16 Sep 2022 03:46:17 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=melexis.com; s=google; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date; bh=p4XqSxlqVYMvnkQENK7Ym8BEQjIxm7QtDHvm9RYN1l0=; b=QPVWM/6P8eRTAl6w+QstLCruhITgo6dCsm1CSFajAzRElroC2BnobrzDLDzXeNHwhd 36kuuUbcAm1LCJPS8KXbpUKkIPGqrnyKFdFQ95t0xAnC2gPIUfpci8CQeJ60dbn4OzD6 1eYnp068wZR/UNvsBV5E+pr4+cJnKcv9LFO71Qj79zVP/ugR1Az7q/EWw31kKeZ/FroC Ur1UMQVRTGM0peNDy9q4e/DsGviO/Wy5U9DbKrRGH13huotv1I6bXWL72xAhDh+1srKv +COVebtJ5AxDjFQwpncfhc3llA0mdVhNPoHWytgt1E1iEI0PYOXw1O1lWMc8I13pDTbA p95A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date; bh=p4XqSxlqVYMvnkQENK7Ym8BEQjIxm7QtDHvm9RYN1l0=; b=bWosDGkBKf/ZIuSvXdlG7FCTBmOzVfwnANMx4AHsEsyNX9E/2fixs+eNNzecFcIn5r o2LLtofkBqVCCkIYLlCEIuopX1iVion/L7tAJt1MJc0BE89/pFFmRaN9JqmjLrpDou1h dHl3RooKhCTmD6Wp3BpkwIRt+an59nEBA1IUxNhyHhkG0KpPSGYn0MNMF/reNJlu7mZk 6GHxBmUjoPDr82QE8Wa2BGorRwsrXlAXiXbpD9gdKQ++agOQuMXysFQVdTN0MjBVBA5I IPfbdEhbOewwKjvqjvNaU4N+1dDNEtxhuoBJxa8h4+HAi0Z2wkrasDlBAJ1xOz+vqorI FG1w== X-Gm-Message-State: ACrzQf3jfLuXqW0xzVwMUMpGcf96zKSkSKNAXzhDLNe8wJgej2PV74by eSAE68E/fBPztUIbzMf0Vumr2b0Q3zjyGA== X-Google-Smtp-Source: AMsMyM4tWsFptJCE3Bkume2/mLP/LFVbEaFVGhh2orjSDVpyMr5kOc2ffnHybo+p59VUjU14qme/VA== X-Received: by 2002:a17:906:9bd3:b0:778:c8e0:fcee with SMTP id de19-20020a1709069bd300b00778c8e0fceemr3106136ejc.275.1663325173308; Fri, 16 Sep 2022 03:46:13 -0700 (PDT) Received: from cmo-ThinkPad-T495.telenet.be (ptr-4xh0y3vvunoijj2gi1k.18120a2.ip6.access.telenet.be. [2a02:1810:a44c:8f00:cef1:93b4:727b:dd58]) by smtp.gmail.com with ESMTPSA id fi24-20020a1709073ad800b0072af4af2f46sm10148434ejc.74.2022.09.16.03.46.12 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 16 Sep 2022 03:46:12 -0700 (PDT) From: cmo@melexis.com To: Jonathan Cameron Cc: linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org, Andy Shevchenko , Crt Mori Subject: [PATCH v4 1/3] iio: temperature: mlx90632 Add runtime powermanagement modes Date: Fri, 16 Sep 2022 12:45:50 +0200 Message-Id: <67684133a45c4da6d4c13f5ee766d35cdae854e6.1663324968.git.cmo@melexis.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" From: Crt Mori The sensor can operate in lower power modes and even make measurements when in those lower powered modes. The decision was taken that if measurement is not requested within 2 seconds the sensor will remain in SLEEP_STEP power mode, where measurements are triggered on request with setting the start of measurement bit (SOB). In this mode the measurements are taking a bit longer because we need to start it and complete it. Currently, in continuous mode we read ready data and this mode is activated if sensor measurement is requested within 2 seconds. The suspend timeout is increased to 6 seconds (instead of 3 before), because that enables more measurements in lower power mode (SLEEP_STEP), with the lowest refresh rate (2 seconds). Signed-off-by: Crt Mori --- drivers/iio/temperature/mlx90632.c | 341 ++++++++++++++++++++++++----- 1 file changed, 287 insertions(+), 54 deletions(-) diff --git a/drivers/iio/temperature/mlx90632.c b/drivers/iio/temperature/m= lx90632.c index 549c0ab5c2be..80497d9bc4e9 100644 --- a/drivers/iio/temperature/mlx90632.c +++ b/drivers/iio/temperature/mlx90632.c @@ -6,11 +6,14 @@ * * Driver for the Melexis MLX90632 I2C 16-bit IR thermopile sensor */ +#include #include +#include #include #include #include #include +#include #include #include #include @@ -55,6 +58,12 @@ #define MLX90632_EE_Ha 0x2481 /* Ha customer calib value reg 16bit */ #define MLX90632_EE_Hb 0x2482 /* Hb customer calib value reg 16bit */ =20 +#define MLX90632_EE_MEDICAL_MEAS1 0x24E1 /* Medical measurement 1 16b= it */ +#define MLX90632_EE_MEDICAL_MEAS2 0x24E2 /* Medical measurement 2 16b= it */ +#define MLX90632_EE_EXTENDED_MEAS1 0x24F1 /* Extended measurement 1 16= bit */ +#define MLX90632_EE_EXTENDED_MEAS2 0x24F2 /* Extended measurement 2 16= bit */ +#define MLX90632_EE_EXTENDED_MEAS3 0x24F3 /* Extended measurement 3 16= bit */ + /* Register addresses - volatile */ #define MLX90632_REG_I2C_ADDR 0x3000 /* Chip I2C address register */ =20 @@ -62,13 +71,16 @@ #define MLX90632_REG_CONTROL 0x3001 /* Control Register address */ #define MLX90632_CFG_PWR_MASK GENMASK(2, 1) /* PowerMode Mask */ #define MLX90632_CFG_MTYP_MASK GENMASK(8, 4) /* Meas select Mask */ +#define MLX90632_CFG_SOB_MASK BIT(11) =20 /* PowerModes statuses */ #define MLX90632_PWR_STATUS(ctrl_val) (ctrl_val << 1) #define MLX90632_PWR_STATUS_HALT MLX90632_PWR_STATUS(0) /* hold */ -#define MLX90632_PWR_STATUS_SLEEP_STEP MLX90632_PWR_STATUS(1) /* sleep ste= p*/ +#define MLX90632_PWR_STATUS_SLEEP_STEP MLX90632_PWR_STATUS(1) /* sleep ste= p */ #define MLX90632_PWR_STATUS_STEP MLX90632_PWR_STATUS(2) /* step */ -#define MLX90632_PWR_STATUS_CONTINUOUS MLX90632_PWR_STATUS(3) /* continuou= s*/ +#define MLX90632_PWR_STATUS_CONTINUOUS MLX90632_PWR_STATUS(3) /* continuou= s */ + +#define MLX90632_EE_RR GENMASK(10, 8) /* Only Refresh Rate bits */ =20 /* Measurement types */ #define MLX90632_MTYP_MEDICAL 0 @@ -116,8 +128,9 @@ #define MLX90632_REF_12 12LL /* ResCtrlRef value of Ch 1 or Ch 2 */ #define MLX90632_REF_3 12LL /* ResCtrlRef value of Channel 3 */ #define MLX90632_MAX_MEAS_NUM 31 /* Maximum measurements in list */ -#define MLX90632_SLEEP_DELAY_MS 3000 /* Autosleep delay */ +#define MLX90632_SLEEP_DELAY_MS 6000 /* Autosleep delay */ #define MLX90632_EXTENDED_LIMIT 27000 /* Extended mode raw value limit */ +#define MLX90632_MEAS_MAX_TIME 2000 /* Max measurement time in ms for the = lowest refresh rate */ =20 /** * struct mlx90632_data - private data for the MLX90632 device @@ -130,6 +143,9 @@ * @object_ambient_temperature: Ambient temperature at object (might diffe= r of * the ambient temperature of sensor. * @regulator: Regulator of the device + * @powerstatus: Current POWER status of the device + * @interaction_ts: Timestamp of the last temperature read that is used + * for power management in jiffies */ struct mlx90632_data { struct i2c_client *client; @@ -139,6 +155,8 @@ struct mlx90632_data { u8 mtyp; u32 object_ambient_temperature; struct regulator *regulator; + int powerstatus; + unsigned long interaction_ts; }; =20 static const struct regmap_range mlx90632_volatile_reg_range[] =3D { @@ -158,6 +176,8 @@ static const struct regmap_range mlx90632_read_reg_rang= e[] =3D { regmap_reg_range(MLX90632_EE_VERSION, MLX90632_EE_Ka), regmap_reg_range(MLX90632_EE_CTRL, MLX90632_EE_I2C_ADDR), regmap_reg_range(MLX90632_EE_Ha, MLX90632_EE_Hb), + regmap_reg_range(MLX90632_EE_MEDICAL_MEAS1, MLX90632_EE_MEDICAL_MEAS2), + regmap_reg_range(MLX90632_EE_EXTENDED_MEAS1, MLX90632_EE_EXTENDED_MEAS3), regmap_reg_range(MLX90632_REG_I2C_ADDR, MLX90632_REG_CONTROL), regmap_reg_range(MLX90632_REG_I2C_CMD, MLX90632_REG_I2C_CMD), regmap_reg_range(MLX90632_REG_STATUS, MLX90632_REG_STATUS), @@ -198,16 +218,38 @@ static const struct regmap_config mlx90632_regmap =3D= { =20 static s32 mlx90632_pwr_set_sleep_step(struct regmap *regmap) { - return regmap_update_bits(regmap, MLX90632_REG_CONTROL, - MLX90632_CFG_PWR_MASK, - MLX90632_PWR_STATUS_SLEEP_STEP); + struct mlx90632_data *data =3D + iio_priv(dev_get_drvdata(regmap_get_device(regmap))); + s32 ret; + + if (data->powerstatus =3D=3D MLX90632_PWR_STATUS_SLEEP_STEP) + return 0; + + ret =3D regmap_write_bits(regmap, MLX90632_REG_CONTROL, MLX90632_CFG_PWR_= MASK, + MLX90632_PWR_STATUS_SLEEP_STEP); + if (ret < 0) + return ret; + + data->powerstatus =3D MLX90632_PWR_STATUS_SLEEP_STEP; + return ret; } =20 static s32 mlx90632_pwr_continuous(struct regmap *regmap) { - return regmap_update_bits(regmap, MLX90632_REG_CONTROL, - MLX90632_CFG_PWR_MASK, - MLX90632_PWR_STATUS_CONTINUOUS); + struct mlx90632_data *data =3D + iio_priv(dev_get_drvdata(regmap_get_device(regmap))); + s32 ret; + + if (data->powerstatus =3D=3D MLX90632_PWR_STATUS_CONTINUOUS) + return 0; + + ret =3D regmap_write_bits(regmap, MLX90632_REG_CONTROL, MLX90632_CFG_PWR_= MASK, + MLX90632_PWR_STATUS_CONTINUOUS); + if (ret < 0) + return ret; + + data->powerstatus =3D MLX90632_PWR_STATUS_CONTINUOUS; + return ret; } =20 /** @@ -219,6 +261,63 @@ static void mlx90632_reset_delay(void) usleep_range(150, 200); } =20 +static int mlx90632_get_measurement_time(struct regmap *regmap, u16 meas) +{ + unsigned int reg; + int ret; + + ret =3D regmap_read(regmap, meas, ®); + if (ret < 0) + return ret; + + return MLX90632_MEAS_MAX_TIME >> FIELD_GET(MLX90632_EE_RR, reg); +} + +static int mlx90632_calculate_dataset_ready_time(struct mlx90632_data *dat= a) +{ + unsigned int refresh_time; + int ret; + + if (data->mtyp =3D=3D MLX90632_MTYP_MEDICAL) { + ret =3D mlx90632_get_measurement_time(data->regmap, + MLX90632_EE_MEDICAL_MEAS1); + if (ret < 0) + return ret; + + refresh_time =3D ret; + + ret =3D mlx90632_get_measurement_time(data->regmap, + MLX90632_EE_MEDICAL_MEAS2); + if (ret < 0) + return ret; + + refresh_time +=3D ret; + } else { + ret =3D mlx90632_get_measurement_time(data->regmap, + MLX90632_EE_EXTENDED_MEAS1); + if (ret < 0) + return ret; + + refresh_time =3D ret; + + ret =3D mlx90632_get_measurement_time(data->regmap, + MLX90632_EE_EXTENDED_MEAS2); + if (ret < 0) + return ret; + + refresh_time +=3D ret; + + ret =3D mlx90632_get_measurement_time(data->regmap, + MLX90632_EE_EXTENDED_MEAS3); + if (ret < 0) + return ret; + + refresh_time +=3D ret; + } + + return refresh_time; +} + /** * mlx90632_perform_measurement() - Trigger and retrieve current measureme= nt cycle * @data: pointer to mlx90632_data object containing regmap information @@ -249,26 +348,75 @@ static int mlx90632_perform_measurement(struct mlx906= 32_data *data) return (reg_status & MLX90632_STAT_CYCLE_POS) >> 2; } =20 -static int mlx90632_set_meas_type(struct regmap *regmap, u8 type) +/** + * mlx90632_perform_measurement_burst() - Trigger and retrieve current mea= surement + * cycle in step sleep mode + * @data: pointer to mlx90632_data object containing regmap information + * + * Perform a measurement and return 2 as measurement cycle position report= ed + * by sensor. This is a blocking function for amount dependent on the sens= or + * refresh rate. + */ +static int mlx90632_perform_measurement_burst(struct mlx90632_data *data) { + unsigned int reg_status; int ret; =20 - if ((type !=3D MLX90632_MTYP_MEDICAL) && (type !=3D MLX90632_MTYP_EXTENDE= D)) - return -EINVAL; + ret =3D regmap_write_bits(data->regmap, MLX90632_REG_CONTROL, + MLX90632_CFG_SOB_MASK, MLX90632_CFG_SOB_MASK); + if (ret < 0) + return ret; =20 - ret =3D regmap_write(regmap, MLX90632_REG_I2C_CMD, MLX90632_RESET_CMD); + ret =3D mlx90632_calculate_dataset_ready_time(data); + if (ret < 0) + return ret; + + msleep(ret); /* Wait minimum time for dataset to be ready */ + + ret =3D regmap_read_poll_timeout(data->regmap, MLX90632_REG_STATUS, + reg_status, + (reg_status & MLX90632_STAT_BUSY) =3D=3D 0, + 10000, 100 * 10000); + if (ret < 0) { + dev_err(&data->client->dev, "data not ready"); + return -ETIMEDOUT; + } + + return 2; +} + +static int mlx90632_set_meas_type(struct mlx90632_data *data, u8 type) +{ + int current_powerstatus; + int ret; + + if (data->mtyp =3D=3D type) + return 0; + + current_powerstatus =3D data->powerstatus; + ret =3D mlx90632_pwr_continuous(data->regmap); + if (ret < 0) + return ret; + + ret =3D regmap_write(data->regmap, MLX90632_REG_I2C_CMD, MLX90632_RESET_C= MD); if (ret < 0) return ret; =20 mlx90632_reset_delay(); =20 - ret =3D regmap_write_bits(regmap, MLX90632_REG_CONTROL, + ret =3D regmap_update_bits(data->regmap, MLX90632_REG_CONTROL, (MLX90632_CFG_MTYP_MASK | MLX90632_CFG_PWR_MASK), (MLX90632_MTYP_STATUS(type) | MLX90632_PWR_STATUS_HALT)); if (ret < 0) return ret; =20 - return mlx90632_pwr_continuous(regmap); + data->mtyp =3D type; + data->powerstatus =3D MLX90632_PWR_STATUS_HALT; + + if (current_powerstatus =3D=3D MLX90632_PWR_STATUS_SLEEP_STEP) + return mlx90632_pwr_set_sleep_step(data->regmap); + + return mlx90632_pwr_continuous(data->regmap); } =20 static int mlx90632_channel_new_select(int perform_ret, uint8_t *channel_n= ew, @@ -355,11 +503,30 @@ static int mlx90632_read_all_channel(struct mlx90632_= data *data, s32 ret, measurement; =20 mutex_lock(&data->lock); - measurement =3D mlx90632_perform_measurement(data); - if (measurement < 0) { - ret =3D measurement; + ret =3D mlx90632_set_meas_type(data, MLX90632_MTYP_MEDICAL); + if (ret < 0) + goto read_unlock; + + switch (data->powerstatus) { + case MLX90632_PWR_STATUS_CONTINUOUS: + measurement =3D mlx90632_perform_measurement(data); + if (measurement < 0) { + ret =3D measurement; + goto read_unlock; + } + break; + case MLX90632_PWR_STATUS_SLEEP_STEP: + measurement =3D mlx90632_perform_measurement_burst(data); + if (measurement < 0) { + ret =3D measurement; + goto read_unlock; + } + break; + default: + ret =3D -EOPNOTSUPP; goto read_unlock; } + ret =3D mlx90632_read_ambient_raw(data->regmap, ambient_new_raw, ambient_old_raw); if (ret < 0) @@ -441,14 +608,20 @@ static int mlx90632_read_all_channel_extended(struct = mlx90632_data *data, s16 *o s32 ret, meas; =20 mutex_lock(&data->lock); - ret =3D mlx90632_set_meas_type(data->regmap, MLX90632_MTYP_EXTENDED); + ret =3D mlx90632_set_meas_type(data, MLX90632_MTYP_EXTENDED); if (ret < 0) goto read_unlock; =20 - ret =3D read_poll_timeout(mlx90632_perform_measurement, meas, meas =3D=3D= 19, - 50000, 800000, false, data); - if (ret !=3D 0) - goto read_unlock; + if (data->powerstatus =3D=3D MLX90632_PWR_STATUS_CONTINUOUS) { + ret =3D read_poll_timeout(mlx90632_perform_measurement, meas, meas =3D= =3D 19, + 50000, 800000, false, data); + if (ret) + goto read_unlock; + } else if (data->powerstatus =3D=3D MLX90632_PWR_STATUS_SLEEP_STEP) { + ret =3D mlx90632_perform_measurement_burst(data); + if (ret < 0) + goto read_unlock; + } =20 ret =3D mlx90632_read_object_raw_extended(data->regmap, object_new_raw); if (ret < 0) @@ -457,8 +630,6 @@ static int mlx90632_read_all_channel_extended(struct ml= x90632_data *data, s16 *o ret =3D mlx90632_read_ambient_raw_extended(data->regmap, ambient_new_raw,= ambient_old_raw); =20 read_unlock: - (void) mlx90632_set_meas_type(data->regmap, MLX90632_MTYP_MEDICAL); - mutex_unlock(&data->lock); return ret; } @@ -743,12 +914,47 @@ static int mlx90632_calc_ambient_dsp105(struct mlx906= 32_data *data, int *val) return ret; } =20 +/** + * mlx90632_pm_interraction_wakeup() - Measure time between user interacti= ons to change powermode + * @data: pointer to mlx90632_data object containing interaction_ts inform= ation + * + * Switch to continuous mode when interaction is faster than MLX90632_MEAS= _MAX_TIME. Update the + * interaction_ts for each function call with the jiffies to enable measur= ement between function + * calls. Initial value of the interaction_ts needs to be set before this = function call. + */ +static int mlx90632_pm_interraction_wakeup(struct mlx90632_data *data) +{ + unsigned long now; + int ret; + + now =3D jiffies; + if (time_in_range(now, data->interaction_ts, + data->interaction_ts + + msecs_to_jiffies(MLX90632_MEAS_MAX_TIME + 100))) { + if (data->powerstatus =3D=3D MLX90632_PWR_STATUS_SLEEP_STEP) { + ret =3D mlx90632_pwr_continuous(data->regmap); + if (ret < 0) + return ret; + } + } + + data->interaction_ts =3D now; + + return 0; +} + static int mlx90632_read_raw(struct iio_dev *indio_dev, struct iio_chan_spec const *channel, int *val, int *val2, long mask) { struct mlx90632_data *data =3D iio_priv(indio_dev); int ret; + int cr; + + pm_runtime_get_sync(&data->client->dev); + ret =3D mlx90632_pm_interraction_wakeup(data); + if (ret < 0) + goto mlx90632_read_raw_pm; =20 switch (mask) { case IIO_CHAN_INFO_PROCESSED: @@ -756,16 +962,22 @@ static int mlx90632_read_raw(struct iio_dev *indio_de= v, case IIO_MOD_TEMP_AMBIENT: ret =3D mlx90632_calc_ambient_dsp105(data, val); if (ret < 0) - return ret; - return IIO_VAL_INT; + goto mlx90632_read_raw_pm; + + ret =3D IIO_VAL_INT; + break; case IIO_MOD_TEMP_OBJECT: ret =3D mlx90632_calc_object_dsp105(data, val); if (ret < 0) - return ret; - return IIO_VAL_INT; + goto mlx90632_read_raw_pm; + + ret =3D IIO_VAL_INT; + break; default: - return -EINVAL; + ret =3D -EINVAL; + break; } + break; case IIO_CHAN_INFO_CALIBEMISSIVITY: if (data->emissivity =3D=3D 1000) { *val =3D 1; @@ -774,13 +986,21 @@ static int mlx90632_read_raw(struct iio_dev *indio_de= v, *val =3D 0; *val2 =3D data->emissivity * 1000; } - return IIO_VAL_INT_PLUS_MICRO; + ret =3D IIO_VAL_INT_PLUS_MICRO; + break; case IIO_CHAN_INFO_CALIBAMBIENT: *val =3D data->object_ambient_temperature; - return IIO_VAL_INT; + ret =3D IIO_VAL_INT; + break; default: - return -EINVAL; + ret =3D -EINVAL; + break; } + +mlx90632_read_raw_pm: + pm_runtime_mark_last_busy(&data->client->dev); + pm_runtime_put_autosuspend(&data->client->dev); + return ret; } =20 static int mlx90632_write_raw(struct iio_dev *indio_dev, @@ -902,6 +1122,7 @@ static int mlx90632_probe(struct i2c_client *client, mlx90632->client =3D client; mlx90632->regmap =3D regmap; mlx90632->mtyp =3D MLX90632_MTYP_MEDICAL; + mlx90632->powerstatus =3D MLX90632_PWR_STATUS_HALT; =20 mutex_init(&mlx90632->lock); indio_dev->name =3D id->name; @@ -961,16 +1182,19 @@ static int mlx90632_probe(struct i2c_client *client, =20 mlx90632->emissivity =3D 1000; mlx90632->object_ambient_temperature =3D 25000; /* 25 degrees milliCelsiu= s */ + mlx90632->interaction_ts =3D jiffies; /* Set initial value */ =20 - pm_runtime_disable(&client->dev); + pm_runtime_get_noresume(&client->dev); ret =3D pm_runtime_set_active(&client->dev); if (ret < 0) { mlx90632_sleep(mlx90632); return ret; } - pm_runtime_enable(&client->dev); + + devm_pm_runtime_enable(&client->dev); pm_runtime_set_autosuspend_delay(&client->dev, MLX90632_SLEEP_DELAY_MS); pm_runtime_use_autosuspend(&client->dev); + pm_runtime_put_autosuspend(&client->dev); =20 return iio_device_register(indio_dev); } @@ -978,16 +1202,8 @@ static int mlx90632_probe(struct i2c_client *client, static int mlx90632_remove(struct i2c_client *client) { struct iio_dev *indio_dev =3D i2c_get_clientdata(client); - struct mlx90632_data *data =3D iio_priv(indio_dev); =20 iio_device_unregister(indio_dev); - - pm_runtime_disable(&client->dev); - pm_runtime_set_suspended(&client->dev); - pm_runtime_put_noidle(&client->dev); - - mlx90632_sleep(data); - return 0; } =20 @@ -1003,30 +1219,47 @@ static const struct of_device_id mlx90632_of_match[= ] =3D { }; MODULE_DEVICE_TABLE(of, mlx90632_of_match); =20 -static int __maybe_unused mlx90632_pm_suspend(struct device *dev) +static int mlx90632_pm_suspend(struct device *dev) { - struct iio_dev *indio_dev =3D i2c_get_clientdata(to_i2c_client(dev)); - struct mlx90632_data *data =3D iio_priv(indio_dev); + struct mlx90632_data *data =3D iio_priv(dev_get_drvdata(dev)); + int ret; + + ret =3D mlx90632_pwr_set_sleep_step(data->regmap); + if (ret < 0) + return ret; + + ret =3D regulator_disable(data->regulator); + if (ret < 0) + dev_err(regmap_get_device(data->regmap), + "Failed to disable power regulator: %d\n", ret); =20 - return mlx90632_sleep(data); + return ret; } =20 -static int __maybe_unused mlx90632_pm_resume(struct device *dev) +static int mlx90632_pm_resume(struct device *dev) { - struct iio_dev *indio_dev =3D i2c_get_clientdata(to_i2c_client(dev)); - struct mlx90632_data *data =3D iio_priv(indio_dev); + struct mlx90632_data *data =3D iio_priv(dev_get_drvdata(dev)); =20 - return mlx90632_wakeup(data); + return mlx90632_enable_regulator(data); } =20 -static UNIVERSAL_DEV_PM_OPS(mlx90632_pm_ops, mlx90632_pm_suspend, - mlx90632_pm_resume, NULL); +static int mlx90632_pm_runtime_suspend(struct device *dev) +{ + struct mlx90632_data *data =3D iio_priv(dev_get_drvdata(dev)); + + return mlx90632_pwr_set_sleep_step(data->regmap); +} + +const struct dev_pm_ops mlx90632_pm_ops =3D { + SYSTEM_SLEEP_PM_OPS(mlx90632_pm_suspend, mlx90632_pm_resume) + RUNTIME_PM_OPS(mlx90632_pm_runtime_suspend, NULL, NULL) +}; =20 static struct i2c_driver mlx90632_driver =3D { .driver =3D { .name =3D "mlx90632", .of_match_table =3D mlx90632_of_match, - .pm =3D &mlx90632_pm_ops, + .pm =3D pm_ptr(&mlx90632_pm_ops), }, .probe =3D mlx90632_probe, .remove =3D mlx90632_remove, --=20 2.34.1 From nobody Fri Apr 3 03:48:29 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 4CC8EC54EE9 for ; Fri, 16 Sep 2022 11:00:02 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231501AbiIPK76 (ORCPT ); Fri, 16 Sep 2022 06:59:58 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:34548 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230265AbiIPK6b (ORCPT ); Fri, 16 Sep 2022 06:58:31 -0400 Received: from mail-ej1-x62e.google.com (mail-ej1-x62e.google.com [IPv6:2a00:1450:4864:20::62e]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id A43A9B72B3 for ; Fri, 16 Sep 2022 03:46:21 -0700 (PDT) Received: by mail-ej1-x62e.google.com with SMTP id lh5so6754098ejb.10 for ; Fri, 16 Sep 2022 03:46:21 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=melexis.com; s=google; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date; bh=JjQOhi02C9AekeMj3LYkyFRWD9UJCpdDKlRtlWP9Usk=; b=omNLplgDHrKq1w8UB4XTWEQZ2Oc3qXewRfnZafK5pYTGg8Jn6oTQ06R2ncNeMCxMsg P+zN81GPV7k5bl12cevmgqi4NMuoZqPybBxFXVgqqHLqoftCz9oBMaGshHVEhcEpDruu FiOEK3ZgcHW5maYNdNTAmrT9Ua0dqKcyEN5kSi+FYIopPi7np874/4cOKnTKTlNKKQm8 e57cW7yocKw3DNTwsAUuWKOHl6AvVBtnkapSK9+ziELauchMj/nlnQWrxEf9rL6aLROM JKjAnB9UQpyZSIAIMj9lLVF828ci4h9YKdjyS8obTiOQg2e0PQ4IuZZ9qYtWbyKHrOwf JFqA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date; bh=JjQOhi02C9AekeMj3LYkyFRWD9UJCpdDKlRtlWP9Usk=; b=lg9MeKx2pm7auUHMpKEaS1ROqWSG4zNTy9Gmx2HUmojuAxvTUoyIMn8o0vHQgJ67tx FHBM8Qt9WhA3IjxWnvDM00+g/h0jzNxt3+MC2FQWo0JyQ5Xu6eTRbsmYzJjAUj474oWT yv+Q+B0jBCC2ByXBovPGlUACDiuR19JRSopO2kE2yp6m7eh3ZFLaBbo6T5dLrNjKCSTD qC/eGQmXTp8TTu2IdBNjRVEmb0p/yJwkjn51nvA1fcHI76uGxfSLjmIEQjfR6R/nxM3O zdV5bYvqtZogz/OSC0fBGYmTBBRaIGlj0kNL6Aq1INx9pf9YRy/GQtzsxamQowi4vOxW WyAw== X-Gm-Message-State: ACrzQf0ieZ6TTX4zPsyX1ZhAWTw12bh7qeoX8qCyXbepoVJv0mnq4aaq zQvJs6khkrk8H1HLVyolkxwKtQ== X-Google-Smtp-Source: AMsMyM7UMdzea5at4yQKZx2yvom3LFjH7jTUideb9x9xxxdGoJLea3GoFR0RXxKAYkBW3iEpWq4wSQ== X-Received: by 2002:a17:907:a4c:b0:77b:ba98:d3e with SMTP id be12-20020a1709070a4c00b0077bba980d3emr3192132ejc.13.1663325178383; Fri, 16 Sep 2022 03:46:18 -0700 (PDT) Received: from cmo-ThinkPad-T495.telenet.be (ptr-4xh0y3vvunoijj2gi1k.18120a2.ip6.access.telenet.be. [2a02:1810:a44c:8f00:cef1:93b4:727b:dd58]) by smtp.gmail.com with ESMTPSA id fi24-20020a1709073ad800b0072af4af2f46sm10148434ejc.74.2022.09.16.03.46.17 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 16 Sep 2022 03:46:17 -0700 (PDT) From: cmo@melexis.com To: Jonathan Cameron Cc: linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org, Andy Shevchenko , Crt Mori Subject: [PATCH v4 2/3] iio: temperature: mlx90632 Read sampling frequency Date: Fri, 16 Sep 2022 12:45:51 +0200 Message-Id: <651b3659f574f9b17de34b774f94c895cbd00dfd.1663324968.git.cmo@melexis.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" From: Crt Mori Allow users to read sensor sampling frequency to better plan the application measurement requests. Signed-off-by: Crt Mori --- drivers/iio/temperature/mlx90632.c | 59 ++++++++++++++++++++++++++++++ 1 file changed, 59 insertions(+) diff --git a/drivers/iio/temperature/mlx90632.c b/drivers/iio/temperature/m= lx90632.c index 80497d9bc4e9..e7946e3ed4fb 100644 --- a/drivers/iio/temperature/mlx90632.c +++ b/drivers/iio/temperature/mlx90632.c @@ -81,6 +81,9 @@ #define MLX90632_PWR_STATUS_CONTINUOUS MLX90632_PWR_STATUS(3) /* continuou= s */ =20 #define MLX90632_EE_RR GENMASK(10, 8) /* Only Refresh Rate bits */ +#define MLX90632_REFRESH_RATE(ee_val) FIELD_GET(MLX90632_EE_RR, ee_val) + /* Extract Refresh Rate from ee register */ +#define MLX90632_REFRESH_RATE_STATUS(refresh_rate) (refresh_rate << 8) =20 /* Measurement types */ #define MLX90632_MTYP_MEDICAL 0 @@ -914,6 +917,32 @@ static int mlx90632_calc_ambient_dsp105(struct mlx9063= 2_data *data, int *val) return ret; } =20 +static int mlx90632_get_refresh_rate(struct mlx90632_data *data, + int *refresh_rate) +{ + unsigned int meas1; + int ret; + + ret =3D regmap_read(data->regmap, MLX90632_EE_MEDICAL_MEAS1, &meas1); + if (ret < 0) + return ret; + + *refresh_rate =3D MLX90632_REFRESH_RATE(meas1); + + return ret; +} + +static const int mlx90632_freqs[][2] =3D { + {0, 500000}, + {1, 0}, + {2, 0}, + {4, 0}, + {8, 0}, + {16, 0}, + {32, 0}, + {64, 0} +}; + /** * mlx90632_pm_interraction_wakeup() - Measure time between user interacti= ons to change powermode * @data: pointer to mlx90632_data object containing interaction_ts inform= ation @@ -992,6 +1021,15 @@ static int mlx90632_read_raw(struct iio_dev *indio_de= v, *val =3D data->object_ambient_temperature; ret =3D IIO_VAL_INT; break; + case IIO_CHAN_INFO_SAMP_FREQ: + ret =3D mlx90632_get_refresh_rate(data, &cr); + if (ret < 0) + goto mlx90632_read_raw_pm; + + *val =3D mlx90632_freqs[cr][0]; + *val2 =3D mlx90632_freqs[cr][1]; + ret =3D IIO_VAL_INT_PLUS_MICRO; + break; default: ret =3D -EINVAL; break; @@ -1025,12 +1063,30 @@ static int mlx90632_write_raw(struct iio_dev *indio= _dev, } } =20 +static int mlx90632_read_avail(struct iio_dev *indio_dev, + struct iio_chan_spec const *chan, + const int **vals, int *type, int *length, + long mask) +{ + switch (mask) { + case IIO_CHAN_INFO_SAMP_FREQ: + *vals =3D (int *)mlx90632_freqs; + *type =3D IIO_VAL_INT_PLUS_MICRO; + *length =3D 2 * ARRAY_SIZE(mlx90632_freqs); + return IIO_AVAIL_LIST; + default: + return -EINVAL; + } +} + static const struct iio_chan_spec mlx90632_channels[] =3D { { .type =3D IIO_TEMP, .modified =3D 1, .channel2 =3D IIO_MOD_TEMP_AMBIENT, .info_mask_separate =3D BIT(IIO_CHAN_INFO_PROCESSED), + .info_mask_shared_by_all =3D BIT(IIO_CHAN_INFO_SAMP_FREQ), + .info_mask_shared_by_all_available =3D BIT(IIO_CHAN_INFO_SAMP_FREQ), }, { .type =3D IIO_TEMP, @@ -1038,12 +1094,15 @@ static const struct iio_chan_spec mlx90632_channels= [] =3D { .channel2 =3D IIO_MOD_TEMP_OBJECT, .info_mask_separate =3D BIT(IIO_CHAN_INFO_PROCESSED) | BIT(IIO_CHAN_INFO_CALIBEMISSIVITY) | BIT(IIO_CHAN_INFO_CALIBAMBIENT), + .info_mask_shared_by_all =3D BIT(IIO_CHAN_INFO_SAMP_FREQ), + .info_mask_shared_by_all_available =3D BIT(IIO_CHAN_INFO_SAMP_FREQ), }, }; =20 static const struct iio_info mlx90632_info =3D { .read_raw =3D mlx90632_read_raw, .write_raw =3D mlx90632_write_raw, + .read_avail =3D mlx90632_read_avail, }; =20 static int mlx90632_sleep(struct mlx90632_data *data) --=20 2.34.1 From nobody Fri Apr 3 03:48:29 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 50CF1C54EE9 for ; Fri, 16 Sep 2022 10:59:47 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230471AbiIPK7n (ORCPT ); Fri, 16 Sep 2022 06:59:43 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:34750 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232066AbiIPK6q (ORCPT ); Fri, 16 Sep 2022 06:58:46 -0400 Received: from mail-ej1-x62e.google.com (mail-ej1-x62e.google.com [IPv6:2a00:1450:4864:20::62e]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id B11CFB72BA for ; Fri, 16 Sep 2022 03:46:23 -0700 (PDT) Received: by mail-ej1-x62e.google.com with SMTP id sb3so18871939ejb.9 for ; Fri, 16 Sep 2022 03:46:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=melexis.com; s=google; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date; bh=2woE7qGhu5XZvBsvA65FmG95p8Vjop1O1RfajPi17so=; b=CJsgnRZy4ibGp/qVozAYH9YQGUi0GqZo6jXKicrm/J2EfkbytaolZ4qicOmkggVQVx 5r0+ImXbwTjrZ7XzdkUDc+OToUw/D7A1xFVey5hMSunj7WXGnBnetm4XTneW95dnE03z jeJt54N0GkBHtDiOAaP3xx1JiopECadgT7pJFLDbtgFFEANKJTxyEGFRBiQTqtgXr2lC hIO5/vNaOggqC8gTV851ENeimBKxFjggijeFZUX3WWpObPH8Ycbf0XJZXr4VAUgNLZ4m psDIImKIQCCKAwNwBZnJxyiU0jrTTTqQhU7wpTO4mp6GlpVjY2YOaFWc4vPBexOAvBPd 0W6A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date; bh=2woE7qGhu5XZvBsvA65FmG95p8Vjop1O1RfajPi17so=; b=vZ5xIkYKA8hUKdmNsNiXFDlGrtnGc824iJtjRje9za9Yz9oCUNVk95xPwSmzf5vPw+ SY4eZaAOhNlvSvboB8yKgzNa5voKmnYkDy2O5wLK315ykeKpsvt2ousmnpEifUlV+lQo Y2208+TnXf2izLn0I5M932xSkuQbqZOvirJtDhPAFECpU+nU3i2+/70RRbCrOp8OfL/D nWFuESX8M5w3jBoZJky5551sj0JRMAYzwXIbrsr/f8PGg59hATajWpSMDCloVoscrulW bhp9oYPaIn15g9M6iUen3lnEETzB9ameNT1+JZGFTLzIqzfCjaKxgOMy1ZbsR2QOAGtg 5LOw== X-Gm-Message-State: ACrzQf1HiqKJj74/f3KNGc9MJMqylEtqhXKLY/XHFKFbLmwqPDCnym/h oG2Nz6NpZdpmGFoqkq9YdTUYxA== X-Google-Smtp-Source: AMsMyM7pt8z4iegjlU5zDlSEhSgDb5SGHgbYGRaGNvAYBizxWdg+NHp0ErlcQLU8n9tXtZUpZ5ZHtw== X-Received: by 2002:a17:906:216:b0:711:f623:8bb0 with SMTP id 22-20020a170906021600b00711f6238bb0mr3246379ejd.174.1663325181425; Fri, 16 Sep 2022 03:46:21 -0700 (PDT) Received: from cmo-ThinkPad-T495.telenet.be (ptr-4xh0y3vvunoijj2gi1k.18120a2.ip6.access.telenet.be. [2a02:1810:a44c:8f00:cef1:93b4:727b:dd58]) by smtp.gmail.com with ESMTPSA id fi24-20020a1709073ad800b0072af4af2f46sm10148434ejc.74.2022.09.16.03.46.20 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 16 Sep 2022 03:46:20 -0700 (PDT) From: cmo@melexis.com To: Jonathan Cameron Cc: linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org, Andy Shevchenko , Crt Mori Subject: [PATCH v4 3/3] iio: temperature: mlx90632 Change return value of sensor measurement channel Date: Fri, 16 Sep 2022 12:45:52 +0200 Message-Id: <65031a769afcb494665bfa6b528cf4fc30a30c8d.1663324968.git.cmo@melexis.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" From: Crt Mori The current EINVAL value is more applicable to embedded library, where user can actually put the fixed value to the sensor. In case of the driver if the value of the channel is invalid it is better in inform userspace that Channel was out of range as that implies more to internal driver error than invalid input. It also makes for easier debugging of where the error comes from during the development. Signed-off-by: Crt Mori --- drivers/iio/temperature/mlx90632.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/iio/temperature/mlx90632.c b/drivers/iio/temperature/m= lx90632.c index e7946e3ed4fb..262b42883fcc 100644 --- a/drivers/iio/temperature/mlx90632.c +++ b/drivers/iio/temperature/mlx90632.c @@ -435,7 +435,7 @@ static int mlx90632_channel_new_select(int perform_ret,= uint8_t *channel_new, *channel_old =3D 1; break; default: - return -EINVAL; + return -ECHRNG; } =20 return 0; --=20 2.34.1