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) <[email protected]> >> >> Signed-off-by: Parvathi Pudi <[email protected]> >> > >> > 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
