Author: stepan
Date: Thu Apr 15 10:26:30 2010
New Revision: 5441
URL: https://tracker.coreboot.org/trac/coreboot/changeset/5441

Log:
the dump function assumed that the mbi data comes right after the header.
Which is not (always) the case.

Signed-off-by: Stefan Reinauer <[email protected]>
Acked-by: Stefan Reinauer <[email protected]>

Modified:
   trunk/src/northbridge/intel/i82830/i82830_smihandler.c

Modified: trunk/src/northbridge/intel/i82830/i82830_smihandler.c
==============================================================================
--- trunk/src/northbridge/intel/i82830/i82830_smihandler.c      Thu Apr 15 
07:19:29 2010        (r5440)
+++ trunk/src/northbridge/intel/i82830/i82830_smihandler.c      Thu Apr 15 
10:26:30 2010        (r5441)
@@ -262,7 +262,8 @@
 
                                getobj->banner.retsts = MSH_OK;
 #ifdef DEBUG_SMI_I82830
-                               dump(banner_id, sizeof(getobj) + len);
+                               dump(banner_id, sizeof(getobj));
+                               dump(getobj->buffer + OBJ_OFFSET, len);
 #endif
                                break;
                        }

-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to