Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41448 )
Change subject: cbfs: add cbfs_boot_locate_file_get_compression_info() ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/41448/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/41448/1//COMMIT_MSG@10 PS1, Line 10: size) with the current API. Therefore, split cbfs_boot_load_file() up Can't you just run cbfs_boot_locate() and then cbfsf_decompression_info() on the resulting file handle? I don't think there's any rule against using <commonlib/cbfs.h> APIs from other files, we're using cbfsf_file_type() all over the place. (Just trying to reduce churn on the CBFS code while I'm trying to change all of it.)