Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47229 )
Change subject: mb/ocp/deltalake: Configure GPIO
......................................................................
Patch Set 7:
(2 comments)
https://review.coreboot.org/c/coreboot/+/47229/6//COMMIT_MSG
Commit Message:
https://review.coreboot.org/c/coreboot/+/47229/6//COMMIT_MSG@7
PS6, Line 7: mb/ocp/deltalake: Configure GPIO
GPIOs
https://review.coreboot.org/c/coreboot/+/47229/6//COMMIT_MSG@13
PS6, Line 13: by Intel ITP and verify the results match with the header file.
This change is hard to review. Could you split it out in two patches. First convert the existing configuration to the new syntax, with `BUILD_TIMELESS=1` verifying nothing changed, and then add the changes from the new inteltool dump?
--
To view, visit https://review.coreboot.org/c/coreboot/+/47229
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ic9837a22bc231a4cb919de316ff6f6ee88411ab8
Gerrit-Change-Number: 47229
Gerrit-PatchSet: 7
Gerrit-Owner: Jingle Hsu <jingle_hsu(a)wiwynn.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Bryant Ou <bryant.ou.q(a)gmail.com>
Gerrit-Reviewer: Johnny Lin <Johnny_Lin(a)wiwynn.com>
Gerrit-Reviewer: Jonathan Zhang <jonzhang(a)fb.com>
Gerrit-Reviewer: Marc Jones <marc(a)marcjonesconsulting.com>
Gerrit-Reviewer: Maxim Polyakov <max.senia.poliak(a)gmail.com>
Gerrit-Reviewer: Morgan Jang <Morgan_Jang(a)wiwynn.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Michael Niewöhner <foss(a)mniewoehner.de>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Comment-Date: Thu, 26 Nov 2020 07:54:33 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Hello build bot (Jenkins), Marc Jones, Maxim Polyakov, Jonathan Zhang, Johnny Lin, Angel Pons, Morgan Jang, Bryant Ou,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/47229
to look at the new patch set (#7).
Change subject: mb/ocp/deltalake: Configure GPIO
......................................................................
mb/ocp/deltalake: Configure GPIO
On Delta Lake DVT with original UEFI firmware, dump GPIO settings
by util/inteltool and generate header file by util/intelp2m.
Tested=On OCP Delta Lake DVT, dump GPIO configurations
by Intel ITP and verify the results match with the header file.
Change-Id: Ic9837a22bc231a4cb919de316ff6f6ee88411ab8
Signed-off-by: Jingle Hsu <jingle_hsu(a)wiwynn.com>
---
M src/mainboard/ocp/deltalake/include/cpxsp_dl_gpio.h
1 file changed, 340 insertions(+), 244 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/29/47229/7
--
To view, visit https://review.coreboot.org/c/coreboot/+/47229
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ic9837a22bc231a4cb919de316ff6f6ee88411ab8
Gerrit-Change-Number: 47229
Gerrit-PatchSet: 7
Gerrit-Owner: Jingle Hsu <jingle_hsu(a)wiwynn.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Bryant Ou <bryant.ou.q(a)gmail.com>
Gerrit-Reviewer: Johnny Lin <Johnny_Lin(a)wiwynn.com>
Gerrit-Reviewer: Jonathan Zhang <jonzhang(a)fb.com>
Gerrit-Reviewer: Marc Jones <marc(a)marcjonesconsulting.com>
Gerrit-Reviewer: Maxim Polyakov <max.senia.poliak(a)gmail.com>
Gerrit-Reviewer: Morgan Jang <Morgan_Jang(a)wiwynn.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-MessageType: newpatchset
Hello build bot (Jenkins), Marc Jones, Maxim Polyakov, Jonathan Zhang, Johnny Lin, Angel Pons, Morgan Jang, Bryant Ou,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/47229
to look at the new patch set (#6).
Change subject: mb/ocp/deltalake: Configure GPIO
......................................................................
mb/ocp/deltalake: Configure GPIO
On Delta Lake DVT with original UEFI firmware, dump GPIO settings
by util/inteltool and generate header file by util/intelp2m.
Tested=On OCP Delta Lake DVT, dump GPIO configurations
by Intel ITP and verify the results match with the header file.
Change-Id: Ic9837a22bc231a4cb919de316ff6f6ee88411ab8
Signed-off-by: Jingle Hsu <jingle_hsu(a)wiwynn.com>
---
M src/mainboard/ocp/deltalake/include/cpxsp_dl_gpio.h
1 file changed, 244 insertions(+), 244 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/29/47229/6
--
To view, visit https://review.coreboot.org/c/coreboot/+/47229
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ic9837a22bc231a4cb919de316ff6f6ee88411ab8
Gerrit-Change-Number: 47229
Gerrit-PatchSet: 6
Gerrit-Owner: Jingle Hsu <jingle_hsu(a)wiwynn.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Bryant Ou <bryant.ou.q(a)gmail.com>
Gerrit-Reviewer: Johnny Lin <Johnny_Lin(a)wiwynn.com>
Gerrit-Reviewer: Jonathan Zhang <jonzhang(a)fb.com>
Gerrit-Reviewer: Marc Jones <marc(a)marcjonesconsulting.com>
Gerrit-Reviewer: Maxim Polyakov <max.senia.poliak(a)gmail.com>
Gerrit-Reviewer: Morgan Jang <Morgan_Jang(a)wiwynn.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-MessageType: newpatchset