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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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ç



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux