From mboxrd@z Thu Jan  1 00:00:00 1970
From: Tom Rini <trini@ti.com>
Date: Wed, 29 Aug 2012 15:28:50 -0700
Subject: [U-Boot] [PATCH v5 4/5] OMAP: networking support for SPL
In-Reply-To: <CANr=Z=YJJH=OrPo4V6_4acFL07W1MirATJ1a_eNrNtTrwze=xA@mail.gmail.com>
References: <1344201713-10291-5-git-send-email-ilya.yanok@cogentembedded.com>
	<1344326860-8772-1-git-send-email-ilya.yanok@cogentembedded.com>
	<CANr=Z=YJJH=OrPo4V6_4acFL07W1MirATJ1a_eNrNtTrwze=xA@mail.gmail.com>
Message-ID: <503E97A2.9060709@ti.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 08/29/2012 02:25 PM, Joe Hershberger wrote:

[snip]
>>  #include <common.h>
>> +#ifdef CONFIG_SPL_BUILD
>> +/* SPL needs only BOOTP + TFTP so undefine other stuff to save space */
>> +#undef CONFIG_CMD_DHCP
>> +#undef CONFIG_CMD_CDP
>> +#undef CONFIG_CMD_DNS
>> +#undef CONFIG_CMD_LINK_LOCAL
>> +#undef CONFIG_CMD_NFS
>> +#undef CONFIG_CMD_PING
>> +#undef CONFIG_CMD_RARP
>> +#undef CONFIG_CMD_SNTP
>> +#undef CONFIG_CMD_TFTPPUT
>> +#undef CONFIG_CMD_TFTPSRV
>> +#endif
> 
> Is this the best place to do this?  Seems it would be clearer to
> modify config_cmd_default.h or add a config_cmd_spl.h that will
> undefine them, and include that.

I was thinking about that too, include/config_cmd_spl.h should probably
be how this happens.

-- 
Tom