Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/2224997/?format=api
{ "id": 2224997, "url": "http://patchwork.ozlabs.org/api/patches/2224997/?format=api", "web_url": "http://patchwork.ozlabs.org/project/uboot/patch/20260420074640.4036119-1-rafal.hibner@secom.com.pl/", "project": { "id": 18, "url": "http://patchwork.ozlabs.org/api/projects/18/?format=api", "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": "<20260420074640.4036119-1-rafal.hibner@secom.com.pl>", "list_archive_url": null, "date": "2026-04-20T07:46:40", "name": "net: zynq_gem: Clear stale speed bits in NWCFG before setting new ones", "commit_ref": null, "pull_url": null, "state": "new", "archived": false, "hash": "90e8c4de6f72f522aa5c52cd2c3cebe63e0addfe", "submitter": { "id": 93195, "url": "http://patchwork.ozlabs.org/api/people/93195/?format=api", "name": "Rafał Hibner", "email": "rafal.hibner@secom.com.pl" }, "delegate": { "id": 1692, "url": "http://patchwork.ozlabs.org/api/users/1692/?format=api", "username": "monstr", "first_name": "Michal", "last_name": "Simek", "email": "monstr@monstr.eu" }, "mbox": "http://patchwork.ozlabs.org/project/uboot/patch/20260420074640.4036119-1-rafal.hibner@secom.com.pl/mbox/", "series": [ { "id": 500565, "url": "http://patchwork.ozlabs.org/api/series/500565/?format=api", "web_url": "http://patchwork.ozlabs.org/project/uboot/list/?series=500565", "date": "2026-04-20T07:46:40", "name": "net: zynq_gem: Clear stale speed bits in NWCFG before setting new ones", "version": 1, "mbox": "http://patchwork.ozlabs.org/series/500565/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/patches/2224997/comments/", "check": "pending", "checks": "http://patchwork.ozlabs.org/api/patches/2224997/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<u-boot-bounces@lists.denx.de>", "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=secom.com.pl header.i=@secom.com.pl header.a=rsa-sha256\n header.s=default header.b=QAJm11bN;\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=reject dis=none) header.from=secom.com.pl", "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=secom.com.pl header.i=@secom.com.pl\n header.b=\"QAJm11bN\";\n\tdkim-atps=neutral", "phobos.denx.de; dmarc=pass (p=reject dis=none)\n header.from=secom.com.pl", "phobos.denx.de;\n spf=pass smtp.mailfrom=rafal.hibner@secom.com.pl" ], "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)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4fzdQZ4qQ0z1yD4\n\tfor <incoming@patchwork.ozlabs.org>; Mon, 20 Apr 2026 18:08:06 +1000 (AEST)", "from h2850616.stratoserver.net (localhost [IPv6:::1])\n\tby phobos.denx.de (Postfix) with ESMTP id D3CCB83E16;\n\tMon, 20 Apr 2026 10:08:03 +0200 (CEST)", "by phobos.denx.de (Postfix, from userid 109)\n id C920C8426E; Mon, 20 Apr 2026 09:46:53 +0200 (CEST)", "from mail.secom.com.pl (mail.secom.com.pl [213.216.87.26])\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 91EF483693\n for <u-boot@lists.denx.de>; Mon, 20 Apr 2026 09:46:51 +0200 (CEST)" ], "X-Spam-Checker-Version": "SpamAssassin 3.4.2 (2018-09-13) on phobos.denx.de", "X-Spam-Level": "", "X-Spam-Status": "No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,\n DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_BLOCKED,\n SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.2", "From": "=?utf-8?q?Rafa=C5=82_Hibner?= <rafal.hibner@secom.com.pl>", "DKIM-Filter": "OpenDKIM Filter v2.10.3 mail.secom.com.pl 346A943FF65", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple; d=secom.com.pl;\n s=default; t=1776671207;\n bh=xORqtN/eo/TCOMmpsDLyJPTyrFpsWAsaEf+WoOwUkxA=;\n h=From:To:Cc:Subject:Date;\n b=QAJm11bN/gFwFDPe9UlfJiNBAU6NXy2yLjO332Nqr5gy7cf5EF7gwKXKz4w1KYx7r\n jxrNVvduWna18Xqw2p7cNAwGZvS6wICqhtKOvTDExgNGrKcB6KSvBZUyO1gajNhM/3\n H10uQpgJxQesLiUZQ1Aul01ik9Zqdg+A6zbsILo0=", "To": "u-boot@lists.denx.de", "Cc": "michal.simek@amd.com, joe.hershberger@ni.com, rfried.dev@gmail.com,\n jerome@forissier.org, trini@konsulko.com, venkatesh.abbarapu@amd.com,\n martin.kaistra@linutronix.de,\n =?utf-8?q?Rafa=C5=82_Hibner?= <rafal.hibner@secom.com.pl>", "Subject": "[PATCH] net: zynq_gem: Clear stale speed bits in NWCFG before setting\n new ones", "Date": "Mon, 20 Apr 2026 07:46:40 +0000", "Message-ID": "<20260420074640.4036119-1-rafal.hibner@secom.com.pl>", "MIME-Version": "1.0", "Content-Type": "text/plain; charset=UTF-8", "Content-Transfer-Encoding": "8bit", "X-MailScanner-ID": "346A943FF65.A1566", "X-MailScanner": "Found to be clean", "X-MailScanner-From": "rafal.hibner@secom.com.pl", "X-Mailman-Approved-At": "Mon, 20 Apr 2026 10:08:02 +0200", "X-BeenThere": "u-boot@lists.denx.de", "X-Mailman-Version": "2.1.39", "Precedence": "list", "List-Id": "U-Boot discussion <u-boot.lists.denx.de>", "List-Unsubscribe": "<https://lists.denx.de/options/u-boot>,\n <mailto:u-boot-request@lists.denx.de?subject=unsubscribe>", "List-Archive": "<https://lists.denx.de/pipermail/u-boot/>", "List-Post": "<mailto:u-boot@lists.denx.de>", "List-Help": "<mailto:u-boot-request@lists.denx.de?subject=help>", "List-Subscribe": "<https://lists.denx.de/listinfo/u-boot>,\n <mailto:u-boot-request@lists.denx.de?subject=subscribe>", "Errors-To": "u-boot-bounces@lists.denx.de", "Sender": "\"U-Boot\" <u-boot-bounces@lists.denx.de>", "X-Virus-Scanned": "clamav-milter 0.103.8 at phobos.denx.de", "X-Virus-Status": "Clean" }, "content": "Commit ecba4380ad26 (\"net: zynq_gem: Update the MDC clock divisor in the\nprobe function\") changed zynq_gem_init() from a direct register write to\na read-modify-write pattern in order to preserve MDC clock divider bits.\nHowever, the old speed selection bits (SPEED100/SPEED1000) are never\ncleared before OR-ing in the new value.\n\nWhen the PHY renegotiates at a different speed between successive calls\nto zynq_gem_init() (e.g. link flapping from 1 Gbps to 100 Mbps on a\nmarginal cable), both SPEED100 and SPEED1000 end up set simultaneously\nin NWCFG. This confuses the GEM hardware and no frames are received.\n\nFix by explicitly clearing both speed bits before merging the new\nconfiguration, so only the currently negotiated speed is ever active.\n\nFixes: ecba4380ad26 (\"net: zynq_gem: Update the MDC clock divisor in the probe function\")\nSigned-off-by: Rafał Hibner <rafal.hibner@secom.com.pl>\n---\n drivers/net/zynq_gem.c | 1 +\n 1 file changed, 1 insertion(+)", "diff": "diff --git a/drivers/net/zynq_gem.c b/drivers/net/zynq_gem.c\nindex 407b022508c..f18458aee98 100644\n--- a/drivers/net/zynq_gem.c\n+++ b/drivers/net/zynq_gem.c\n@@ -612,6 +612,7 @@ static int zynq_gem_init(struct udevice *dev)\n \t\tbreak;\n \t}\n \tnwcfg = readl(®s->nwcfg);\n+\tnwcfg &= ~(ZYNQ_GEM_NWCFG_SPEED100 | ZYNQ_GEM_NWCFG_SPEED1000);\n \tnwcfg |= nwconfig;\n \tif (nwcfg)\n \t\twritel(nwcfg, ®s->nwcfg);\n", "prefixes": [] }