Peichao Li has uploaded a new patch set (#2). ( https://review.coreboot.org/28669 )
Change subject: mb/google/octopus: Close unused I2C bus 7 include SCL and SDA for Phaser
......................................................................
mb/google/octopus: Close unused I2C bus 7 include SCL and SDA for Phaser
Since I2C bus 7 attached the touchscreen device however Phaser units
that haven't it. So for avoiding side effects, we need close I2C bus
7 SCL and SDA respectilvely.
BUG=
TEST=according to sku_id(Phaser: 0x1, Phaser360: 0x2, Phaser360s: 0x3)
distinguish whether close these gpios.
Change-Id: I8ad17761f2a053dc329bbec0a0a3284d47289666
Signed-off-by: peichao.wang <peichao.wang(a)bitland.corp-partner.google.com>
---
M src/mainboard/google/octopus/variants/phaser/gpio.c
1 file changed, 54 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/69/28669/2
--
To view, visit https://review.coreboot.org/28669
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I8ad17761f2a053dc329bbec0a0a3284d47289666
Gerrit-Change-Number: 28669
Gerrit-PatchSet: 2
Gerrit-Owner: Peichao Li <peichao.wang(a)bitland.corp-partner.google.com>
Hello Pratikkumar V Prajapati, Subrata Banik, Bora Guvendik, build bot (Jenkins), Hannah Williams, Krzysztof M Sywula,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/28367
to look at the new patch set (#4).
Change subject: mb/intel/coffeelake_rvp: GPIO support for whiskey board
......................................................................
mb/intel/coffeelake_rvp: GPIO support for whiskey board
Add gpio programming difference for whiskeylake rvp platform.
BUG=N/A
TEST=N/A
Change-Id: I35a0384f828fd3219e0c3adb4830f5bdab800e32
Signed-off-by: Lijian Zhao <lijian.zhao(a)intel.com>
---
M src/mainboard/intel/coffeelake_rvp/variants/baseboard/gpio.c
1 file changed, 8 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/67/28367/4
--
To view, visit https://review.coreboot.org/28367
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I35a0384f828fd3219e0c3adb4830f5bdab800e32
Gerrit-Change-Number: 28367
Gerrit-PatchSet: 4
Gerrit-Owner: Lijian Zhao <lijian.zhao(a)intel.com>
Gerrit-Reviewer: Bora Guvendik <bora.guvendik(a)intel.com>
Gerrit-Reviewer: Hannah Williams <hannah.williams(a)intel.com>
Gerrit-Reviewer: Krzysztof M Sywula <krzysztof.m.sywula(a)intel.com>
Gerrit-Reviewer: Lijian Zhao <lijian.zhao(a)intel.com>
Gerrit-Reviewer: Pratikkumar V Prajapati <pratikkumar.v.prajapati(a)intel.com>
Gerrit-Reviewer: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Richard Spiegel has posted comments on this change. ( https://review.coreboot.org/28608 )
Change subject: amd/stoneyridge: Sync PSP base to MSR
......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/#/c/28608/1/src/soc/amd/stoneyridge/cpu.c
File src/soc/amd/stoneyridge/cpu.c:
https://review.coreboot.org/#/c/28608/1/src/soc/amd/stoneyridge/cpu.c@123
PS1, Line 123: setup_lapic();
> It'd be nice to comment properly what's really needed. […]
Yes, I did tested it against S3 cycle and reboot. It's write once, it survives reboot and S3 cycle. Original Marshall's code caused errors both ways, reboot and S3 cycle. Once I added the protection, problem was gone. Also I tried without the code and using iotools to write it once... then it survived reboot and S3 cycle. Only S5 clears it!
Will add the comment.
--
To view, visit https://review.coreboot.org/28608
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I30027d3b0a6fbd540375e96001beb9c25bf3a678
Gerrit-Change-Number: 28608
Gerrit-PatchSet: 1
Gerrit-Owner: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Gerrit-Reviewer: Garrett Kirkendall <garrett.kirkendall(a)amd.corp-partner.google.com>
Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Gerrit-Reviewer: Richard Spiegel <richard.spiegel(a)silverbackltd.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Tue, 18 Sep 2018 20:48:37 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No