Re: [PATCH v3 2/8] dt-bindings: gpio: cdns: add Axiado AX3005 GPIO variant
[email protected] Fri, 17 Jul 2026 04:00:55 +0000
| Newsgroups | org.infradead.lists.linux-i3c,org.kernel.vger.linux-devicetree |
|---|---|
| Message-ID | <[email protected]> |
Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider: - [High] The schema enforces a 3-item compatible string chain that unintentionally forces AX3005 devices to inherit an AX3000-specific driver quirk, contradicting the commit message. -- --- Patch [2]: [PATCH v3 2/8] dt-bindings: gpio: cdns: add Axiado AX3005 GPIO variant --- commit 53c67e7dc4609ee9e61f57a1d43c9c6ca04d123b Author: Swark Yang <[email protected]> dt-bindings: gpio: cdns: add Axiado AX3005 GPIO variant Add binding for the Axiado AX3005 GPIO controller. The commit message states there are no known changes and it should fall back directly to the cdns,gpio-r1p02 compatible string. > diff --git a/Documentation/devicetree/bindings/gpio/cdns,gpio.yaml b/Documentation/devicetree/bindings/gpio/cdns,gpio.yaml > --- a/Documentation/devicetree/bindings/gpio/cdns,gpio.yaml > +++ b/Documentation/devicetree/bindings/gpio/cdns,gpio.yaml > @@ -14,8 +14,11 @@ properties: > oneOf: > - const: cdns,gpio-r1p02 > - items: > - - enum: > - - axiado,ax3000-gpio > + - const: axiado,ax3000-gpio > + - const: cdns,gpio-r1p02 > + - items: > + - const: axiado,ax3005-gpio > + - const: axiado,ax3000-gpio > - const: cdns,gpio-r1p02 [Severity: High] Does forcing axiado,ax3000-gpio into the 3-item compatible string chain unintentionally cause AX3005 devices to inherit AX3000-specific driver quirks? By requiring the axiado,ax3000-gpio compatible string in the fallback chain, any AX3005 device tree must include it to pass schema validation. The underlying driver will then match this intermediate string via device_get_match_data() and apply ax3000_gpio_quirks, such as skipping initialization of CDNS_GPIO_DIRECTION_MODE and CDNS_GPIO_OUTPUT_EN. The commit message states the intent is to fall back to the cdns,gpio-r1p02 compatible. Should axiado,ax3005-gpio instead be added to an enum alongside axiado,ax3000-gpio in a 2-item fallback chain, avoiding the intermediate AX3000 string entirely? -- Sashiko AI review ยท https://sashiko.dev/#/patchset/20260716-upstream-axiado-ax3005-upstream-v3-0-c429095143ec@axiado.com?part=2 -- linux-i3c mailing list [email protected] http://lists.infradead.org/mailman/listinfo/linux-i3c