diff options
author | Julius Werner <jwerner@chromium.org> | 2020-12-29 17:33:30 -0800 |
---|---|---|
committer | Julius Werner <jwerner@chromium.org> | 2021-03-08 22:29:51 +0000 |
commit | 723e3b10afe40a8fff2fa85c6ec2e10852533425 (patch) | |
tree | 13513406c7003a441b16d05494023b775553b65f /src/include | |
parent | abc69712c267034bee0613baac80084659b8f65e (diff) |
cbfs: Reflow cbfs.c and cbfs.h to 96-character line lengths
Doing this all in one go keeps the files consistent and should make
future refactoring easier.
Signed-off-by: Julius Werner <jwerner@chromium.org>
Change-Id: I4a701d24fc9ccd68dce8789aab15fd21964a55f9
Reviewed-on: https://review.coreboot.org/c/coreboot/+/49330
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/include')
-rw-r--r-- | src/include/cbfs.h | 67 |
1 files changed, 33 insertions, 34 deletions
diff --git a/src/include/cbfs.h b/src/include/cbfs.h index cad01c623d..431f6e5fea 100644 --- a/src/include/cbfs.h +++ b/src/include/cbfs.h @@ -17,35 +17,36 @@ void *cbfs_boot_map_optionrom(uint16_t vendor, uint16_t device); /* Return mapping of option ROM with revision number. Returns NULL on error. */ void *cbfs_boot_map_optionrom_revision(uint16_t vendor, uint16_t device, uint8_t rev); + /* Locate file by name and optional type. Return 0 on success. < 0 on error. */ int cbfs_boot_locate(struct cbfsf *fh, const char *name, uint32_t *type); -/* Map file into memory, returning a pointer to the mapping or NULL on error. - If |size_out| is not NULL, it will pass out the size of the mapped file. - NOTE: Since this may return a direct pointer to memory-mapped hardware, - compressed files are NOT transparently decompressed (unlike cbfs_load()). */ -void *cbfs_map(const char *name, size_t *size_out); -/* Like cbfs_map(), except that it will always read from the read-only CBFS - ("COREBOOT" FMAP region), even when CONFIG(VBOOT) is enabled. */ -void *cbfs_ro_map(const char *name, size_t *size_out); -/* Removes a previously allocated CBFS mapping. Should try to unmap mappings in - strict LIFO order where possible, since mapping backends often don't support - more complicated cases. */ -int cbfs_unmap(void *mapping); /* Locate file in a specific region of fmap. Return 0 on success. < 0 on error*/ int cbfs_locate_file_in_region(struct cbfsf *fh, const char *region_name, const char *name, uint32_t *type); -/* Load a file from CBFS into a buffer. Returns amount of loaded bytes on - success or 0 on error. File will get decompressed as necessary. Same - decompression requirements as cbfs_load_and_decompress(). */ + +/* Map file into memory, returning a pointer to the mapping or NULL on error. If |size_out| is + not NULL, it will pass out the size of the mapped file. + NOTE: Since this may return a direct pointer to memory-mapped hardware, compressed files are + NOT transparently decompressed (unlike cbfs_load()). */ +void *cbfs_map(const char *name, size_t *size_out); +/* Like cbfs_map(), except that it will always read from the read-only CBFS (the "COREBOOT" FMAP + region), even when CONFIG(VBOOT) is enabled. */ +void *cbfs_ro_map(const char *name, size_t *size_out); +/* Removes a previously allocated CBFS mapping. Should try to unmap mappings in strict LIFO + order where possible, since mapping backends often don't support more complicated cases. */ +int cbfs_unmap(void *mapping); + +/* Load a file from CBFS into a buffer. Returns amount of loaded bytes on success or 0 on error. + File will get decompressed as necessary. */ size_t cbfs_load(const char *name, void *buf, size_t buf_size); -/* Like cbfs_load(), except that it will always read from the read-only CBFS - ("COREBOOT" FMAP region), even when CONFIG(VBOOT) is enabled. */ +/* Like cbfs_load(), except that it will always read from the read-only CBFS (the "COREBOOT" + FMAP region), even when CONFIG(VBOOT) is enabled. */ size_t cbfs_ro_load(const char *name, void *buf, size_t buf_size); -/* Load |in_size| bytes from |rdev| at |offset| to the |buffer_size| bytes - * large |buffer|, decompressing it according to |compression| in the process. - * Returns the decompressed file size, or 0 on error. - * LZMA files will be mapped for decompression. LZ4 files will be decompressed - * in-place with the buffer size requirements outlined in compression.h. */ + +/* Load |in_size| bytes from |rdev| at |offset| to the |buffer_size| bytes large |buffer|, + decompressing it according to |compression| in the process. Returns the decompressed file + size, or 0 on error. LZMA files will be mapped for decompression. LZ4 files will be + decompressed in-place with the buffer size requirements outlined in compression.h. */ size_t cbfs_load_and_decompress(const struct region_device *rdev, size_t offset, size_t in_size, void *buffer, size_t buffer_size, uint32_t compression); @@ -53,10 +54,9 @@ size_t cbfs_load_and_decompress(const struct region_device *rdev, size_t offset, int cbfs_prog_stage_load(struct prog *prog); /* - * Data structure that represents "a" CBFS boot device, with optional metadata - * cache. Generally we only have one of these, or two (RO and RW) when - * CONFIG(VBOOT) is set. The region device stored here must always be a - * subregion of boot_device_ro(). + * Data structure that represents "a" CBFS boot device, with optional metadata cache. Generally + * we only have one of these, or two (RO and RW) when CONFIG(VBOOT) is set. The region device + * stored here must always be a subregion of boot_device_ro(). */ struct cbfs_boot_device { struct region_device rdev; @@ -68,18 +68,17 @@ struct cbfs_boot_device { void cbfs_boot_device_find_mcache(struct cbfs_boot_device *cbd, uint32_t id); /* - * Retrieves the currently active CBFS boot device. If |force_ro| is set, will - * always return the read-only CBFS instead (this only makes a difference when - * CONFIG(VBOOT) is enabled). May perform certain CBFS initialization tasks. - * Returns NULL on error (e.g. boot device IO error). + * Retrieves the currently active CBFS boot device. If |force_ro| is set, will always return the + * read-only CBFS instead (this only makes a difference when CONFIG(VBOOT) is enabled). May + * perform certain CBFS initialization tasks. Returns NULL on error (e.g. boot device IO error). */ const struct cbfs_boot_device *cbfs_get_boot_device(bool force_ro); /* - * Builds the mcache (if |cbd->mcache| is set) and verifies |metadata_hash| (if - * it is not NULL). If CB_CBFS_CACHE_FULL is returned, the mcache is incomplete - * but still valid and the metadata hash was still verified. Should be called - * once per *boot* (not once per stage) before the first CBFS access. + * Builds the mcache (if |cbd->mcache| is set) and verifies |metadata_hash| (if it is not NULL). + * If CB_CBFS_CACHE_FULL is returned, the mcache is incomplete but still valid and the metadata + * hash was still verified. Should be called once per *boot* (not once per stage) before the + * first CBFS access. */ cb_err_t cbfs_init_boot_device(const struct cbfs_boot_device *cbd, struct vb2_hash *metadata_hash); |