[U-Boot] SPL framework re-design

Scott Wood
2011-06-20T16:19:29Z

On Sun, 19 Jun 2011 15:52:29 +0530
"V, Aneesh" <aneesh@ti.com> wrote:

> On Sat, Jun 18, 2011 at 3:58 AM, Scott Wood <scottwood@freescale.com> wrote:
> > On Fri, 17 Jun 2011 22:18:57 +0530
> > Aneesh V <aneesh@ti.com> wrote:
> >
> >> @@ -1158,6 +1164,7 @@ clobber: ? ? ? ?clean
> >> ? ? ? @[ ! -d $(obj)nand_spl ] || find $(obj)nand_spl -name "*" -type l
> >> -print | xargs rm -f
> >> ? ? ? @[ ! -d $(obj)onenand_ipl ] || find $(obj)onenand_ipl -name "*" -type
> >> l -print | xargs rm -f
> >> ? ? ? @[ ! -d $(obj)mmc_spl ] || find $(obj)mmc_spl -name "*" -type l
> >> -print | xargs rm -f
> >> + ? ? @[ ! -d $(obj)spl ] || find $(obj)mmc_spl -name "*" -type l -print |
> >> xargs rm -f
> >
> > That last mmc_spl should just be spl.
> >
> >> +LIBS-$(CONFIG_SYS_SPL_NAND_SUPPORT) += mmc/libnand.o
> >> +LIBS-$(CONFIG_SYS_SPL_ONENAND_SUPPORT) += mmc/libonenand.o
>
> Oops!! That was a copy paste error. It was intended to be:
> +LIBS-$(CONFIG_SYS_SPL_NAND_SUPPORT) += nand/libnand.o
> +LIBS-$(CONFIG_SYS_SPL_ONENAND_SUPPORT) += onenand/libonenand.o

Still, what would go in those files?

It'd have to be something more specific, like:

LIBS-$(CONFIG_SYS_SPL_NAND_SIMPLE) += nand/nand_boot.o
LIBS-$(CONFIG_SYS_SPL_NAND_FSL_ELBC) += nand/nand_boot_fsl_elbc.o
...

Hmm, I guess you'd stick this in a recursive makefile. Seems like overkill.

> The top-level Makefile doesn't include any source files by itself.
> All SPL content comes from one or more of libraries. So, there
> will be at least one library defined, I believe(unless there
> is an error, of course).

Couldn't there be only OBJS?

-Scott