From mboxrd@z Thu Jan  1 00:00:00 1970
From: York Sun <yorksun@freescale.com>
Date: Tue, 26 May 2015 08:54:04 -0700
Subject: [U-Boot] [PATCH 05/20] armv8/ls2085a: call ft_pcie_setup() to
 change dts status
In-Reply-To: <1431932937-10120-5-git-send-email-prabhakar@freescale.com>
References: <1431932937-10120-1-git-send-email-prabhakar@freescale.com>
 <1431932937-10120-5-git-send-email-prabhakar@freescale.com>
Message-ID: <5564971C.9080906@freescale.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Prabhakar and Minghuan,

On 05/18/2015 12:08 AM, Prabhakar Kushwaha wrote:
> From: Minghuan Lian <Minghuan.Lian@freescale.com>
> 
>  1. The patch call ft_pcie_setup() to disable PCIe dts node if
>     corresponding PCIe controller is disabled according to RCW.
>  2. Fix LS2085a PCIe compatible
> 
> Signed-off-by: Minghuan Lian <Minghuan.Lian@freescale.com>
> Signed-off-by: Prabhakar Kushwaha <prabhakar@freescale.com>
> ---
>  board/freescale/ls2085a/ls2085a.c       | 4 ++++
>  board/freescale/ls2085aqds/ls2085aqds.c | 4 ++++
>  board/freescale/ls2085ardb/ls2085ardb.c | 4 ++++
>  include/configs/ls2085a_common.h        | 3 ++-
>  4 files changed, 14 insertions(+), 1 deletion(-)
> 
> diff --git a/board/freescale/ls2085a/ls2085a.c b/board/freescale/ls2085a/ls2085a.c
> index dd0acf2..afb99d1 100644
> --- a/board/freescale/ls2085a/ls2085a.c
> +++ b/board/freescale/ls2085a/ls2085a.c
> @@ -142,6 +142,10 @@ int ft_board_setup(void *blob, bd_t *bd)
>  	fsl_mc_ldpaa_exit(bd);
>  #endif
>  
> +#ifdef CONFIG_PCI
> +	ft_pcie_setup(blob, bd);
> +#endif
> +

Shouldn't this function be called from SoC function? It is not a board-dependent
setup, but rather depending on RCW which is an SoC feature.

York