netdev - Re: [PATCH v4 2/2] arm: dts: ti: Adds support for AM335x and AM437x

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <2a977fd4-910a-4838-9ed6-97224d6ab775@lunn.ch>
Date: Fri, 16 Jan 2026 14:55:20 +0100
From: Andrew Lunn <andrew@...n.ch>
To: Parvathi Pudi <parvathi@...thit.com>
Cc: Andrew Davis <afd@...com>, nm <nm@...com>,
	Vignesh Raghavendra <vigneshr@...com>,
	Kevin Hilman <khilman@...libre.com>, rogerq <rogerq@...nel.org>,
	tony <tony@...mide.com>, robh <robh@...nel.org>,
	krzk+dt <krzk+dt@...nel.org>, conor+dt <conor+dt@...nel.org>,
	richardcochran <richardcochran@...il.com>,
	aaro koskinen <aaro.koskinen@....fi>,
	andreas <andreas@...nade.info>,
	linux-omap <linux-omap@...r.kernel.org>,
	devicetree <devicetree@...r.kernel.org>,
	linux-kernel <linux-kernel@...r.kernel.org>,
	netdev <netdev@...r.kernel.org>, danishanwar <danishanwar@...com>,
	pratheesh <pratheesh@...com>, j-rameshbabu <j-rameshbabu@...com>,
	praneeth <praneeth@...com>, srk <srk@...com>,
	rogerq <rogerq@...com>, krishna <krishna@...thit.com>,
	mohan <mohan@...thit.com>, pmohan <pmohan@...thit.com>,
	basharath <basharath@...thit.com>,
	Murali Karicheri <m-karicheri2@...com>
Subject: Re: [PATCH v4 2/2] arm: dts: ti: Adds support for AM335x and AM437x

> On the AM335x board, the CPSW MDIO and PRUSS MDIO signals are routed to the same physical
> pins (as shown in the schematic, see page 10 “MII_MUX” in tmdxice3359_sch_3h0013_v2_1a.pdf
> from https://www.ti.com/lit/zip/TIDR336 ). Because of this shared routing, the pinmux
> configuration applied by U-Boot for CPSW MDIO remains active even if the CPSW MDIO node is
> later disabled in Linux, and Linux does not automatically revert the pins to their reset state.

It is generally a bad idea to rely on the bootloader. I would make the
CPSW MDIO configure the pins how it needs it. The PRUSS MDIO should
also configure the pins how it needs them. However, it is not as
simple as that...

Looking at the schematic, what you have is ugly. You literally wire
the outputs together, without a hardware mux. For MDC you assume one
is Hi-Z, while the other drives the line. For MDIO it does not matter,
both are inputs. so Hi-Z.

I actually think you might need to represent this in Linux, with
something i would call a pinmux-mux. You give it two sets of pinmux
configurations. The active device claims the mux and gets it to set
the two sets of pinmux as needed. Also, just setting the pinmux to
GPIO is not sufficient, you also need to ensure the GPIO is configured
for input, so the lines go Hi-Z. Often pinmux and GPIO controllers are
interconnected, so the pinmux subsystem might be able to do that for
you.

I don't know if a pinmux-mux already exists in Linux. You probably
want to ask on the pinmux mailing list, or they might have a different
idea how to cleanly do this.

	Andrew