GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

util/nvramtool: Fix typos & remove unneeded whitespace 

Change-Id: I0a704cba80d0439ae95db34a6b73df7be5b3b862
Signed-off-by: Elyes HAOUAS <ehaouas@noos.fr>
Reviewed-on: https://review.coreboot.org/28290
Reviewed-by: Felix Held <felix-coreboot@felixheld.de>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
Elyes HAOUAS 2018-08-23 18:19:52 +02:00 committed by Felix Held
parent 982c7555ad
commit fcd70085c2
1 changed files with 49 additions and 49 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

54
util/nvramtool/coreboot_tables.h
View File

@ -15,28 +15,28 @@
/* The coreboot table information is for conveying information /* The coreboot table information is for conveying information
* from the firmware to the loaded OS image. Primarily this * from the firmware to the loaded OS image. Primarily this
* is expected to be information that cannot be discovered by * is expected to be information that cannot be discovered by
* other means, such as quering the hardware directly. * other means, such as querying the hardware directly.
* *
* All of the information should be Position Independent Data. * All of the information should be Position Independent Data.
* That is it should be safe to relocated any of the information * That is, it should be safe to relocate any of the information
* without it's meaning/correctnes changing. For table that * without changing its meaning/correctness. For tables that
* can reasonably be used on multiple architectures the data * can reasonably be used on multiple architectures the data
* size should be fixed. This should ease the transition between * size should be fixed. This should ease the transition between
* 32 bit and 64 bit architectures etc. * 32 bit and 64 bit architectures etc.
* *
* The completeness test for the information in this table is: * The completeness test for the information in this table is:
* - Can all of the hardware be detected? * - Can all of the hardware be detected?
* - Are the per motherboard constants available? * - Are the per-motherboard constants available?
* - Is there enough to allow a kernel to run that was written before * - Is there enough to allow a kernel to run that was written before
* a particular motherboard is constructed? (Assuming the kernel * a particular motherboard is constructed? (Assuming the kernel
* has drivers for all of the hardware but it does not have * has drivers for all of the hardware but it does not have
* assumptions on how the hardware is connected together). * assumptions on how the hardware is connected together).
* *
* With this test it should be straight forward to determine if a * With this test it should be straightforward to determine if a
* table entry is required or not. This should remove much of the * table entry is required or not. This should remove much of the
* long term compatibility burden as table entries which are * long-term compatibility burden as table entries which are
* irrelevant or have been replaced by better alternatives may be * irrelevant or have been replaced by better alternatives may be
* dropped. Of course it is polite and expidite to include extra * dropped. Of course it is polite and expedite to include extra
* table entries and be backwards compatible, but it is not required. * table entries and be backwards compatible, but it is not required.
*/ */
@ -85,10 +85,10 @@ struct lb_header {
uint32_t table_entries; uint32_t table_entries;
}; };
/* Every entry in the boot enviroment list will correspond to a boot /* Every entry in the boot environment list will correspond to a boot
* info record. Encoding both type and size. The type is obviously * info record, encoding both type and size. The type is obviously
* so you can tell what it is. The size allows you to skip that * so you can tell what it is. The size allows you to skip that
* boot enviroment record if you don't know what it easy. This allows * boot environment record if you don't know what it is. This allows
* forward compatibility with records not yet defined. * forward compatibility with records not yet defined.
*/ */
struct lb_record { struct lb_record {
@ -106,7 +106,7 @@ struct lb_memory_range {
uint32_t type; uint32_t type;
#define LB_MEM_RAM 1 /* Memory anyone can use */ #define LB_MEM_RAM 1 /* Memory anyone can use */
#define LB_MEM_RESERVED 2 /* Don't use this memory region */ #define LB_MEM_RESERVED 2 /* Don't use this memory region */
#define LB_MEM_TABLE 16 /* Ram configuration tables are kept in */ #define LB_MEM_TABLE 16 /* RAM configuration tables are kept in */
}; };
struct lb_memory { struct lb_memory {
@ -155,22 +155,22 @@ struct lb_forward {
uint64_t forward; uint64_t forward;
}; };
/* The following structures are for the cmos definitions table */ /* The following structures are for the CMOS definitions table */
#define LB_TAG_CMOS_OPTION_TABLE 200 #define LB_TAG_CMOS_OPTION_TABLE 200
/* cmos header record */ /* CMOS header record */
struct cmos_option_table { struct cmos_option_table {
uint32_t tag; /* CMOS definitions table type */ uint32_t tag; /* CMOS definitions table type */
uint32_t size; /* size of the entire table */ uint32_t size; /* size of the entire table */
uint32_t header_length; /* length of header */ uint32_t header_length; /* length of header */
}; };
/* cmos entry record /* CMOS entry record
This record is variable length. The name field may be * This record has a variable length. The name field may be
shorter than CMOS_MAX_NAME_LENGTH. The entry may start * shorter than CMOS_MAX_NAME_LENGTH. The entry may start
anywhere in the byte, but can not span bytes unless it * anywhere in the byte, but can not span bytes unless it
starts at the beginning of the byte and the length is * starts at the beginning of the byte and the length
fills complete bytes. * fills complete bytes.
*/ */
#define LB_TAG_OPTION 201 #define LB_TAG_OPTION 201
struct cmos_entries { struct cmos_entries {
uint32_t tag; /* entry type */ uint32_t tag; /* entry type */
@ -184,10 +184,10 @@ struct cmos_entries {
variable length int aligned */ variable length int aligned */
}; };
/* cmos enumerations record /* CMOS enumerations record
This record is variable length. The text field may be * This record has a variable length. The text field may be
shorter than CMOS_MAX_TEXT_LENGTH. * shorter than CMOS_MAX_TEXT_LENGTH.
*/ */
#define LB_TAG_OPTION_ENUM 202 #define LB_TAG_OPTION_ENUM 202
struct cmos_enums { struct cmos_enums {
uint32_t tag; /* enumeration type */ uint32_t tag; /* enumeration type */
@ -199,9 +199,9 @@ struct cmos_enums {
variable length int aligned */ variable length int aligned */
}; };
/* cmos defaults record /* CMOS default record
This record contains default settings for the cmos ram. * This record contains default settings for the CMOS RAM.
*/ */
#define LB_TAG_OPTION_DEFAULTS 203 #define LB_TAG_OPTION_DEFAULTS 203
struct cmos_defaults { struct cmos_defaults {
uint32_t tag; /* default type */ uint32_t tag; /* default type */