[U-Boot] [PATCH] dfu: Limit dfu_mmc to non-SPL case

Tom Rini trini at konsulko.com
Wed Feb 14 14:34:03 UTC 2018


On Wed, Feb 14, 2018 at 07:59:46PM +0530, Faiz Abbas wrote:

> Since 82b9dc63 ("Only build cli_readline.o for CMDLINE on non-SPL"),
> cli_simple.c is only being built for non-SPL case. However, dfu_mmc
> requires cli_simple.c to build and dfu fails to build for SPL when
> CONFIG_HUSH_PARSER is disabled.
> 
> Therefore, limit dfu_mmc to non-SPL case only.
> 
> Signed-off-by: Faiz Abbas <faiz_abbas at ti.com>
> ---
>  drivers/dfu/Makefile | 2 ++
>  drivers/dfu/dfu.c    | 2 ++
>  2 files changed, 4 insertions(+)

Does DFU for MMC make sense inside of SPL?  If so, you should follow
82b9dc63 and CONFIG_FSL_DDR_INTERACTIVE and bring back in cli_simple.o
(and cli_readline.o, if needed) for CONFIG_SPL_DFU_MMC or so.  If not,
we should more cleanly and clearly block out all non-DFU RAM cases for
SPL.  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/20180214/f891fac4/attachment.sig>


More information about the U-Boot mailing list