[U-Boot] [PATCH v1 01/10] common: do not compile common fastboot code when building the SPL

Tom Rini
2017-12-21T14:46:23Z

On Thu, Dec 21, 2017 at 12:53:47PM +0100, Jean-Jacques Hiblot wrote:

> This is not required as fastboot can't be started from SPL.
>
> Signed-off-by: Jean-Jacques Hiblot <jjhiblot@ti.com>
> ---
>
> common/Makefile | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/common/Makefile b/common/Makefile
> index 1416620..c7bde23 100644
> --- a/common/Makefile
> +++ b/common/Makefile
> @@ -109,6 +109,7 @@ obj-$(CONFIG_IO_TRACE) += iotrace.o
> obj-y += memsize.o
> obj-y += stdio.o
>
> +ifndef CONFIG_SPL_BUILD
> # This option is not just y/n - it can have a numeric value
> ifdef CONFIG_FASTBOOT_FLASH
> obj-y += image-sparse.o
> @@ -119,6 +120,7 @@ ifdef CONFIG_FASTBOOT_FLASH_NAND_DEV
> obj-y += fb_nand.o
> endif
> endif
> +endif
>
> ifdef CONFIG_CMD_EEPROM_LAYOUT
> obj-y += eeprom/eeprom_field.o eeprom/eeprom_layout.o

We don't need this as all of the code (and text) is discarded. Checked
with am335x_evm_usbspl as that does set FASTBOOT. Thanks!

--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20171221/a7f9db5b/attachment.sig>