u-boot.git/include/cbfs.h, branch v2020.10

cbfs: Don't require the CBFS size with cbfs_init_mem()

2020-05-27T06:40:09+00:00

The size is not actually used since it is present in the header. Drop this
parameter. Also tidy up error handling while we are here.

Signed-off-by: Simon Glass 
Reviewed-by: Bin Meng

cbfs: Allow reading a file from a CBFS given its base addr

2020-05-27T06:40:09+00:00

Currently we support reading a file from CBFS given the address of the end
of the ROM. Sometimes we only know the start of the CBFS. Add a function
to find a file given that.

Signed-off-by: Simon Glass 
Reviewed-by: Bin Meng

cbfs: Change file_cbfs_find_uncached() to return an error

2020-05-27T06:40:09+00:00

This function currently returns a node pointer so there is no way to know
the error code. Also it uses data in BSS which seems unnecessary since the
caller might prefer to use a local variable.

Update the function and split its body out into a separate function so we
can use it later.

Signed-off-by: Simon Glass 
Reviewed-by: Bin Meng

cbfs: Return the error code from file_cbfs_init()

2020-05-27T06:40:09+00:00

We may as well return the error code and use it directly in the command
code. CBFS still uses its own error enum which we may be able to remove,
but leave it for now.

Signed-off-by: Simon Glass 
Reviewed-by: Bin Meng

cbfs: Use ulong consistently

2020-05-27T06:40:09+00:00

U-Boot uses ulong for addresses but there are a few places in this driver
that don't use it. Convert this driver over to follow this convention
fully.

Signed-off-by: Simon Glass 
Reviewed-by: Bin Meng

cbfs: drop file_cbfs_result declaration

2020-05-27T06:40:09+00:00

It is not definded anywhere.

Signed-off-by: Christian Gmeiner 
Reviewed-by: Heinrich Schuchardt 
Reviewed-by: Bin Meng

x86: cbfs: Drop unwanted declaration

2020-04-30T09:16:12+00:00

The intention here is add a forward declaration, not actually declare a
variable. Fix it.

Signed-off-by: Simon Glass 
Reviewed-by: Bin Meng

cbfs: do not pack struct cbfs_cachenode

2019-11-02T10:00:51+00:00

With the __packed attribute sandbox_defconfig cannot be compiled with GCC
9.2.1:

fs/cbfs/cbfs.c: In function ‘file_cbfs_fill_cache’:
fs/cbfs/cbfs.c:164:16: error: taking address of packed member of
‘struct cbfs_cachenode’ may result in an unaligned pointer value
[-Werror=address-of-packed-member]
  164 |   cache_tail = &new_node->next;
      |                ^~~~~~~~~~~~~~~

struct cbfs_cachenode is only an internal structure. So let's rearrange the
fields such that the structure is naturally packed and remove the __packed
attribute.

Signed-off-by: Heinrich Schuchardt 
Reviewed-by: Bin Meng 
Reviewed-by: Simon Glass

cbfs: Add functions to support multiple CBFSs

2019-08-18T13:54:10+00:00

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:10+00:00

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