coreboot-gerrit March 2017

coreboot-gerrit@coreboot.org

1 participants
2925 discussions

Change in coreboot[master]: nb/i945/raminit: Use common ddr2 decode functions
by build bot (Jenkins) (Code Review) 31 Mar '17

31 Mar '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/18305 ) Change subject: nb/i945/raminit: Use common ddr2 decode functions ...................................................................... Patch Set 29: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/6991/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/51024/ : SUCCESS -- To view, visit https://review.coreboot.org/18305 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I97c93939d11807752797785dd88c70b43a236ee3 Gerrit-PatchSet: 29 Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: HAOUAS Elyes <ehaouas(a)noos.fr> Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) Gerrit-Reviewer: coreboot org <coreboot.org(a)gmail.com> Gerrit-HasComments: No

1 0

Change in coreboot[master]: device/dram/ddr2: Add functions to select common CAS en freq
by build bot (Jenkins) (Code Review) 31 Mar '17

31 Mar '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/18320 ) Change subject: device/dram/ddr2: Add functions to select common CAS en freq ...................................................................... Patch Set 22: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/6989/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/51022/ : SUCCESS -- To view, visit https://review.coreboot.org/18320 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3ab39d38a243edddfde8f70ebd23f79ff774e90e Gerrit-PatchSet: 22 Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) Gerrit-HasComments: No

1 0

Change in coreboot[master]: libpayload-x86: Export keyboard modifiers
by build bot (Jenkins) (Code Review) 31 Mar '17

31 Mar '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/18601 ) Change subject: libpayload-x86: Export keyboard modifiers ...................................................................... Patch Set 3: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-gerrit/51013/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/6995/ : SUCCESS -- To view, visit https://review.coreboot.org/18601 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ifc7bd4aa86f20d67c5b542d0458b966e605c5499 Gerrit-PatchSet: 3 Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Owner: Patrick Rudolph <siro(a)das-labor.org> Gerrit-Reviewer: Martin Roth <martinroth(a)google.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) Gerrit-HasComments: No

1 0

Change in coreboot[master]: libpayload: Fill reg_base for debugging purposes
by build bot (Jenkins) (Code Review) 31 Mar '17

31 Mar '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/18600 ) Change subject: libpayload: Fill reg_base for debugging purposes ...................................................................... Patch Set 3: Build Successful https://qa.coreboot.org/job/coreboot-gerrit/51012/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/6994/ : SUCCESS -- To view, visit https://review.coreboot.org/18600 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: If42135c8e10b70d5ac9626521abd9cca3cf40053 Gerrit-PatchSet: 3 Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Owner: Patrick Rudolph <siro(a)das-labor.org> Gerrit-Reviewer: Martin Roth <martinroth(a)google.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) Gerrit-HasComments: No

1 0

Change in coreboot[master]: libpayload: Always set pciaddr
by build bot (Jenkins) (Code Review) 31 Mar '17

31 Mar '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/18599 ) Change subject: libpayload: Always set pciaddr ...................................................................... Patch Set 3: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-gerrit/51011/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/6993/ : SUCCESS -- To view, visit https://review.coreboot.org/18599 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I83a0e7f7196ed251fa0becc4e56bef3ca68f20f4 Gerrit-PatchSet: 3 Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Owner: Patrick Rudolph <siro(a)das-labor.org> Gerrit-Reviewer: Martin Roth <martinroth(a)google.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) Gerrit-HasComments: No

1 0

Change in coreboot[master]: libpayload: Fill reg_base for debugging purposes
by build bot (Jenkins) (Code Review) 31 Mar '17

31 Mar '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/18600 ) Change subject: libpayload: Fill reg_base for debugging purposes ...................................................................... Patch Set 3: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-gerrit/51012/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/6992/ : SUCCESS -- To view, visit https://review.coreboot.org/18600 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: If42135c8e10b70d5ac9626521abd9cca3cf40053 Gerrit-PatchSet: 3 Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Owner: Patrick Rudolph <siro(a)das-labor.org> Gerrit-Reviewer: Martin Roth <martinroth(a)google.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) Gerrit-HasComments: No

1 0

Change in coreboot[master]: mainboard/neoware/g170: disable use of upper memory for SeaBIOS
by Martin Roth (Code Review) 31 Mar '17

31 Mar '17

Martin Roth has posted comments on this change. ( https://review.coreboot.org/18905 ) Change subject: mainboard/neoware/g170: disable use of upper memory for SeaBIOS ...................................................................... Patch Set 1: You need a signed-off-by line in the commit message: Signed-off-by: Lubomir Rintel <lkundrak(a)v3.sk> -- To view, visit https://review.coreboot.org/18905 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I69f1fe38503b0f8d6015b515637d8376726490c0 Gerrit-PatchSet: 1 Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Owner: Lubomir Rintel <lkundrak(a)v3.sk> Gerrit-Reviewer: Martin Roth <martinroth(a)google.com> Gerrit-Reviewer: build bot (Jenkins) Gerrit-HasComments: No

1 0

Change in coreboot[master]: drivers/i2c/tpm: remove unused variable in tpm_transmit()
by Philippe Mathieu-Daudé (Code Review) 31 Mar '17

31 Mar '17

Philippe Mathieu-Daudé has posted comments on this change. ( https://review.coreboot.org/19060 ) Change subject: drivers/i2c/tpm: remove unused variable in tpm_transmit() ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://review.coreboot.org/19060 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I015a6633c0951980658b3c879e48bc84d604d62e Gerrit-PatchSet: 1 Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Owner: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com> Gerrit-Reviewer: Philippe Mathieu-Daudé <philippe.mathieu.daude(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) Gerrit-HasComments: No

1 0

Change in coreboot[master]: nb/i945/raminit: Use common ddr2 decode functions
by Arthur Heymans (Code Review) 31 Mar '17

31 Mar '17

Hello build bot (Jenkins), coreboot org, I'd like you to reexamine a change. Please visit https://review.coreboot.org/18305 to look at the new patch set (#29). Change subject: nb/i945/raminit: Use common ddr2 decode functions ...................................................................... nb/i945/raminit: Use common ddr2 decode functions This simplifies computing dram timings a lot. This removes computation of rank size based on columns, rows, banks,... and uses the information in SPD byte 31. The result of this is that dimms with multiple asymmetric ranks are not supported anymore. These however are very rare and most likely never tested on this platform. Change-Id: I97c93939d11807752797785dd88c70b43a236ee3 Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz> --- M src/northbridge/intel/i945/raminit.c M src/northbridge/intel/i945/raminit.h 2 files changed, 149 insertions(+), 616 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/05/18305/29 -- To view, visit https://review.coreboot.org/18305 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I97c93939d11807752797785dd88c70b43a236ee3 Gerrit-PatchSet: 29 Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: HAOUAS Elyes <ehaouas(a)noos.fr> Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) Gerrit-Reviewer: coreboot org <coreboot.org(a)gmail.com>

1 0

Change in coreboot[master]: device/dram/ddr2: Add functions to select common CAS en freq
by Arthur Heymans (Code Review) 31 Mar '17

31 Mar '17

Hello build bot (Jenkins), I'd like you to reexamine a change. Please visit https://review.coreboot.org/18320 to look at the new patch set (#22). Change subject: device/dram/ddr2: Add functions to select common CAS en freq ...................................................................... device/dram/ddr2: Add functions to select common CAS en freq This adds two functions: - one to find the highest common tCLK per CAS; - another to find to select tCLK and CAS based on the results of the previous function and the limits on minimal tCLK the memory controller can impose. Change-Id: I3ab39d38a243edddfde8f70ebd23f79ff774e90e Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz> --- M src/device/dram/ddr2.c M src/include/device/dram/ddr2.h 2 files changed, 83 insertions(+), 1 deletion(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/20/18320/22 -- To view, visit https://review.coreboot.org/18320 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I3ab39d38a243edddfde8f70ebd23f79ff774e90e Gerrit-PatchSet: 22 Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins)

1 0

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

coreboot-gerrit March 2017