Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42795 )
Change subject: src: Remove unneeded white spaces ......................................................................
Patch Set 2:
(4 comments)
https://review.coreboot.org/c/coreboot/+/42795/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/42795/2//COMMIT_MSG@9 PS2, Line 9: Remove white spaces before tabs. Please use this line as the commit message summary.
https://review.coreboot.org/c/coreboot/+/42795/2/src/mainboard/asus/p2b/vari... File src/mainboard/asus/p2b/variants/p2b-ls/overridetree.cb:
https://review.coreboot.org/c/coreboot/+/42795/2/src/mainboard/asus/p2b/vari... PS2, Line 7: # SCSI-68 port is always terminated Interesting case to deal with. The indentation in device tree is done with spaces here, so it could be kept?
https://review.coreboot.org/c/coreboot/+/42795/2/src/soc/intel/tigerlake/acp... File src/soc/intel/tigerlake/acpi/tcss.asl:
https://review.coreboot.org/c/coreboot/+/42795/2/src/soc/intel/tigerlake/acp... PS2, Line 359: * 0x00 - Ready Spaces instead of the tab should be used for the alignment here.
https://review.coreboot.org/c/coreboot/+/42795/2/src/soc/intel/tigerlake/acp... PS2, Line 388: * 0x01 - TCSS Deven is cleared by BIOS Mailbox request : * 0x1x - TCSS Deven is in progress of switching state according to given request : * 0xFE - Command timeout : * 0xFF - Command corrupt Spaces instead of tabs should be used for alignment here.