From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Wed, 26 Apr 2017 09:35:56 -0400 Subject: [U-Boot] [PATCH 3/3] spl: dfu: reduce spl-dfu MLO size In-Reply-To: <1493212449-26863-4-git-send-email-ravibabu@ti.com> References: <1493212449-26863-1-git-send-email-ravibabu@ti.com> <1493212449-26863-4-git-send-email-ravibabu@ti.com> Message-ID: <20170426133556.GX12511@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Wed, Apr 26, 2017 at 06:44:09PM +0530, Ravi Babu wrote: > Since spl-dfu does not dfu-reset, there is no need > of run_command_cli, hence compiling out cli.c and > cli_hush.c to reduce the spl-dfu memory foot print. > > Signed-off-by: Ravi Babu [snip] > diff --git a/drivers/dfu/Makefile b/drivers/dfu/Makefile > index 61f2b71..ef48f36 100644 > --- a/drivers/dfu/Makefile > +++ b/drivers/dfu/Makefile > @@ -6,8 +6,10 @@ > # > > obj-$(CONFIG_USB_FUNCTION_DFU) += dfu.o > +ifndef CONFIG_SPL_BUILD > obj-$(CONFIG_DFU_MMC) += dfu_mmc.o > obj-$(CONFIG_DFU_NAND) += dfu_nand.o > -obj-$(CONFIG_DFU_RAM) += dfu_ram.o > obj-$(CONFIG_DFU_SF) += dfu_sf.o > obj-$(CONFIG_DFU_TFTP) += dfu_tftp.o > +endif > +obj-$(CONFIG_DFU_RAM) += dfu_ram.o We should discard at link time the unreachable parts here in SPL, no? > diff --git a/include/dfu.h b/include/dfu.h > index f39d3f1..b53ae80 100644 > --- a/include/dfu.h > +++ b/include/dfu.h > @@ -203,7 +203,7 @@ static inline void dfu_set_defer_flush(struct dfu_entity *dfu) > int dfu_write_from_mem_addr(struct dfu_entity *dfu, void *buf, int size); > > /* Device specific */ > -#ifdef CONFIG_DFU_MMC > +#if defined(CONFIG_DFU_MMC) && !defined(CONFIG_SPL_BUILD) I don't like the initial condition we have here, adding the !SPL_BUILD test makes this even worse, lets not do that. -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: