On 21.07.2020 20:13, Codrin Ciubotariu wrote:
> Adding the PHY nodes directly under the Ethernet node became deprecated,
> so the aim of this patch series is to make MACB use an MDIO node as
> container for MDIO devices.
> This patch series starts with a small patch to use the device-managed
> devm_mdiobus_alloc(). In the next two patches we update the bindings and
> adapt macb driver to parse the device-tree PHY nodes from under an MDIO
> node. The last patches add the MDIO node in the device-trees of sama5d2,
> sama5d3, samad4 and sam9x60 boards.
>
Tested this series on sama5d2_xplained in the following scenarios:
1/ PHY bindings from patch 4/7:
mdio {
#address-cells = <1>;
#size-cells = <0>;
ethernet-phy@1 {
reg = <0x1>;
interrupt-parent = <&pioA>;
interrupts = <PIN_PC9 IRQ_TYPE_LEVEL_LOW>;
};
2/ PHY bindings before this series:
ethernet-phy@1 {
reg = <0x1>;
interrupt-parent = <&pioA>;
interrupts = <PIN_PC9 IRQ_TYPE_LEVEL_LOW>;
};
3/ No PHY bindings at all.
All 3 cases went OK.
You can add:
Tested-by: Claudiu Beznea <claudiu.beznea@xxxxxxxxxxxxx>
Acked-by: Claudiu Beznea <claudiu.beznea@xxxxxxxxxxxxx>
Thank you,
Claudiu Beznea
> Changes in v2:
> - renamed patch 2/7 from "macb: bindings doc: use an MDIO node as a
> container for PHY nodes" to "dt-bindings: net: macb: use an MDIO
> node as a container for PHY nodes"
> - added back a newline removed by mistake in patch 3/7
>
> Codrin Ciubotariu (7):
> net: macb: use device-managed devm_mdiobus_alloc()
> dt-bindings: net: macb: use an MDIO node as a container for PHY nodes
> net: macb: parse PHY nodes found under an MDIO node
> ARM: dts: at91: sama5d2: add an mdio sub-node to macb
> ARM: dts: at91: sama5d3: add an mdio sub-node to macb
> ARM: dts: at91: sama5d4: add an mdio sub-node to macb
> ARM: dts: at91: sam9x60: add an mdio sub-node to macb
>
> Documentation/devicetree/bindings/net/macb.txt | 15 ++++++++++++---
> arch/arm/boot/dts/at91-sam9x60ek.dts | 8 ++++++--
> arch/arm/boot/dts/at91-sama5d27_som1.dtsi | 16 ++++++++++------
> arch/arm/boot/dts/at91-sama5d27_wlsom1.dtsi | 17 ++++++++++-------
> arch/arm/boot/dts/at91-sama5d2_ptc_ek.dts | 13 ++++++++-----
> arch/arm/boot/dts/at91-sama5d2_xplained.dts | 12 ++++++++----
> arch/arm/boot/dts/at91-sama5d3_xplained.dts | 16 ++++++++++++----
> arch/arm/boot/dts/at91-sama5d4_xplained.dts | 12 ++++++++----
> drivers/net/ethernet/cadence/macb_main.c | 18 ++++++++++++------
> 9 files changed, 86 insertions(+), 41 deletions(-)
>
