Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/2226528/?format=api
{ "id": 2226528, "url": "http://patchwork.ozlabs.org/api/patches/2226528/?format=api", "web_url": "http://patchwork.ozlabs.org/project/uboot/patch/20260422123353.2902537-1-parvathi@couthit.com/", "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": "<20260422123353.2902537-1-parvathi@couthit.com>", "list_archive_url": null, "date": "2026-04-22T12:33:03", "name": "[v3] board: ti: am335x: Conditional MDIO PAD configuration instead of static for AM335_ICE", "commit_ref": null, "pull_url": null, "state": "new", "archived": false, "hash": "58d81192a918c6740ca2390ba22d23121b90f3fd", "submitter": { "id": 90270, "url": "http://patchwork.ozlabs.org/api/people/90270/?format=api", "name": "Parvathi Pudi", "email": "parvathi@couthit.com" }, "delegate": { "id": 3651, "url": "http://patchwork.ozlabs.org/api/users/3651/?format=api", "username": "trini", "first_name": "Tom", "last_name": "Rini", "email": "trini@ti.com" }, "mbox": "http://patchwork.ozlabs.org/project/uboot/patch/20260422123353.2902537-1-parvathi@couthit.com/mbox/", "series": [ { "id": 501023, "url": "http://patchwork.ozlabs.org/api/series/501023/?format=api", "web_url": "http://patchwork.ozlabs.org/project/uboot/list/?series=501023", "date": "2026-04-22T12:33:03", "name": "[v3] board: ti: am335x: Conditional MDIO PAD configuration instead of static for AM335_ICE", "version": 3, "mbox": "http://patchwork.ozlabs.org/series/501023/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/patches/2226528/comments/", "check": "pending", "checks": "http://patchwork.ozlabs.org/api/patches/2226528/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=fail reason=\"signature verification failed\" (2048-bit key;\n unprotected) header.d=couthit.com header.i=@couthit.com header.a=rsa-sha256\n header.s=default header.b=OxDZjdTG;\n\tdkim-atps=neutral", "legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=lists.denx.de\n (client-ip=85.214.62.61; helo=phobos.denx.de;\n envelope-from=u-boot-bounces@lists.denx.de; receiver=patchwork.ozlabs.org)", "phobos.denx.de;\n dmarc=fail (p=none dis=none) header.from=couthit.com", "phobos.denx.de;\n spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de", "phobos.denx.de;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n unprotected) header.d=couthit.com header.i=@couthit.com header.b=\"OxDZjdTG\";\n\tdkim-atps=neutral", "phobos.denx.de;\n dmarc=pass (p=none dis=none) header.from=couthit.com", "phobos.denx.de;\n spf=pass smtp.mailfrom=parvathi@couthit.com" ], "Received": [ "from phobos.denx.de (phobos.denx.de [85.214.62.61])\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 4g127l47m4z1yD5\n\tfor <incoming@patchwork.ozlabs.org>; Thu, 23 Apr 2026 00:45:07 +1000 (AEST)", "from h2850616.stratoserver.net (localhost [IPv6:::1])\n\tby phobos.denx.de (Postfix) with ESMTP id 530EB84307;\n\tWed, 22 Apr 2026 16:45:01 +0200 (CEST)", "by phobos.denx.de (Postfix, from userid 109)\n id D7CCD842B7; Wed, 22 Apr 2026 14:34:28 +0200 (CEST)", "from server.couthit.com (server.couthit.com [162.240.164.96])\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 66EEC842A4\n for <u-boot@lists.denx.de>; Wed, 22 Apr 2026 14:34:24 +0200 (CEST)", "from [115.246.246.98] (port=47680 helo=cypher.couthit.local)\n by server.couthit.com with esmtpsa (TLS1.3) tls TLS_AES_256_GCM_SHA384\n (Exim 4.98.1) (envelope-from <parvathi@couthit.com>)\n id 1wFWma-0000000AooY-1k2a; Wed, 22 Apr 2026 08:34:20 -0400" ], "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", "DKIM-Signature": "v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=couthit.com\n ; s=default;\n h=Content-Transfer-Encoding:MIME-Version:Message-ID:Date:Subject\n :Cc:To:From:Sender:Reply-To:Content-Type:Content-ID:Content-Description:\n Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n In-Reply-To:References:List-Id:List-Help:List-Unsubscribe:List-Subscribe:\n List-Post:List-Owner:List-Archive;\n bh=RRGjAcf5OwZ78xd1SedMWvS3jAraH42UH49sDp8RFZM=; b=OxDZjdTGVCZV0yPfhEOsAi0MoR\n YdFgFhTnyyMJ0t4KAUjf/iIhEgUppDmYjHeHojz3JFchVb5Ra36XkQvz9ClzkmBOr7SCN/ByDBde0\n C4fejUgIGU9xUQnf8htjMstpOUySiSFfFeoj5NcFzmks66GndKrMU0q/4NgIrBNC8vfwZS/8RyoIH\n w41wyi66XBJ2V2FNGxc9rXei9PHRDWeKWveAVQv5tnfz/F930kighyjZrMuMbdV0rN1d5jqrZCJiA\n BYugP09YOndkfZ6ba+r2HZuvzbXv3Ke9HQ51KQ5LHwOlxgltnM7sX/vgdBrDssD0bKAe0jrTsdHBN\n j8Odd9mw==;", "From": "Parvathi Pudi <parvathi@couthit.com>", "To": "u-boot@lists.denx.de, trini@konsulko.com, Maarten.Brock@sttls.nl,\n kory.maincent@bootlin.com, msp@baylibre.com, parvathi@couthit.com,\n sbellary@baylibre.com, romain.gantois@bootlin.com", "Cc": "pratheesh@ti.com, j-rameshbabu@ti.com, praneeth@ti.com, vigneshr@ti.com,\n srk@ti.com, rogerq@ti.com, danishanwar@ti.com, m-malladi@ti.com,\n krishna@couthit.com, mohan@couthit.com, pmohan@couthit.com,\n basharath@couthit.com", "Subject": "[PATCH v3] board: ti: am335x: Conditional MDIO PAD configuration\n instead of static for AM335_ICE", "Date": "Wed, 22 Apr 2026 18:03:03 +0530", "Message-ID": "<20260422123353.2902537-1-parvathi@couthit.com>", "X-Mailer": "git-send-email 2.43.0", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "X-AntiAbuse": [ "This header was added to track abuse,\n please include it with any abuse report", "Primary Hostname - server.couthit.com", "Original Domain - lists.denx.de", "Originator/Caller UID/GID - [47 12] / [47 12]", "Sender Address Domain - couthit.com" ], "X-Get-Message-Sender-Via": "server.couthit.com: authenticated_id:\n parvathi@couthit.com", "X-Authenticated-Sender": "server.couthit.com: parvathi@couthit.com", "X-Source": "", "X-Source-Args": "", "X-Source-Dir": "", "X-Mailman-Approved-At": "Wed, 22 Apr 2026 16:44:59 +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": "This patch removes the static MDIO pinmux configuration from\nrmii1_pin_mux[] and instead configures the MDIO pins conditionally\nduring board_init(). Previously, the MDIO_CLK and MDIO_DATA pins\nwere always configured for CPSW in mux.c, which could lead to\nunnecessary pin ownership and conflicts in scenarios where CPSW\nis not used.\n\nWith this change, the MDIO pins are configured only when required,\nensuring that CPSW Ethernet functionality in U-Boot remains unaffected.\nThis approach keeps Ethernet boot behavior intact and provides cleaner\nseparation between CPSW and other Ethernet use cases.\n\nReviewed-by: Markus Schneider-Pargmann (TI) <msp@baylibre.com>\nSigned-off-by: Parvathi Pudi <parvathi@couthit.com>\n---\nChanges from v2 to v3 :\n*) Addressed a comment from Markus Schneider-Pargmann on this patch.\n*) Added Reviewed-by: tag from Markus Schneider-Pargmann (TI).\n\nChanges from v1 to v2 :\n*) Addressed Markus Schneider-Pargmann's comments on this patch.\n\n board/ti/am335x/board.c | 10 ++++++++++\n board/ti/am335x/mux.c | 2 --\n 2 files changed, 10 insertions(+), 2 deletions(-)", "diff": "diff --git a/board/ti/am335x/board.c b/board/ti/am335x/board.c\nindex 90e37a8d913..9446a92c3dd 100644\n--- a/board/ti/am335x/board.c\n+++ b/board/ti/am335x/board.c\n@@ -28,6 +28,7 @@\n #include <asm/arch/mmc_host_def.h>\n #include <asm/arch/sys_proto.h>\n #include <asm/arch/mem.h>\n+#include <asm/arch/mux.h>\n #include <asm/global_data.h>\n #include <asm/io.h>\n #include <asm/emif.h>\n@@ -72,6 +73,12 @@ static struct ctrl_dev *cdev = (struct ctrl_dev *)CTRL_DEVICE_BASE;\n #define GPIO0_IRQSTATUSRAW\t(AM33XX_GPIO0_BASE + 0x024)\n #define GPIO1_IRQSTATUSRAW\t(AM33XX_GPIO1_BASE + 0x024)\n \n+static struct module_pin_mux rmii1_mdio_pin_mux[] = {\n+\t{OFFSET(mdio_clk), MODE(0) | PULLUP_EN},\t/* MDIO_CLK */\n+\t{OFFSET(mdio_data), MODE(0) | RXACTIVE | PULLUP_EN}, /* MDIO_DATA */\n+\t{-1},\n+};\n+\n /*\n * Read header information from EEPROM into global structure.\n */\n@@ -779,6 +786,9 @@ int board_init(void)\n \t\t\thang();\n \t\t}\n \n+\t\tif (!eth0_is_mii)\n+\t\t\tconfigure_module_pin_mux(rmii1_mdio_pin_mux);\n+\n \t\tprueth_is_mii = eth0_is_mii;\n \n \t\t/* disable rising edge IRQs */\ndiff --git a/board/ti/am335x/mux.c b/board/ti/am335x/mux.c\nindex d2d87c304f6..36d849d2119 100644\n--- a/board/ti/am335x/mux.c\n+++ b/board/ti/am335x/mux.c\n@@ -190,8 +190,6 @@ static struct module_pin_mux mii1_pin_mux[] = {\n };\n \n static struct module_pin_mux rmii1_pin_mux[] = {\n-\t{OFFSET(mdio_clk), MODE(0) | PULLUP_EN},\t/* MDIO_CLK */\n-\t{OFFSET(mdio_data), MODE(0) | RXACTIVE | PULLUP_EN}, /* MDIO_DATA */\n \t{OFFSET(mii1_crs), MODE(1) | RXACTIVE},\t\t/* MII1_CRS */\n \t{OFFSET(mii1_rxerr), MODE(1) | RXACTIVE},\t/* MII1_RXERR */\n \t{OFFSET(mii1_txen), MODE(1)},\t\t\t/* MII1_TXEN */\n", "prefixes": [ "v3" ] }