coreboot-gerrit September 2017

coreboot-gerrit@coreboot.org

1 participants
1960 discussions

Change in coreboot[master]: nb/intel/i945: Update comments
by build bot (Jenkins) (Code Review) 16 Sep '17

16 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21563 ) Change subject: nb/intel/i945: Update comments ...................................................................... Patch Set 3: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/15782/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/60576/ : SUCCESS -- To view, visit https://review.coreboot.org/21563 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: If4c7aa4723c0a9715e2931e40a041bfcfd55b8b6 Gerrit-Change-Number: 21563 Gerrit-PatchSet: 3 Gerrit-Owner: HAOUAS Elyes <ehaouas(a)noos.fr> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sat, 16 Sep 2017 17:49:55 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: nb/intel/i945: Update comments
by build bot (Jenkins) (Code Review) 16 Sep '17

16 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21563 ) Change subject: nb/intel/i945: Update comments ...................................................................... Patch Set 2: Verified-1 Build Failed https://qa.coreboot.org/job/coreboot-gerrit/60575/ : ABORTED -- To view, visit https://review.coreboot.org/21563 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: If4c7aa4723c0a9715e2931e40a041bfcfd55b8b6 Gerrit-Change-Number: 21563 Gerrit-PatchSet: 2 Gerrit-Owner: HAOUAS Elyes <ehaouas(a)noos.fr> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sat, 16 Sep 2017 17:43:17 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: nb/intel/i945: Update comments
by build bot (Jenkins) (Code Review) 16 Sep '17

16 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21563 ) Change subject: nb/intel/i945: Update comments ...................................................................... Patch Set 1: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/15781/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/60574/ : SUCCESS -- To view, visit https://review.coreboot.org/21563 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: If4c7aa4723c0a9715e2931e40a041bfcfd55b8b6 Gerrit-Change-Number: 21563 Gerrit-PatchSet: 1 Gerrit-Owner: HAOUAS Elyes <ehaouas(a)noos.fr> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sat, 16 Sep 2017 14:50:56 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: nb/intel/i945: Update comments
by HAOUAS Elyes (Code Review) 16 Sep '17

16 Sep '17

HAOUAS Elyes has uploaded this change for review. ( https://review.coreboot.org/21563 Change subject: nb/intel/i945: Update comments ...................................................................... nb/intel/i945: Update comments i945M supports two DIMMs, however i945G* supports four DIMMs and have their SPD at 0x50/0x51/0x52/0x53. Change-Id: If4c7aa4723c0a9715e2931e40a041bfcfd55b8b6 Signed-off-by: Elyes HAOUAS <ehaouas(a)noos.fr> --- M src/northbridge/intel/i945/raminit.c 1 file changed, 3 insertions(+), 1 deletion(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/63/21563/1 diff --git a/src/northbridge/intel/i945/raminit.c b/src/northbridge/intel/i945/raminit.c index f10f108..14960b8 100644 --- a/src/northbridge/intel/i945/raminit.c +++ b/src/northbridge/intel/i945/raminit.c @@ -324,7 +324,7 @@ int i; /** - * i945 supports two DIMMs, in two configurations: + * i945M supports two DIMMs, in two configurations: * * - single channel with two DIMMs * - dual channel with one DIMM per channel @@ -332,6 +332,8 @@ * In practice dual channel mainboards have their SPD at 0x50/0x52 * whereas single channel configurations have their SPD at 0x50/0x51. * + * i945G* supports four DIMMs, and have their SPD at 0x50/0x51/0x52/0x53. + * * The capability register knows a lot about the channel configuration * but for now we stick with the information we gather via SPD. */ -- To view, visit https://review.coreboot.org/21563 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: If4c7aa4723c0a9715e2931e40a041bfcfd55b8b6 Gerrit-Change-Number: 21563 Gerrit-PatchSet: 1 Gerrit-Owner: HAOUAS Elyes <ehaouas(a)noos.fr>

1 0

Change in coreboot[master]: ifdtool: refactor region_name(s) and new_layout
by build bot (Jenkins) (Code Review) 16 Sep '17

16 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21315 ) Change subject: ifdtool: refactor region_name(s) and new_layout ...................................................................... Patch Set 36: Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/15777/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/60571/ : SUCCESS -- To view, visit https://review.coreboot.org/21315 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ia2b92c3f835f9849a243a48c37588c2dbe7449bf Gerrit-Change-Number: 21315 Gerrit-PatchSet: 36 Gerrit-Owner: Bill XIE <persmule(a)gmail.com> Gerrit-Reviewer: Bill XIE <persmule(a)gmail.com> Gerrit-Reviewer: Nicola Corna <nicola(a)corna.info> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Philippe Mathieu-Daudé <f4bug(a)amsat.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sat, 16 Sep 2017 04:02:16 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: ifdtool: port the feature to set AltMeDisable/HAP bit here
by build bot (Jenkins) (Code Review) 16 Sep '17

16 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21437 ) Change subject: ifdtool: port the feature to set AltMeDisable/HAP bit here ...................................................................... Patch Set 21: Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/15779/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/60572/ : SUCCESS -- To view, visit https://review.coreboot.org/21437 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I9a2ecc60cfbb9ee9d96f15be3d53226cb428729a Gerrit-Change-Number: 21437 Gerrit-PatchSet: 21 Gerrit-Owner: Bill XIE <persmule(a)gmail.com> Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Nicola Corna <nicola(a)corna.info> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sat, 16 Sep 2017 04:00:09 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: ifdtool: refactor region-permission-related feature
by build bot (Jenkins) (Code Review) 16 Sep '17

16 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21416 ) Change subject: ifdtool: refactor region-permission-related feature ...................................................................... Patch Set 27: Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/15778/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/60570/ : SUCCESS -- To view, visit https://review.coreboot.org/21416 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I4d34606545db4d970ce3ec202da4bf4c5cbad18c Gerrit-Change-Number: 21416 Gerrit-PatchSet: 27 Gerrit-Owner: Bill XIE <persmule(a)gmail.com> Gerrit-Reviewer: Bill XIE <persmule(a)gmail.com> Gerrit-Reviewer: Nicola Corna <nicola(a)corna.info> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Philippe Mathieu-Daudé <f4bug(a)amsat.org> Gerrit-Reviewer: Shawn Chang <citypw(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sat, 16 Sep 2017 04:00:09 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: ifdtool: refacter some code
by build bot (Jenkins) (Code Review) 16 Sep '17

16 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21511 ) Change subject: ifdtool: refacter some code ...................................................................... Patch Set 10: Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/15776/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/60569/ : SUCCESS -- To view, visit https://review.coreboot.org/21511 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I98965711e4cb9792e5cc86cc4c1035559e0274f5 Gerrit-Change-Number: 21511 Gerrit-PatchSet: 10 Gerrit-Owner: Bill XIE <persmule(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: Sat, 16 Sep 2017 03:54:45 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: ifdtool: redesign some structures
by build bot (Jenkins) (Code Review) 16 Sep '17

16 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21510 ) Change subject: ifdtool: redesign some structures ...................................................................... Patch Set 10: Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/15775/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/60567/ : SUCCESS -- To view, visit https://review.coreboot.org/21510 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I161c9a2ae83d26e658d67d0804e943fff95fe076 Gerrit-Change-Number: 21510 Gerrit-PatchSet: 10 Gerrit-Owner: Bill XIE <persmule(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: Sat, 16 Sep 2017 03:54:28 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: ifdtool: merge region_filenames with region_name(s)
by build bot (Jenkins) (Code Review) 16 Sep '17

16 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21509 ) Change subject: ifdtool: merge region_filenames with region_name(s) ...................................................................... Patch Set 10: Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/15773/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/60566/ : SUCCESS -- To view, visit https://review.coreboot.org/21509 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I38489c6d3b3c161e9b0281188e6cdd0b62e38335 Gerrit-Change-Number: 21509 Gerrit-PatchSet: 10 Gerrit-Owner: Bill XIE <persmule(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: Sat, 16 Sep 2017 03:51:32 +0000 Gerrit-HasComments: No

1 0

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

coreboot-gerrit September 2017