Re: [PATCH 1/1] dt-bindings: net: dsa: Convert lan9303.txt to yaml format

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] Unresolved phandles in YAML examples will cause `dt_binding_check` to fail.
--

commit ff1d6b54c66febb1ac2492f6699ba1d476db3541
Author: Frank Li <Frank.Li@nxp.com>

dt-bindings: net: dsa: Convert lan9303.txt to yaml format

This commit converts the SMSC/MicroChip LAN9303 device tree binding from
text to YAML format. It also renames switch-phy to switch in the example
to resolve warnings during device tree checks.

> diff --git a/Documentation/devicetree/bindings/net/dsa/smsc,lan9303.yaml b/Documentation/devicetree/bindings/net/dsa/smsc,lan9303.yaml
> new file mode 100644
> index 0000000000000..42f8473538a07
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/net/dsa/smsc,lan9303.yaml
[ ... ]
> +examples:
> +  - |
> +    #include <dt-bindings/gpio/gpio.h>
> +
> +    /* I2C managed mode */
> +    i2c {
> +        #address-cells = <1>;
> +        #size-cells = <0>;
> +
> +        switch@a {
> +            compatible = "smsc,lan9303-i2c";
> +            reg = <0xa>;
> +            reset-gpios = <&gpio7 6 GPIO_ACTIVE_LOW>;

Will this example fail dt_binding_check? The &gpio7 phandle is referenced
here, but there does not appear to be a corresponding gpio7 label defined
within the scope of the example.

> +            reset-duration = <200>;
> +
> +            ports {
> +                #address-cells = <1>;
> +                #size-cells = <0>;
> +
> +                port@0 {
> +                    reg = <0>;
> +                    label = "cpu";
> +                    ethernet = <&master>;

Similarly, the &master phandle is used here but is not defined. When the
example is compiled into a standalone DTS file during dt_binding_check,
does this result in a reference to non-existent node or label error from
dtc? 

Defining dummy nodes for these labels within the examples might be necessary
to allow the schema to pass validation.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260512214704.1084890-1-Frank.Li@nxp.com?part=1