| Message ID | 20211206174139.2296497-1-robh@kernel.org (mailing list archive) |
|---|---|
| State | Not Applicable |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | dt-bindings: net: mdio: Allow any child node name | expand |
| Context | Check | Description |
|---|---|---|
| netdev/tree_selection | success | Not a local patch |
On 12/6/21 9:41 AM, Rob Herring wrote: > An MDIO bus can have devices other than ethernet PHYs on it, so it > should allow for any node name rather than just 'ethernet-phy'. > > Cc: Andrew Lunn <andrew@lunn.ch> > Cc: Heiner Kallweit <hkallweit1@gmail.com> > Cc: Russell King <linux@armlinux.org.uk> > Cc: "David S. Miller" <davem@davemloft.net> > Cc: Jakub Kicinski <kuba@kernel.org> > Cc: Florian Fainelli <f.fainelli@gmail.com> > Cc: netdev@vger.kernel.org > Signed-off-by: Rob Herring <robh@kernel.org> Acked-by: Florian Fainelli <f.fainelli@gmail.com> Thanks!
On Mon, 6 Dec 2021 11:41:39 -0600 Rob Herring wrote: > An MDIO bus can have devices other than ethernet PHYs on it, so it > should allow for any node name rather than just 'ethernet-phy'. Hi Rob, what's your preference for merging these?
On Mon, Dec 06, 2021 at 11:41:39AM -0600, Rob Herring wrote: > An MDIO bus can have devices other than ethernet PHYs on it, so it > should allow for any node name rather than just 'ethernet-phy'. > > Cc: Andrew Lunn <andrew@lunn.ch> > Cc: Heiner Kallweit <hkallweit1@gmail.com> > Cc: Russell King <linux@armlinux.org.uk> > Cc: "David S. Miller" <davem@davemloft.net> > Cc: Jakub Kicinski <kuba@kernel.org> > Cc: Florian Fainelli <f.fainelli@gmail.com> > Cc: netdev@vger.kernel.org > Signed-off-by: Rob Herring <robh@kernel.org> > --- > Documentation/devicetree/bindings/net/mdio.yaml | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Makes sense, Reviewed-by: Thierry Reding <treding@nvidia.com>
On Mon, Dec 6, 2021 at 2:57 PM Jakub Kicinski <kuba@kernel.org> wrote: > > On Mon, 6 Dec 2021 11:41:39 -0600 Rob Herring wrote: > > An MDIO bus can have devices other than ethernet PHYs on it, so it > > should allow for any node name rather than just 'ethernet-phy'. > > Hi Rob, what's your preference for merging these? I can take them. Rob
diff --git a/Documentation/devicetree/bindings/net/mdio.yaml b/Documentation/devicetree/bindings/net/mdio.yaml index 08e15fb1584f..53206e4a7a14 100644 --- a/Documentation/devicetree/bindings/net/mdio.yaml +++ b/Documentation/devicetree/bindings/net/mdio.yaml @@ -59,7 +59,7 @@ properties: type: boolean patternProperties: - "^ethernet-phy@[0-9a-f]+$": + '@[0-9a-f]+$': type: object properties:
An MDIO bus can have devices other than ethernet PHYs on it, so it should allow for any node name rather than just 'ethernet-phy'. Cc: Andrew Lunn <andrew@lunn.ch> Cc: Heiner Kallweit <hkallweit1@gmail.com> Cc: Russell King <linux@armlinux.org.uk> Cc: "David S. Miller" <davem@davemloft.net> Cc: Jakub Kicinski <kuba@kernel.org> Cc: Florian Fainelli <f.fainelli@gmail.com> Cc: netdev@vger.kernel.org Signed-off-by: Rob Herring <robh@kernel.org> --- Documentation/devicetree/bindings/net/mdio.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)