linux-kernel - Re: [PATCH v2] dt-bindings: pinctrl: convert pinctrl-mcp23s08.txt to yaml format

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: <usqmeunejf44l6wjw67ocv4idyxfpw5ivt5v4hqkputd7d7xsk@3ies2iwutzsz>
Date: Sat, 26 Oct 2024 14:18:50 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Himanshu Bhavani <himanshu.bhavani@...iconsignals.io>
Cc: linus.walleij@...aro.org, robh@...nel.org, krzk+dt@...nel.org, 
	tarang.raval@...iconsignals.io, Conor Dooley <conor+dt@...nel.org>, linux-gpio@...r.kernel.org, 
	devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2] dt-bindings: pinctrl: convert pinctrl-mcp23s08.txt to
 yaml format

On Thu, Oct 24, 2024 at 06:16:18PM +0530, Himanshu Bhavani wrote:
> +  pinmux:
> +    type: object
> +    properties:
> +      pins:
> +        description:
> +          The list of GPIO pins controlled by this node. Each pin name
> +          corresponds to a physical pin on the GPIO expander.
> +        items:
> +          pattern: "^gpio([0-9]|[1][0-5])$"

Since I expect resend, correct also quotes - use consistently either '
or ".

> +        maxItems: 16
> +
> +      bias-pull-up:
> +        type: boolean
> +        description:
> +          Configures pull-up resistors for the GPIO pins. Absence of this
> +          property will leave the configuration in its default state.
> +
> +    required:
> +      - pins
> +
> +    additionalProperties: false
> +
> +required:
> +  - compatible
> +  - reg
> +  - gpio-controller
> +  - '#gpio-cells'
> +
> +unevaluatedProperties: false
> +
> +examples:
> +  - |
> +    #include <dt-bindings/interrupt-controller/irq.h>
> +
> +    i2c {
> +        #address-cells = <1>;
> +        #size-cells = <0>;
> +
> +        mcp23018: gpio@20 {
> +            compatible = "microchip,mcp23018";
> +            reg = <0x20>;
> +            gpio-controller;
> +            #gpio-cells = <2>;
> +
> +            interrupt-parent = <&gpio1>;
> +            interrupts = <17 IRQ_TYPE_LEVEL_LOW>;
> +            interrupt-controller;
> +            #interrupt-cells = <2>;
> +            microchip,irq-mirror;
> +        };
> +    };

Drop this example.

> +
> +  - |
> +    spi {
> +        #address-cells = <1>;
> +        #size-cells = <0>;
> +
> +        mcp23s17: gpio@0 {

Drop unused label

> +            compatible = "microchip,mcp23s17";
> +            reg = <0>;
> +            gpio-controller;
> +            #gpio-cells = <2>;
> +            spi-max-frequency = <1000000>;
> +            microchip,spi-present-mask = /bits/ 8 <0x01>;
> +        };
> +    };
> +
> +  - |
> +    #include <dt-bindings/interrupt-controller/irq.h>
> +    #include <dt-bindings/gpio/gpio.h>
> +
> +    i2c {

Keep one complete example for i2c and one for spi. This was not in
previous patch and changelog does not explain why you need three
examples.

> +        #address-cells = <1>;
> +        #size-cells = <0>;
> +
> +        mcp23017: gpio@21 {

Drop unused label

Best regards,
Krzysztof