[PATCH v3] Let the EQoS in imx8mp handle fixed-phy too.
Elmar Psilog
epsi at gmx.de
Sun Mar 19 11:43:07 CET 2023
> Without that patch it lost track to the node to scan
> speed and duplex.
> Patch was created by Marek Vasut, just tested by me.
>
> Signed-off-by: Elmar Psilog <epsi at gmx.de>
> Reviewed-by: Marek Vasut <marex at denx.de>
> ---
> changes v2: fix format issues
> changes v3: remove {} around single if, add reviewd..
>
> drivers/net/dwc_eth_qos.c | 18 +++++++++++++++---
> 1 file changed, 15 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/net/dwc_eth_qos.c b/drivers/net/dwc_eth_qos.c
> index 112deb546d..b9de205b8a 100644
> --- a/drivers/net/dwc_eth_qos.c
> +++ b/drivers/net/dwc_eth_qos.c
> @@ -788,9 +788,21 @@ static int eqos_start(struct udevice *dev)
> */
> if (!eqos->phy) {
> int addr = -1;
> - addr = eqos_get_phy_addr(eqos, dev);
> - eqos->phy = phy_connect(eqos->mii, addr, dev,
> - eqos->config->interface(dev));
> + ofnode fixed_node;
> +
> + if (IS_ENABLED(CONFIG_PHY_FIXED)) {
> + fixed_node = ofnode_find_subnode(dev_ofnode(dev),
> + "fixed-link");
> + if (ofnode_valid(fixed_node))
> + eqos->phy = fixed_phy_create(dev_ofnode(dev));
> + }
> +
> + if (!eqos->phy) {
> + addr = eqos_get_phy_addr(eqos, dev);
> + eqos->phy = phy_connect(eqos->mii, addr, dev,
> + eqos->config->interface(dev));
> + }
> +
> if (!eqos->phy) {
> pr_err("phy_connect() failed");
> goto err_stop_resets;
> --
> 2.34.1
I would give a short reminder about the patch. Reviewed by Marek, ok for
Ramon. Would be glad to see it in 2023.04.
Thanks
Elmar
More information about the U-Boot
mailing list