[coreboot-gerrit] New patch to review for coreboot: selfboot: Move the usable-RAM check into a function

[Martin Roth]

Martin Roth ([e-mail]) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/16585

-gerrit

commit 826b9279eb99d0260cf295622603dfb1312f3606
Author: Simon Glass 
Date:   Sun Sep 4 16:37:04 2016 -0600

selfboot: Move the usable-RAM check into a function

In preparation for making this check optional, move it into its own
    function. load_self_segments() is already long and we don't want to make
    it longer.

BUG=chrome-os-partner:56314
    BRANCH=none
    TEST=boot on gru and see that BL31 loads and runs correctly

Change-Id: If48d2bf485a23f21c5599670e77a7b8b098f1a88
    Signed-off-by: Martin Roth 
    Original-Commit-Id: 2381e02efa2033857ac06acbc4f0c0dd08de1080
    Original-Change-Id: I005e5e4d9b2136605bdd95e9060655df7a8238cb
    Original-Signed-off-by: Simon Glass 
    Original-Reviewed-on: https://chromium-review.googlesource.com/381092
    Original-Reviewed-by: Aaron Durbin 
---
 src/lib/selfboot.c | 26 ++++++++++++++++++--------
 1 file changed, 18 insertions(+), 8 deletions(-)

diff --git a/src/lib/selfboot.c b/src/lib/selfboot.c
index 2fdf8ce..1ce7f94 100644
--- a/src/lib/selfboot.c
+++ b/src/lib/selfboot.c
@@ -328,23 +328,20 @@ static int build_self_segment_list(
    return 1;
 }

-static int load_self_segments(
-	struct segment *head,
-	struct prog *payload)
+static int payload_targets_usable_ram(struct segment *head)
 {
-	struct segment *ptr;
    const unsigned long one_meg = (1UL << 20);
-	unsigned long bounce_high = lb_end;
+	struct segment *ptr;

-	for(ptr = head->next; ptr != head; ptr = ptr->next) {
+	for (ptr = head->next; ptr != head; ptr = ptr->next) {
    	if (bootmem_region_targets_usable_ram(ptr->s_dstaddr,
-							ptr->s_memsz))
+						      ptr->s_memsz))
    		continue;

if (ptr->s_dstaddr < one_meg &&
    	    (ptr->s_dstaddr + ptr->s_memsz) <= one_meg) {
    		printk(BIOS_DEBUG,
-				"Payload being loaded below 1MiB "
+				"Payload being loaded at below 1MiB "
    			"without region being marked as RAM usable.\n");
    		continue;
    	}
@@ -357,6 +354,19 @@ static int load_self_segments(
    	return 0;
    }

+	return 1;
+}
+
+static int load_self_segments(
+	struct segment *head,
+	struct prog *payload)
+{
+	struct segment *ptr;
+	unsigned long bounce_high = lb_end;
+
+	if (!payload_targets_usable_ram(head))
+		return 0;
+
    for(ptr = head->next; ptr != head; ptr = ptr->next) {
    	/*
    	 * Add segments to bootmem memory map before a bounce buffer is