[coreboot-gerrit] Patch set updated for coreboot: util/amdfwtool: Increase space used for structures

Marc Jones (marc@marcjonesconsulting.com) gerrit at coreboot.org
Fri Oct 28 19:30:53 CEST 2016


Marc Jones (marc at marcjonesconsulting.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17148

-gerrit

commit 41f4198a1bc46d3032a842787bfdf5a78351ad12
Author: Marc Jones <marcj303 at gmail.com>
Date:   Tue Sep 20 22:55:54 2016 -0600

    util/amdfwtool: Increase space used for structures
    
    Double the space for psp2dir to 0x200.
    
    Based on advice from AMD, increase the region containing
    the signature to 4K.
    
    Original-Signed-off-by: Marc Jones <marcj303 at gmail.com>
    Original-Signed-off-by: Marshall Dawson <marshalldawson3rd at gmail.com>
    (cherry picked from commit e03a9402711c3a210816d0aa32865491a0523639)
    
    Change-Id: If60132f913928bab0c2fe4aacedf342080929599
    Signed-off-by: Marc Jones <marcj303 at gmail.com>
---
 util/amdfwtool/amdfwtool.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/util/amdfwtool/amdfwtool.c b/util/amdfwtool/amdfwtool.c
index 3af0078..ef8795b 100644
--- a/util/amdfwtool/amdfwtool.c
+++ b/util/amdfwtool/amdfwtool.c
@@ -538,7 +538,7 @@ int main(int argc, char **argv)
 		}
 	}
 
-	current = ALIGN(current, 0x100);
+	current = ALIGN(current, 0x1000);
 	current = integrate_firmwares(rom, current, amd_romsig, amd_fw_table);
 
 	if (pspflag == 1) {
@@ -555,7 +555,7 @@ int main(int argc, char **argv)
 		current = ALIGN(current, 0x10000); /* PSP2 dir */
 		psp2dir = rom + current;
 		amd_romsig[5] = current + ROM_BASE_ADDRESS;
-		current += 0x100;	/* Add conservative size of psp2dir. */
+		current += 0x200;	/* Add conservative size of psp2dir. */
 
 #if PSP_COMBO
 		/* TODO: remove the hardcode. */



More information about the coreboot-gerrit mailing list