From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.1 (2015-04-28) on sa.local.altlinux.org X-Spam-Level: X-Spam-Status: No, score=-3.4 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,RP_MATCHES_RCVD autolearn=ham autolearn_force=no version=3.4.1 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=basealt.ru; s=dkim; t=1772188375; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=/GtySMX3xm8WS8B+Pyzbdgf1MJMUyLgLV0wehrV6U1o=; b=AB51TwhZDwpv4iZfbgA5oOCf9wZDPtbyd1R0fWpXZzSKbxwh0luJB844Xm7Oc1sPWnI2Q3 Ln38zOlJl8PY9L536ibQw+EeoVXXCzFxt3m5RHOryHlx/RSY+0mL9gA6rQCOawawRAoeKx 9C95Aa88pEmiuOGRiIH/+7DhWpEInHwOKY48QbhvwmTTLEFC7xe836Z8uO7ne3DzyLZDsG 9twppy1S+9ovOigqNotDCKsOHhp6dc73CmI93XQU4P3KmWn3kEY1zqeolwAYyiROl0+KvT MmdFbY8wtMrPvKIzAJLFbXNfDZfEkLWUHUX2UHJ6fzvbF2LDMl5zr9LWbw/90Q== From: Daniil Gnusarev To: gnusarevda@basealt.ru, devel-kernel@lists.altlinux.org Date: Fri, 27 Feb 2026 14:32:11 +0400 Message-ID: <20260227103236.785736-11-gnusarevda@basealt.ru> X-Mailer: git-send-email 2.42.2 In-Reply-To: <20260227103236.785736-1-gnusarevda@basealt.ru> References: <20260227103236.785736-1-gnusarevda@basealt.ru> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [d-kernel] [PATCH 10/35] net: fwnode_get_phy_id: consider all compatible strings X-BeenThere: devel-kernel@lists.altlinux.org X-Mailman-Version: 2.1.12 Precedence: list Reply-To: ALT Linux kernel packages development List-Id: ALT Linux kernel packages development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 27 Feb 2026 10:33:01 -0000 Archived-At: List-Archive: List-Post: From: Alexey Sheplyakov Commit cf99686072a1b7037a1d782b66037b2b722bf2c9 ("of: mdio: Refactor of_get_phy_id()") has broken Ethernet on TF307 board (and possibly other boards based on Baikal-M/T1 SoCs). That commit replaces `of_get_phy_id` with `fwnode_get_phy_id`. And `fwnode_get_phy_id` considers only the 1st compatible string to find out phy_id. This works well for all schema compliant device trees, since the `compatible` property of PHY nodes is supposed to be "ethernet-phy-idNNNN.MMMM". However DTB embedded in TF307 firmware describes PHY like this: gmac0_phy: ethernet-phy@3 { compatible = "micrel,ksz9031", "ethernet-phy-id0022.1620", "ethernet-phy-ieee802.3-c22"; reg = <0x3>; }; That is, the 1st compatible string is "micrel,ksz9031". Thus `fwnode_get_phy_id` is unable to parse phy_id, and `stmmac_mdio_register` fails. As a result Ethernet driver is unable to attach to PHY, and can't send/receive anything. To avoid the problem this patch adjusts `fwnode_get_phy_id` to consider *all* compatible strings. Signed-off-by: Alexey Sheplyakov X-DONTUPSTREAM X-feature-Baikal-M --- drivers/net/phy/phy_device.c | 41 ++++++++++++++++++++++++++---------- 1 file changed, 30 insertions(+), 11 deletions(-) diff --git a/drivers/net/phy/phy_device.c b/drivers/net/phy/phy_device.c index 7a67c900e79a51..8f7127e2b331b4 100644 --- a/drivers/net/phy/phy_device.c +++ b/drivers/net/phy/phy_device.c @@ -1070,18 +1070,37 @@ static int get_phy_c22_id(struct mii_bus *bus, int addr, u32 *phy_id) int fwnode_get_phy_id(struct fwnode_handle *fwnode, u32 *phy_id) { unsigned int upper, lower; - const char *cp; - int ret; - - ret = fwnode_property_read_string(fwnode, "compatible", &cp); - if (ret) - return ret; - - if (sscanf(cp, "ethernet-phy-id%4x.%4x", &upper, &lower) != 2) - return -EINVAL; + const char **compat; + int ret, count, i; + + /* FIXME: where is fwnode_property_for_each_string? */ + count = fwnode_property_read_string_array(fwnode, "compatible", NULL, 0); + if (count < 0) + return count; + else if (count == 0) + return -ENODATA; + + compat = kcalloc(count, sizeof(*compat), GFP_KERNEL); + if (!compat) + return -ENOMEM; + ret = fwnode_property_read_string_array(fwnode, "compatible", compat, count); + if (ret < 0) + goto out; - *phy_id = ((upper & GENMASK(15, 0)) << 16) | (lower & GENMASK(15, 0)); - return 0; + ret = -EINVAL; + for (i = 0; i < count; i++) { + pr_info("%s: considering '%s'\n", __func__, compat[i]); + if (sscanf(compat[i], "ethernet-phy-id%4x.%4x", &upper, &lower) != 2) { + continue; + } else { + *phy_id = ((upper & GENMASK(15, 0)) << 16) | (lower & GENMASK(15, 0)); + ret = 0; + break; + } + } +out: + kfree(compat); + return ret; } EXPORT_SYMBOL(fwnode_get_phy_id); -- 2.42.2