[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