bloblist: Use LOG_CATEGORY to simply logging

Use the convenience functions to improve readability.

Signed-off-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
Simon Glass 2022-01-12 19:26:20 -07:00
parent f16ec77784
commit 1d8bbd76f0

View file

@ -4,6 +4,8 @@
* Written by Simon Glass <sjg@chromium.org> * Written by Simon Glass <sjg@chromium.org>
*/ */
#define LOG_CATEGORY LOGC_BLOBLIST
#include <common.h> #include <common.h>
#include <bloblist.h> #include <bloblist.h>
#include <log.h> #include <log.h>
@ -133,8 +135,7 @@ static int bloblist_addrec(uint tag, int size, int align,
new_alloced = data_start + ALIGN(size, align); new_alloced = data_start + ALIGN(size, align);
if (new_alloced > hdr->size) { if (new_alloced > hdr->size) {
log(LOGC_BLOBLIST, LOGL_ERR, log_err("Failed to allocate %x bytes size=%x, need size=%x\n",
"Failed to allocate %x bytes size=%x, need size=%x\n",
size, hdr->size, new_alloced); size, hdr->size, new_alloced);
return log_msg_ret("bloblist add", -ENOSPC); return log_msg_ret("bloblist add", -ENOSPC);
} }
@ -249,13 +250,12 @@ static int bloblist_resize_rec(struct bloblist_hdr *hdr,
expand_by = ALIGN(new_size - rec->size, BLOBLIST_ALIGN); expand_by = ALIGN(new_size - rec->size, BLOBLIST_ALIGN);
new_alloced = ALIGN(hdr->alloced + expand_by, BLOBLIST_ALIGN); new_alloced = ALIGN(hdr->alloced + expand_by, BLOBLIST_ALIGN);
if (new_size < 0) { if (new_size < 0) {
log(LOGC_BLOBLIST, LOGL_DEBUG, log_debug("Attempt to shrink blob size below 0 (%x)\n",
"Attempt to shrink blob size below 0 (%x)\n", new_size); new_size);
return log_msg_ret("size", -EINVAL); return log_msg_ret("size", -EINVAL);
} }
if (new_alloced > hdr->size) { if (new_alloced > hdr->size) {
log(LOGC_BLOBLIST, LOGL_ERR, log_err("Failed to allocate %x bytes size=%x, need size=%x\n",
"Failed to allocate %x bytes size=%x, need size=%x\n",
new_size, hdr->size, new_alloced); new_size, hdr->size, new_alloced);
return log_msg_ret("alloc", -ENOSPC); return log_msg_ret("alloc", -ENOSPC);
} }
@ -347,8 +347,7 @@ int bloblist_check(ulong addr, uint size)
return log_msg_ret("Bad size", -EFBIG); return log_msg_ret("Bad size", -EFBIG);
chksum = bloblist_calc_chksum(hdr); chksum = bloblist_calc_chksum(hdr);
if (hdr->chksum != chksum) { if (hdr->chksum != chksum) {
log(LOGC_BLOBLIST, LOGL_ERR, "Checksum %x != %x\n", hdr->chksum, log_err("Checksum %x != %x\n", hdr->chksum, chksum);
chksum);
return log_msg_ret("Bad checksum", -EIO); return log_msg_ret("Bad checksum", -EIO);
} }
gd->bloblist = hdr; gd->bloblist = hdr;
@ -446,7 +445,7 @@ int bloblist_init(void)
} }
ret = bloblist_new(addr, CONFIG_BLOBLIST_SIZE, 0); ret = bloblist_new(addr, CONFIG_BLOBLIST_SIZE, 0);
} else { } else {
log(LOGC_BLOBLIST, LOGL_DEBUG, "Found existing bloblist\n"); log_debug("Found existing bloblist\n");
} }
return ret; return ret;