build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34485 )
Change subject: libpayload: usbmsc: Skip zero-length packets at end of data ......................................................................
Patch Set 4:
(8 comments)
https://review.coreboot.org/c/coreboot/+/34485/4/payloads/libpayload/drivers... File payloads/libpayload/drivers/usb/usbmsc.c:
https://review.coreboot.org/c/coreboot/+/34485/4/payloads/libpayload/drivers... PS4, Line 222: int ret = ctrlr->bulk (ep, sizeof (csw_t), (u8 *) csw, 1); space prohibited between function name and open parenthesis '('
https://review.coreboot.org/c/coreboot/+/34485/4/payloads/libpayload/drivers... PS4, Line 222: int ret = ctrlr->bulk (ep, sizeof (csw_t), (u8 *) csw, 1); space prohibited between function name and open parenthesis '('
https://review.coreboot.org/c/coreboot/+/34485/4/payloads/libpayload/drivers... PS4, Line 227: ret = ctrlr->bulk (ep, sizeof (csw_t), (u8 *) csw, 1); space prohibited between function name and open parenthesis '('
https://review.coreboot.org/c/coreboot/+/34485/4/payloads/libpayload/drivers... PS4, Line 227: ret = ctrlr->bulk (ep, sizeof (csw_t), (u8 *) csw, 1); space prohibited between function name and open parenthesis '('
https://review.coreboot.org/c/coreboot/+/34485/4/payloads/libpayload/drivers... PS4, Line 231: if (ctrlr->bulk (ep, sizeof (csw_t), (u8 *) csw, 1) < 0) { space prohibited between function name and open parenthesis '('
https://review.coreboot.org/c/coreboot/+/34485/4/payloads/libpayload/drivers... PS4, Line 231: if (ctrlr->bulk (ep, sizeof (csw_t), (u8 *) csw, 1) < 0) { space prohibited between function name and open parenthesis '('
https://review.coreboot.org/c/coreboot/+/34485/4/payloads/libpayload/drivers... PS4, Line 231: if (ctrlr->bulk (ep, sizeof (csw_t), (u8 *) csw, 1) < 0) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/c/coreboot/+/34485/4/payloads/libpayload/drivers... PS4, Line 237: usb_debug ("MSC: received malformed CSW\n"); space prohibited between function name and open parenthesis '('