Re: [PATCH] configs: Layerscape: Remove the 'fdt_addr' env

[Wolfgang Denk <wd@denx.de>]

Dear Zhiqiang Hou,

In message <20210812072846.39305-1-Zhiqiang.Hou@nxp.com> you wrote:

> diff --git a/include/configs/ls1012afrdm.h b/include/configs/ls1012afrdm.h
> index 2711f651d7..c7fdd10cf5 100644
> --- a/include/configs/ls1012afrdm.h
> +++ b/include/configs/ls1012afrdm.h
> @@ -1,6 +1,7 @@
>  /* SPDX-License-Identifier: GPL-2.0+ */
>  /*
>   * Copyright 2016 Freescale Semiconductor, Inc.
> + * Copyright 2021 NXP
>   */
>  
>  #ifndef __LS1012ARDB_H__
> @@ -24,7 +25,6 @@
>  #define CONFIG_EXTRA_ENV_SETTINGS		\
>  	"verify=no\0"				\
>  	"fdt_high=0xffffffffffffffff\0"		\
> -	"fdt_addr=0x00f00000\0"			\
>  	"kernel_addr=0x01000000\0"		\
>  	"scriptaddr=0x80000000\0"		\
>  	"fdtheader_addr_r=0x80100000\0"		\
> diff --git a/include/configs/ls1012afrwy.h b/include/configs/ls1012afrwy.h

Adding or - like here, even worse - removing just a few lines of
data definitions is certainly nothing where you can claim a
copyright on.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd@denx.de
HR Manager to job candidate "I see you've had no  computer  training.
Although  that  qualifies  you  for upper management, it means you're
under-qualified for our entry level positions."