[coreboot-gerrit] New patch to review for coreboot: lib/selfboot: remove duplicate prog_set_area()

Aaron Durbin (adurbin@chromium.org) gerrit at coreboot.org
Mon Jul 11 21:30:43 CEST 2016


Aaron Durbin (adurbin at chromium.org) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/15602

-gerrit

commit ac6506311bbe2ef2435626a3868d17e2fbcffe0c
Author: Aaron Durbin <adurbin at chromium.org>
Date:   Mon Jul 11 14:22:16 2016 -0500

    lib/selfboot: remove duplicate prog_set_area()
    
    There were two successive calls to prog_set_area() which
    duplicated the same logic. Remove the unnecessary redundancy.
    
    Change-Id: I594577f8e7e78d403e7a5656f78e784e98c2c859
    Signed-off-by: Aaron Durbin <adurbin at chromium.org>
---
 src/lib/selfboot.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/src/lib/selfboot.c b/src/lib/selfboot.c
index 6178415..832dad5 100644
--- a/src/lib/selfboot.c
+++ b/src/lib/selfboot.c
@@ -508,9 +508,6 @@ void *selfload(struct prog *payload)
 	/* Update the payload's area with the bounce buffer information. */
 	prog_set_area(payload, (void *)(uintptr_t)bounce_buffer, bounce_size);
 
-	/* Update the payload's area with the bounce buffer information. */
-	prog_set_area(payload, (void *)(uintptr_t)bounce_buffer, bounce_size);
-
 	return (void *)entry;
 
 out:



More information about the coreboot-gerrit mailing list