{"id":2220072,"url":"http://patchwork.ozlabs.org/api/patches/2220072/?format=json","web_url":"http://patchwork.ozlabs.org/project/uboot/patch/20260406084204.1274302-1-pranav.vinaytilak@amd.com/","project":{"id":18,"url":"http://patchwork.ozlabs.org/api/projects/18/?format=json","name":"U-Boot","link_name":"uboot","list_id":"u-boot.lists.denx.de","list_email":"u-boot@lists.denx.de","web_url":null,"scm_url":null,"webscm_url":null,"list_archive_url":"","list_archive_url_format":"","commit_url_format":""},"msgid":"<20260406084204.1274302-1-pranav.vinaytilak@amd.com>","list_archive_url":null,"date":"2026-04-06T08:41:50","name":"net: phy: ethernet_id: fix duplicate eth_phy binding","commit_ref":null,"pull_url":null,"state":"new","archived":false,"hash":"e020395841a11b3ebf1ca92ae26a55a8e04f8b73","submitter":{"id":92380,"url":"http://patchwork.ozlabs.org/api/people/92380/?format=json","name":"Pranav Tilak","email":"pranav.vinaytilak@amd.com"},"delegate":{"id":157425,"url":"http://patchwork.ozlabs.org/api/users/157425/?format=json","username":"jforissier","first_name":"Jerome","last_name":"Forissier","email":"jerome.forissier@linaro.org"},"mbox":"http://patchwork.ozlabs.org/project/uboot/patch/20260406084204.1274302-1-pranav.vinaytilak@amd.com/mbox/","series":[{"id":498832,"url":"http://patchwork.ozlabs.org/api/series/498832/?format=json","web_url":"http://patchwork.ozlabs.org/project/uboot/list/?series=498832","date":"2026-04-06T08:41:50","name":"net: phy: ethernet_id: fix duplicate eth_phy binding","version":1,"mbox":"http://patchwork.ozlabs.org/series/498832/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/patches/2220072/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/2220072/checks/","tags":{},"related":[],"headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@legolas.ozlabs.org","Authentication-Results":["legolas.ozlabs.org;\n\tdkim=pass (1024-bit key;\n unprotected) header.d=amd.com header.i=@amd.com header.a=rsa-sha256\n header.s=selector1 header.b=bhoQ4+eb;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=lists.denx.de\n (client-ip=2a01:238:438b:c500:173d:9f52:ddab:ee01; helo=phobos.denx.de;\n envelope-from=u-boot-bounces@lists.denx.de; receiver=patchwork.ozlabs.org)","phobos.denx.de;\n dmarc=pass (p=quarantine dis=none) header.from=amd.com","phobos.denx.de;\n spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de","phobos.denx.de;\n\tdkim=pass (1024-bit key;\n unprotected) header.d=amd.com header.i=@amd.com header.b=\"bhoQ4+eb\";\n\tdkim-atps=neutral","phobos.denx.de;\n dmarc=pass (p=quarantine dis=none) header.from=amd.com","phobos.denx.de;\n spf=fail smtp.mailfrom=Pranav.VinayTilak@amd.com"],"Received":["from phobos.denx.de (phobos.denx.de\n [IPv6:2a01:238:438b:c500:173d:9f52:ddab:ee01])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519 server-signature ECDSA (secp384r1) server-digest SHA384)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4fq2rk00R1z1yFt\n\tfor ; Mon, 06 Apr 2026 18:42:29 +1000 (AEST)","from h2850616.stratoserver.net (localhost [IPv6:::1])\n\tby phobos.denx.de (Postfix) with ESMTP id 586D78394E;\n\tMon, 6 Apr 2026 10:42:27 +0200 (CEST)","by phobos.denx.de (Postfix, from userid 109)\n id 143BB83C2B; Mon, 6 Apr 2026 10:42:26 +0200 (CEST)","from CY7PR03CU001.outbound.protection.outlook.com\n (mail-westcentralusazlp170100005.outbound.protection.outlook.com\n [IPv6:2a01:111:f403:c112::5])\n (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits))\n (No client certificate requested)\n by phobos.denx.de (Postfix) with ESMTPS id A463C8352B\n for ; Mon, 6 Apr 2026 10:42:23 +0200 (CEST)","from LV3P220CA0026.NAMP220.PROD.OUTLOOK.COM (2603:10b6:408:234::27)\n by DM4PR12MB6640.namprd12.prod.outlook.com (2603:10b6:8:8f::15) with\n Microsoft SMTP Server (version=TLS1_2,\n cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9769.17; Mon, 6 Apr\n 2026 08:42:19 +0000","from BN2PEPF000055DF.namprd21.prod.outlook.com\n (2603:10b6:408:234:cafe::5b) by LV3P220CA0026.outlook.office365.com\n (2603:10b6:408:234::27) with Microsoft SMTP Server (version=TLS1_3,\n cipher=TLS_AES_256_GCM_SHA384) id 15.20.9769.31 via Frontend Transport; Mon,\n 6 Apr 2026 08:42:19 +0000","from satlexmb07.amd.com (165.204.84.17) by\n BN2PEPF000055DF.mail.protection.outlook.com (10.167.245.9) with Microsoft\n SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id\n 15.20.9818.0 via Frontend Transport; Mon, 6 Apr 2026 08:42:18 +0000","from satlexmb07.amd.com (10.181.42.216) by satlexmb07.amd.com\n (10.181.42.216) with Microsoft SMTP Server (version=TLS1_2,\n cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.17; Mon, 6 Apr\n 2026 03:42:17 -0500","from xhdkummari40.xilinx.com (10.180.168.240) by satlexmb07.amd.com\n (10.181.42.216) with Microsoft SMTP Server (version=TLS1_2,\n cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.17 via Frontend\n Transport; Mon, 6 Apr 2026 03:42:15 -0500"],"X-Spam-Checker-Version":"SpamAssassin 3.4.2 (2018-09-13) on phobos.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-2.6 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,\n DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,\n RCVD_IN_DNSWL_BLOCKED,SPF_HELO_PASS,SPF_PASS autolearn=ham\n autolearn_force=no version=3.4.2","ARC-Seal":"i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none;\n b=GmmI5A9DtE43/+WUsVvvdWdPmqzh2BZ46N1FurnrzLBwHNdFGhQvOn6iTj0mYC/ojGeEcjxBYkHcOrmQf55Og9/eAyvr65n34xApntFhmHA1WUmDaLUJEAeEGnJda8dyds1XWDyhlWyCOPfmab3EIdTipA3JBI+QaUSDfHWlnbnJWBQdiLCe3oz0E8x6AnXLB+UTx92EwjBp03IF+D+4ozKjl5cxW0S8I9/+s2DNVkpcf/hIaQ/x8F5UrYl7d1B2aWitBUvlIXA/c2uvGC7usz0qJJX5aty/+JnO4JwjJEaFemcJ2ZwezsQJvXlbP78sBQtQcIP1L8EXoT9us2I5yA==","ARC-Message-Signature":"i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com;\n s=arcselector10001;\n h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1;\n bh=/nepva75pzjecnpucGZHmuRUP5dxbJXohbGZETgojLc=;\n b=CxQpPHBAyTDSKjc4KzcSiQFgJ1U3+VxMhP/5d+N5vMLSb0ZNpe+5HJ/H6TvxJaneIpBZD41ZVMWk+gLG4jtuH8u9HRIUJnygRcEN4/SpKEJwQ7jS+6xDx8FvOyyL5reioTQCHJYS63cK2CmlCXEo4ed9513wnSb1V6I3wbR63FYZQoQOjTcEWxOKg+47Uj1HOHKESe6LFlPPCr3p6hurlfoKeKu0IsgSGFkOTbLHPlCEUEClumSv4O01lUwntFvxhPL4KUPPcAvCRZgqi0fe0E7c2mOeLvyuG2/fKnqtGWvx2DgcOtWgXWzl2gzxR36pgsa2SDIbNTajtpfq+1ATaw==","ARC-Authentication-Results":"i=1; mx.microsoft.com 1; spf=pass (sender ip is\n 165.204.84.17) smtp.rcpttodomain=lists.denx.de smtp.mailfrom=amd.com;\n dmarc=pass (p=quarantine sp=quarantine pct=100) action=none\n header.from=amd.com; dkim=none (message not signed); arc=none (0)","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1;\n h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck;\n bh=/nepva75pzjecnpucGZHmuRUP5dxbJXohbGZETgojLc=;\n b=bhoQ4+ebQpjST839raO9UTyZabn3fVJi+pbDQeGi8RYk1SdR+WBlPESzeaSOEI7axpYeJMwNupGA7tKcMzgVigWvTRCqR375ZA0CjeKUXoxc7qFwAyrkMmrHT/OCLzdVEEgKsZ7pbSdXt8preGtxsUscUn/mC4PtGXN/RVMaDjE=","X-MS-Exchange-Authentication-Results":"spf=pass (sender IP is 165.204.84.17)\n smtp.mailfrom=amd.com; dkim=none (message not signed)\n header.d=none;dmarc=pass action=none header.from=amd.com;","Received-SPF":"Pass (protection.outlook.com: domain of amd.com designates\n 165.204.84.17 as permitted sender) receiver=protection.outlook.com;\n client-ip=165.204.84.17; helo=satlexmb07.amd.com; pr=C","From":"Pranav Tilak ","To":", ","CC":", , Pranav Tilak\n , Jerome Forissier ,\n \"Tom Rini\" , Marek Vasut\n , Ramon Fried ","Subject":"[PATCH] net: phy: ethernet_id: fix duplicate eth_phy binding","Date":"Mon, 6 Apr 2026 14:11:50 +0530","Message-ID":"<20260406084204.1274302-1-pranav.vinaytilak@amd.com>","X-Mailer":"git-send-email 2.34.1","MIME-Version":"1.0","Content-Transfer-Encoding":"8bit","Content-Type":"text/plain","X-EOPAttributedMessage":"0","X-MS-PublicTrafficType":"Email","X-MS-TrafficTypeDiagnostic":"BN2PEPF000055DF:EE_|DM4PR12MB6640:EE_","X-MS-Office365-Filtering-Correlation-Id":"763f1240-b903-40f5-be2f-08de93b86997","X-MS-Exchange-SenderADCheck":"1","X-MS-Exchange-AntiSpam-Relay":"0","X-Microsoft-Antispam":"BCL:0;\n ARA:13230040|82310400026|36860700016|376014|1800799024|56012099003|18002099003;","X-Microsoft-Antispam-Message-Info":"\n CAd+cJJ8KtiBcXqlQSDqVr02TYzMw7a0pxeP6BWGehdWSXoo0bxMpYn6xKNNd5mX74RkpyC64JmbGbM6exz8Mhznh9GLuVeWGvcau/HkzI9vmiEaGyMQcw3gYW1+97VEtci4LflfwvL7Yvysc+Hun5a1PnlN3WHy3QF+O9Sn0sw2tBx6uiqxmBVmWW+3b+O9w3FwYv6DsjYoo9NQnjpmxmJcFZQgQEBBZ6SPXIRx6m2a374dhDJe+e9UGz22tWXhfXL4SESXsfNtGu66RveJS/oYQM4xsWVv5XUfS8SFLm2UJFTRFjo6REYUAuihK9q6jeMe27jFdHaBy55jxTc49ySlUFLwMHGhaSXwIRSl17z+BvLI+XaLfy/LLrpU5uhL/t52MSkyrV31afVK3cR4Qo+XmA21pVcz9deOODYQeqxfTXZyu8ACa4wexjrOGRFqkCojmKXB4gA8iNy3HzXO/9VuZSL6RlWGSDCuw5G0SgDgSt5gX/w1wWCYqDob9UKnp/7iS/bga0vMgDpzBUQjuFpJxN/4y556cD96VibxBUslIepm7lydz3OV478JwfjHt2Eyrg3xkutAAOeOXwAKDxcw+b6dUnfRNwubing3lzafEG8iOxKy9dCMd6JSXGZzOIILYYGCVuxD93PgWPsjNNH5iGAcr3ZNLq38d+GsFKzOBWAK5QUeLQSpJSSshuCXPF8Zk7yEVU1zS/eqUTFgWyZiGIlmGZqhP3nBjWEJDjQ=","X-Forefront-Antispam-Report":"CIP:165.204.84.17; CTRY:; LANG:en; SCL:1; SRV:;\n IPV:NLI; SFV:NSPM; H:satlexmb07.amd.com; PTR:InfoDomainNonexistent; CAT:NONE;\n SFS:(13230040)(82310400026)(36860700016)(376014)(1800799024)(56012099003)(18002099003);\n DIR:OUT; SFP:1101;","X-MS-Exchange-AntiSpam-MessageData-ChunkCount":"1","X-MS-Exchange-AntiSpam-MessageData-0":"\n zlBhpwPOUvAoqK7AupK7gWLPmBnxWLetR9O79/ux2EW5Kb7esr5OQKUopW/56LVkGr5oSBascjv0wBxx9W4xxN6CzK50y7F1wKbpVdti9N4dzAFjVRsnpLy8dfTAVL9y4Br7yFqnY78EKLPuKNFsq29eUtFQWLhaadmshfOK1H77YLpkLHJqa9RYkN2a+jyq5eShXFUX0X1ZtTff/C4SNz9wi91f2ls2xhJTAGeGck5vuY0GI+819qYlUgcolG6WkNnsprzY46eEPrpUVkcoPQVqXmT0iRtwdJuWhIbS0aaKjZHIX/bQzVSMxXbzmsemuqbZrCYPRKKiMLjJ/8pRBiR99IsdpL/dDoau5q3eaZt7Wtehp5/yRlrylspAeXaQ7tIk0EiaoBtNKidqNPzAOQWkQtPGv1j/jygjZ9nf4OsXcSgf1SPyvUM2CX2dNecY","X-OriginatorOrg":"amd.com","X-MS-Exchange-CrossTenant-OriginalArrivalTime":"06 Apr 2026 08:42:18.7588 (UTC)","X-MS-Exchange-CrossTenant-Network-Message-Id":"\n 763f1240-b903-40f5-be2f-08de93b86997","X-MS-Exchange-CrossTenant-Id":"3dd8961f-e488-4e60-8e11-a82d994e183d","X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp":"\n TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d; Ip=[165.204.84.17];\n Helo=[satlexmb07.amd.com]","X-MS-Exchange-CrossTenant-AuthSource":"\n BN2PEPF000055DF.namprd21.prod.outlook.com","X-MS-Exchange-CrossTenant-AuthAs":"Anonymous","X-MS-Exchange-CrossTenant-FromEntityHeader":"HybridOnPrem","X-MS-Exchange-Transport-CrossTenantHeadersStamped":"DM4PR12MB6640","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.39","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n ","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n ","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" ","X-Virus-Scanned":"clamav-milter 0.103.8 at phobos.denx.de","X-Virus-Status":"Clean"},"content":"When both CONFIG_PHY_ETHERNET_ID and CONFIG_DM_ETH_PHY are enabled,\neth_phy_binds_nodes() called from eth_post_bind() already binds the\nethernet PHY node to eth_phy_generic_drv. However, phy_connect_phy_id()\ncalled via zynq_phy_init() -> phy_connect() cannot attach to the\nalready bound PHY device and creates a second instance, resulting\nin duplicate entries in the dm tree.\n\nFix this by guarding the bind/probe block in phy_connect_phy_id()\nwith uclass_get_device_by_ofnode() to skip re-binding if the PHY\nnode is already registered in UCLASS_ETH_PHY.\n\nFixes: 68a4d1506109 (\"net: phy: Bind ETH_PHY uclass driver to each new PHY\")\nSigned-off-by: Pranav Tilak \n---\n drivers/net/phy/ethernet_id.c | 4 +++-\n 1 file changed, 3 insertions(+), 1 deletion(-)","diff":"diff --git a/drivers/net/phy/ethernet_id.c b/drivers/net/phy/ethernet_id.c\nindex 2f8454ca27d..7b232ab37f6 100644\n--- a/drivers/net/phy/ethernet_id.c\n+++ b/drivers/net/phy/ethernet_id.c\n@@ -8,6 +8,7 @@\n #include \n #include \n #include \n+#include \n #include \n #include \n #include \n@@ -51,7 +52,8 @@ struct phy_device *phy_connect_phy_id(struct mii_dev *bus, struct udevice *dev,\n \tif (phydev)\n \t\tphydev->node = node;\n \n-\tif (IS_ENABLED(CONFIG_DM_ETH_PHY) && ofnode_valid(node)) {\n+\tif (IS_ENABLED(CONFIG_DM_ETH_PHY) && ofnode_valid(node) &&\n+\t uclass_get_device_by_ofnode(UCLASS_ETH_PHY, node, &pdev)) {\n \t\tnode_name = ofnode_get_name(node);\n \t\tret = device_bind_driver_to_node(dev, \"eth_phy_generic_drv\",\n \t\t\t\t\t\t node_name, node,\n","prefixes":[]}