From mboxrd@z Thu Jan  1 00:00:00 1970
From: Prabhakar Kushwaha <prabhakar@freescale.com>
Date: Sat, 6 Apr 2013 07:58:14 +0530
Subject: [U-Boot] [PATCH 1/5][v2] powerpc/mpc85xx:No NOR boot,
 do not compile IFC errata A003399
In-Reply-To: <1365194033.17535.9@snotra>
References: <1363936068-17107-1-git-send-email-prabhakar@freescale.com>
	<1365194033.17535.9@snotra>
Message-ID: <515F883E.8090107@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

On 04/06/2013 02:03 AM, Scott Wood wrote:
> On 03/22/2013 02:07:48 AM, Prabhakar Kushwaha wrote:
>> IFC errata A003399 is valid for IFC NOR boot i.e.if no on-board NOR 
>> flash or
>> no NOR boot, do not compile its workaround.
>>
>> Signed-off-by: Prabhakar Kushwaha <prabhakar@freescale.com>
>> ---
>>  Based upon git://git.denx.de/u-boot.git branch master
>>  Changes for v2:
>>      - Add CONFIG_A003399_NOR_WORKAROUND for errata workaround
>>     - Add CONFIG_A003399_NOR_WORKAROUND in README
>>
>>  README                                    |    4 ++++
>>  arch/powerpc/cpu/mpc85xx/cpu_init_early.c |    9 ++++++---
>>  arch/powerpc/cpu/mpc8xxx/fsl_ifc.c        |    2 +-
>>  3 files changed, 11 insertions(+), 4 deletions(-)
>>
>> diff --git a/README b/README
>> index 7f2506a..c4ad69d 100644
>> --- a/README
>> +++ b/README
>> @@ -414,6 +414,10 @@ The following options need to be configured:
>>          See Freescale App Note 4493 for more information about
>>          this erratum.
>>
>> +        CONFIG_A003399_NOR_WORKAROUND
>> +        Enables a workaround for IFC erratum A003399. It is only
>> +        requred during NOR boot.
>> +
>>          CONFIG_SYS_FSL_CORENET_SNOOPVEC_COREONLY
>>
>>          This is the value to write into CCSR offset 0x18600
>> diff --git a/arch/powerpc/cpu/mpc85xx/cpu_init_early.c 
>> b/arch/powerpc/cpu/mpc85xx/cpu_init_early.c
>> index dacfdd1..3a4bfc5 100644
>> --- a/arch/powerpc/cpu/mpc85xx/cpu_init_early.c
>> +++ b/arch/powerpc/cpu/mpc85xx/cpu_init_early.c
>> @@ -25,7 +25,8 @@
>>
>>  DECLARE_GLOBAL_DATA_PTR;
>>
>> -#if defined(CONFIG_SYS_FSL_ERRATUM_IFC_A003399) && 
>> !defined(CONFIG_SYS_RAMBOOT)
>> +#if defined(CONFIG_SYS_FSL_ERRATUM_IFC_A003399)\
>> +    && defined(CONFIG_A003399_NOR_WORKAROUND)
>
> Why would CONFIG_A003399_NOR_WORKAROUND be enabled if 
> CONFIG_SYS_FSL_ERRATUM_IFC_A003399 isn't?


originally i tried the same but it did not work. As P1010RDB.h does not 
include config_mpc85xx.h.   Only arch/powerpc/cpu/mpc85xx/*.c files 
includes both P1010RDB.h and config_mpc85xx.h

Regards,
Prabhakar