From mboxrd@z Thu Jan  1 00:00:00 1970
From: Scott Wood <scottwood@freescale.com>
Date: Tue, 25 Jun 2013 10:08:21 -0500
Subject: [U-Boot] [PATCH] powerpc/mpc85xx:Disable Debug TLB entry for
 non-minimal SPL
In-Reply-To: <1372136636-12909-1-git-send-email-prabhakar@freescale.com>
	(from prabhakar@freescale.com on Tue Jun 25 00:03:56 2013)
References: <1372136636-12909-1-git-send-email-prabhakar@freescale.com>
Message-ID: <1372172901.8183.14@snotra>
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

On 06/25/2013 12:03:56 AM, Prabhakar Kushwaha wrote:
> CONFIG_SPL_BUILD creates debug TLB entry, so disable it before  
> init_tlbs.
> 
> CONFIG_SPL_INIT_MINIMAL never creates any debug TLB entry, so no need
> of disable_tlb().
> 
> Signed-off-by: Prabhakar Kushwaha <prabhakar@freescale.com>
> ---
>  arch/powerpc/cpu/mpc85xx/cpu_init_early.c |    3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/arch/powerpc/cpu/mpc85xx/cpu_init_early.c  
> b/arch/powerpc/cpu/mpc85xx/cpu_init_early.c
> index 837c034..0cb2717 100644
> --- a/arch/powerpc/cpu/mpc85xx/cpu_init_early.c
> +++ b/arch/powerpc/cpu/mpc85xx/cpu_init_early.c
> @@ -180,7 +180,8 @@ void cpu_init_early_f(void)
> 
>  	invalidate_tlb(1);
> 
> -#if defined(CONFIG_SYS_PPC_E500_DEBUG_TLB) &&  
> !defined(CONFIG_SPL_BUILD) && !defined(CONFIG_NAND_SPL)
> +#if defined(CONFIG_SYS_PPC_E500_DEBUG_TLB) && \
> +		!defined(CONFIG_SPL_INIT_MINIMAL) &&  
> !defined(CONFIG_NAND_SPL)
>  	disable_tlb(CONFIG_SYS_PPC_E500_DEBUG_TLB);
>  #endif

This fails to clear the debug TLB in non-SPL_BUILD portion of a build  
that has CONFIG_SPL_INIT_MINIMAL set.

-Scott