View Change
2 comments:
Commit Message:
Patch Set #5, Line 8:
Please elaborate, what DPM is, what it is used for, and where it’s documented.
DPM is used for power manager, which is mediatek proprietary, it's binary release. Sorry for that we can't document it.
Patch Set #5, Line 10: TEST=Boots correctly on Asurada
Tested with what firmware version, and how much time did it add? Maybe add the related coreboot log […]
Dpm is tested pass with firmware 13457.0.
Dpm log is added already, for example:
DPM: load finish in 37 msecs
To view, visit change 46393. To unsubscribe, or for help writing mail filters, visit settings.
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I16b341ad63940b45b886c4a7fd733c1970624e40
Gerrit-Change-Number: 46393
Gerrit-PatchSet: 6
Gerrit-Owner: Yidi Lin <yidi.lin@mediatek.com>
Gerrit-Reviewer: Duan huayang <huayang.duan@mediatek.com>
Gerrit-Reviewer: Martin Roth <martinroth@google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: Xi Chen <xixi.chen@mediatek.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-CC: Xixi Chen <xixi.chen@mediatek.corp-partner.google.com>
Gerrit-Comment-Date: Tue, 20 Oct 2020 03:02:11 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-MessageType: comment