[coreboot-gerrit] Patch set updated for coreboot: 5b36fab i82801ix: Move to implicit length patching

Vladimir Serbinenko (phcoder@gmail.com) gerrit at coreboot.org
Tue Nov 4 22:23:23 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/7329

-gerrit

commit 5b36fabb1429a484b5713dc7d0a873ebc40b4249
Author: Vladimir Serbinenko <phcoder at gmail.com>
Date:   Tue Nov 4 21:21:06 2014 +0100

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

diff --git a/src/southbridge/intel/i82801ix/lpc.c b/src/southbridge/intel/i82801ix/lpc.c
index 664088c..dc2cfe8 100644
--- a/src/southbridge/intel/i82801ix/lpc.c
+++ b/src/southbridge/intel/i82801ix/lpc.c
@@ -544,16 +544,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();
 	}
 }
 #endif



More information about the coreboot-gerrit mailing list