Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31616 )
Change subject: superio/ite/common: add option for enabling 5 FANs ......................................................................
Patch Set 1: Code-Review+1
(3 comments)
Very nice.
https://review.coreboot.org/#/c/31616/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/#/c/31616/1//COMMIT_MSG@17 PS1, Line 17: was is
https://review.coreboot.org/#/c/31616/1/src/superio/ite/common/Kconfig File src/superio/ite/common/Kconfig:
https://review.coreboot.org/#/c/31616/1/src/superio/ite/common/Kconfig@48 PS1, Line 48: FAN controller always works in SmartGuardian mode Please add a dot/period at the end of the sentence.
https://review.coreboot.org/#/c/31616/1/src/superio/ite/common/Kconfig@53 PS1, Line 53: ITE FAN controller has 5 independent outputs Please add a dot/period at the end of the sentence.