March 2018 - flashrom-gerrit - mail.coreboot.org

Change in flashrom[master]: Add support for Atmel/Adesto AT25SF161 and Winbond W25Q80EW
by David Hendricks (Code Review) 28 Mar '18

28 Mar '18

David Hendricks has posted comments on this change. ( https://review.coreboot.org/25099 ) Change subject: Add support for Atmel/Adesto AT25SF161 and Winbond W25Q80EW ...................................................................... Patch Set 3: Code-Review+2 Made trivial fixes to address the remaining comments, carrying forward the +2 from earlier. -- To view, visit https://review.coreboot.org/25099 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: Ia9e8f7f23896f7002401c6b1e616c0dc102198e2 Gerrit-Change-Number: 25099 Gerrit-PatchSet: 3 Gerrit-Owner: David Hendricks <david.hendricks(a)gmail.com> Gerrit-Reviewer: David Hendricks <david.hendricks(a)gmail.com> Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Stefan Tauner <stefan.tauner(a)gmx.at> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Wed, 28 Mar 2018 19:34:54 +0000 Gerrit-HasComments: No Gerrit-HasLabels: Yes

1 0

Change in flashrom[master]: Add support for Atmel/Adesto AT25SF161 and Winbond W25Q80EW
by David Hendricks (Code Review) 28 Mar '18

28 Mar '18

David Hendricks has posted comments on this change. ( https://review.coreboot.org/25099 ) Change subject: Add support for Atmel/Adesto AT25SF161 and Winbond W25Q80EW ...................................................................... Patch Set 3: (2 comments) https://review.coreboot.org/#/c/25099/2//COMMIT_MSG Commit Message: https://review.coreboot.org/#/c/25099/2//COMMIT_MSG@7 PS2, Line 7: > still Done https://review.coreboot.org/#/c/25099/2/flashchips.c File flashchips.c: https://review.coreboot.org/#/c/25099/2/flashchips.c@a1523 PS2, Line 1523: > lost empty line? Done -- To view, visit https://review.coreboot.org/25099 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: Ia9e8f7f23896f7002401c6b1e616c0dc102198e2 Gerrit-Change-Number: 25099 Gerrit-PatchSet: 3 Gerrit-Owner: David Hendricks <david.hendricks(a)gmail.com> Gerrit-Reviewer: David Hendricks <david.hendricks(a)gmail.com> Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Stefan Tauner <stefan.tauner(a)gmx.at> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Wed, 28 Mar 2018 19:33:29 +0000 Gerrit-HasComments: Yes Gerrit-HasLabels: No

1 0

Change in flashrom[master]: Add support for Atmel/Adesto AT25SF161 and Winbond W25Q80EW
by build bot (Jenkins) (Code Review) 28 Mar '18

28 Mar '18

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/25099 ) Change subject: Add support for Atmel/Adesto AT25SF161 and Winbond W25Q80EW ...................................................................... Patch Set 3: Verified+1 Build Successful https://qa.coreboot.org/job/flashrom-customrules/1225/ : SUCCESS https://qa.coreboot.org/job/flashrom_gerrit/1019/ : SUCCESS -- To view, visit https://review.coreboot.org/25099 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: Ia9e8f7f23896f7002401c6b1e616c0dc102198e2 Gerrit-Change-Number: 25099 Gerrit-PatchSet: 3 Gerrit-Owner: David Hendricks <david.hendricks(a)gmail.com> Gerrit-Reviewer: David Hendricks <david.hendricks(a)gmail.com> Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Stefan Tauner <stefan.tauner(a)gmx.at> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Wed, 28 Mar 2018 19:33:21 +0000 Gerrit-HasComments: No Gerrit-HasLabels: Yes

1 0

Change in flashrom[master]: Add support for Atmel/Adesto AT25SF161 and Winbond W25Q80EW
by David Hendricks (Code Review) 28 Mar '18

28 Mar '18

Hello Stefan Tauner, Paul Menzel, build bot (Jenkins), Nico Huber, I'd like you to reexamine a change. Please visit https://review.coreboot.org/25099 to look at the new patch set (#3). Change subject: Add support for Atmel/Adesto AT25SF161 and Winbond W25Q80EW ...................................................................... Add support for Atmel/Adesto AT25SF161 and Winbond W25Q80EW Change-Id: Ia9e8f7f23896f7002401c6b1e616c0dc102198e2 Signed-off-by: Stanislav Sedov <ssedov(a)fb.com> --- M flashchips.c M flashchips.h 2 files changed, 78 insertions(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/99/25099/3 -- To view, visit https://review.coreboot.org/25099 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ia9e8f7f23896f7002401c6b1e616c0dc102198e2 Gerrit-Change-Number: 25099 Gerrit-PatchSet: 3 Gerrit-Owner: David Hendricks <david.hendricks(a)gmail.com> Gerrit-Reviewer: David Hendricks <david.hendricks(a)gmail.com> Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Stefan Tauner <stefan.tauner(a)gmx.at> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>

1 0

Change in flashrom[master]: [PATCH 1/1] board_enable: add AOpen i965GMt-LA
by build bot (Jenkins) (Code Review) 28 Mar '18

28 Mar '18

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/25396 ) Change subject: [PATCH 1/1] board_enable: add AOpen i965GMt-LA ...................................................................... Patch Set 2: Build Successful https://qa.coreboot.org/job/flashrom-customrules/1224/ : SUCCESS https://qa.coreboot.org/job/flashrom_gerrit/1018/ : SUCCESS -- To view, visit https://review.coreboot.org/25396 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: I8899bbe06707fe76256539f90f5b670301228d52 Gerrit-Change-Number: 25396 Gerrit-PatchSet: 2 Gerrit-Owner: Idwer Vollering <vidwer(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: Wed, 28 Mar 2018 18:01:16 +0000 Gerrit-HasComments: No Gerrit-HasLabels: No

1 0

Change in flashrom[master]: [PATCH 1/1] board_enable: add AOpen i965GMt-LA
by Idwer Vollering (Code Review) 28 Mar '18

28 Mar '18

Hello Paul Menzel, build bot (Jenkins), I'd like you to reexamine a change. Please visit https://review.coreboot.org/25396 to look at the new patch set (#2). Change subject: [PATCH 1/1] board_enable: add AOpen i965GMt-LA ...................................................................... [PATCH 1/1] board_enable: add AOpen i965GMt-LA Change-Id: I8899bbe06707fe76256539f90f5b670301228d52 Signed-off-by: Luc Verhaegen <libv(a)skynet.be> --- M board_enable.c 1 file changed, 2 insertions(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/96/25396/2 -- To view, visit https://review.coreboot.org/25396 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I8899bbe06707fe76256539f90f5b670301228d52 Gerrit-Change-Number: 25396 Gerrit-PatchSet: 2 Gerrit-Owner: Idwer Vollering <vidwer(a)gmail.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>

1 0

Change in flashrom[master]: [PATCH 1/1] board_enable: add AOpen i965GMt-LA
by build bot (Jenkins) (Code Review) 28 Mar '18

28 Mar '18

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/25396 ) Change subject: [PATCH 1/1] board_enable: add AOpen i965GMt-LA ...................................................................... Patch Set 1: Verified+1 Build Successful https://qa.coreboot.org/job/flashrom-customrules/1223/ : SUCCESS https://qa.coreboot.org/job/flashrom_gerrit/1017/ : SUCCESS -- To view, visit https://review.coreboot.org/25396 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: I8899bbe06707fe76256539f90f5b670301228d52 Gerrit-Change-Number: 25396 Gerrit-PatchSet: 1 Gerrit-Owner: Idwer Vollering <vidwer(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Wed, 28 Mar 2018 10:44:20 +0000 Gerrit-HasComments: No Gerrit-HasLabels: Yes

1 0

Change in flashrom[master]: [PATCH 1/1] board_enable: add AOpen i965GMt-LA
by Idwer Vollering (Code Review) 28 Mar '18

28 Mar '18

Idwer Vollering has uploaded this change for review. ( https://review.coreboot.org/25396 Change subject: [PATCH 1/1] board_enable: add AOpen i965GMt-LA ...................................................................... [PATCH 1/1] board_enable: add AOpen i965GMt-LA Change-Id: I8899bbe06707fe76256539f90f5b670301228d52 Signed-off-by: Luc Verhaegen <libv(a)skynet.be> --- M board_enable.c 1 file changed, 2 insertions(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/96/25396/1 diff --git a/board_enable.c b/board_enable.c index 5e8f4dd..182a455 100644 --- a/board_enable.c +++ b/board_enable.c @@ -1692,6 +1692,7 @@ /* * Suited for: * - ASUS P5BV-R: LGA775 + 3200 + ICH7 + * - AOpen i965GMt-LA: Intel Socket479 + 965GM + ICH8M */ static int intel_ich_gpio20_raise(void) { @@ -2324,6 +2325,7 @@ {0x1022, 0x2090, 0, 0, 0x1022, 0x2080, 0, 0, NULL, "artecgroup", "dbe61", P3, "Artec Group", "DBE61", 0, OK, board_artecgroup_dbe6x}, {0x1022, 0x2090, 0, 0, 0x1022, 0x2080, 0, 0, NULL, "artecgroup", "dbe62", P3, "Artec Group", "DBE62", 0, OK, board_artecgroup_dbe6x}, {0x8086, 0x27b9, 0xa0a0, 0x0632, 0x8086, 0x27da, 0xa0a0, 0x0632, NULL, NULL, NULL, P3, "AOpen", "i945GMx-VFX", 0, OK, intel_ich_gpio38_raise}, + {0x8086, 0x2a00, 0xa0a0, 0x063e, 0x8086, 0x2815, 0xa0a0, 0x063e, NULL, NULL, NULL, P3, "AOpen", "i965GMt-LA", 0, OK, intel_ich_gpio20_raise}, {0x8086, 0x277c, 0xa0a0, 0x060b, 0x8086, 0x27da, 0xa0a0, 0x060b, NULL, NULL, NULL, P3, "AOpen", "i975Xa-YDG", 0, OK, board_aopen_i975xa_ydg}, {0x8086, 0x27A0, 0x8086, 0x7270, 0x8086, 0x27B9, 0x8086, 0x7270, "^iMac5,2$", NULL, NULL, P2, "Apple", "iMac5,2", 0, OK, p2_whitelist_laptop}, {0x8086, 0x27A0, 0x8086, 0x7270, 0x8086, 0x27B9, 0x8086, 0x7270, "^MacBook2,1$", NULL, NULL, P2, "Apple", "MacBook2,1", 0, OK, p2_whitelist_laptop}, -- To view, visit https://review.coreboot.org/25396 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I8899bbe06707fe76256539f90f5b670301228d52 Gerrit-Change-Number: 25396 Gerrit-PatchSet: 1 Gerrit-Owner: Idwer Vollering <vidwer(a)gmail.com>

1 0

Change in flashrom[master]: Remove address from GPLv2 headers
by build bot (Jenkins) (Code Review) 27 Mar '18

27 Mar '18

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/25381 ) Change subject: Remove address from GPLv2 headers ...................................................................... Patch Set 1: Verified+1 Build Successful https://qa.coreboot.org/job/flashrom-customrules/1222/ : SUCCESS https://qa.coreboot.org/job/flashrom_gerrit/1016/ : SUCCESS -- To view, visit https://review.coreboot.org/25381 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: I7bfc339673cbf5ee2d2ff7564c4db04ca088d0a4 Gerrit-Change-Number: 25381 Gerrit-PatchSet: 1 Gerrit-Owner: Elyes HAOUAS <ehaouas(a)noos.fr> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Tue, 27 Mar 2018 10:57:08 +0000 Gerrit-HasComments: No Gerrit-HasLabels: Yes

1 0

Change in flashrom[master]: Remove unneeded white spaces
by build bot (Jenkins) (Code Review) 27 Mar '18

27 Mar '18

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/25380 ) Change subject: Remove unneeded white spaces ...................................................................... Patch Set 1: Verified+1 Build Successful https://qa.coreboot.org/job/flashrom-customrules/1221/ : SUCCESS https://qa.coreboot.org/job/flashrom_gerrit/1015/ : SUCCESS -- To view, visit https://review.coreboot.org/25380 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: Ic2d3bb9d8581a0471a8568a130f893b34dddf113 Gerrit-Change-Number: 25380 Gerrit-PatchSet: 1 Gerrit-Owner: Elyes HAOUAS <ehaouas(a)noos.fr> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Tue, 27 Mar 2018 10:56:47 +0000 Gerrit-HasComments: No Gerrit-HasLabels: Yes

1 0

2024

2023

2022

2021

2020

2019

2018

2017

flashrom-gerrit March 2018