[PATCH v3] board: ti: am335x: Conditional MDIO PAD configuration instead of static for AM335_ICE

[Tom Rini]

On Fri, May 01, 2026 at 03:49:42PM +0530, Parvathi Pudi wrote:
> Hi,
> 
> > On Wed, Apr 22, 2026 at 06:03:03PM +0530, Parvathi Pudi wrote:
> > 
> >> This patch removes the static MDIO pinmux configuration from
> >> rmii1_pin_mux[] and instead configures the MDIO pins conditionally
> >> during board_init(). Previously, the MDIO_CLK and MDIO_DATA pins
> >> were always configured for CPSW in mux.c, which could lead to
> >> unnecessary pin ownership and conflicts in scenarios where CPSW
> >> is not used.
> >> 
> >> With this change, the MDIO pins are configured only when required,
> >> ensuring that CPSW Ethernet functionality in U-Boot remains unaffected.
> >> This approach keeps Ethernet boot behavior intact and provides cleaner
> >> separation between CPSW and other Ethernet use cases.
> >> 
> >> Reviewed-by: Markus Schneider-Pargmann (TI) <msp at baylibre.com>
> >> Signed-off-by: Parvathi Pudi <parvathi at couthit.com>
> > 
> > This leads to failure to build on:
> > am335x_evm_spiboot am335x_hs_evm am335x_hs_evm_spi am335x_hs_evm_uart
> > 
> 
> We tested the patch locally with the mentioned defconfigs (am335x_evm_spiboot,
> am335x_hs_evm, am335x_hs_evm_spi, am335x_hs_evm_uart), and we get the following
> warning:
> 
> ‘rmii1_mdio_pin_mux’ defined but not used [-Wunused-variable]
> 
> Are you referring to this warning or do you see any other build errors?

Yes, warnings are treated as errors by CI and so must be fixed before
being applied.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20260501/060624af/attachment.sig>