coreboot-gerrit December 2017

coreboot-gerrit@coreboot.org

1 participants
1557 discussions

Change in coreboot[master]: autoport: Add Intel PCIe Root Port and Bridge
by build bot (Jenkins) (Code Review) 09 Dec '17

09 Dec '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/22794 ) Change subject: autoport: Add Intel PCIe Root Port and Bridge ...................................................................... Patch Set 2: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/19141/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/64325/ : SUCCESS -- To view, visit https://review.coreboot.org/22794 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I4111b73adc0f08d643c940cd43ab7fd4c0af7668 Gerrit-Change-Number: 22794 Gerrit-PatchSet: 2 Gerrit-Owner: Jean Lucas <jean(a)4ray.co> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sat, 09 Dec 2017 11:18:16 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: [NOTFORMERGE] nb/sandybridge/raminit: print write training statistics
by build bot (Jenkins) (Code Review) 09 Dec '17

09 Dec '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/22795 ) Change subject: [NOTFORMERGE] nb/sandybridge/raminit: print write training statistics ...................................................................... Patch Set 1: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/19140/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/64324/ : SUCCESS -- To view, visit https://review.coreboot.org/22795 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I8033517f1bc877875ccd7ef950329b8b251fc2c2 Gerrit-Change-Number: 22795 Gerrit-PatchSet: 1 Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sat, 09 Dec 2017 10:58:53 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: intel: Use MSR_EBC_FREQUENCY_ID instead of 0x2c
by build bot (Jenkins) (Code Review) 09 Dec '17

09 Dec '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/22603 ) Change subject: intel: Use MSR_EBC_FREQUENCY_ID instead of 0x2c ...................................................................... Patch Set 2: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/19139/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/64323/ : SUCCESS -- To view, visit https://review.coreboot.org/22603 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ib1b761fc417f1bb000f408d3bed5e8666963f51d Gerrit-Change-Number: 22603 Gerrit-PatchSet: 2 Gerrit-Owner: Elyes HAOUAS <ehaouas(a)noos.fr> Gerrit-Reviewer: Elyes HAOUAS <ehaouas(a)noos.fr> Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sat, 09 Dec 2017 10:52:39 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: [NOTFORMERGE] nb/sandybridge/raminit: print write training statistics
by Arthur Heymans (Code Review) 09 Dec '17

09 Dec '17

Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/22795 Change subject: [NOTFORMERGE] nb/sandybridge/raminit: print write training statistics ...................................................................... [NOTFORMERGE] nb/sandybridge/raminit: print write training statistics for debug purposes... Change-Id: I8033517f1bc877875ccd7ef950329b8b251fc2c2 Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz> --- M src/northbridge/intel/sandybridge/raminit_common.c 1 file changed, 8 insertions(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/95/22795/1 diff --git a/src/northbridge/intel/sandybridge/raminit_common.c b/src/northbridge/intel/sandybridge/raminit_common.c index eaef5f7..64a4808 100644 --- a/src/northbridge/intel/sandybridge/raminit_common.c +++ b/src/northbridge/intel/sandybridge/raminit_common.c @@ -1551,6 +1551,7 @@ int timC; int statistics[NUM_LANES][MAX_TIMC + 1]; int lane; + int i; wait_428c(channel); @@ -1577,6 +1578,13 @@ } } FOR_ALL_LANES { + printk(RAM_SPEW, "statistics lane %d:\n", lane); + for (i = 0; i < MAX_TIMC + 1; i++) { + printk(RAM_SPEW, " 0x%x", statistics[lane][i]); + if (!(i % 16)) + printk(RAM_SPEW, "\n"); + } + printk(RAM_SPEW, "\n"); struct run rn = get_longest_zero_run(statistics[lane], MAX_TIMC + 1); ctrl->timings[channel][slotrank].lanes[lane].timC = rn.middle; -- To view, visit https://review.coreboot.org/22795 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I8033517f1bc877875ccd7ef950329b8b251fc2c2 Gerrit-Change-Number: 22795 Gerrit-PatchSet: 1 Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>

1 0

Change in coreboot[master]: cpu/intel/speedstep/acpi.c: Add Netburst CPUs
by build bot (Jenkins) (Code Review) 09 Dec '17

09 Dec '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/22604 ) Change subject: cpu/intel/speedstep/acpi.c: Add Netburst CPUs ...................................................................... Patch Set 5: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/19138/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/64322/ : SUCCESS -- To view, visit https://review.coreboot.org/22604 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I06e162d3260dedeb3b16583460633507fbcbd52a Gerrit-Change-Number: 22604 Gerrit-PatchSet: 5 Gerrit-Owner: Elyes HAOUAS <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, 09 Dec 2017 10:37:03 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: autoport: Add Intel PCIe Root Port and Bridge
by build bot (Jenkins) (Code Review) 09 Dec '17

09 Dec '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/22794 ) Change subject: autoport: Add Intel PCIe Root Port and Bridge ...................................................................... Patch Set 1: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/19137/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/64321/ : SUCCESS -- To view, visit https://review.coreboot.org/22794 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I4111b73adc0f08d643c940cd43ab7fd4c0af7668 Gerrit-Change-Number: 22794 Gerrit-PatchSet: 1 Gerrit-Owner: Jean Lucas <jean(a)4ray.co> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sat, 09 Dec 2017 10:04:35 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: mainboard: Add Dell Inspiron 660s
by build bot (Jenkins) (Code Review) 09 Dec '17

09 Dec '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/22682 ) Change subject: mainboard: Add Dell Inspiron 660s ...................................................................... Patch Set 6: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/19136/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/64320/ : SUCCESS -- To view, visit https://review.coreboot.org/22682 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I8dabb1e1d14e60957e8c8800b0c30921d6bb5d8e Gerrit-Change-Number: 22682 Gerrit-PatchSet: 6 Gerrit-Owner: Jean Lucas <jean(a)4ray.co> Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Idwer Vollering <vidwer(a)gmail.com> Gerrit-Reviewer: Jean Lucas <jean(a)4ray.co> Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Vladimir Serbinenko <phcoder(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sat, 09 Dec 2017 09:54:45 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: autoport: Add Intel PCIe Root Port and Bridge
by Jean Lucas (Code Review) 09 Dec '17

09 Dec '17

Jean Lucas has uploaded this change for review. ( https://review.coreboot.org/22794 Change subject: autoport: Add Intel PCIe Root Port and Bridge ...................................................................... autoport: Add Intel PCIe Root Port and Bridge 0x0151 corresponds to the PCIe Root Port for the Xeon E3-1200. 0x244e corresponds to the 82801 PCI Bridge. Change-Id: I4111b73adc0f08d643c940cd43ab7fd4c0af7668 Signed-off-by: Jean Lucas <jean(a)4ray.co> --- M util/autoport/bd82x6x.go 1 file changed, 1 insertion(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/94/22794/1 diff --git a/util/autoport/bd82x6x.go b/util/autoport/bd82x6x.go index ce06a81..1e64470 100644 --- a/util/autoport/bd82x6x.go +++ b/util/autoport/bd82x6x.go @@ -443,6 +443,7 @@ 0x1c18, 0x1c1a, 0x1c1c, 0x1c1e, 0x1e10, 0x1e12, 0x1e14, 0x1e16, 0x1e18, 0x1e1a, 0x1e1c, 0x1e1e, + 0x0151, 0x244e, } { RegisterPCI(0x8086, id, GenericPCI{}) } -- To view, visit https://review.coreboot.org/22794 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I4111b73adc0f08d643c940cd43ab7fd4c0af7668 Gerrit-Change-Number: 22794 Gerrit-PatchSet: 1 Gerrit-Owner: Jean Lucas <jean(a)4ray.co>

1 0

Change in coreboot[master]: google/scarlet: support INNOLUX P097PFG panel
by Julius Werner (Code Review) 09 Dec '17

09 Dec '17

Julius Werner has posted comments on this change. ( https://review.coreboot.org/22780 ) Change subject: google/scarlet: support INNOLUX P097PFG panel ...................................................................... Patch Set 1: (1 comment) https://review.coreboot.org/#/c/22780/1/src/mainboard/google/gru/mainboard.c File src/mainboard/google/gru/mainboard.c: https://review.coreboot.org/#/c/22780/1/src/mainboard/google/gru/mainboard.… PS1, Line 568: if (sku_id() == 6) { > So how to translate sku_id still do not have final decision, right? We figured out the problem (see CL:22791). With that fix the SKU IDs are as expected (7 for existing rev2s, and the rev3 with INX should be 6). -- To view, visit https://review.coreboot.org/22780 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I43fa5d859a9a529a84c58a953b37d03953ce648a Gerrit-Change-Number: 22780 Gerrit-PatchSet: 1 Gerrit-Owner: Lin Huang <hl(a)rock-chips.com> Gerrit-Reviewer: Brian Norris <briannorris(a)chromium.org> Gerrit-Reviewer: Julius Werner <jwerner(a)chromium.org> Gerrit-Reviewer: Lin Huang <hl(a)rock-chips.com> Gerrit-Reviewer: Matthias Kaehlcke <mka(a)chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sat, 09 Dec 2017 06:59:54 +0000 Gerrit-HasComments: Yes

1 0

Change in coreboot[master]: soc/amd/common: Add amdblocks/spi.h
by Martin Roth (Code Review) 09 Dec '17

09 Dec '17

Martin Roth has posted comments on this change. ( https://review.coreboot.org/22793 ) Change subject: soc/amd/common: Add amdblocks/spi.h ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://review.coreboot.org/22793 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I0dbb5bf84cc3462a7aa58a5531d6b8b8bc8ca4df Gerrit-Change-Number: 22793 Gerrit-PatchSet: 1 Gerrit-Owner: Marc Jones <marc(a)marcjonesconsulting.com> Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd(a)gmail.com> Gerrit-Reviewer: Martin Roth <martinroth(a)google.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sat, 09 Dec 2017 04:04:10 +0000 Gerrit-HasComments: No

1 0

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

coreboot-gerrit December 2017