Shelley Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45407 )
Change subject: region_file_update_data: Change API to accept array of structs ......................................................................
Patch Set 4:
(6 comments)
https://review.coreboot.org/c/coreboot/+/45407/3/src/drivers/mrc_cache/mrc_c... File src/drivers/mrc_cache/mrc_cache.c:
https://review.coreboot.org/c/coreboot/+/45407/3/src/drivers/mrc_cache/mrc_c... PS3, Line 426: 1
ARRAY_SIZE(cbmem_entry)
Made region_file_update_data a wrapper for region_file_update_data_arr(), which takes multiple arrays of data.
https://review.coreboot.org/c/coreboot/+/45407/3/src/drivers/mrc_cache/mrc_c... PS3, Line 553: 1
ARRAY_SIZE(invalid_entry)
Made this a wrapper for region_file_update_data_arr.
https://review.coreboot.org/c/coreboot/+/45407/3/src/include/region_file.h File src/include/region_file.h:
https://review.coreboot.org/c/coreboot/+/45407/3/src/include/region_file.h@4... PS3, Line 40: int
size_t ?
Done
https://review.coreboot.org/c/coreboot/+/45407/3/src/include/region_file.h@4... PS3, Line 47: struct
const struct
Done
https://review.coreboot.org/c/coreboot/+/45407/3/src/lib/region_file.c File src/lib/region_file.c:
https://review.coreboot.org/c/coreboot/+/45407/3/src/lib/region_file.c@369 PS3, Line 369: struct
const struct
Done
https://review.coreboot.org/c/coreboot/+/45407/3/src/lib/region_file.c@407 PS3, Line 407: struct
const struct
Done