[coreboot-gerrit] Change in ...coreboot[master]: mb/emulation/qemu: Fix fw_cfg regression

[Patrick Rudolph (Code Review)]

Patrick Rudolph has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/31593

Change subject: mb/emulation/qemu: Fix fw_cfg regression
......................................................................

mb/emulation/qemu: Fix fw_cfg regression

The change I48cc943aaa999e4323e9d7e5dd666c5316533dcc
"mb/emulation/qemu-i440fx: change file handling"

introduced a regression where it loads only 4 bytes of the
ACPI table, instead of the whole table.

Load the whole ACPI table.

Tested on Qemu using GNU/Linux.

Change-Id: Ibacbf7caab9be5f181c12e9dd39a2893b13cf6c9
Signed-off-by: Patrick Rudolph 
---
M src/mainboard/emulation/qemu-i440fx/fw_cfg.c
1 file changed, 1 insertion(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/93/31593/1

diff --git a/src/mainboard/emulation/qemu-i440fx/fw_cfg.c b/src/mainboard/emulation/qemu-i440fx/fw_cfg.c
index 6a9ef03..a029b38 100644
--- a/src/mainboard/emulation/qemu-i440fx/fw_cfg.c
+++ b/src/mainboard/emulation/qemu-i440fx/fw_cfg.c
@@ -251,7 +251,7 @@

printk(BIOS_DEBUG, "QEMU: loading "%s" to 0x%lx (len %d)\n",
    		       s[i].alloc.file, current, f.size);
-			fw_cfg_get(f.select, (void *)current, sizeof(current));
+			fw_cfg_get(f.select, (void *)current, f.size);
    		addrs[i] = current;
    		current += f.size;
    		break;