[U-Boot] [PATCH] armv8: Remove dependency of SERDES for LS CHASIS3

[Sriram Dash]

>From: York Sun
>Subject: Re: [PATCH] armv8: Remove dependency of SERDES for LS CHASIS3
>
>On 09/04/2017 03:14 AM, Sriram Dash wrote:
>> Remove dependency of SYS_HAS_SERDES for Layerscape Chasis 3.
>>
>> Signed-off-by: Sriram Dash <sriram.dash at nxp.com>
>> ---
>>   arch/arm/cpu/armv8/fsl-layerscape/Kconfig | 4 ++--
>>   arch/arm/cpu/armv8/fsl-layerscape/cpu.c   | 2 ++
>>   arch/arm/cpu/armv8/fsl-layerscape/soc.c   | 2 ++
>>   3 files changed, 6 insertions(+), 2 deletions(-)
>>
>> diff --git a/arch/arm/cpu/armv8/fsl-layerscape/Kconfig
>> b/arch/arm/cpu/armv8/fsl-layerscape/Kconfig
>> index cdeef26..355f8bb 100644
>> --- a/arch/arm/cpu/armv8/fsl-layerscape/Kconfig
>> +++ b/arch/arm/cpu/armv8/fsl-layerscape/Kconfig
>> @@ -58,6 +58,8 @@ config ARCH_LS2080A
>>   	select ARM_ERRATA_829520
>>   	select ARM_ERRATA_833471
>>   	select FSL_LSCH3
>> +	select SYS_FSL_SRDS_1
>> +	select SYS_HAS_SERDES
>>   	select SYS_FSL_DDR
>>   	select SYS_FSL_DDR_LE
>>   	select SYS_FSL_DDR_VER_50
>> @@ -93,8 +95,6 @@ config FSL_LSCH2
>>
>>   config FSL_LSCH3
>>   	bool
>> -	select SYS_FSL_SRDS_1
>> -	select SYS_HAS_SERDES
>>
>>   config FSL_MC_ENET
>>   	bool "Management Complex network"
>> diff --git a/arch/arm/cpu/armv8/fsl-layerscape/cpu.c
>> b/arch/arm/cpu/armv8/fsl-layerscape/cpu.c
>> index c6fede3..3ce2ffc 100644
>> --- a/arch/arm/cpu/armv8/fsl-layerscape/cpu.c
>> +++ b/arch/arm/cpu/armv8/fsl-layerscape/cpu.c
>> @@ -12,7 +12,9 @@
>>   #include <asm/system.h>
>>   #include <asm/armv8/mmu.h>
>>   #include <asm/io.h>
>> +#ifdef CONFIG_SYS_HAS_SERDES
>>   #include <asm/arch/fsl_serdes.h>
>> +#endif
>
>Do not use ifdef here. Instead, fix fsl_serdes.h for proper include.
>CONFIG_SYS_HAS_SERDES is already being check there. You can adjust it if needed.
>

Sure. This will be better. I will make necessary changes in the next rev.

>York