[U-Boot] [PATCH 1/5][v2] powerpc/mpc85xx:No NOR boot, do not compile IFC errata A003399

[Scott Wood <scottwood@freescale.com>]

On 04/09/2013 10:57:04 PM, Prabhakar Kushwaha wrote:
> On 04/10/2013 05:11 AM, Scott Wood wrote:
>> On 04/08/2013 11:56:25 PM, Prabhakar Kushwaha wrote:
>>> I tried just after CONFIG_P1010 but it is giving compilation error.
>> 
>> What error?
> 
> 
> make[1]: Entering directory  
> `/home/b32579/Workspace/MERGE/OPENSRC/U-BOOT/u-boot'
> /opt/freescale/usr/local/gcc-4.5.55-eglibc-2.11.55/powerpc-linux-gnu/bin/powerpc-linux-gnu-gcc  
> -DDO_DEPS_ONLY \
>         -g  -Os   -fpic -mrelocatable -ffunction-sections  
> -fdata-sections -meabi -D__KERNEL__ -DCONFIG_SYS_TEXT_BASE=0x11000000  
> -I/home/b32579/Workspace/MERGE/OPENSRC/U-BOOT/u-boot/include  
> -fno-builtin -ffreestanding -nostdinc -isystem  
> /opt/freescale/usr/local/gcc-4.5.55-eglibc-2.11.55/powerpc-linux-gnu/lib/gcc/powerpc-linux-gnu/4.5.1/include  
> -pipe  -DCONFIG_PPC -D__powerpc__ -ffixed-r2 -Wa,-me500 -msoft-float  
> -mno-string  -mno-spe -Wall -Wstrict-prototypes -fno-stack-protector  
> -Wno-format-nonliteral -Wno-format-security    \
>         -o lib/asm-offsets.s lib/asm-offsets.c -c -S
> In file included from  
> /home/b32579/Workspace/MERGE/OPENSRC/U-BOOT/u-boot/include/common.h:174:0,
>                  from lib/asm-offsets.c:18:
> /home/b32579/Workspace/MERGE/OPENSRC/U-BOOT/u-boot/include/asm/global_data.h:98:21:  
> error: 'CONFIG_SYS_NUM_TLBCAMS' undeclared here (not in a function)
> make[1]: *** [lib/asm-offsets.s] Error 1
> 
> because of these defines in config_mpc85xx.h
> #if defined(CONFIG_E500MC)
> #define CONFIG_SYS_NUM_TLBCAMS        64
> #elif defined(CONFIG_E500)
> #define CONFIG_SYS_NUM_TLBCAMS        16
> #endif

You also need CONFIG_E500 defined before you include config_mpc85xx.h  
-- although it might be better to define CONFIG_E500[MC] in  
config_mpc85xx.h based on the SoC type, and move the  
CONFIG_SYS_NUM_TLBCAMS setting to the bottom of the file.

>>> Just to mention, I need to provide relative path in order to  
>>> compile.
>> 
>> What specifically did you try and what error did it produce?
> 
> I did not try anything.

If you didn't try a normal include patch and see a failure, why do you  
say you need to provide a relative path?

#include <asm/config_mpc85xx.h> works for me.

-Scott