From mboxrd@z Thu Jan  1 00:00:00 1970
From: York Sun <yorksun@freescale.com>
Date: Tue, 14 Oct 2014 13:29:16 -0700
Subject: [U-Boot] [PATCH v2 4/7] common: spl: Add interactive DDR
 debugger support for SPL
In-Reply-To: <1411727622-26557-5-git-send-email-b18965@freescale.com>
References: <1411727622-26557-1-git-send-email-b18965@freescale.com>
	<1411727622-26557-5-git-send-email-b18965@freescale.com>
Message-ID: <543D879C.8000806@freescale.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On 09/26/2014 03:33 AM, Alison Wang wrote:
> For LS102xA, interactive DDR debugger is still needed in SPL part.
> So build the needed files in SPL image too.
> 
> Signed-off-by: Alison Wang <alison.wang@freescale.com>
> ---
> Change log:
>  v2: No change.
> 
>  common/Makefile | 20 ++++++++++----------
>  1 file changed, 10 insertions(+), 10 deletions(-)
> 
> diff --git a/common/Makefile b/common/Makefile
> index b19d379..a166e4c 100644
> --- a/common/Makefile
> +++ b/common/Makefile
> @@ -8,22 +8,12 @@
>  # core
>  ifndef CONFIG_SPL_BUILD
>  obj-y += main.o
> -obj-y += command.o
>  obj-y += exports.o
>  obj-y += hash.o
>  ifdef CONFIG_SYS_HUSH_PARSER
>  obj-y += cli_hush.o
>  endif
>  
> -# We always have this since drivers/ddr/fs/interactive.c needs it
> -obj-y += cli_simple.o
> -
> -obj-y += cli.o
> -obj-y += cli_readline.o
> -obj-y += s_record.o
> -obj-y += xyzModem.o
> -obj-y += cmd_disk.o
> -
>  # This option is not just y/n - it can have a numeric value
>  ifdef CONFIG_BOOTDELAY
>  obj-y += autoboot.o
> @@ -265,4 +255,14 @@ obj-y += aboot.o
>  obj-y += fb_mmc.o
>  endif
>  
> +# We always have this since drivers/ddr/fs/interactive.c needs it
> +obj-y += cli_simple.o
> +
> +obj-y += cli.o
> +obj-y += cli_readline.o
> +obj-y += command.o
> +obj-y += s_record.o
> +obj-y += xyzModem.o
> +obj-y += cmd_disk.o
> +
>  CFLAGS_env_embedded.o := -Wa,--no-warn -DENV_CRC=$(shell tools/envcrc 2>/dev/null)
> 


Can you explain more in commit message why moving them around adds interactive
DDR debugging?

York