coreboot-gerrit September 2017

coreboot-gerrit@coreboot.org

1 participants
1960 discussions

Change in coreboot[master]: util/board_status: do a spell check
by build bot (Jenkins) (Code Review) 01 Sep '17

01 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21322 ) Change subject: util/board_status: do a spell check ...................................................................... Patch Set 1: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/14969/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/59696/ : SUCCESS -- To view, visit https://review.coreboot.org/21322 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie39be471851586076343b8e9454a9140d4664b8d Gerrit-Change-Number: 21322 Gerrit-PatchSet: 1 Gerrit-Owner: Idwer Vollering <vidwer(a)gmail.com> Gerrit-Reviewer: Sumeet R Pawnikar <sumeet.r.pawnikar(a)intel.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Fri, 01 Sep 2017 15:34:15 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: util/lint: Exclude external payloads from coreboot lowercase check
by build bot (Jenkins) (Code Review) 01 Sep '17

01 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21321 ) Change subject: util/lint: Exclude external payloads from coreboot lowercase check ...................................................................... Patch Set 1: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/14968/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/59695/ : SUCCESS -- To view, visit https://review.coreboot.org/21321 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I70bbf37fad67fd9bda5724811f4bbdcc53779a42 Gerrit-Change-Number: 21321 Gerrit-PatchSet: 1 Gerrit-Owner: Martin Roth <martinroth(a)google.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Fri, 01 Sep 2017 15:29:08 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: util/board_status: do a spell check
by Sumeet R Pawnikar (Code Review) 01 Sep '17

01 Sep '17

Sumeet R Pawnikar has posted comments on this change. ( https://review.coreboot.org/21322 ) Change subject: util/board_status: do a spell check ...................................................................... Patch Set 1: (1 comment) https://review.coreboot.org/#/c/21322/1//COMMIT_MSG Commit Message: https://review.coreboot.org/#/c/21322/1//COMMIT_MSG@7 PS1, Line 7: do a spell check I would say Typo fixed. -- To view, visit https://review.coreboot.org/21322 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie39be471851586076343b8e9454a9140d4664b8d Gerrit-Change-Number: 21322 Gerrit-PatchSet: 1 Gerrit-Owner: Idwer Vollering <vidwer(a)gmail.com> Gerrit-Reviewer: Sumeet R Pawnikar <sumeet.r.pawnikar(a)intel.com> Gerrit-Comment-Date: Fri, 01 Sep 2017 15:28:49 +0000 Gerrit-HasComments: Yes

1 0

Change in coreboot[master]: util/board_status: do a spell check
by Idwer Vollering (Code Review) 01 Sep '17

01 Sep '17

Idwer Vollering has uploaded this change for review. ( https://review.coreboot.org/21322 Change subject: util/board_status: do a spell check ...................................................................... util/board_status: do a spell check Change-Id: Ie39be471851586076343b8e9454a9140d4664b8d Signed-off-by: Idwer Vollering <vidwer(a)gmail.com> --- M util/board_status/README 1 file changed, 4 insertions(+), 4 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/22/21322/1 diff --git a/util/board_status/README b/util/board_status/README index 2337826..97fd688 100644 --- a/util/board_status/README +++ b/util/board_status/README @@ -13,9 +13,9 @@ ------------ [1] in make menuconfig you should have: [*] Use CMOS for configuration values This option is also known as CONFIG_USE_OPTION_TABLE in Kconfig. -Information trough SSH +Information through SSH ====================== -board_status.sh can gather information trough ssh with the -r +board_status.sh can gather information through ssh with the -r option. When using "-r <host>", The script will attempt to log into @@ -28,10 +28,10 @@ board_status.sh expects the remote host to have the following programs in its path: cbmem, dmesg -Boot log gathering trough a serial port +Boot log gathering through a serial port ======================================= When using -s </dev/xxx>, board_status.sh starts by retrieving the -boot log trough the serial port. +boot log through the serial port. To produce such logs, power off the board, run board_status.sh with the right arguments, power on the board. -- To view, visit https://review.coreboot.org/21322 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ie39be471851586076343b8e9454a9140d4664b8d Gerrit-Change-Number: 21322 Gerrit-PatchSet: 1 Gerrit-Owner: Idwer Vollering <vidwer(a)gmail.com>

1 0

Change in coreboot[master]: util/lint: Exclude external payloads from coreboot lowercase check
by Martin Roth (Code Review) 01 Sep '17

01 Sep '17

Martin Roth has uploaded this change for review. ( https://review.coreboot.org/21321 Change subject: util/lint: Exclude external payloads from coreboot lowercase check ...................................................................... util/lint: Exclude external payloads from coreboot lowercase check Change-Id: I70bbf37fad67fd9bda5724811f4bbdcc53779a42 Signed-off-by: Martin Roth <martinroth(a)google.com> --- M util/lint/lint-stable-021-coreboot-lowercase 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/21/21321/1 diff --git a/util/lint/lint-stable-021-coreboot-lowercase b/util/lint/lint-stable-021-coreboot-lowercase index ef83a66..82e2dd4 100755 --- a/util/lint/lint-stable-021-coreboot-lowercase +++ b/util/lint/lint-stable-021-coreboot-lowercase @@ -16,7 +16,7 @@ LC_ALL=C export LC_ALL -EXCLUDE='^3rdparty/\|util/crossgcc/xgcc\|Binary file\|coreboot\|COREBOOT\|CorebootPayload\|CorebootModule\|minnowboard.org/…' +EXCLUDE='^3rdparty/\|util/crossgcc/xgcc\|Binary file\|coreboot\|COREBOOT\|CorebootPayload\|CorebootModule\|minnowboard.org/…' # Use git grep if the code is in a git repo, otherwise use grep. if [ -n "$(command -v git)" ] && [ -d .git ]; then -- To view, visit https://review.coreboot.org/21321 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I70bbf37fad67fd9bda5724811f4bbdcc53779a42 Gerrit-Change-Number: 21321 Gerrit-PatchSet: 1 Gerrit-Owner: Martin Roth <martinroth(a)google.com>

1 0

Change in coreboot[master]: nb/intel/x4x: Select LAPIC_MONOTONIC_TIMER
by Sumeet R Pawnikar (Code Review) 01 Sep '17

01 Sep '17

Sumeet R Pawnikar has posted comments on this change. ( https://review.coreboot.org/21320 ) Change subject: nb/intel/x4x: Select LAPIC_MONOTONIC_TIMER ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://review.coreboot.org/21320 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I01059c8cbdc6a002dfd75b6da3a629811b137702 Gerrit-Change-Number: 21320 Gerrit-PatchSet: 1 Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: Sumeet R Pawnikar <sumeet.r.pawnikar(a)intel.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Fri, 01 Sep 2017 15:22:12 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: Kconfig: Move and rename ADD_VBT_DATA_FILE
by build bot (Jenkins) (Code Review) 01 Sep '17

01 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20619 ) Change subject: Kconfig: Move and rename ADD_VBT_DATA_FILE ...................................................................... Patch Set 3: Verified-1 Build Unstable https://qa.coreboot.org/job/coreboot-gerrit/59694/ : UNSTABLE https://qa.coreboot.org/job/coreboot-checkpatch/14967/ : SUCCESS -- To view, visit https://review.coreboot.org/20619 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I688bac339c32e9c856642a0f4bd5929beef06409 Gerrit-Change-Number: 20619 Gerrit-PatchSet: 3 Gerrit-Owner: Patrick Rudolph <siro(a)das-labor.org> Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Matt DeVillier <matt.devillier(a)gmail.com> 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: Fri, 01 Sep 2017 15:07:59 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: mb/*/*/romstage.c: Clean up targets with i82801gx
by build bot (Jenkins) (Code Review) 01 Sep '17

01 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21065 ) Change subject: mb/*/*/romstage.c: Clean up targets with i82801gx ...................................................................... Patch Set 4: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/14964/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/59691/ : SUCCESS -- To view, visit https://review.coreboot.org/21065 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I9d6a0ff3d391947f611a2f3c65684f4ee57bc263 Gerrit-Change-Number: 21065 Gerrit-PatchSet: 4 Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Fri, 01 Sep 2017 14:57:37 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: ec/lenovo/h8/acpi/thermal: Add ACPI fan control
by build bot (Jenkins) (Code Review) 01 Sep '17

01 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21227 ) Change subject: ec/lenovo/h8/acpi/thermal: Add ACPI fan control ...................................................................... Patch Set 3: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/14966/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/59692/ : SUCCESS -- To view, visit https://review.coreboot.org/21227 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I075ff5c69676927db1c5e731294e18796884f97e Gerrit-Change-Number: 21227 Gerrit-PatchSet: 3 Gerrit-Owner: Patrick Rudolph <siro(a)das-labor.org> Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz> 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: Philipp Deppenwiese <zaolin.daisuki(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Fri, 01 Sep 2017 14:37:20 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: ec/lenovo/h8/acpi/thermal: Add support for passive cooling
by build bot (Jenkins) (Code Review) 01 Sep '17

01 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21223 ) Change subject: ec/lenovo/h8/acpi/thermal: Add support for passive cooling ...................................................................... Patch Set 3: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/14965/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/59693/ : SUCCESS -- To view, visit https://review.coreboot.org/21223 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I922923a9029de77158988ac254bab4aad9536935 Gerrit-Change-Number: 21223 Gerrit-PatchSet: 3 Gerrit-Owner: Patrick Rudolph <siro(a)das-labor.org> Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz> 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: Philipp Deppenwiese <zaolin.daisuki(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Fri, 01 Sep 2017 14:37:14 +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