build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/26657 )
Change subject: [WIP] Hack out a lot of stuff
......................................................................
Patch Set 1: Verified+1
Build Successful
https://qa.coreboot.org/job/flashrom-customrules/1309/ : SUCCESS
https://qa.coreboot.org/job/flashrom_gerrit/1086/ : SUCCESS
--
To view, visit https://review.coreboot.org/26657
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Iae8ac4c9006d2f5a488a100e8d878f9069ad7e54
Gerrit-Change-Number: 26657
Gerrit-PatchSet: 1
Gerrit-Owner: David Hendricks <david.hendricks(a)gmail.com>
Gerrit-Reviewer: David Hendricks <david.hendricks(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Tue, 29 May 2018 02:32:05 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: Yes
David Hendricks has posted comments on this change. ( https://review.coreboot.org/26657 )
Change subject: [WIP] Hack out a lot of stuff
......................................................................
Patch Set 1: Code-Review-2
Move along, nothing to see here...
--
To view, visit https://review.coreboot.org/26657
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Iae8ac4c9006d2f5a488a100e8d878f9069ad7e54
Gerrit-Change-Number: 26657
Gerrit-PatchSet: 1
Gerrit-Owner: David Hendricks <david.hendricks(a)gmail.com>
Gerrit-Reviewer: David Hendricks <david.hendricks(a)gmail.com>
Gerrit-Comment-Date: Tue, 29 May 2018 02:32:03 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: Yes
David Hendricks has posted comments on this change. ( https://review.coreboot.org/26232 )
Change subject: chipset_enable: Add PCI IDs for discrete Kaby Lake PCHs
......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/#/c/26232/1//COMMIT_MSG
Commit Message:
https://review.coreboot.org/#/c/26232/1//COMMIT_MSG@21
PS1, Line 21: Revision 003
It seems you were looking at the same version of spec as me. If you want to leave out the Z370 stuff for now, that's fine too.
--
To view, visit https://review.coreboot.org/26232
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ida545d69ec998a5d3ae4dc88e76adbb13952bceb
Gerrit-Change-Number: 26232
Gerrit-PatchSet: 1
Gerrit-Owner: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: David Hendricks <david.hendricks(a)gmail.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Mon, 28 May 2018 23:37:51 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No
David Hendricks has posted comments on this change. ( https://review.coreboot.org/26232 )
Change subject: chipset_enable: Add PCI IDs for discrete Kaby Lake PCHs
......................................................................
Patch Set 1: Code-Review+2
(1 comment)
Looks good, though you may wanna add the Z370 PCI ID while you're at it.
https://review.coreboot.org/#/c/26232/1/chipset_enable.c
File chipset_enable.c:
https://review.coreboot.org/#/c/26232/1/chipset_enable.c@1952
PS1, Line 1952: {0x8086, 0xa2c8, NT, "Intel", "B250", enable_flash_pch100},
Revision 003 of the spec also mentions 0xa2c9 for the Z370.
--
To view, visit https://review.coreboot.org/26232
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ida545d69ec998a5d3ae4dc88e76adbb13952bceb
Gerrit-Change-Number: 26232
Gerrit-PatchSet: 1
Gerrit-Owner: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: David Hendricks <david.hendricks(a)gmail.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Mon, 28 May 2018 23:35:56 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: Yes
David Hendricks has posted comments on this change. ( https://review.coreboot.org/26582 )
Change subject: flashchips: AT25SL128A tested PREW
......................................................................
Patch Set 1: Code-Review-1
Please squash this into the previous patch.
--
To view, visit https://review.coreboot.org/26582
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If32900f742a3b3fb9951eac052cea7631748635e
Gerrit-Change-Number: 26582
Gerrit-PatchSet: 1
Gerrit-Owner: Hal Martin <hal.martin+coreboot(a)gmail.com>
Gerrit-Reviewer: David Hendricks <david.hendricks(a)gmail.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Mon, 28 May 2018 23:17:43 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: Yes
David Hendricks has posted comments on this change. ( https://review.coreboot.org/26576 )
Change subject: flashchips: Add ATMEL AT25SL128A
......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/#/c/26576/1/flashchips.c
File flashchips.c:
https://review.coreboot.org/#/c/26576/1/flashchips.c@2261
PS1, Line 2261: TEST_UNTESTED
Update this too since you have tested it.
--
To view, visit https://review.coreboot.org/26576
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I60c433ffe9e34663c2cfc608b8b76943cd92a8ba
Gerrit-Change-Number: 26576
Gerrit-PatchSet: 1
Gerrit-Owner: Hal Martin <hal.martin+coreboot(a)gmail.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: David Hendricks <david.hendricks(a)gmail.com>
Gerrit-Comment-Date: Mon, 28 May 2018 23:17:29 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No
David Hendricks has posted comments on this change. ( https://review.coreboot.org/26576 )
Change subject: flashchips: Add ATMEL AT25SL128A
......................................................................
Patch Set 1:
(1 comment)
One small nit, otherwise looks good. Thanks!
https://review.coreboot.org/#/c/26576/1/flashchips.c
File flashchips.c:
https://review.coreboot.org/#/c/26576/1/flashchips.c@2287
PS1, Line 2287: 1650, 1950
Voltage tolerance is 1.7-2.0 according to the datasheet, so this should be {1700, 2000}.
--
To view, visit https://review.coreboot.org/26576
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I60c433ffe9e34663c2cfc608b8b76943cd92a8ba
Gerrit-Change-Number: 26576
Gerrit-PatchSet: 1
Gerrit-Owner: Hal Martin <hal.martin+coreboot(a)gmail.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: David Hendricks <david.hendricks(a)gmail.com>
Gerrit-Comment-Date: Mon, 28 May 2018 23:16:38 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No
Nico Huber has posted comments on this change. ( https://review.coreboot.org/26652 )
Change subject: Remove white space before tab
......................................................................
Patch Set 2: Code-Review+2
--
To view, visit https://review.coreboot.org/26652
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I90f171924790ced74a62ca344fee8607607aa480
Gerrit-Change-Number: 26652
Gerrit-PatchSet: 2
Gerrit-Owner: Elyes HAOUAS <ehaouas(a)noos.fr>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Mon, 28 May 2018 18:37:43 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: Yes
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/26261 )
Change subject: Enable writes with active ME
......................................................................
Patch Set 4: Code-Review+2
--
To view, visit https://review.coreboot.org/26261
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I94a5e7074b845c227e43d76d04dd1a71082a1cef
Gerrit-Change-Number: 26261
Gerrit-PatchSet: 4
Gerrit-Owner: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Mon, 28 May 2018 17:50:22 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: Yes