John Su has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38658 )
Change subject: mb/google/drallion/variants/drallion: Update thermal configuration for DPTF ......................................................................
Patch Set 2:
(2 comments)
https://review.coreboot.org/c/coreboot/+/38658/2/src/mainboard/google/dralli... File src/mainboard/google/drallion/variants/drallion/include/variant/acpi/dptf.asl:
https://review.coreboot.org/c/coreboot/+/38658/2/src/mainboard/google/dralli... PS2, Line 34: 90
How did you arrive to such a high temperature thershold value for Ambient sensor for passive thermal […]
Yes, for our test item request the thermal sensor trigger point is all in control.
https://review.coreboot.org/c/coreboot/+/38658/2/src/mainboard/google/dralli... PS2, Line 60: 12
Did you check the performance impact due to reduction of this PL1 and PL2 values ? […]
Yes, we have already check the performance test by WebGL aquarium, the spec is FPS:60 with 1000 fishes, and can meet target.