Re: [PATCH 2/4] dt-bindings: net: dsa: document internal MDIO bus

[Arınç ÜNAL]

On 13.09.2023 16:00, Vladimir Oltean wrote:
> On Wed, Sep 13, 2023 at 02:35:11PM +0300, Arınç ÜNAL wrote:
> > On 13.09.2023 14:04, Vladimir Oltean wrote:
> > > I don't think they're for switch ports only. Any driver which uses
> > > phylink_fwnode_phy_connect() or its derivatives gets subject to the same
> > > bindings. But putting the sub-schema in ethernet-controller.yaml makes
> > > sense, just maybe not naming it "phylink-switch".
> >
> > Got it. Should we disallow managed altogether when fixed-link is also
> > defined, or just with in-band-status value?
>
> Just with the "in-band-status" value - just like phylink_parse_mode()
> implies. If not possible, just leave that condition out.

I can rewrite the property to allow values other than in-band-status.

      - if:
          required: [ fixed-link ]
        then:
          properties:
            managed:
              const: auto

Arınç