Re: [PATCH net-next v2 1/2] dt-bindings: net: dsa: lantiq,gswip: add reset-gpios

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential
issue to consider:
- [Low] dt-bindings: net: dsa: lantiq,gswip: split GSW1xx into separate
  schema
--

commit cbde9490cffae75d1433f9bd1e4ef36f7a9097cd
Author: Joseph Telaak <joseph@telaak.dev>

dt-bindings: net: dsa: lantiq,gswip: add reset-gpios

This commit adds the reset-gpios property to the lantiq,gswip binding for
standalone GSW1xx switches and disallows it for SoC-integrated variants.

> diff --git a/Documentation/devicetree/bindings/net/dsa/lantiq,gswip.yaml b/Documentation/devicetree/bindings/net/dsa/lantiq,gswip.yaml
> index 49af5573e452d..9a74ea2a6dcdd 100644
> --- a/Documentation/devicetree/bindings/net/dsa/lantiq,gswip.yaml
> +++ b/Documentation/devicetree/bindings/net/dsa/lantiq,gswip.yaml
> @@ -40,6 +40,11 @@ properties:
>        - const: mdio
>        - const: mii
>  
> +  reset-gpios:
> +    maxItems: 1
> +    description:
> +      GPIO connected to the hardware reset input of a standalone switch.
> +
>    mdio:
>      $ref: /schemas/net/mdio.yaml#
>      unevaluatedProperties: false
> @@ -173,6 +178,7 @@ allOf:
>          mdio:
>            required:
>              - compatible
> +        reset-gpios: false
>      else:
>        properties:
>          reg:

[Severity: Low]
Would it make sense to split the standalone MDIO-connected GSW1xx switches
into a separate binding schema instead of adding another conditional
override here?

The SoC-integrated MMIO switches and standalone MDIO switches sit on
different buses and already use the allOf if-then-else block to constrain
or disallow reg, reg-names, mdio, and gphy-fw between the two families.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261108-mxl-gsw1xx-reset-v2-0-46c1da093893@telaak.dev?part=1