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

[Himanshu Bhavani <himanshu.bhavani@xxxxxxxxxxxxxxxxx>]

Hi Krzysztof,

>Hm? Left-over?

>BTW, did you test this before sending?

Yes, I have tested. Below are the logs for the reference.
But I guess I have to upgrade dtschema

SCHEMA  Documentation/devicetree/bindings/processed-schema.json
/data/opensourcenxp/tarang/mainline_kernel/Documentation/devicetree/bindings/net/snps,dwmac.yaml: mac-mode: missing type definition
  CHKDT   ../Documentation/devicetree/bindings
  LINT    ../Documentation/devicetree/bindings
  DTEX    Documentation/devicetree/bindings/pinctrl/pinctrl-mcp23s08.example.dts
  DTC [C] Documentation/devicetree/bindings/pinctrl/pinctrl-mcp23s08.example.dtb

Best Regards,
Himanshu
________________________________________
From: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
Sent: 22 October 2024 12:58
To: Himanshu Bhavani <himanshu.bhavani@xxxxxxxxxxxxxxxxx>; linus.walleij@xxxxxxxxxx <linus.walleij@xxxxxxxxxx>; robh@xxxxxxxxxx <robh@xxxxxxxxxx>; krzk+dt@xxxxxxxxxx <krzk+dt@xxxxxxxxxx>; conor+dt@xxxxxxxxxx <conor+dt@xxxxxxxxxx>
Cc: linux-gpio@xxxxxxxxxxxxxxx <linux-gpio@xxxxxxxxxxxxxxx>; devicetree@xxxxxxxxxxxxxxx <devicetree@xxxxxxxxxxxxxxx>; linux-kernel@xxxxxxxxxxxxxxx <linux-kernel@xxxxxxxxxxxxxxx>
Subject: Re: [PATCH] dt-bindings: pinctrl: convert pinctrl-mcp23s08.txt to yaml format
 
CAUTION: This email originated from outside the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.

On 22/10/2024 08:01, Himanshu Bhavani wrote:
> +
> +examples:
> +  - |
> +    #include <dt-bindings/interrupt-controller/irq.h>
> +    #include <dt-bindings/gpio/gpio.h>
> +
> +    i2c {
> +        #address-cells = <1>;
> +        #size-cells = <0>;
> +
> +        mcp23017: gpio@20 {
> +            compatible = "microchip,mcp23017";
> +            reg = <0x20>;
> +            gpio-controller;
> +            #gpio-cells = <2>;
> +
> +            interrupt-parent = <&gpio1>;
> +            interrupts = <17 IRQ_TYPE_LEVEL_LOW>; // Check this line

Hm? Left-over?

BTW, did you test this before sending?

Best regards,
Krzysztof