Re: [PATCH net-next v2 05/10] dt-bindings: net: dsa: define MDIO bus child node

From: Arınç ÜNAL
Date: Sat Sep 16 2023 - 11:37:55 EST


On 16.09.2023 17:11, Andrew Lunn wrote:
On Sat, Sep 16, 2023 at 02:08:57PM +0300, Arınç ÜNAL wrote:
Some DSA subdrivers register the MDIO bus of the switch they control. Or
let the DSA driver register the MDIO bus. The node for these buses are
either required or optional, depending on the subdriver. Document this on
all of the affected DSA switch schemas.

The attributes of a DSA subdriver that lets the DSA driver register the
bus:
- ds->ops->phy_read() and ds->ops->phy_write() are present.
- ds->slave_mii_bus is not populated by the DSA subdriver.
- The bus is registered non-OF-based or OF-based. Registered OF-based if
"mdio" child node is defined.

The affected DSA switch schemas are documented below.

- brcm,b53.yaml

drivers/net/dsa/b53/b53_common.c:
- The DSA subdriver lets the DSA driver register the bus.

---

git uses --- to separate the commit message from additional comments
for the reviewers. Anything after the --- will not be merged. Is that
your intention?

No, they're vital information. I'll find something better to separate them,
thanks.

Arınç