coreboot-gerrit January 2018

coreboot-gerrit@coreboot.org

1 participants
2315 discussions

Change in coreboot[master]: soc/intel/cannonlake: Add more HDA Audio Link settings
by build bot (Jenkins) (Code Review) 29 Jan '18

29 Jan '18

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/23400 ) Change subject: soc/intel/cannonlake: Add more HDA Audio Link settings ...................................................................... Patch Set 4: Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/21314/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/66707/ : SUCCESS -- To view, visit https://review.coreboot.org/23400 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ib2a75f554ddf9919a62c78a162ec1b9e602f1f5d Gerrit-Change-Number: 23400 Gerrit-PatchSet: 4 Gerrit-Owner: Lijian Zhao <lijian.zhao(a)intel.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Lijian Zhao <lijian.zhao(a)intel.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Sathyanarayana Nujella <sathyanarayana.nujella(a)intel.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Mon, 29 Jan 2018 10:47:33 +0000 Gerrit-HasComments: No Gerrit-HasLabels: No

1 0

Change in coreboot[master]: mainboard/google/zoombini: Add config for meowth audio
by build bot (Jenkins) (Code Review) 29 Jan '18

29 Jan '18

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/23334 ) Change subject: mainboard/google/zoombini: Add config for meowth audio ...................................................................... Patch Set 10: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/21317/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/66710/ : SUCCESS -- To view, visit https://review.coreboot.org/23334 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ic89bf669c7ab2ef39ce64e4da6a57a7069ee75f9 Gerrit-Change-Number: 23334 Gerrit-PatchSet: 10 Gerrit-Owner: Sathyanarayana Nujella <sathyanarayana.nujella(a)intel.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Bora Guvendik <bora.guvendik(a)intel.com> Gerrit-Reviewer: HARSHAPRIYA N <harshapriya.n(a)intel.com> Gerrit-Reviewer: Hannah Williams <hannah.williams(a)intel.com> Gerrit-Reviewer: Lijian Zhao <lijian.zhao(a)intel.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Sathyanarayana Nujella <sathyanarayana.nujella(a)intel.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Mon, 29 Jan 2018 10:41:03 +0000 Gerrit-HasComments: No Gerrit-HasLabels: Yes

1 0

Change in coreboot[master]: nb/intel/*.h: Remove left-over register definitions
by build bot (Jenkins) (Code Review) 29 Jan '18

29 Jan '18

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/23487 ) Change subject: nb/intel/*.h: Remove left-over register definitions ...................................................................... Patch Set 1: Verified-1 Build Unstable https://qa.coreboot.org/job/coreboot-gerrit/66709/ : UNSTABLE https://qa.coreboot.org/job/coreboot-checkpatch/21316/ : SUCCESS -- To view, visit https://review.coreboot.org/23487 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I122affffd5108052ed7a95b34d0d66a6d3279d41 Gerrit-Change-Number: 23487 Gerrit-PatchSet: 1 Gerrit-Owner: Patrick Rudolph <siro(a)das-labor.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Mon, 29 Jan 2018 10:32:22 +0000 Gerrit-HasComments: No Gerrit-HasLabels: Yes

1 0

Change in coreboot[master]: mainboard/google/zoombini: Add config for meowth audio
by build bot (Jenkins) (Code Review) 29 Jan '18

29 Jan '18

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/23334 ) Change subject: mainboard/google/zoombini: Add config for meowth audio ...................................................................... Patch Set 9: Verified-1 Build Failed https://qa.coreboot.org/job/coreboot-gerrit/66708/ : ABORTED https://qa.coreboot.org/job/coreboot-checkpatch/21315/ : SUCCESS -- To view, visit https://review.coreboot.org/23334 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ic89bf669c7ab2ef39ce64e4da6a57a7069ee75f9 Gerrit-Change-Number: 23334 Gerrit-PatchSet: 9 Gerrit-Owner: Sathyanarayana Nujella <sathyanarayana.nujella(a)intel.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Bora Guvendik <bora.guvendik(a)intel.com> Gerrit-Reviewer: HARSHAPRIYA N <harshapriya.n(a)intel.com> Gerrit-Reviewer: Hannah Williams <hannah.williams(a)intel.com> Gerrit-Reviewer: Lijian Zhao <lijian.zhao(a)intel.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Sathyanarayana Nujella <sathyanarayana.nujella(a)intel.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Mon, 29 Jan 2018 10:27:43 +0000 Gerrit-HasComments: No Gerrit-HasLabels: Yes

1 0

Change in coreboot[master]: nb/intel/*.h: Remove left-over register definitions
by Patrick Rudolph (Code Review) 29 Jan '18

29 Jan '18

Patrick Rudolph has uploaded this change for review. ( https://review.coreboot.org/23487 Change subject: nb/intel/*.h: Remove left-over register definitions ...................................................................... nb/intel/*.h: Remove left-over register definitions The code has been moved into drivers folder. Change-Id: I122affffd5108052ed7a95b34d0d66a6d3279d41 Signed-off-by: Patrick Rudolph <siro(a)das-labor.org> --- M src/northbridge/intel/fsp_sandybridge/northbridge.h M src/northbridge/intel/haswell/haswell.h M src/northbridge/intel/nehalem/nehalem.h M src/northbridge/intel/sandybridge/sandybridge.h 4 files changed, 0 insertions(+), 28 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/87/23487/1 diff --git a/src/northbridge/intel/fsp_sandybridge/northbridge.h b/src/northbridge/intel/fsp_sandybridge/northbridge.h index c0194f2..ddd56e1 100644 --- a/src/northbridge/intel/fsp_sandybridge/northbridge.h +++ b/src/northbridge/intel/fsp_sandybridge/northbridge.h @@ -105,13 +105,6 @@ #define BCTRL1 0x3e /* 16bit */ - -/* Device 0:2.0 PCI configuration space (Graphics Device) */ - -#define MSAC 0x62 /* Multi Size Aperture Control */ -#define SWSCI 0xe8 /* SWSCI enable */ -#define ASLS 0xfc /* OpRegion Base */ - /* * MCHBAR */ diff --git a/src/northbridge/intel/haswell/haswell.h b/src/northbridge/intel/haswell/haswell.h index 2d03a68..cd6a537 100644 --- a/src/northbridge/intel/haswell/haswell.h +++ b/src/northbridge/intel/haswell/haswell.h @@ -92,13 +92,6 @@ #define BCTRL1 0x3e /* 16bit */ - -/* Device 0:2.0 PCI configuration space (Graphics Device) */ - -#define MSAC 0x62 /* Multi Size Aperture Control */ -#define SWSCI 0xe8 /* SWSCI enable */ -#define ASLS 0xfc /* OpRegion Base */ - /* * MCHBAR */ diff --git a/src/northbridge/intel/nehalem/nehalem.h b/src/northbridge/intel/nehalem/nehalem.h index a693e12..886cd00 100644 --- a/src/northbridge/intel/nehalem/nehalem.h +++ b/src/northbridge/intel/nehalem/nehalem.h @@ -191,13 +191,6 @@ #define BCTRL1 0x3e /* 16bit */ - -/* Device 0:2.0 PCI configuration space (Graphics Device) */ - -#define MSAC 0x62 /* Multi Size Aperture Control */ -#define SWSCI 0xe8 /* SWSCI enable */ -#define ASLS 0xfc /* OpRegion Base */ - /* * MCHBAR */ diff --git a/src/northbridge/intel/sandybridge/sandybridge.h b/src/northbridge/intel/sandybridge/sandybridge.h index 1f56585..591cc3f 100644 --- a/src/northbridge/intel/sandybridge/sandybridge.h +++ b/src/northbridge/intel/sandybridge/sandybridge.h @@ -110,13 +110,6 @@ #define BCTRL1 0x3e /* 16bit */ - -/* Device 0:2.0 PCI configuration space (Graphics Device) */ - -#define MSAC 0x62 /* Multi Size Aperture Control */ -#define SWSCI 0xe8 /* SWSCI enable */ -#define ASLS 0xfc /* OpRegion Base */ - /* * MCHBAR */ -- To view, visit https://review.coreboot.org/23487 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I122affffd5108052ed7a95b34d0d66a6d3279d41 Gerrit-Change-Number: 23487 Gerrit-PatchSet: 1 Gerrit-Owner: Patrick Rudolph <siro(a)das-labor.org>

1 0

Change in coreboot[master]: soc/intel/common/opregion: Get rid of opregion.c
by build bot (Jenkins) (Code Review) 29 Jan '18

29 Jan '18

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21703 ) Change subject: soc/intel/common/opregion: Get rid of opregion.c ...................................................................... Patch Set 9: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/21313/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/66706/ : SUCCESS -- To view, visit https://review.coreboot.org/21703 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I5f78e9030df12da5369d142dda5c59e576ebcef7 Gerrit-Change-Number: 21703 Gerrit-PatchSet: 9 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: Patrick Rudolph <siro(a)das-labor.org> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Pratikkumar V Prajapati <pratikkumar.v.prajapati(a)intel.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Mon, 29 Jan 2018 10:22:48 +0000 Gerrit-HasComments: No Gerrit-HasLabels: Yes

1 0

Change in coreboot[master]: autoport: Don't do writes to FD
by build bot (Jenkins) (Code Review) 29 Jan '18

29 Jan '18

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/23486 ) Change subject: autoport: Don't do writes to FD ...................................................................... Patch Set 2: Verified-1 Build Failed https://qa.coreboot.org/job/coreboot-gerrit/66705/ : FAILURE -- To view, visit https://review.coreboot.org/23486 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I4edb75569ceec2d2f1308755a66d286202ca0ae6 Gerrit-Change-Number: 23486 Gerrit-PatchSet: 2 Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Mon, 29 Jan 2018 09:53:32 +0000 Gerrit-HasComments: No Gerrit-HasLabels: Yes

1 0

Change in coreboot[master]: autoport: Don't do writes to FD
by build bot (Jenkins) (Code Review) 29 Jan '18

29 Jan '18

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/23486 ) Change subject: autoport: Don't do writes to FD ...................................................................... Patch Set 1: Verified-1 Build Failed https://qa.coreboot.org/job/coreboot-checkpatch/21312/ : ABORTED https://qa.coreboot.org/job/coreboot-gerrit/66704/ : ABORTED -- To view, visit https://review.coreboot.org/23486 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I4edb75569ceec2d2f1308755a66d286202ca0ae6 Gerrit-Change-Number: 23486 Gerrit-PatchSet: 1 Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Mon, 29 Jan 2018 09:53:09 +0000 Gerrit-HasComments: No Gerrit-HasLabels: Yes

1 0

Change in coreboot[master]: autoport: Don't do writes to FD
by Arthur Heymans (Code Review) 29 Jan '18

29 Jan '18

Arthur Heymans has uploaded a new patch set (#2). ( https://review.coreboot.org/23486 ) Change subject: autoport: Don't do writes to FD ...................................................................... autoport: Don't do writes to FD This is properly done in ramstage, so no need to do it in romstage. Change-Id: I4edb75569ceec2d2f1308755a66d286202ca0ae6 Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz> --- M util/autoport/bd82x6x.go 1 file changed, 1 insertion(+), 5 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/86/23486/2 -- To view, visit https://review.coreboot.org/23486 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I4edb75569ceec2d2f1308755a66d286202ca0ae6 Gerrit-Change-Number: 23486 Gerrit-PatchSet: 2 Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>

1 0

Change in coreboot[master]: autoport: Don't do write to FD
by Arthur Heymans (Code Review) 29 Jan '18

29 Jan '18

Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/23486 Change subject: autoport: Don't do write to FD ...................................................................... autoport: Don't do write to FD This is properly done in ramstage, so no need to do it in romstage. Change-Id: I4edb75569ceec2d2f1308755a66d286202ca0ae6 Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz> --- M util/autoport/bd82x6x.go 1 file changed, 1 insertion(+), 5 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/86/23486/1 diff --git a/util/autoport/bd82x6x.go b/util/autoport/bd82x6x.go index d342a79..c7862cf 100644 --- a/util/autoport/bd82x6x.go +++ b/util/autoport/bd82x6x.go @@ -336,12 +336,8 @@ void rcba_config(void) { - /* Disable devices. */ `) - RestoreRCBA32(sb, inteltool, 0x3414) - RestoreRCBA32(sb, inteltool, 0x3418) - - sb.WriteString("\n}\n") + sb.WriteString("}\n\n") sb.WriteString("const struct southbridge_usb_port mainboard_usb_ports[] = {\n") -- To view, visit https://review.coreboot.org/23486 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I4edb75569ceec2d2f1308755a66d286202ca0ae6 Gerrit-Change-Number: 23486 Gerrit-PatchSet: 1 Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>

1 0

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

coreboot-gerrit January 2018