build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46730 )
Change subject: [RFC] libpayload/keyboard: Dynamically toggle controller translation ......................................................................
Patch Set 1:
(4 comments)
https://review.coreboot.org/c/coreboot/+/46730/1/payloads/libpayload/drivers... File payloads/libpayload/drivers/i8042/keyboard.c:
https://review.coreboot.org/c/coreboot/+/46730/1/payloads/libpayload/drivers... PS1, Line 365: switch (actual_set) { switch and case should be at the same indent
https://review.coreboot.org/c/coreboot/+/46730/1/payloads/libpayload/drivers... PS1, Line 366: case 0x43: actual_set = 1; break; trailing statements should be on next line
https://review.coreboot.org/c/coreboot/+/46730/1/payloads/libpayload/drivers... PS1, Line 367: case 0x41: actual_set = 2; break; trailing statements should be on next line
https://review.coreboot.org/c/coreboot/+/46730/1/payloads/libpayload/drivers... PS1, Line 368: case 0x3f: actual_set = 3; break; trailing statements should be on next line