build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/23834 )
Change subject: Include <sys/io.h> in intelmetool/intelmetool.c
......................................................................
Patch Set 1: Verified+1
Build Successful
https://qa.coreboot.org/job/coreboot-gerrit/67755/ : SUCCESS
https://qa.coreboot.org/job/coreboot-checkpatch/22256/ : SUCCESS
--
To view, visit https://review.coreboot.org/23834
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: Ia27203cf47b9be3f7bf1ad422c8f490caeae8f56
Gerrit-Change-Number: 23834
Gerrit-PatchSet: 1
Gerrit-Owner: Ivan J. <parazyd(a)dyne.org>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Wed, 21 Feb 2018 10:07:42 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: Yes
Ivan J. has uploaded this change for review. ( https://review.coreboot.org/23834
Change subject: Include <sys/io.h> in intelmetool/intelmetool.c
......................................................................
Include <sys/io.h> in intelmetool/intelmetool.c
This allows compiling the program using musl-libc, since otherwise
iopl(2) is undeclared.
Change-Id: Ia27203cf47b9be3f7bf1ad422c8f490caeae8f56
Signed-off-by: Ivan J. <parazyd(a)dyne.org>
---
M util/intelmetool/intelmetool.c
1 file changed, 1 insertion(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/34/23834/1
diff --git a/util/intelmetool/intelmetool.c b/util/intelmetool/intelmetool.c
index 0b0e509..14cb035 100644
--- a/util/intelmetool/intelmetool.c
+++ b/util/intelmetool/intelmetool.c
@@ -18,6 +18,7 @@
#include <unistd.h>
#include <string.h>
#include <cpuid.h>
+#include <sys/io.h>
#ifdef __NetBSD__
#include <machine/sysarch.h>
--
To view, visit https://review.coreboot.org/23834
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: Ia27203cf47b9be3f7bf1ad422c8f490caeae8f56
Gerrit-Change-Number: 23834
Gerrit-PatchSet: 1
Gerrit-Owner: Ivan J. <parazyd(a)dyne.org>
Furquan Shaikh has uploaded a new patch set (#2). ( https://review.coreboot.org/23833 )
Change subject: soc/intel/skylake: Do not set ACPI_FADT_LEGACY_DEVICES
......................................................................
soc/intel/skylake: Do not set ACPI_FADT_LEGACY_DEVICES
SKL/KBL PCH does not support legacy devices. This change removes the
setting of ACPI_FADT_LEGACY_DEVICES flag in FADT for SKL/KBL.
It helps Linux kernel to disable controllers required to support legacy
devices only e.g. i8237 DMA controller.
BUG=b:72679357
Change-Id: Ie2a85a719997157f52b0eab7254689f5a56ba05b
Signed-off-by: Furquan Shaikh <furquan(a)chromium.org>
---
M src/soc/intel/skylake/acpi.c
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/33/23833/2
--
To view, visit https://review.coreboot.org/23833
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: Ie2a85a719997157f52b0eab7254689f5a56ba05b
Gerrit-Change-Number: 23833
Gerrit-PatchSet: 2
Gerrit-Owner: Furquan Shaikh <furquan(a)google.com>
Furquan Shaikh has uploaded this change for review. ( https://review.coreboot.org/23833
Change subject: soc/intel/skylake: Do not set ACPI_FADT_LEGACY_DEVICES
......................................................................
soc/intel/skylake: Do not set ACPI_FADT_LEGACY_DEVICES
SKL/KBL PCH does not support legacy devices. This change removes the
setting of ACPI_FADT_LEGACY_DEVICES flag in FADT for SKL/KBL.
BUG=b:72679357
Change-Id: Ie2a85a719997157f52b0eab7254689f5a56ba05b
Signed-off-by: Furquan Shaikh <furquan(a)chromium.org>
---
M src/soc/intel/skylake/acpi.c
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/33/23833/1
diff --git a/src/soc/intel/skylake/acpi.c b/src/soc/intel/skylake/acpi.c
index a0be5f5..6e1a886 100644
--- a/src/soc/intel/skylake/acpi.c
+++ b/src/soc/intel/skylake/acpi.c
@@ -267,7 +267,7 @@
fadt->day_alrm = 0xd;
fadt->mon_alrm = 0x00;
fadt->century = 0x00;
- fadt->iapc_boot_arch = ACPI_FADT_LEGACY_DEVICES;
+ fadt->iapc_boot_arch = 0;
if (!IS_ENABLED(CONFIG_NO_FADT_8042))
fadt->iapc_boot_arch |= ACPI_FADT_8042;
--
To view, visit https://review.coreboot.org/23833
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: Ie2a85a719997157f52b0eab7254689f5a56ba05b
Gerrit-Change-Number: 23833
Gerrit-PatchSet: 1
Gerrit-Owner: Furquan Shaikh <furquan(a)google.com>