Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32222 )
Change subject: mb/facebook/watson: Disable turbo ......................................................................
Patch Set 2:
(2 comments)
https://review.coreboot.org/#/c/32222/2/src/mainboard/facebook/watson/Kconfi... File src/mainboard/facebook/watson/Kconfig:
https://review.coreboot.org/#/c/32222/2/src/mainboard/facebook/watson/Kconfi... PS2, Line 45: def_bool n Why the Kconfig?
https://review.coreboot.org/#/c/32222/2/src/mainboard/facebook/watson/mainbo... File src/mainboard/facebook/watson/mainboard.c:
https://review.coreboot.org/#/c/32222/2/src/mainboard/facebook/watson/mainbo... PS2, Line 6: -present A copyright notice should state each year when (a part of) the work was first published. If `-present` is supposed to mean present at the time of writing that'd be useless, I guess. If it's supposed to mean present at the time of reading, it would become a lie next year (unless you'd add something noticeable at each January 1st?).
So what's the point here?