Attention is currently required from: Wisley Chen. Hello Wisley Chen,
I'd like you to do a code review. Please visit
https://review.coreboot.org/c/coreboot/+/62822
to review the following change.
Change subject: driver/intel/usb4/retimer: Change loglevel prefix ......................................................................
driver/intel/usb4/retimer: Change loglevel prefix
In usb4_retimer_fill_ssdt(), it search all dpf ports and shows message in not support dpf ports. It's not error and changes the loglevel prefix to BIOS_INFO.
BUG=b:222038287 BRANCH=firmware-brya-14505.B TEST=emerge-brya coreboot
Signed-off-by: Wisley Chen wisley.chen@quanta.corp-partner.google.com Change-Id: I508ec7662e078893f944edb3d68364c57d5c5a73 --- M src/drivers/intel/usb4/retimer/retimer.c 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/22/62822/1
diff --git a/src/drivers/intel/usb4/retimer/retimer.c b/src/drivers/intel/usb4/retimer/retimer.c index 0c027eb..58ef2a8 100644 --- a/src/drivers/intel/usb4/retimer/retimer.c +++ b/src/drivers/intel/usb4/retimer/retimer.c @@ -358,7 +358,7 @@ for (dfp_port = 0; dfp_port < DFP_NUM_MAX; dfp_port++) {
if (!config->dfp[dfp_port].power_gpio.pin_count) { - printk(BIOS_ERR, "%s: No DFP%1d power GPIO for %s\n", + printk(BIOS_INFO, "%s: No DFP%1d power GPIO for %s\n", __func__, dfp_port, dev_path(dev)); continue; }