[U-Boot] [PATCH 1/2 v2] board: freescale: ls1012afrx:Common files to support

[York Sun]

On 06/06/2018 04:16 AM, Pramod Kumar wrote:
> LS1012A-FRDM and LS1012A-FRWY board.
> 
> 1-Move all common files applicable for both boards LS1012A-FRDM
>   and LS1012A-FRWY into board directory ls1012afrx.
> 2-Restructure LS1012A-FRDM code. Only board specific files are
>   in LS1012A-FRDM board directory.
> 
> Signed-off-by: Pramod Kumar <pramod.kumar_1 at nxp.com>
> ---
> Depends on:
> http://patchwork.ozlabs.org/patch/918935/
> http://patchwork.ozlabs.org/patch/918933/
> http://patchwork.ozlabs.org/patch/918932/
> 
> Changes for v2:
>  - Rebased patch to above dependency patches 
> 
>  arch/arm/Kconfig                                   |  2 +
>  board/freescale/ls1012afrdm/Kconfig                | 60 ----------------------
>  board/freescale/ls1012afrdm/MAINTAINERS            | 12 +----
>  board/freescale/ls1012afrdm/Makefile               |  4 +-
>  board/freescale/ls1012afrx/Kconfig                 | 27 ++++++++++
>  board/freescale/{ls1012afrdm => ls1012afrx}/eth.c  |  0
>  .../ls1012afrdm.c => ls1012afrx/ls1012afrx.c}      |  0
>  7 files changed, 32 insertions(+), 73 deletions(-)
>  create mode 100644 board/freescale/ls1012afrx/Kconfig
>  rename board/freescale/{ls1012afrdm => ls1012afrx}/eth.c (100%)
>  rename board/freescale/{ls1012afrdm/ls1012afrdm.c => ls1012afrx/ls1012afrx.c} (100%)
> 
> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> index 22234cd..ffff190 100644
> --- a/arch/arm/Kconfig
> +++ b/arch/arm/Kconfig
> @@ -1417,6 +1417,8 @@ source "board/freescale/ls1046ardb/Kconfig"
>  source "board/freescale/ls1012aqds/Kconfig"
>  source "board/freescale/ls1012ardb/Kconfig"
>  source "board/freescale/ls1012afrdm/Kconfig"
> +source "board/freescale/ls1012afrx/Kconfig"
> +source "board/freescale/ls1012afrwy/Kconfig"
>  source "board/freescale/mx35pdk/Kconfig"
>  source "board/freescale/s32v234evb/Kconfig"
>  source "board/gdsys/a38x/Kconfig"
> diff --git a/board/freescale/ls1012afrdm/Kconfig b/board/freescale/ls1012afrdm/Kconfig
> index f52a896..58e592d 100644
> --- a/board/freescale/ls1012afrdm/Kconfig
> +++ b/board/freescale/ls1012afrdm/Kconfig
> @@ -21,63 +21,3 @@ config SYS_LS_PPA_FW_ADDR
>  	default 0x40400000
>  
>  endif
> -
> -if FSL_PFE
> -
> -config BOARD_SPECIFIC_OPTIONS # dummy
> -	def_bool y
> -	select PHYLIB
> -	imply PHY_REALTEK
> -	imply PHY_ATHEROS
> -
> -config DDR_PFE_PHYS_BASEADDR
> -	hex "PFE DDR physical base address"
> -	default 0x03800000
> -
> -config DDR_PFE_BASEADDR
> -	hex "PFE DDR base address"
> -	default 0x83800000
> -
> -config PFE_EMAC1_PHY_ADDR
> -	hex "PFE DDR base address"
> -	default 0x2
> -
> -config PFE_EMAC2_PHY_ADDR
> -	hex "PFE DDR base address"
> -	default 0x1
> -
> -endif
> -
> -if TARGET_LS1012AFRWY
> -
> -config SYS_BOARD
> -	default "ls1012afrdm"
> -
> -config SYS_VENDOR
> -	default "freescale"
> -
> -config SYS_SOC
> -	default "fsl-layerscape"
> -
> -config SYS_CONFIG_NAME
> -	default "ls1012afrwy"
> -
> -config SYS_LS_PFE_FW_ADDR
> -	hex "Flash address of PFE firmware"
> -	default 0x40020000
> -
> -config SYS_LS_PPA_FW_ADDR
> -	hex "PPA Firmware Addr"
> -	default 0x40060000
> -
> -config SYS_LS_PPA_ESBC_ADDR
> -	hex "PPA Firmware HDR Addr"
> -	default 0x401f4000
> -
> -config SYS_LS_PFE_ESBC_ADDR
> -	hex "PFE Firmware HDR Addr"
> -	default 0x401f8000
> -
> -endif
> -
> -source "board/freescale/common/Kconfig"

Why do you remove this file, but only add it for ls1012afrx below?
Doesn't ls1012afrdm use the same common file?

Please rebase before sending another version.

York