lkml.org 
[lkml]   [2023]   [Jan]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH V3 7/9] dt-bindings: i2c: xiic: Add 'xlnx,axi-iic-2.1' to
From
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@xilinx.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@xilinx.com>
> Signed-off-by: Manikanta Guntupalli <manikanta.guntupalli@amd.com>
> Acked-by: Michal Simek <michal.simek@amd.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

\
 
 \ /
  Last update: 2023-03-26 23:53    [W:0.105 / U:0.568 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site