Am 28. September 2020 02:46:13 MESZ schrieb Simon Glass <s...@chromium.org>:
>Add some category names that were missed in recent changes. Update the
>comment as a reminder.
>
>Signed-off-by: Simon Glass <s...@chromium.org>
>---
>
> common/log.c | 5 +++++
> include/log.h | 4 +++-
> 2 files changed, 8 insertions(+), 1 deletion(-)
>
>diff --git a/common/log.c b/common/log.c
>index 9a5f100da34..38336d84e3c 100644
>--- a/common/log.c
>+++ b/common/log.c
>@@ -21,6 +21,11 @@ static const char *log_cat_name[LOGC_COUNT -
>LOGC_NONE] = {
> "driver-model",
> "device-tree",
> "efi",
>+ "alloc",
>+ "sandbox",
>+ "bloblist",
>+ "devres",
>+ "acpi",
> };
Should we add a compile time check that the array size matches the number of
non-uclass categories?
Best regards
Heinrich
>
> static const char *log_level_name[LOGL_COUNT] = {
>diff --git a/include/log.h b/include/log.h
>index 2859ce1f2e7..00885004a36 100644
>--- a/include/log.h
>+++ b/include/log.h
>@@ -39,7 +39,9 @@ enum log_level_t {
>
> /**
> * Log categories supported. Most of these correspond to uclasses (i.e.
>- * enum uclass_id) but there are also some more generic categories
>+ * enum uclass_id) but there are also some more generic categories.
>+ *
>+ * Remember to update log_cat_name[] after adding a new category.
> */
> enum log_category_t {
> LOGC_FIRST = 0, /* First part mirrors UCLASS_... */
