linux-kernel - Re: [PATCH V3 7/9] dt-bindings: i2c: xiic: Add 'xlnx,axi-iic-2.1' to

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]

Message-ID: <c15b377a-a07e-c43e-55f2-c0f6dd8cd75f@kernel.org>
Date:   Tue, 24 Jan 2023 12:33:20 +0100
From:   Krzysztof Kozlowski <krzk@...nel.org>
To:     Manikanta Guntupalli <manikanta.guntupalli@....com>,
        michal.simek@...inx.com, michal.simek@....com,
        linux-arm-kernel@...ts.infradead.org, linux-i2c@...r.kernel.org,
        linux-kernel@...r.kernel.org
Cc:     git@....com, srinivas.goud@....com, shubhrajyoti.datta@....com,
        manion05gk@...il.com,
        Raviteja Narayanam <raviteja.narayanam@...inx.com>
Subject: Re: [PATCH V3 7/9] dt-bindings: i2c: xiic: Add 'xlnx,axi-iic-2.1' to

On 24/01/2023 06:04, Manikanta Guntupalli wrote:

Please use scripts/get_maintainers.pl to get a list of necessary people
and lists to CC.  It might happen, that command when run on an older
kernel, gives you outdated entries.  Therefore please be sure you base
your patches on recent Linux kernel.

You missed several entries, so this also will mean patch won't get tested.


> From: Raviteja Narayanam <raviteja.narayanam@...inx.com>
> 
> compatible

Messed up subject and commit msg.

> 
> Added the xilinx I2C new version 'xlnx,axi-iic-2.1' string to compatible
> Added clock-frequency as optional property.
> 
> Signed-off-by: Raviteja Narayanam <raviteja.narayanam@...inx.com>
> Signed-off-by: Manikanta Guntupalli <manikanta.guntupalli@....com>
> Acked-by: Michal Simek <michal.simek@....com>
> ---
>  .../devicetree/bindings/i2c/xlnx,xps-iic-2.00.a.yaml       | 7 ++++++-
>  1 file changed, 6 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/i2c/xlnx,xps-iic-2.00.a.yaml b/Documentation/devicetree/bindings/i2c/xlnx,xps-iic-2.00.a.yaml
> index 8d241a703d85..42488a67260c 100644
> --- a/Documentation/devicetree/bindings/i2c/xlnx,xps-iic-2.00.a.yaml
> +++ b/Documentation/devicetree/bindings/i2c/xlnx,xps-iic-2.00.a.yaml
> @@ -14,7 +14,9 @@ allOf:
>  
>  properties:
>    compatible:
> -    const: xlnx,xps-iic-2.00.a
> +    enum:
> +      - xlnx,xps-iic-2.00.a
> +      - xlnx,axi-iic-2.1
>  
>    reg:
>      maxItems: 1
> @@ -36,6 +38,9 @@ required:
>    - interrupts
>    - clocks
>  
> +optional:
> +  - clock-frequency

Nope. Just test your bindings...

Does not look like you tested the bindings. Please run `make
dt_binding_check` (see
Documentation/devicetree/bindings/writing-schema.rst for instructions).

> +
>  unevaluatedProperties: false
>  
>  examples:

Best regards,
Krzysztof