linux-kernel - Re: [PATCH V11 2/9] dt-bindings: mfd: pm8008: Change the address cells

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:   Wed, 11 May 2022 09:11:22 -0500
From:   Rob Herring <robh@...nel.org>
To:     Satya Priya <quic_c_skakit@...cinc.com>
Cc:     Bjorn Andersson <bjorn.andersson@...aro.org>,
        Lee Jones <lee.jones@...aro.org>,
        Liam Girdwood <lgirdwood@...il.com>,
        Mark Brown <broonie@...nel.org>, linux-arm-msm@...r.kernel.org,
        devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
        swboyd@...omium.org, quic_collinsd@...cinc.com,
        quic_subbaram@...cinc.com, quic_jprakash@...cinc.com
Subject: Re: [PATCH V11 2/9] dt-bindings: mfd: pm8008: Change the address
 cells

On Thu, May 05, 2022 at 02:55:32PM +0530, Satya Priya wrote:
> Change the address cells as '2' so that the first cell
> describes the i2c address offset of the clients. 
> This helps us to define the child nodes of all
> clients under the same parent mfd node, instead of
> adding separate mfd DT nodes.
> 
> Change the gpios reg value accordingly.
> 
> Signed-off-by: Satya Priya <quic_c_skakit@...cinc.com>
> ---
> Changes in V11:
>  - New patch added from V11.
> 
>  Documentation/devicetree/bindings/mfd/qcom,pm8008.yaml | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/mfd/qcom,pm8008.yaml b/Documentation/devicetree/bindings/mfd/qcom,pm8008.yaml
> index a89649c..a41618e 100644
> --- a/Documentation/devicetree/bindings/mfd/qcom,pm8008.yaml
> +++ b/Documentation/devicetree/bindings/mfd/qcom,pm8008.yaml
> @@ -39,7 +39,7 @@ properties:
>    interrupt-controller: true
>  
>    "#address-cells":
> -    const: 1
> +    const: 2
>  
>    "#size-cells":
>      const: 0
> @@ -48,7 +48,7 @@ properties:
>      maxItems: 1
>  
>  patternProperties:
> -  "^gpio@[0-9a-f]+$":
> +  "^gpio@[0],[0-9a-f]+$":

^gpio@0,[0-9a-f]+$

>      type: object
>  
>      description: |
> @@ -61,7 +61,7 @@ patternProperties:
>            - const: qcom,spmi-gpio
>  
>        reg:
> -        description: Peripheral address of one of the two GPIO peripherals.
> +        description: Peripheral offset and address of one of the two GPIO peripherals.
>          maxItems: 1
>  
>        gpio-controller: true
> @@ -110,7 +110,7 @@ examples:
>        pm8008i@8 {
>          compatible = "qcom,pm8008";
>          reg = <0x8>;
> -        #address-cells = <1>;
> +        #address-cells = <2>;
>          #size-cells = <0>;
>          interrupt-controller;
>          #interrupt-cells = <2>;
> @@ -120,9 +120,9 @@ examples:
>  
>          reset-gpios = <&pm8350c_gpios 4 GPIO_ACTIVE_LOW>;
>  
> -        pm8008_gpios: gpio@...0 {
> +        pm8008_gpios: gpio@0,c000 {
>            compatible = "qcom,pm8008-gpio", "qcom,spmi-gpio";
> -          reg = <0xc000>;
> +          reg = <0x0 0xc000>;
>            gpio-controller;
>            gpio-ranges = <&pm8008_gpios 0 0 2>;
>            #gpio-cells = <2>;
> -- 
> 2.7.4
> 
>