Re: [U-Boot] [PATCH 1/6] ARM: keystone2: Split monitor code / command code
Hello Tom, On Wed, 16 Mar 2016 11:03:03 -0400, Tom Riniwrote: > 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
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 Riniwrote: > > 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
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