| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-Id: <20210607144718.1724413-1-liambeguin@gmail.com>
Date: Mon, 7 Jun 2021 10:47:10 -0400
From: Liam Beguin <liambeguin@...il.com>
To: liambeguin@...il.com, peda@...ntia.se, jic23@...nel.org,
lars@...afoo.de, pmeerw@...erw.net
Cc: linux-kernel@...r.kernel.org, linux-iio@...r.kernel.org,
devicetree@...r.kernel.org, robh+dt@...nel.org
Subject: [PATCH v2 0/8] iio: afe: add temperature rescaling support
From: Liam Beguin <lvb@...hos.com>
Add temperature rescaling support to the IIO Analog Front End driver.
This series includes minor bugfixes and adds support for a generic
temperature front end circuit.
As Peter suggested in v1, if the upstream channel has an offset, the
rescaler will default to using the processed value. This was done to
avoid having to process all supported offset an scale type combinations.
At first I tried to use iio_convert_raw_to_processed() to get more
precision out of processed values but ran into issues when one of my
ADCs didn't provide a scale. I tried to address this in the first two
patches.
When adding offset support to iio-rescale, I also noticed that
iio_read_channel_processed() assumes that the offset is always an
integer which I tried to address in the third patch without breaking
valid implicit truncations.
Related to: https://patchwork.kernel.org/project/linux-iio/list/?series=483087
Changes since v1:
- rebase on latest iio `testing` branch
- also apply consumer scale on integer channel scale types
- don't break implicit truncation in processed channel offset
calculation
- drop temperature AFE flavors in favor of a simpler generic
implementation
Thanks for your time
Liam Beguin (8):
iio: inkern: apply consumer scale on IIO_VAL_INT cases
iio: inkern: apply consumer scale when no channel scale is available
iio: inkern: error out on unsupported offset type
iio: inkern: return valid type on raw to processed conversion
iio: afe: rescale: add upstream offset support
iio: afe: rescale: add offset support
iio: afe: rescale: add temperature sensor support
dt-bindings: iio: afe: add binding for temperature-sense-amplifier
.../iio/afe/temperature-sense-amplifier.yaml | 57 +++++++++++++++++++
MAINTAINERS | 1 +
drivers/iio/afe/iio-rescale.c | 39 ++++++++++++-
drivers/iio/inkern.c | 46 +++++++++++----
4 files changed, 131 insertions(+), 12 deletions(-)
create mode 100644 Documentation/devicetree/bindings/iio/afe/temperature-sense-amplifier.yaml
Range-diff against v1:
1: 36f038f93537 < -: ------------ iio: inkern: always apply scale requested by consumer
2: fd3e9a69841a < -: ------------ iio: inkern: error out on unsupported offset type
3: 91c473be7098 < -: ------------ iio: afe: rescale: use core to get processed value
4: 1097973f3bf7 < -: ------------ iio: afe: rescale: add offset support
5: 786badf92421 < -: ------------ iio: afe: rescale: add support for temperature sensors
6: 0cae8abf6a06 < -: ------------ dt-bindings: iio: afe: update MAINTAINERS file
7: e806c73122f8 < -: ------------ dt-bindings: iio: afe: add binding for temperature-sense-rtd
8: f156b16ba01a < -: ------------ dt-bindings: iio: afe: add binding for temperature-sense-current
-: ------------ > 1: 8ebae9e606a2 iio: inkern: apply consumer scale on IIO_VAL_INT cases
-: ------------ > 2: 4d6e4d772f94 iio: inkern: apply consumer scale when no channel scale is available
-: ------------ > 3: dd26ddb49658 iio: inkern: error out on unsupported offset type
-: ------------ > 4: 56e9e00cab9d iio: inkern: return valid type on raw to processed conversion
-: ------------ > 5: d86fabc43985 iio: afe: rescale: add upstream offset support
-: ------------ > 6: 332fdb2d59ae iio: afe: rescale: add offset support
-: ------------ > 7: 3eabc81fb9aa iio: afe: rescale: add temperature sensor support
9: 9bfdfe7d86b7 ! 8: 37980da320b2 dt-bindings: iio: afe: add binding for temperature-sense-amplifier
@@ Commit message
dt-bindings: iio: afe: add binding for temperature-sense-amplifier
An ADC is often used to measure other quantities indirectly. This
- binding describe one cases, the measurement of a temperature through a
- voltage sense amplifier such as the LTC2997.
+ binding describe such a use case, the measurement of a temperature
+ through an analog front end connected to a voltage channel.
Signed-off-by: Liam Beguin <lvb@...hos.com>
@@ Documentation/devicetree/bindings/iio/afe/temperature-sense-amplifier.yaml (new)
+ - Liam Beguin <lvb@...hos.com>
+
+description: |
-+ When an io-channel measures the output voltage of a temperature IC such as
-+ the LTC2997, the interesting measurement is almost always the corresponding
++ When an io-channel measures the output voltage of a temperature analog front
++ end such as an RTD (resistance thermometer) or a temperature to current
++ sensor, the interesting measurement is almost always the corresponding
+ temperature, not the voltage output. This binding describes such a circuit.
+
+properties:
@@ Documentation/devicetree/bindings/iio/afe/temperature-sense-amplifier.yaml (new)
+ '#io-channel-cells':
+ const: 1
+
-+ alpha-micro-volts-per-degree:
-+ description: |
-+ Output voltage gain of the temperature IC.
++ sense-gain-mult:
++ $ref: /schemas/types.yaml#/definitions/uint32
++ description: Amplifier gain multiplier. The default is <1>.
+
-+ use-kelvin-scale:
-+ type: boolean
-+ description: |
-+ Boolean indicating if alpha uses Kelvin degrees instead of Celsius.
++ sense-gain-div:
++ $ref: /schemas/types.yaml#/definitions/uint32
++ description: Amplifier gain divider. The default is <1>.
++
++ sense-offset-millicelsius:
++ description: Amplifier offset. The default is <0>.
+
+additionalProperties: false
+required:
+ - compatible
+ - io-channels
-+ - alpha-micro-volts-per-degree
+
+examples:
+ - |
-+ znq_temp: iio-rescale0 {
++ pt1000_1: temperature-sensor {
+ compatible = "temperature-sense-amplifier";
+ #io-channel-cells = <1>;
+ io-channels = <&temp_adc 3>;
+
-+ use-kelvin-scale;
-+ alpha-micro-volts-per-degree = <4000>;
++ sense-gain-mult = <1000000>;
++ sense-gain-div = <3908>;
++ sense-offset-millicelsius = <(-255885)>;
+ };
-+
+...
## MAINTAINERS ##
@@ MAINTAINERS: L: linux-iio@...r.kernel.org
F: Documentation/devicetree/bindings/iio/afe/current-sense-amplifier.yaml
F: Documentation/devicetree/bindings/iio/afe/current-sense-shunt.yaml
+F: Documentation/devicetree/bindings/iio/afe/temperature-sense-amplifier.yaml
- F: Documentation/devicetree/bindings/iio/afe/temperature-sense-current.yaml
- F: Documentation/devicetree/bindings/iio/afe/temperature-sense-rtd.yaml
F: Documentation/devicetree/bindings/iio/afe/voltage-divider.yaml
+ F: drivers/iio/afe/iio-rescale.c
+
base-commit: 41340965b4f8055f975f73e1e3d23eff8038f013
--
2.30.1.489.g328c10930387
Powered by blists - more mailing lists