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

[Parvathi Pudi]

Hi,

> 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.
> 

Sure, we will fix this in the next version.

Thanks and regards,
Parvathi