[PATCH v2 1/1] dt-bindings: iio: imx8qxp-adc: add missing vref-supply
Bough Chen
haibo.chen at nxp.com
Mon Apr 24 20:29:17 PDT 2023
> -----Original Message-----
> From: Alexander Stein <alexander.stein at ew.tq-group.com>
> Sent: 2023年4月24日 17:23
> To: Cai Huoqing <cai.huoqing at linux.dev>; Bough Chen <haibo.chen at nxp.com>;
> Jonathan Cameron <jic23 at kernel.org>; Lars-Peter Clausen <lars at metafoo.de>;
> Rob Herring <robh+dt at kernel.org>; Krzysztof Kozlowski
> <krzysztof.kozlowski+dt at linaro.org>; Shawn Guo <shawnguo at kernel.org>;
> Sascha Hauer <s.hauer at pengutronix.de>; Fabio Estevam
> <festevam at gmail.com>
> Cc: Alexander Stein <alexander.stein at ew.tq-group.com>; Pengutronix Kernel
> Team <kernel at pengutronix.de>; dl-linux-imx <linux-imx at nxp.com>;
> linux-iio at vger.kernel.org; devicetree at vger.kernel.org;
> linux-arm-kernel at lists.infradead.org; Marco Felsch <m.felsch at pengutronix.de>
> Subject: [PATCH v2 1/1] dt-bindings: iio: imx8qxp-adc: add missing vref-supply
>
> Although this property is used right now for IIO_CHAN_INFO_SCALE, this ADC
> has two internal reference voltages, which the driver currently doesn't make use
> of.
>
> Fixes: db73419d8c06 ("dt-bindings: iio: adc: Add binding documentation for NXP
> IMX8QXP ADC")
> Signed-off-by: Alexander Stein <alexander.stein at ew.tq-group.com>
Thanks!
Reviewed-by: Haibo Chen <haibo.chen at nxp.com>
> ---
> Thanks for your inputs. This improved descritpion should make it clear that this
> property is only about the external reference voltage, not about the optional,
> internal voltages.
>
> Changes in v2:
> * Improved commit message subject as suggested
> * Add hint about feature flag regarding multiple, internal, reference
> voltages
>
> .../devicetree/bindings/iio/adc/nxp,imx8qxp-adc.yaml | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/iio/adc/nxp,imx8qxp-adc.yaml
> b/Documentation/devicetree/bindings/iio/adc/nxp,imx8qxp-adc.yaml
> index 63369ba388e4..878e67054d7c 100644
> --- a/Documentation/devicetree/bindings/iio/adc/nxp,imx8qxp-adc.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/nxp,imx8qxp-adc.yaml
> @@ -39,6 +39,12 @@ properties:
> power-domains:
> maxItems: 1
>
> + vref-supply:
> + description: |
> + External ADC reference voltage supply on VREFH pad. If VERID[MVI] is
> + set, there are additional, internal reference voltages selectable.
> + VREFH1 is always from VREFH pad.
> +
> "#io-channel-cells":
> const: 1
>
> @@ -72,6 +78,7 @@ examples:
> assigned-clocks = <&clk IMX_SC_R_ADC_0>;
> assigned-clock-rates = <24000000>;
> power-domains = <&pd IMX_SC_R_ADC_0>;
> + vref-supply = <®_1v8>;
> #io-channel-cells = <1>;
> };
> };
> --
> 2.34.1
More information about the linux-arm-kernel
mailing list