Chen Wisley has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38212 )
Change subject: mb/google/hatch: Add noise mitigation setting for dratini/jinlon ......................................................................
Patch Set 5:
(3 comments)
https://review.coreboot.org/c/coreboot/+/38212/3//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/38212/3//COMMIT_MSG@9 PS3, Line 9: mitgation
What is the impact of the changes in the slew rates, aside from acoustic noise mitigation? Is there […]
The SW option is provided in intel doc 575216 and the same solution is applied on hatch board.
Intel replied on issue tracker: The acoustic noise mitigation option may impact interrupt response time, device latency to memory, exit latency from C-states, IO configuration, power and performance…etc.
https://review.coreboot.org/c/coreboot/+/38212/3//COMMIT_MSG@10 PS3, Line 10: Ramp(IA, GT, SA)
Space before (.
Done
https://review.coreboot.org/c/coreboot/+/38212/3//COMMIT_MSG@14 PS3, Line 14: TEST=build and verify that noise reduce.
+1
let me check with our power team