| Message ID | 20211022163548.3380625-1-sean.anderson@seco.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 25790844006af9538e1a01554bc043b404fd8845 |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | [net-next,1/2] dt-bindings: net: macb: Add mdio bus child node | expand |
| Context | Check | Description |
|---|---|---|
| netdev/cover_letter | success | Single patches do not need cover letters |
| netdev/fixes_present | success | Fixes tag not required for -next series |
| netdev/patch_count | success | Link |
| netdev/tree_selection | success | Clearly marked for net-next |
| netdev/subject_prefix | success | Link |
| netdev/cc_maintainers | warning | 1 maintainers not CCed: robh+dt@kernel.org |
| netdev/source_inline | success | Was 0 now: 0 |
| netdev/verify_signedoff | success | Signed-off-by tag matches author and committer |
| netdev/module_param | success | Was 0 now: 0 |
| netdev/build_32bit | success | Errors and warnings before: 0 this patch: 0 |
| netdev/kdoc | success | Errors and warnings before: 0 this patch: 0 |
| netdev/verify_fixes | success | No Fixes tag |
| netdev/checkpatch | success | total: 0 errors, 0 warnings, 0 checks, 10 lines checked |
| netdev/build_allmodconfig_warn | success | Errors and warnings before: 0 this patch: 0 |
| netdev/header_inline | success | No static functions without inline keyword in header files |
On Fri, Oct 22, 2021 at 12:35:47PM -0400, Sean Anderson wrote: > This adds an optional mdio bus child node. If present, the mac will > look for PHYs there instead of directly under the top-level node. This > eliminates any ambiguity about whether child nodes are PHYs, and allows > the MDIO bus to contain non-PHY devices. > > Signed-off-by: Sean Anderson <sean.anderson@seco.com> Reviewed-by: Andrew Lunn <andrew@lunn.ch> Andrew
On Fri, Oct 22, 2021 at 12:35:47PM -0400, Sean Anderson wrote: > This adds an optional mdio bus child node. If present, the mac will > look for PHYs there instead of directly under the top-level node. This > eliminates any ambiguity about whether child nodes are PHYs, and allows > the MDIO bus to contain non-PHY devices. Hi Sean Please always have a patch 0/X for patchsets, which explains the big picture of the patchset. This is also used as the merge commit message. Andrew
Hello: This series was applied to netdev/net-next.git (master) by David S. Miller <davem@davemloft.net>: On Fri, 22 Oct 2021 12:35:47 -0400 you wrote: > This adds an optional mdio bus child node. If present, the mac will > look for PHYs there instead of directly under the top-level node. This > eliminates any ambiguity about whether child nodes are PHYs, and allows > the MDIO bus to contain non-PHY devices. > > Signed-off-by: Sean Anderson <sean.anderson@seco.com> > > [...] Here is the summary with links: - [net-next,1/2] dt-bindings: net: macb: Add mdio bus child node https://git.kernel.org/netdev/net-next/c/25790844006a - [net-next,2/2] net: macb: Use mdio child node for MDIO bus if it exists https://git.kernel.org/netdev/net-next/c/4d98bb0d7ec2 You are awesome, thank you!
diff --git a/Documentation/devicetree/bindings/net/macb.txt b/Documentation/devicetree/bindings/net/macb.txt index af9df2f01a1c..a1b06fd1962e 100644 --- a/Documentation/devicetree/bindings/net/macb.txt +++ b/Documentation/devicetree/bindings/net/macb.txt @@ -30,6 +30,10 @@ Required properties: Optional elements: 'tsu_clk' - clocks: Phandles to input clocks. +Optional properties: +- mdio: node containing PHY children. If this node is not present, then PHYs + will be direct children. + The MAC address will be determined using the optional properties defined in ethernet.txt.
This adds an optional mdio bus child node. If present, the mac will look for PHYs there instead of directly under the top-level node. This eliminates any ambiguity about whether child nodes are PHYs, and allows the MDIO bus to contain non-PHY devices. Signed-off-by: Sean Anderson <sean.anderson@seco.com> --- Documentation/devicetree/bindings/net/macb.txt | 4 ++++ 1 file changed, 4 insertions(+)