[PATCH 6/6] siemens: factoryset: use correct config for soc specific implementation

Sverdlin, Alexander alexander.sverdlin at siemens.com
Wed Nov 29 08:52:42 CET 2023


Hi Enrico,

On Tue, 2023-11-21 at 15:21 +0100, Enrico Leto wrote:
> Adding the capricorn board family some parts diverge from draco family.
> The switches used were not pertinent and need to be enhanced for each new
> board of the capricorn family. Replace them through the SOC name 'AM33XX'
> and 'IMX8'.
> 
> Signed-off-by: Enrico Leto <enrico.leto at siemens.com>
> ---
>  board/siemens/common/factoryset.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/board/siemens/common/factoryset.c b/board/siemens/common/factoryset.c
> index 9adc42ce0b7..493834d8806 100644
> --- a/board/siemens/common/factoryset.c
> +++ b/board/siemens/common/factoryset.c
> @@ -11,7 +11,7 @@
>  #include <g_dnl.h>
>  #include <net.h>
>  #include <asm/io.h>
> -#if !CONFIG_IS_ENABLED(TARGET_GIEDI) && !CONFIG_IS_ENABLED(TARGET_DENEB)
> +#if CONFIG_IS_ENABLED(AM33XX)
>  #include <asm/arch/cpu.h>
>  #endif
>  #include "board.h"
> @@ -197,7 +197,7 @@ int factoryset_read_eeprom(int i2c_addr)
>                 cp1 += 3;
>         }
>  
> -#if CONFIG_IS_ENABLED(TARGET_GIEDI) || CONFIG_IS_ENABLED(TARGET_DENEB)
> +#if CONFIG_IS_ENABLED(CONFIG_IMX8)

did you mean CONFIG_IS_ENABLED(IMX8) here?

>         /* get mac address for WLAN */
>         ret = get_factory_record_val(cp, size, (uchar *)"WLAN1", (uchar *)"mac",
>                                      buf, MAX_STRING_LENGTH);
> @@ -305,7 +305,7 @@ static int factoryset_mac_env_set(void)
>  
>         eth_env_set_enetaddr("ethaddr", mac_addr);
>  
> -#if CONFIG_IS_ENABLED(TARGET_GIEDI) || CONFIG_IS_ENABLED(TARGET_DENEB)
> +#if CONFIG_IS_ENABLED(CONFIG_IMX8)

ditto

>         eth_env_set_enetaddr("eth1addr", mac_addr);
>  
>         /* wlan mac */

-- 
Alexander Sverdlin
Siemens AG
www.siemens.com


More information about the U-Boot mailing list