Hi Tom, On 22.12.2017 04:13, Tom Rini wrote: > There are no CMDs to be run in SPL and no need for any of their lists. > > CC: Stefano Babic <[email protected]> > Cc: Fabio Estevam <[email protected]> > Cc: Jagan Teki <[email protected]> > Cc: Maxime Ripard <[email protected]> > Cc: "Andreas Bießmann" <[email protected]> > Cc: Michal Simek <[email protected]> > Cc: Daniel Schwierzeck <[email protected]> > Cc: Mario Six <[email protected]> > Cc: Wolfgang Denk <[email protected]> > Cc: York Sun <[email protected]> > Cc: Simon Glass <[email protected]> > Cc: Bin Meng <[email protected]> > Cc:Wenyou Yang <[email protected]> > Cc: Lokesh Vutla <[email protected]> > Signed-off-by: Tom Rini <[email protected]> > --- > arch/arm/cpu/arm1136/u-boot-spl.lds | 3 +++ > arch/arm/cpu/arm926ejs/mxs/u-boot-spl.lds | 3 +++ > arch/arm/cpu/arm926ejs/orion5x/u-boot-spl.lds | 3 +++ > arch/arm/cpu/arm926ejs/spear/u-boot-spl.lds | 3 +++ > arch/arm/cpu/armv7/sunxi/u-boot-spl.lds | 3 +++ > arch/arm/cpu/armv8/u-boot-spl.lds | 3 +++ > arch/arm/cpu/u-boot-spl.lds | 3 +++ > arch/arm/mach-at91/arm926ejs/u-boot-spl.lds | 3 +++ > arch/arm/mach-at91/armv7/u-boot-spl.lds | 3 +++ > arch/arm/mach-omap2/u-boot-spl.lds | 3 +++ > arch/arm/mach-zynq/u-boot-spl.lds | 3 +++ > arch/microblaze/cpu/u-boot-spl.lds | 3 +++ > arch/mips/cpu/u-boot-spl.lds | 3 +++ > arch/powerpc/cpu/mpc83xx/u-boot-spl.lds | 3 +++ > arch/powerpc/cpu/mpc85xx/u-boot-spl.lds | 3 +++ > arch/sandbox/cpu/u-boot-spl.lds | 2 ++ > arch/x86/cpu/u-boot-spl.lds | 3 +-- > 17 files changed, 48 insertions(+), 2 deletions(-) >
...
> diff --git a/arch/mips/cpu/u-boot-spl.lds b/arch/mips/cpu/u-boot-spl.lds
> index 07004ea11155..19444e3e600e 100644
> --- a/arch/mips/cpu/u-boot-spl.lds
> +++ b/arch/mips/cpu/u-boot-spl.lds
> @@ -11,6 +11,9 @@ OUTPUT_ARCH(mips)
> ENTRY(_start)
> SECTIONS
> {
> + /* There are no CMDs in SPL */
> + /DISCARD/ : { *(.u_boot_list_2_cmd_*) }
> +
> . = 0x00000000;
>
> . = ALIGN(4);
does a DISCARD() work together with the KEEP() in this block?
#ifdef CONFIG_SPL_DM
. = ALIGN(4);
.u_boot_list : {
KEEP(*(SORT(.u_boot_list*)));
} > .spl_mem
#endif
I suggest to do something like this:
KEEP(*(SORT(.u_boot_list_2_uclass*)));
KEEP(*(SORT(.u_boot_list_2_driver*)));
Then all u_boot_list_* would be discarded by default except the two
lists above when a board selects CONFIG_SPL_DM. Also this would have the
benefit of discarding all u_boot_list_2_env* entries.
--
- Daniel
signature.asc
Description: OpenPGP digital signature
_______________________________________________ U-Boot mailing list [email protected] https://lists.denx.de/listinfo/u-boot

