cbfs: correct types used for accessing files
In commit 72a8e5e751
the
Makefile's were updated to use named types for cbfs
file addition. However, the call sites were not checked to
ensure the types matched. Correct all call sites to use the
named types.
Change-Id: Ib9fa693ef517e3196a3f04e9c06db52a9116fee7
Signed-off-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: http://review.coreboot.org/9195
Tested-by: build bot (Jenkins)
Reviewed-by: Marc Jones <marc.jones@se-eng.com>
This commit is contained in:
parent
2a567eeb1d
commit
a30f7e667c
|
@ -36,8 +36,8 @@ int read_spd_from_cbfs(u8 *buf, int idx)
|
|||
size_t min_len = (idx + 1) * SPD_SIZE;
|
||||
|
||||
printk(BIOS_DEBUG, "read SPD\n");
|
||||
spd_file = cbfs_get_file_content(CBFS_DEFAULT_MEDIA, "spd.bin", 0xab,
|
||||
&spd_file_len);
|
||||
spd_file = cbfs_get_file_content(CBFS_DEFAULT_MEDIA, "spd.bin",
|
||||
CBFS_TYPE_SPD, &spd_file_len);
|
||||
if (!spd_file)
|
||||
printk(BIOS_EMERG, "file [spd.bin] not found in CBFS");
|
||||
if (spd_file_len < min_len)
|
||||
|
|
|
@ -64,7 +64,8 @@ static u32 get_mrc_cache_region(struct mrc_data_container **mrc_region_ptr)
|
|||
{
|
||||
size_t region_size;
|
||||
*mrc_region_ptr = cbfs_get_file_content(CBFS_DEFAULT_MEDIA,
|
||||
"mrc.cache", 0xac,
|
||||
"mrc.cache",
|
||||
CBFS_TYPE_MRC_CACHE,
|
||||
®ion_size);
|
||||
|
||||
return region_size;
|
||||
|
|
|
@ -74,7 +74,11 @@
|
|||
#define CBFS_TYPE_VSA 0x51
|
||||
#define CBFS_TYPE_MBI 0x52
|
||||
#define CBFS_TYPE_MICROCODE 0x53
|
||||
#define CBFS_TYPE_FSP 0x60
|
||||
#define CBFS_TYPE_MRC 0x61
|
||||
#define CBFS_COMPONENT_CMOS_DEFAULT 0xaa
|
||||
#define CBFS_TYPE_SPD 0xab
|
||||
#define CBFS_TYPE_MRC_CACHE 0xac
|
||||
#define CBFS_COMPONENT_CMOS_LAYOUT 0x01aa
|
||||
|
||||
#define CBFS_HEADER_MAGIC 0x4F524243
|
||||
|
|
|
@ -78,8 +78,8 @@ static void copy_spd(struct pei_data *peid)
|
|||
int spd_index = 0; /* No GPIO selection, force index 0 for now */
|
||||
|
||||
printk(BIOS_DEBUG, "SPD index %d\n", spd_index);
|
||||
spd_file = cbfs_get_file_content(CBFS_DEFAULT_MEDIA, "spd.bin", 0xab,
|
||||
&spd_file_len);
|
||||
spd_file = cbfs_get_file_content(CBFS_DEFAULT_MEDIA, "spd.bin",
|
||||
CBFS_TYPE_SPD, &spd_file_len);
|
||||
if (!spd_file)
|
||||
die("SPD data not found.");
|
||||
|
||||
|
|
|
@ -79,8 +79,8 @@ static void copy_spd(struct pei_data *peid)
|
|||
size_t spd_file_len;
|
||||
|
||||
printk(BIOS_DEBUG, "SPD index %d\n", spd_index);
|
||||
spd_file = cbfs_get_file_content(CBFS_DEFAULT_MEDIA, "spd.bin", 0xab,
|
||||
&spd_file_len);
|
||||
spd_file = cbfs_get_file_content(CBFS_DEFAULT_MEDIA, "spd.bin",
|
||||
CBFS_TYPE_SPD, &spd_file_len);
|
||||
if (!spd_file)
|
||||
die("SPD data not found.");
|
||||
|
||||
|
|
|
@ -95,8 +95,8 @@ static void copy_spd(struct pei_data *peid)
|
|||
int spd_index = get_gpios(gpio_vector);
|
||||
|
||||
printk(BIOS_DEBUG, "spd index %d\n", spd_index);
|
||||
spd_file = cbfs_get_file_content(CBFS_DEFAULT_MEDIA, "spd.bin", 0xab,
|
||||
&spd_file_len);
|
||||
spd_file = cbfs_get_file_content(CBFS_DEFAULT_MEDIA, "spd.bin",
|
||||
CBFS_TYPE_SPD, &spd_file_len);
|
||||
if (!spd_file)
|
||||
die("SPD data not found.");
|
||||
|
||||
|
|
|
@ -82,8 +82,8 @@ static void copy_spd(struct pei_data *peid)
|
|||
size_t spd_file_len;
|
||||
|
||||
printk(BIOS_DEBUG, "SPD index %d\n", spd_index);
|
||||
spd_file = cbfs_get_file_content(CBFS_DEFAULT_MEDIA, "spd.bin", 0xab,
|
||||
&spd_file_len);
|
||||
spd_file = cbfs_get_file_content(CBFS_DEFAULT_MEDIA, "spd.bin",
|
||||
CBFS_TYPE_SPD, &spd_file_len);
|
||||
if (!spd_file)
|
||||
die("SPD data not found.");
|
||||
|
||||
|
|
|
@ -80,8 +80,8 @@ static void copy_spd(struct pei_data *peid)
|
|||
size_t spd_file_len;
|
||||
|
||||
printk(BIOS_DEBUG, "SPD index %d\n", spd_index);
|
||||
spd_file = cbfs_get_file_content(CBFS_DEFAULT_MEDIA, "spd.bin", 0xab,
|
||||
&spd_file_len);
|
||||
spd_file = cbfs_get_file_content(CBFS_DEFAULT_MEDIA, "spd.bin",
|
||||
CBFS_TYPE_SPD, &spd_file_len);
|
||||
if (!spd_file)
|
||||
die("SPD data not found.");
|
||||
|
||||
|
|
|
@ -237,8 +237,8 @@ void main(unsigned long bist)
|
|||
break;
|
||||
}
|
||||
|
||||
spd_data = cbfs_get_file_content(CBFS_DEFAULT_MEDIA, "spd.bin", 0xab,
|
||||
&spd_file_len);
|
||||
spd_data = cbfs_get_file_content(CBFS_DEFAULT_MEDIA, "spd.bin",
|
||||
CBFS_TYPE_SPD, &spd_file_len);
|
||||
if (!spd_data)
|
||||
die("SPD data not found.");
|
||||
if (spd_file_len < (spd_index + 1) * 256)
|
||||
|
|
|
@ -71,7 +71,9 @@ static u32 get_mrc_cache_region(struct mrc_data_container **mrc_region_ptr)
|
|||
#else
|
||||
size_t region_size;
|
||||
*mrc_region_ptr = cbfs_get_file_content(CBFS_DEFAULT_MEDIA,
|
||||
"mrc.cache", 0xac, ®ion_size);
|
||||
"mrc.cache",
|
||||
CBFS_TYPE_MRC_CACHE,
|
||||
®ion_size);
|
||||
return region_size;
|
||||
#endif
|
||||
}
|
||||
|
|
|
@ -163,7 +163,7 @@ void sdram_initialize(struct pei_data *pei_data)
|
|||
|
||||
/* Locate and call UEFI System Agent binary. */
|
||||
entry = (unsigned long)cbfs_get_file_content(
|
||||
CBFS_DEFAULT_MEDIA, "mrc.bin", 0xab, NULL);
|
||||
CBFS_DEFAULT_MEDIA, "mrc.bin", CBFS_TYPE_MRC, NULL);
|
||||
if (entry) {
|
||||
int rv;
|
||||
asm volatile (
|
||||
|
|
|
@ -71,7 +71,9 @@ static u32 get_mrc_cache_region(struct mrc_data_container **mrc_region_ptr)
|
|||
#else
|
||||
size_t region_size;
|
||||
*mrc_region_ptr = cbfs_get_file_content(CBFS_DEFAULT_MEDIA,
|
||||
"mrc.cache", 0xac, ®ion_size);
|
||||
"mrc.cache",
|
||||
CBFS_TYPE_MRC_CACHE,
|
||||
®ion_size);
|
||||
return region_size;
|
||||
#endif
|
||||
|
||||
|
|
|
@ -246,9 +246,8 @@ void sdram_initialize(struct pei_data *pei_data)
|
|||
pei_data->tx_byte = do_putchar;
|
||||
|
||||
/* Locate and call UEFI System Agent binary. */
|
||||
/* TODO make MRC blob (0xab?) defined in cbfs_core.h. */
|
||||
entry = cbfs_get_file_content(
|
||||
CBFS_DEFAULT_MEDIA, "mrc.bin", 0xab, NULL);
|
||||
CBFS_DEFAULT_MEDIA, "mrc.bin", CBFS_TYPE_MRC, NULL);
|
||||
if (entry) {
|
||||
int rv;
|
||||
rv = entry (pei_data);
|
||||
|
|
|
@ -147,8 +147,8 @@ void raminit(struct mrc_params *mp, int prev_sleep_state)
|
|||
}
|
||||
|
||||
/* Determine if mrc.bin is in the cbfs. */
|
||||
if (cbfs_get_file_content(CBFS_DEFAULT_MEDIA, "mrc.bin", 0xab, NULL) ==
|
||||
NULL) {
|
||||
if (cbfs_get_file_content(CBFS_DEFAULT_MEDIA, "mrc.bin", CBFS_TYPE_MRC,
|
||||
NULL) == NULL) {
|
||||
printk(BIOS_DEBUG, "Couldn't find mrc.bin\n");
|
||||
return;
|
||||
}
|
||||
|
|
|
@ -86,7 +86,7 @@ void raminit(struct pei_data *pei_data)
|
|||
|
||||
/* Determine if mrc.bin is in the cbfs. */
|
||||
entry = (pei_wrapper_entry_t)cbfs_get_file_content(
|
||||
CBFS_DEFAULT_MEDIA, "mrc.bin", 0xab, NULL);
|
||||
CBFS_DEFAULT_MEDIA, "mrc.bin", CBFS_TYPE_MRC, NULL);
|
||||
if (entry == NULL) {
|
||||
printk(BIOS_DEBUG, "Couldn't find mrc.bin\n");
|
||||
return;
|
||||
|
|
Loading…
Reference in New Issue