linux-kernel - Re: [PATCH] dt-bindings: iio/adc: qcom,spmi-iadc: Fix 'reg' property in example

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Sun, 30 Jan 2022 12:17:21 +0000
From:   Jonathan Cameron <jic23@...nel.org>
To:     Rob Herring <robh@...nel.org>
Cc:     Lars-Peter Clausen <lars@...afoo.de>, linux-iio@...r.kernel.org,
        devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] dt-bindings: iio/adc: qcom,spmi-iadc: Fix 'reg'
 property in example

On Wed, 26 Jan 2022 17:12:17 -0600
Rob Herring <robh@...nel.org> wrote:

> The QCom SPMI PMIC child nodes are defined to have a single address cell,
> but the example has an erroneous size cell. Remove it.
> 
> Signed-off-by: Rob Herring <robh@...nel.org>
Probably should have a fixes tag...
Fixes: a4e6bf69418c ("dt-bindings:iio:adc:qcom,spmi-iadc: txt to yaml format conversion.")

Given it's been there a long time I'm guessing no great rush on this so I'll
queue it up for then next merge window.

Shout if you want to pick it up sooner.

Thanks,

Jonathan

> ---
>  Documentation/devicetree/bindings/iio/adc/qcom,spmi-iadc.yaml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/iio/adc/qcom,spmi-iadc.yaml b/Documentation/devicetree/bindings/iio/adc/qcom,spmi-iadc.yaml
> index 27e3108661c0..2a94db688830 100644
> --- a/Documentation/devicetree/bindings/iio/adc/qcom,spmi-iadc.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/qcom,spmi-iadc.yaml
> @@ -51,7 +51,7 @@ examples:
>          #size-cells = <0>;
>          pmic_iadc: adc@...0 {
>              compatible = "qcom,spmi-iadc";
> -            reg = <0x3600 0x100>;
> +            reg = <0x3600>;
>              interrupts = <0x0 0x36 0x0 IRQ_TYPE_EDGE_RISING>;
>              qcom,external-resistor-micro-ohms = <10000>;
>              #io-channel-cells  = <1>;