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

[Lukasz Majewski <lukma@denx.de>]

Hi Faiz,

> Hi,
> 
> On Wednesday 14 February 2018 08:04 PM, Tom Rini wrote:
> > 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@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?   
> 
> No.
> 
> 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!  
> 
> Do you mean non-RAM DFU cases? Block out everything but RAM?

I think yes - only TI is using DFU RAM to download u-boot to configured
DDRAM and execute it from there.

So I agree with Tom, other gargets and DFU parts shall _not_ be enabled
for SPL (as it consumes a lot of space).

> 
> 
> Thanks,
> Faiz


Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180214/5f4ebe41/attachment.sig>