diff options
author | Nico Huber <nico.huber@secunet.com> | 2012-12-07 15:50:40 +0100 |
---|---|---|
committer | Nico Huber <nico.huber@secunet.com> | 2012-12-10 16:45:24 +0100 |
commit | 593f577ffab9663fdf95fc7cc0b5f5bc623b52ae (patch) | |
tree | b26dc237b508281ab62676c54f02930dda276374 /payloads/libpayload/arch | |
parent | c95f2f5615963deb911684b321c3e14dd2edd4d9 (diff) | |
download | coreboot-593f577ffab9663fdf95fc7cc0b5f5bc623b52ae.tar.xz |
libpayload: Fix use of virtual pointers in sysinfo
In I4c456f5, I falsely identified struct cb_string.string as a pointer
which it is not. So we don't need phys_to_virt() here.
Change-Id: I3e2b6226ae2b0672dfc6e0fa4f6990e14e1b7089
Signed-off-by: Nico Huber <nico.huber@secunet.com>
Reviewed-on: http://review.coreboot.org/1987
Tested-by: build bot (Jenkins)
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Diffstat (limited to 'payloads/libpayload/arch')
-rw-r--r-- | payloads/libpayload/arch/i386/coreboot.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/payloads/libpayload/arch/i386/coreboot.c b/payloads/libpayload/arch/i386/coreboot.c index fc22d9a05a..d39af4ab5d 100644 --- a/payloads/libpayload/arch/i386/coreboot.c +++ b/payloads/libpayload/arch/i386/coreboot.c @@ -150,7 +150,7 @@ static void cb_parse_framebuffer(void *ptr, struct sysinfo_t *info) static void cb_parse_string(unsigned char *ptr, char **info) { - *info = (char *)phys_to_virt(((struct cb_string *)ptr)->string); + *info = (char *)((struct cb_string *)ptr)->string; } static int cb_parse_header(void *addr, int len, struct sysinfo_t *info) |