On Mon, 11 Mar 2024 15:35:07 +0100, Niklas Söderlund wrote: > The design for this driver followed that of other Renesas Ethernet > drivers and thus did not force a child-node for the MDIO bus. As there > are no upstream drivers or users of this binding yet take the > opportunity to correct this and force the usage of a child-node for the > MDIO bus. > > Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx> > --- > Hello, > > Learn from the mistake made with the Renesas AVB bindings being worked > on in [1] where no mdio node was used to describe the MDIO bus and the > PHY was added directly as a child node to the AVB node. > > The Ethernet TSN driver is still in review and have not been merged and > no usage of the bindings merged either. So while this breaks the binding > it effects no one so we can correct this mistake without breaking any > use-cases before we need to support any backward compatibility. > > 1. https://patchwork.kernel.org/project/netdevbpf/list/?series=834331 > --- > .../bindings/net/renesas,ethertsn.yaml | 33 ++++++++----------- > 1 file changed, 14 insertions(+), 19 deletions(-) > Reviewed-by: Rob Herring <robh@xxxxxxxxxx>