Attention is currently required from: Lance Zhao, Subrata Banik, Caveh Jalali, Tim Wawrzynczak, Sumeet R Pawnikar, Boris Mittelberg.
Hello Lance Zhao, build bot (Jenkins), Subrata Banik, Caveh Jalali, Reka Norman, Tim Wawrzynczak, Boris Mittelberg,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/73249
to look at the new patch set (#2).
Change subject: Reland drivers/intel/dptf: Add multiple fan support under dptf ......................................................................
Reland drivers/intel/dptf: Add multiple fan support under dptf
This reverts commit 4dba71fd25c91a9e610287c61238a8fe24452e4e.
Add multiple fan support for dptf policies.
This also fixes the Gmeet resolution drop issue as per b:246535768 comment#12. When system starts Gmeet video call, it uses the hardware accelerated encoder as per the expectation. But, as soon as another system connects to the call, the immediate fallback observed from hardware to software encoder. Due to this, Gmeet resolution dropped from 720p to 180p. This issue observed on AlderLake-N SoC based fanless platforms. This same issue was not seen on fan based systems. With the fix in dptf driver where fan configures appropriate setting for only fan participant, able to see consistent 720p resolution.
BUG=b:246535768,b:235254828 BRANCH=None TEST=Built and tested on Alderlake-P Redrix system for two fans support and on Alderlake-N fanless systems. With this code change Gmeet resolution drop not observed.
Signed-off-by: Sumeet Pawnikar sumeet.r.pawnikar@intel.com Change-Id: Id07d279ff962253c22be9d395ed7be0d732aeaa7 --- M src/acpi/acpigen_dptf.c M src/drivers/intel/dptf/chip.h M src/drivers/intel/dptf/dptf.c M src/ec/google/chromeec/acpi/emem.asl M src/ec/google/chromeec/chip.h M src/ec/google/chromeec/ec_acpi.c M src/ec/google/chromeec/ec_dptf_helpers.c M src/ec/google/common/dptf.h M src/include/acpi/acpigen_dptf.h 9 files changed, 238 insertions(+), 47 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/49/73249/2