Re: [PATCH 1/3] dt-bindings: gpio: add Axiado SGPIO controller

From: Krzysztof Kozlowski

Date: Tue Apr 14 2026 - 10:07:14 EST


On 14/04/2026 15:48, Petar Stepanovic wrote:
> +properties:
> + compatible:
> + enum:
> + - axiado,sgpio

That's a SoC no? Where is SoC compatible?

> +
> + reg:
> + maxItems: 1
> +
> + gpio-controller: true
> +
> + '#gpio-cells':
> + const: 2
> +
> + interrupts:
> + maxItems: 1
> +
> + interrupt-controller: true
> +
> + '#interrupt-cells':
> + const: 2
> +
> + design-variant:

Sorry, but no, none of this and further properties apply to DT. Drop all
of them.

Please also read writing bindings so you won't make trivial mistakes.

...


> + sgpio@a000 {
> + compatible = "axiado,sgpio";

Don't come with own style, please. Look at other files.

> + reg = <0xa000 0x800>;
> + gpio-controller;
> + #gpio-cells = <2>;
Best regards,
Krzysztof