Small bug somehow slipped there. The method body length is incorrectly computed.
authorRudolf Marek <r.marek@assembler.cz>
Mon, 2 Mar 2009 22:45:31 +0000 (22:45 +0000)
committerRudolf Marek <r.marek@assembler.cz>
Mon, 2 Mar 2009 22:45:31 +0000 (22:45 +0000)
The attached patch fixes this. I did not spotted that because the return arg is
moved just outside of method and I have overseen the closing }

Signed-off-by: Rudolf Marek <r.marek@assembler.cz>
Acked-by: Peter Stuge <peter@stuge.se>
git-svn-id: svn://svn.coreboot.org/coreboot/trunk@3968 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1

src/arch/i386/boot/acpigen.c

index 53bf6d0841c5910a626be2db0982ae17eaa86cc0..f1132f610b19e7d106d470c462571cf2832059f6 100644 (file)
@@ -239,8 +239,10 @@ int acpigen_write_PPC(u8 nr)
        acpigen_emit_byte(0xa4);
        /* arg */
        len += acpigen_write_byte(nr);
+       /* add all single bytes */
+       len += 3;
        acpigen_patch_len(len - 1);
-       return len + 3;
+       return len;
 }
 
 int acpigen_write_PSS_package(u32 coreFreq, u32 power, u32 transLat, u32 busmLat,