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

Parvathi Pudi parvathi at couthit.com
Fri May 1 12:19:42 CEST 2026


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?

Thanks and Regards,
Parvathi


More information about the U-Boot mailing list