Re: [U-Boot] [PATCH 1/6] ARM: keystone2: Split monitor code / command code

2016-03-19 Thread Albert ARIBAUD
Hello Tom,

On Wed, 16 Mar 2016 11:03:03 -0400, Tom Rini  wrote:
> When we switch to including all linker lists in U-Boot it is important
> to not include commands as that may lead to link errors due to other
> things we have already discarded.  In this case, we split the code for
> supporting the monitor out from the code for loading it.

Not sure I'm understanding this commit message. Can you clarify?

Amicalement,
-- 
Albert.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 1/6] ARM: keystone2: Split monitor code / command code

2016-03-19 Thread Tom Rini
On Wed, Mar 16, 2016 at 06:15:19PM +0100, Albert ARIBAUD wrote:
> Hello Tom,
> 
> On Wed, 16 Mar 2016 11:03:03 -0400, Tom Rini  wrote:
> > When we switch to including all linker lists in U-Boot it is important
> > to not include commands as that may lead to link errors due to other
> > things we have already discarded.  In this case, we split the code for
> > supporting the monitor out from the code for loading it.
> 
> Not sure I'm understanding this commit message. Can you clarify?

sigh, I missed a word in this series, "... linker lists in U-Boot SPL it is
..."

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 1/6] ARM: keystone2: Split monitor code / command code

2016-03-19 Thread Tom Rini
On Wed, Mar 16, 2016 at 11:03:03AM -0400, Tom Rini wrote:

> When we switch to including all linker lists in U-Boot it is important
> to not include commands as that may lead to link errors due to other
> things we have already discarded.  In this case, we split the code for
> supporting the monitor out from the code for loading it.
> 
> Cc: Vitaly Andrianov 
> Cc: Nishanth Menon 
> Cc: Lokesh Vutla 
> Signed-off-by: Tom Rini 

Reworded slightly and applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot