[coreboot-gerrit] Patch set updated for coreboot: f0ac067 i82801gx: Move to implicit length patching

Vladimir Serbinenko (phcoder@gmail.com) gerrit at coreboot.org
Wed Nov 5 09:36:14 CET 2014


Vladimir Serbinenko (phcoder at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/7328

-gerrit

commit f0ac0679f68111e6fb707e65fc11b6f10b69a936
Author: Vladimir Serbinenko <phcoder at gmail.com>
Date:   Tue Nov 4 21:20:56 2014 +0100

    i82801gx: Move to implicit length patching
    
    Change-Id: Idba0f33d231084d02392e23026f567c30f77b316
    Signed-off-by: Vladimir Serbinenko <phcoder at gmail.com>
---
 src/southbridge/intel/i82801gx/lpc.c | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/src/southbridge/intel/i82801gx/lpc.c b/src/southbridge/intel/i82801gx/lpc.c
index 10e40be..0618676 100644
--- a/src/southbridge/intel/i82801gx/lpc.c
+++ b/src/southbridge/intel/i82801gx/lpc.c
@@ -606,16 +606,15 @@ static void southbridge_inject_dsdt(void)
 	global_nvs_t *gnvs = cbmem_add (CBMEM_ID_ACPI_GNVS, sizeof (*gnvs));
 
 	if (gnvs) {
-		int scopelen;
 		memset(gnvs, 0, sizeof(*gnvs));
 		acpi_create_gnvs(gnvs);
 		/* And tell SMI about it */
 		smm_setup_structures(gnvs, NULL, NULL);
 
 		/* Add it to SSDT.  */
-		scopelen = acpigen_write_scope("\\");
-		scopelen += acpigen_write_name_dword("NVSA", (u32) gnvs);
-		acpigen_patch_len(scopelen - 1);
+		acpigen_write_scope("\\");
+		acpigen_write_name_dword("NVSA", (u32) gnvs);
+		acpigen_pop_len();
 	}
 }
 



More information about the coreboot-gerrit mailing list