linux-kernel - Re: [PATCH v6 3/6] dt-bindings: mfd: Add Realtek RTL9300 switch peripherals

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <5o77wkohvujnfnm4xm73b65gpx5by7chhyhdbuw3dkpota53us@5x6jlcabjoes>
Date: Wed, 16 Oct 2024 08:41:48 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Chris Packham <chris.packham@...iedtelesis.co.nz>
Cc: andi.shyti@...nel.org, robh@...nel.org, krzk+dt@...nel.org, 
	conor+dt@...nel.org, lee@...nel.org, sre@...nel.org, tsbogend@...ha.franken.de, 
	markus.stockhausen@....de, linux-i2c@...r.kernel.org, devicetree@...r.kernel.org, 
	linux-kernel@...r.kernel.org, linux-pm@...r.kernel.org, linux-mips@...r.kernel.org
Subject: Re: [PATCH v6 3/6] dt-bindings: mfd: Add Realtek RTL9300 switch
 peripherals

On Wed, Oct 16, 2024 at 11:59:45AM +1300, Chris Packham wrote:
> +properties:
> +  compatible:
> +    oneOf:
> +      - items:
> +          - enum:
> +              - realtek,rtl9302b-i2c
> +              - realtek,rtl9302c-i2c
> +              - realtek,rtl9303-i2c
> +          - const: realtek,rtl9301-i2c
> +      - const: realtek,rtl9301-i2c
> +
> +  reg:
> +    description: Register offset and size this I2C controller.
> +
> +  "#address-cells":
> +    const: 1
> +
> +  "#size-cells":
> +    const: 0
> +
> +patternProperties:
> +  '^i2c@[0-7]$':
> +    $ref: /schemas/i2c/i2c-controller.yaml
> +    unevaluatedProperties: false
> +
> +    properties:
> +      reg:
> +        description: The SDA pin associated with the I2C bus.
> +        maxItems: 1
> +
> +    required:
> +      - reg
> +
> +required:
> +  - compatible
> +  - reg
> +
> +unevaluatedProperties: false

This has to be: additionalProperties: false

> +
> +examples:
> +  - |
> +    i2c@36c {
> +      compatible = "realtek,rtl9301-i2c";
> +      reg = <0x36c 0x14>;
> +      #address-cells = <1>;
> +      #size-cells = <0>;
> +
> +      i2c@0 {
> +        reg = <0>;
> +        #address-cells = <1>;
> +        #size-cells = <0>;
> +        gpio@20 {
> +          compatible = "nxp,pca9555";
> +          gpio-controller;
> +          #gpio-cells = <2>;
> +          reg = <0x20>;
> +        };
> +      };
> +
> +      i2c@2 {
> +        reg = <2>;
> +        #address-cells = <1>;
> +        #size-cells = <0>;
> +        gpio@20 {
> +          compatible = "nxp,pca9555";
> +          gpio-controller;
> +          #gpio-cells = <2>;
> +          reg = <0x20>;
> +        };
> +      };
> +    };
> +    i2c@388 {
> +      compatible = "realtek,rtl9301-i2c";
> +      reg = <0x388 0x14>;
> +      #address-cells = <1>;
> +      #size-cells = <0>;
> +
> +      i2c@7 {
> +        reg = <7>;
> +        #address-cells = <1>;
> +        #size-cells = <0>;
> +      };
> +    };

Yoy have now multiple same examples. Keep only one, complete in the parent
schema.

> diff --git a/Documentation/devicetree/bindings/mfd/realtek,rtl9301-switch.yaml b/Documentation/devicetree/bindings/mfd/realtek,rtl9301-switch.yaml
> new file mode 100644
> index 000000000000..f053303ab1e6
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/mfd/realtek,rtl9301-switch.yaml
> @@ -0,0 +1,114 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2

Best regards,
Krzysztof