Re: [PATCH 4/8] configs: am62ax_evm_a53_defconfig: Enable networking

[Roger Quadros <rogerq@kernel.org>]

On 13/11/2023 16:51, Nishanth Menon wrote:
> Enable networking
> 
> Signed-off-by: Nishanth Menon <nm@ti.com>
> ---
>  configs/am62ax_evm_a53_defconfig | 7 ++++++-
>  1 file changed, 6 insertions(+), 1 deletion(-)
> 
> diff --git a/configs/am62ax_evm_a53_defconfig b/configs/am62ax_evm_a53_defconfig
> index d0a34c75505d..c94708a7bf85 100644
> --- a/configs/am62ax_evm_a53_defconfig
> +++ b/configs/am62ax_evm_a53_defconfig
> @@ -42,7 +42,6 @@ CONFIG_SPL_OF_CONTROL=y
>  CONFIG_MULTI_DTB_FIT=y
>  CONFIG_SPL_MULTI_DTB_FIT=y
>  CONFIG_SPL_MULTI_DTB_FIT_NO_COMPRESSION=y
> -# CONFIG_NET is not set
>  CONFIG_SPL_DM=y
>  CONFIG_SPL_DM_SEQ_ALIAS=y
>  CONFIG_REGMAP=y
> @@ -51,6 +50,8 @@ CONFIG_SPL_OF_TRANSLATE=y
>  CONFIG_CLK=y
>  CONFIG_SPL_CLK=y
>  CONFIG_CLK_TI_SCI=y
> +CONFIG_DMA_CHANNELS=y
> +CONFIG_TI_K3_NAVSS_UDMA=y
>  CONFIG_TI_SCI_PROTOCOL=y
>  # CONFIG_GPIO is not set
>  # CONFIG_I2C is not set
> @@ -60,6 +61,10 @@ CONFIG_MMC_SDHCI=y
>  CONFIG_MMC_SDHCI_ADMA=y
>  CONFIG_SPL_MMC_SDHCI_ADMA=y
>  CONFIG_MMC_SDHCI_AM654=y
> +CONFIG_PHY_TI_DP83867=y
> +CONFIG_PHY_FIXED=y

Why do we need PHY_FIXED?

"
config PHY_FIXED
        bool "Fixed-Link PHY"
        help
          Fixed PHY is used for having a 'fixed-link' to another MAC with a direct
          connection (MII, RGMII, ...).
          There is nothing like autoneogation and so
          on, the link is always up with fixed speed and fixed duplex-setting.
          More information: doc/device-tree-bindings/net/fixed-link.txt
"


> +CONFIG_TI_AM65_CPSW_NUSS=y
> +CONFIG_PHY=y
>  CONFIG_PINCTRL=y
>  CONFIG_SPL_PINCTRL=y
>  CONFIG_PINCTRL_SINGLE=y

-- 
cheers,
-roger