Attention is currently required from: Felix Singer.
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/80168?usp=email )
Change subject: superio/ite/env_ctrl: Allow to override the fan PWM clock ......................................................................
Patch Set 1: Code-Review+1
(2 comments)
File src/superio/ite/common/env_ctrl.c:
https://review.coreboot.org/c/coreboot/+/80168/comment/5401a6a0_2e6bd010 : PS1, Line 134: conf->fan_pwm_clock & 1 Missing parentheses.
File src/superio/ite/common/env_ctrl_chip.h:
https://review.coreboot.org/c/coreboot/+/80168/comment/e80a20f5_b17233b3 : PS1, Line 72: /* bit 0 is used to mark non-default values */ What I did so leaving it unset in the dt results in the default. That's worth mentioning in the commit message, otherwise the patch looks a bit confusing.