u-boot.git/fs/cbfs, branch v2020.01

cbfs: Rename camel-case variables

2019-08-18T13:54:10Z

Rename some camel-case variables to match U-Boot style. Camel case is not generally allowed in U-Boot. Rename this variable to fit in with the style. Signed-off-by: Simon Glass Reviewed-by: Bin Meng Tested-by: Bin Meng

cbfs: Add functions to support multiple CBFSs

2019-08-18T13:54:10Z

Sometimes an image has multiple CBFS. The current CBFS API is limited to handling only one at time. Also it keeps track of the CBFS internally in BSS, which does not work before relocation, for example. Add a few new functions to overcome these limitations. Signed-off-by: Simon Glass Reviewed-by: Bin Meng Tested-by: Bin Meng

cbfs: Move result variable into the struct

2019-08-18T13:54:10Z

Move the result variable into the struct also, so that it can be used when BSS is not available. Add a function to read it. Note that all functions sill use the BSS version of the data. Signed-off-by: Simon Glass Reviewed-by: Bin Meng Tested-by: Bin Meng

cbfs: Move static variables into a struct

2019-08-18T13:54:10Z

At present there are a number of static variables in BSS. This cannot work with SPL, at least until BSS is available in board_init_r(). Move the variables into a struct, so it is possible to malloc() it and use it before BSS is available. Signed-off-by: Simon Glass Reviewed-by: Bin Meng Tested-by: Bin Meng

cbfs: Move declarations above functions

2019-08-18T13:54:10Z

At present this file has a function at the top, above declarations. This is normally avoided, so fix it. Signed-off-by: Simon Glass Reviewed-by: Bin Meng Tested-by: Bin Meng

cbfs: Allow CBFS to be used in SPL

2019-08-18T13:54:10Z

Add a new Kconfig option to enable CBFS in SPL. This can be useful when the memory-init code is in CBFS. Signed-off-by: Simon Glass Reviewed-by: Bin Meng Tested-by: Bin Meng

cbfs: Rename checksum to attributes_offset

2019-07-24T03:27:57Z

It seems that this field has been renamed in later version of coreboot. Update it. Signed-off-by: Simon Glass

fs: cbfs: Fix out of bound access during CBFS walking through

2018-12-31T01:42:41Z

The call to file_cbfs_fill_cache() is given with the parameter 'start' pointing to the offset by the CBFS base address, but with the parameter 'size' that equals to the whole CBFS size. During CBFS walking through, it checks files one by one and after it pass over the end of the CBFS which is 4GiB boundary it tries to check files from address 0 and so on, until the overall size the codes checked hits to the given 'size'. Fix this by passing 'start' pointing to the CBFS base address. Signed-off-by: Bin Meng Reviewed-by: Simon Glass

fs: cbfs: remove wrong header validation

2018-12-31T01:42:41Z

cbfs_fileheader.len indicates the content size of the file in the cbfs, and it has nothing to do with cbfs_fileheader.offset which is the starting address of the file in the cbfs. Remove such check in file_cbfs_next_file(). Before this change 'cbfsinit' failed with 'Bad CBFS file'. After this change all cbfs commands are working as expected. Signed-off-by: Christian Gmeiner [bmeng: keep the necessary header sanity check] Signed-off-by: Bin Meng Reviewed-by: Simon Glass

SPDX: Convert all of our single license tags to Linux Kernel style

2018-05-07T13:34:12Z

When U-Boot started using SPDX tags we were among the early adopters and there weren't a lot of other examples to borrow from. So we picked the area of the file that usually had a full license text and replaced it with an appropriate SPDX-License-Identifier: entry. Since then, the Linux Kernel has adopted SPDX tags and they place it as the very first line in a file (except where shebangs are used, then it's second line) and with slightly different comment styles than us. In part due to community overlap, in part due to better tag visibility and in part for other minor reasons, switch over to that style. This commit changes all instances where we have a single declared license in the tag as both the before and after are identical in tag contents. There's also a few places where I found we did not have a tag and have introduced one. Signed-off-by: Tom Rini