From mboxrd@z Thu Jan  1 00:00:00 1970
From: Scott Wood <scottwood@freescale.com>
Date: Tue, 9 Apr 2013 16:57:25 -0500
Subject: [U-Boot] [PATCH 2/5][v2] powerpc/mpc85xx: new SPL support for
	IFC NAND
In-Reply-To: <5163A152.5080304@freescale.com> (from prabhakar@freescale.com
	on Tue Apr  9 00:04:18 2013)
References: <1365468905.28843.30@snotra> <5163A152.5080304@freescale.com>
Message-ID: <1365544645.31043.10@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 04/09/2013 12:04:18 AM, Prabhakar Kushwaha wrote:
> On 04/09/2013 06:25 AM, Scott Wood wrote:
>> On 04/05/2013 09:38:32 PM, Prabhakar Kushwaha wrote:
>>> On 04/06/2013 02:10 AM, Scott Wood wrote:
>>>> On 03/22/2013 02:08:05 AM, Prabhakar Kushwaha wrote:
>>>>> diff --git a/board/freescale/common/Makefile  
>>>>> b/board/freescale/common/Makefile
>>>>> index 75725b4..a4d521c 100644
>>>>> --- a/board/freescale/common/Makefile
>>>>> +++ b/board/freescale/common/Makefile
>>>>> @@ -29,6 +29,15 @@ endif
>>>>> 
>>>>>  LIB    = $(obj)libfreescale.o
>>>>> 
>>>>> +MINIMAL=
>>>>> +
>>>>> +ifdef CONFIG_SPL_BUILD
>>>>> +ifdef CONFIG_SPL_INIT_MINIMAL
>>>>> +MINIMAL=y
>>>>> +endif
>>>>> +endif
>>>>> +
>>>>> +ifndef MINIMAL
>>>>>  COBJS-$(CONFIG_FSL_CADMUS)    += cadmus.o
>>>>>  COBJS-$(CONFIG_FSL_VIA)        += cds_via.o
>>>>>  COBJS-$(CONFIG_FMAN_ENET)    += fman.o
>>>>> @@ -62,6 +71,7 @@ SUBLIB-$(CONFIG_P3041DS)    +=  
>>>>> p_corenet/libp_corenet.o
>>>>>  SUBLIB-$(CONFIG_P4080DS)    += p_corenet/libp_corenet.o
>>>>>  SUBLIB-$(CONFIG_P5020DS)    += p_corenet/libp_corenet.o
>>>>>  SUBLIB-$(CONFIG_P5040DS)    += p_corenet/libp_corenet.o
>>>>> +endif
>>>>> 
>>>>>  SRCS    := $(SOBJS:.o=.S) $(COBJS-y:.o=.c)
>>>>>  OBJS    := $(addprefix $(obj),$(COBJS-y))
>>>> 
>>>> If this is needed to avoid pulling in any files from this  
>>>> directory, why did p1_p2_rdb_pc not need this?  Is there a  
>>>> particular file that is failing to build or failing to get  
>>>> optimized away
>>> 
>>> I cannot avoid this directory :(.
>>> because new SPL compile this directory even after undef QIXIS  
>>> defines
>>> 
>>> #ifndef CONFIG_SPL_BUILD
>>> #define CONFIG_FSL_QIXIS
>>> #endif
>> 
>> Yeah, the makefiles never see CONFIG_SPL_BUILD -- it only works for  
>> C code.
>> 
>> The P1022 SPL patch just does this in the makefile:
>> 
>> +ifndef CONFIG_SPL_BUILD
>>  COBJS-$(CONFIG_FSL_NGPIXIS)    += ngpixis.o
>> +endif
>> 
>> ...which is less intrusive.  We can revisit it if we end up actually  
>> needing pixis/qixis inside an SP
> 
> Only putting qixis.c in ifndef CONFIG_SPI_BUILD may not work for  
> 9132QDS. following change is also required i.e. to put  
> CONFIG_ID_EEPROM in ifndef CONFIG_SPI_BUILD
> 
> +ifndef CONFIG_SPL_BUILD
>  COBJS-$(CONFIG_FSL_QIXIS)    += qixis.o
> -COBJS-$(CONFIG_PQ_MDS_PIB)    += pq-mds-pib.o
>  COBJS-$(CONFIG_ID_EEPROM)    += sys_eeprom.o
> +endif
> +COBJS-$(CONFIG_PQ_MDS_PIB)    += pq-mds-pib.o
> 
> It will be good to have original patch which do not compile any file  
> for SPL. if any file needs to be compiled for SPL, it can be added in  
> ifdef MINIMAL.

OK.

-Scott