Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38911 )
Change subject: payloads: Fix typos ......................................................................
Patch Set 2:
(7 comments)
https://review.coreboot.org/c/coreboot/+/38911/2/payloads/libpayload/drivers... File payloads/libpayload/drivers/i8042/keyboard.c:
https://review.coreboot.org/c/coreboot/+/38911/2/payloads/libpayload/drivers... PS2, Line 352: ps/2 PS/2
https://review.coreboot.org/c/coreboot/+/38911/2/payloads/libpayload/drivers... PS2, Line 356: parameters nit: 'params' is a common short form of 'parameters', not a typo. The complete form now fits in one line because of the increased 96 character width.
https://review.coreboot.org/c/coreboot/+/38911/2/payloads/libpayload/drivers... File payloads/libpayload/drivers/serial/ipq40xx.c:
https://review.coreboot.org/c/coreboot/+/38911/2/payloads/libpayload/drivers... PS2, Line 553: For simplicity sake Missing a `'s` and a comma:
For simplicity's sake, let's rely...
https://review.coreboot.org/c/coreboot/+/38911/2/payloads/libpayload/drivers... File payloads/libpayload/drivers/serial/ipq806x.c:
https://review.coreboot.org/c/coreboot/+/38911/2/payloads/libpayload/drivers... PS2, Line 343: /* For simplicity sake let's rely on coreboot initializing the UART. */ Same as before
https://review.coreboot.org/c/coreboot/+/38911/2/payloads/libpayload/drivers... File payloads/libpayload/drivers/serial/qcs405.c:
https://review.coreboot.org/c/coreboot/+/38911/2/payloads/libpayload/drivers... PS2, Line 541: /* For simplicity sake let's rely on coreboot initializing the UART. */ Same as before
https://review.coreboot.org/c/coreboot/+/38911/2/payloads/libpayload/drivers... File payloads/libpayload/drivers/udc/dwc2.c:
https://review.coreboot.org/c/coreboot/+/38911/2/payloads/libpayload/drivers... PS2, Line 256: /*Already disabled*/ Please add spaces like on the other comments
https://review.coreboot.org/c/coreboot/+/38911/2/payloads/libpayload/libc/re... File payloads/libpayload/libc/readline.c:
https://review.coreboot.org/c/coreboot/+/38911/2/payloads/libpayload/libc/re... PS2, Line 132: unprintable 'unprintables' was referring to 'unprintable characters'