New patch to review for coreboot: 11c098e Print segment clean up information only when required.

[Marc Jones]

Marc Jones ([e-mail]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/7937

-gerrit

commit 11c098e407d91ddfd5b23c0b7929b834a8f40756
Author: Vadim Bendebury 
Date:   Tue May 13 15:43:58 2014 -0700

Print segment clean up information only when required.

Eliminate duplicated printout and if needed, print only changed
    information.

BUG=none
    TEST=verified that the 'New segment dstaddr...' message is not
         duplicated anymore

Original-Change-Id: Ia13593394fccbb225f2bd9ab2b9228bac29d50fb
    Original-Signed-off-by: Vadim Bendebury 
    Original-Reviewed-on: https://chromium-review.googlesource.com/199672
    Original-Reviewed-by: Aaron Durbin 
    (cherry picked from commit aadf018821ebfa63d6ac9d2429ae1fb483dd6cb3)
    Signed-off-by: Marc Jones

Change-Id: I7544bddc4026191395cfe3b8ac66256ec223391e
---
 src/lib/selfboot.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/lib/selfboot.c b/src/lib/selfboot.c
index 8e9e0de..de059b0 100644
--- a/src/lib/selfboot.c
+++ b/src/lib/selfboot.c
@@ -252,9 +252,10 @@ static int build_self_segment_list(
    		/* Clean up the values */
    		if (new->s_filesz > new->s_memsz)  {
    			new->s_filesz = new->s_memsz;
+				printk(BIOS_DEBUG,
+				       "  cleaned up filesize 0x%lx\n",
+				       new->s_filesz);
    		}
-			printk(BIOS_DEBUG, "  (cleaned up) New segment addr 0x%lx size 0x%lx offset 0x%lx filesize 0x%lx\n",
-				new->s_dstaddr, new->s_memsz, new->s_srcaddr, new->s_filesz);
    		break;

case PAYLOAD_SEGMENT_BSS: