[coreboot-gerrit] Change in coreboot[master]: security/intel/cbnt: Add logging

[Angel Pons (Code Review)]

Attention is currently required from: Arthur Heymans, Anjaneya "Reddy" Chagam, Jonathan Zhang, Johnny Lin, Morgan Jang, Patrick Rudolph.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/54093 )

Change subject: security/intel/cbnt: Add logging
......................................................................

Patch Set 10: Code-Review+1

(2 comments)

File src/security/intel/cbnt/logging.c:

https://review.coreboot.org/c/coreboot/+/54093/comment/de70534e_92ceb6a4 
PS10, Line 121: 	printk(BIOS_DEBUG, "CBNT:\tSACM INFO MSR (0x13A) raw: 0x%016llx\n", acm_info.raw);
> printk(BIOS_DEBUG, "CBNT: could be replaced with a macro

Maybe borrow the CBFS macros, amd adapt them as needed?

#define ERROR(...) printk(BIOS_ERR, "CBnT ERROR: " __VA_ARGS__)
 #define LOG(...) printk(BIOS_INFO, "CBnT: " __VA_ARGS__)
 #define DEBUG(...) do { \
    if (CONFIG(DEBUG_CBFS)) \
    	printk(BIOS_SPEW, "CBnT DEBUG: " __VA_ARGS__); \
 } while (0)

https://review.coreboot.org/c/coreboot/+/54093/comment/37be020a_8911142d 
PS10, Line 122: \
> for proper tables please use fixed width using %-20s notation instead of \t

I generally use spaces to indent these