[U-Boot] [PATCH v5 5/5] configs: am65x_evm_a53: enable networking

[Keerthy]

On 09/07/19 4:27 AM, Joe Hershberger wrote:
> On Thu, Jun 6, 2019 at 7:15 AM Keerthy <j-keerthy at ti.com> wrote:
>>
>> From: Grygorii Strashko <grygorii.strashko at ti.com>
>>
>> Enable TI K3 AM65x CPSW NUSS driver.
>>
>> Signed-off-by: Grygorii Strashko <grygorii.strashko at ti.com>
>> Signed-off-by: Keerthy <j-keerthy at ti.com>
>> Reviewed-by: Tom Rini <trini at konsulko.com>
> 
> Question below, but...
> 
> Acked-by: Joe Hershberger <joe.hershberger at ni.com>
> 
>> ---
>>   configs/am65x_evm_a53_defconfig | 7 +++++++
>>   1 file changed, 7 insertions(+)
>>
>> diff --git a/configs/am65x_evm_a53_defconfig b/configs/am65x_evm_a53_defconfig
>> index 41cf0100fa..5d33b127b1 100644
>> --- a/configs/am65x_evm_a53_defconfig
>> +++ b/configs/am65x_evm_a53_defconfig
>> @@ -44,6 +44,7 @@ CONFIG_SPL_MULTI_DTB_FIT_NO_COMPRESSION=y
>>   CONFIG_ENV_IS_IN_FAT=y
>>   CONFIG_ENV_FAT_INTERFACE="mmc"
>>   CONFIG_ENV_FAT_DEVICE_AND_PART="1:1"
>> +CONFIG_NET_RANDOM_ETHADDR=y
>>   CONFIG_DM=y
>>   CONFIG_SPL_DM=y
>>   CONFIG_SPL_DM_SEQ_ALIAS=y
>> @@ -58,6 +59,11 @@ CONFIG_K3_SEC_PROXY=y
>>   CONFIG_DM_MMC=y
>>   CONFIG_MMC_SDHCI=y
>>   CONFIG_MMC_SDHCI_K3_ARASAN=y
>> +CONFIG_PHY_TI=y
>> +CONFIG_PHY_FIXED=y
>> +CONFIG_DM_ETH=y
>> +CONFIG_TI_AM65_CPSW_NUSS=y
>> +CONFIG_PHY=y
>>   CONFIG_PINCTRL=y
>>   # CONFIG_PINCTRL_GENERIC is not set
>>   CONFIG_SPL_PINCTRL=y
>> @@ -74,3 +80,4 @@ CONFIG_SOC_TI=y
>>   CONFIG_SYSRESET=y
>>   CONFIG_SPL_SYSRESET=y
>>   CONFIG_SYSRESET_TI_SCI=y
>> +CONFIG_HEXDUMP=y
> 
> Why is this needed?

Ah this was needed when some debug prints were enabled in prior 
versions. I will remove this in v6.

Thanks for the review.

Best Regards,
Keerthy

> 
>> --
>> 2.17.1
>>
>> _______________________________________________
>> U-Boot mailing list
>> U-Boot at lists.denx.de
>> https://lists.denx.de/listinfo/u-boot