[coreboot-gerrit] [M] Change in coreboot[main]: drivers/intel/fsp2_0: Add option to control debug log level using CBFS

[Julius Werner (Code Review)]

Attention is currently required from: Andrey Petrov, Intel coreboot Reviewers, Jérémy Compostella, Paul Menzel, Ronak Kanabar, Subrata Banik.

Julius Werner has posted comments on this change by Subrata Banik. ( https://review.coreboot.org/c/coreboot/+/86001?usp=email )

Change subject: drivers/intel/fsp2_0: Add option to control debug log level using CBFS
......................................................................

Patch Set 7:

(1 comment)

File src/drivers/intel/fsp2_0/debug.c:

https://review.coreboot.org/c/coreboot/+/86001/comment/e7da1cbf_cf4fc8f7?usp... :
PS6, Line 185: 	if (!CONFIG(USE_CBFS_FILE_OPTION_BACKEND))
Why? Why shouldn't this just work for all option backends equally?

To be clear, I think this function should just be:
```
enum fsp_log_level fsp_get_pcd_debug_log_level(void)
{
  return get_uint_option("fsp_pcd_debug_level", fsp_map_console_log_level());
}
```

[Sorry, looks like I forgot to publish this comment yesterday.]