[PATCH 06/15] board: ti: j721e: evm: Drop board based serdes initialization
Kumar, Udit
u-kumar1 at ti.com
Sat Nov 4 11:35:47 CET 2023
On 11/3/2023 6:07 AM, Nishanth Menon wrote:
> Serdes initialization should be compile flag based instead of being
> eeprom based.
>
> Signed-off-by: Nishanth Menon <nm at ti.com>
> ---
> board/ti/j721e/evm.c | 10 ++--------
> 1 file changed, 2 insertions(+), 8 deletions(-)
>
> diff --git a/board/ti/j721e/evm.c b/board/ti/j721e/evm.c
> index 17640ae79665..d49993fffe39 100644
> --- a/board/ti/j721e/evm.c
> +++ b/board/ti/j721e/evm.c
> @@ -359,9 +359,6 @@ void configure_serdes_torrent(void)
> struct phy serdes;
> int ret;
>
> - if (!IS_ENABLED(CONFIG_PHY_CADENCE_TORRENT))
> - return;
> -
> ret = uclass_get_device_by_driver(UCLASS_PHY,
> DM_DRIVER_GET(torrent_phy_provider),
> &dev);
> @@ -393,9 +390,6 @@ void configure_serdes_sierra(void)
> int ret, count, i;
> int link_count = 0;
>
> - if (!IS_ENABLED(CONFIG_PHY_CADENCE_SIERRA))
> - return;
> -
> ret = uclass_get_device_by_driver(UCLASS_MISC,
> DM_DRIVER_GET(sierra_phy_provider),
> &dev);
> @@ -477,10 +471,10 @@ int board_late_init(void)
> probe_daughtercards();
> }
>
> - if (board_is_j7200_som())
> + if (IS_ENABLED(CONFIG_PHY_CADENCE_TORRENT))
> configure_serdes_torrent();
>
> - if (board_is_j721e_som())
> + if (IS_ENABLED(CONFIG_PHY_CADENCE_SIERRA))
> configure_serdes_sierra();
>
Could be merged into one if condition
> return 0;
More information about the U-Boot
mailing list