Change in ...coreboot[master]: sdm845: Add USB support on cheza platform

[mturney mturney (Code Review)]

mturney mturney has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/25213 )

Change subject: sdm845: Add USB support on cheza platform
......................................................................

Patch Set 67:

(2 comments)

https://review.coreboot.org/#/c/25213/66/src/soc/qualcomm/sdm845/usb.c 
File src/soc/qualcomm/sdm845/usb.c:

https://review.coreboot.org/#/c/25213/66/src/soc/qualcomm/sdm845/usb.c@646 
PS66, Line 646: 	printk(BIOS_INFO, "Starting DWC3 and PHY resets for USB\n");
> nit: should move this into reset_usb0()/reset_usb1() to be able to print the right controller name

Ack

https://review.coreboot.org/#/c/25213/66/src/soc/qualcomm/sdm845/usb.c@788 
PS66, Line 788: 	/* DEBUG_STAT5: wait for 160uS for PLL lock;
> Correct multi-line comment style is either […]

Ack