Philippe Mathieu-Daudé has posted comments on this change. ( https://review.coreboot.org/19987 )
Change subject: src/cpu/amd/quadcore Fix checkpatch errors/warnings ......................................................................
Patch Set 1: Code-Review+1
(2 comments)
https://review.coreboot.org/#/c/19987/1/src/cpu/amd/quadcore/amd_sibling.c File src/cpu/amd/quadcore/amd_sibling.c:
PS1, Line 108: 8 * siblings + sysconf.nodes; isn't it cleaner to align just after the "="?
https://review.coreboot.org/#/c/19987/1/src/cpu/amd/quadcore/quadcore.c File src/cpu/amd/quadcore/quadcore.c:
PS1, Line 62: "Start same here, align after the "("? and so on