diff options
| author | Vadim Bendebury <vbendeb@chromium.org> | 2011-10-03 16:41:50 -0700 |
|---|---|---|
| committer | Vadim Bendebury <vbendeb@chromium.org> | 2011-10-03 17:15:16 -0700 |
| commit | c80db17943527bdb9c4bdb434343389426449a77 (patch) | |
| tree | bc6c5625279f54ebb8f730620846a20db976b860 /include/coreboot | |
| parent | 5cac8a8221c9470c75f81399e3a92eeda28a9851 (diff) | |
Process all cbmem table references consistently.
A few CBMEM table entries are included in coreboot to let u-boot
discover the locations of those entries. They all are passed
using the same structure within coreboot table. This patch makes
use of that structure for all communicated entries (timestam,
console and MRC cache).
It does not make sense to keep types of pointers in the sysinfo
table, as nobody but the respective users of those fields use the
types. Let's keep the pointers as void *, this also allows to
reduce the amount of include files in systinfo.h and hide the
timestamp structures in timestamp.c.
BUG=chrome-os-partner:4200
TEST=manual
. build the new firmware and program it on a stumpy
. restart the machine
. after it comes up to ChromeOS, run the cbmem.py utility
Observe that timstamp values are displayed, and there is an entry
with index 1000 (added by u-boot to the coreboot timestamp
table).
Change-Id: Icb808145c4c62cee939eceb3d5bf5afb3bcd00d9
Signed-off-by: Vadim Bendebury <vbendeb@chromium.org>
Reviewed-on: http://gerrit.chromium.org/gerrit/8711
Reviewed-by: Stefan Reinauer <reinauer@google.com>
Diffstat (limited to 'include/coreboot')
| -rw-r--r-- | include/coreboot/timestamp.h | 16 |
1 files changed, 2 insertions, 14 deletions
diff --git a/include/coreboot/timestamp.h b/include/coreboot/timestamp.h index 8b08e8fadf..f8184f43f6 100644 --- a/include/coreboot/timestamp.h +++ b/include/coreboot/timestamp.h @@ -17,20 +17,8 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA, 02110-1301 USA */ -#ifndef __TIMESTAMP_H__ -#define __TIMESTAMP_H__ - -struct timestamp_entry { - uint32_t entry_id; - uint64_t entry_stamp; -} __attribute__((packed)); - -struct timestamp_table { - uint64_t base_time; - uint32_t max_entries; - uint32_t num_entries; - struct timestamp_entry entries[0]; /* Variable number of entries */ -} __attribute__((packed)); +#ifndef __COREBOOT_TIMESTAMP_H__ +#define __COREBOOT_TIMESTAMP_H__ enum timestamp_id { TS_BEFORE_INITRAM = 1, |