build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45024 )
Change subject: libpayload/usb: Fix printf format string mismatches in debug messages ......................................................................
Patch Set 4:
(3 comments)
https://review.coreboot.org/c/coreboot/+/45024/4/payloads/libpayload/drivers... File payloads/libpayload/drivers/usb/ohci.c:
https://review.coreboot.org/c/coreboot/+/45024/4/payloads/libpayload/drivers... PS4, Line 102: usb_debug("| O | Function Address | [%04"PRIx32"] |\n", cur->config & 0x7F); line over 96 characters
https://review.coreboot.org/c/coreboot/+/45024/4/payloads/libpayload/drivers... PS4, Line 480: usb_debug("ohci_control(): doing transfer with %x. first_td at %"PRIxPTR"\n", Prefer using '"%s...", __func__' to using 'ohci_control', this function's name, in a string
https://review.coreboot.org/c/coreboot/+/45024/4/payloads/libpayload/drivers... PS4, Line 518: usb_debug("bulk: %x bytes from %p, finalize: %x, maxpacketsize: %x\n", dalen, src, finalize, ep->maxpacketsize); line over 96 characters