On Mon, Sep 28, 2026 at 09:43:48AM +0800, Kyle Switch wrote: > > > +unevaluatedProperties: false > > > + > > > +examples: > > > + - | > > > + mdio { > > > + #address-cells = <1>; > > > + #size-cells = <0>; > > > + > > > + ethernet-phy-package@9 { > > > + #address-cells = <1>; > > > + #size-cells = <0>; > > > + compatible = "motorcomm,yt8824-package"; > > > + reg = <9>; > > > + > > > + phy-mode = "internal"; > > > + > > > + ethernet-phy@4 { > > > + reg = <4>; > > > + }; > > > + > > > + ethernet-phy@5 { > > > + reg = <5>; > > > + }; > > > + > > > + ethernet-phy@6 { > > > + reg = <6>; > > > + }; > > > + > > > + ethernet-phy@7 { > > > + reg = <7>; > > > + }; > > Unless the addresses and numbers of phys are entirely unconstrained, I > > think you should add some pattern properties for them. > > Ans: I'm somewhat confused on this point. What might the pattern properties There's no need for this "ans" business, the quoting makes it clear already. >         you mentioned include? Could you provide more detail? thank you. On second thoughts, there's some basic constraints already provided by ethernet-phy-package for this. Probably don't need to anyhting specific.