Bora Guvendik has posted comments on this change. ( https://review.coreboot.org/29274 )
Change subject: soc/intel/cannonlake: Enable ISH from device
......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/#/c/29274/1/src/soc/intel/cannonlake/romstage/f…
File src/soc/intel/cannonlake/romstage/fsp_params.c:
https://review.coreboot.org/#/c/29274/1/src/soc/intel/cannonlake/romstage/f…
PS1, Line 59: const struct device *dev = dev_find_slot(0, PCH_DEVFN_ISH);
move this to the beginning of the function
--
To view, visit https://review.coreboot.org/29274
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: I6889634bf65e7ce5cc3e3393c57c86d622f1ac68
Gerrit-Change-Number: 29274
Gerrit-PatchSet: 1
Gerrit-Owner: Lijian Zhao <lijian.zhao(a)intel.com>
Gerrit-Reviewer: Bora Guvendik <bora.guvendik(a)intel.com>
Gerrit-Reviewer: Hannah Williams <hannah.williams(a)intel.com>
Gerrit-Reviewer: Krzysztof M Sywula <krzysztof.m.sywula(a)intel.com>
Gerrit-Reviewer: Kyoung Il Kim <kyoung.il.kim(a)intel.com>
Gerrit-Reviewer: Li1 Feng <li1.feng(a)intel.com>
Gerrit-Reviewer: Lijian Zhao <lijian.zhao(a)intel.com>
Gerrit-Reviewer: Pratikkumar V Prajapati <pratikkumar.v.prajapati(a)intel.com>
Gerrit-Reviewer: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Tue, 30 Oct 2018 20:40:47 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No
Richard Spiegel has posted comments on this change. ( https://review.coreboot.org/29350 )
Change subject: soc/amd/stoneyridge: Create MMIO ACPI access functions
......................................................................
Patch Set 1:
Yes... my mistake.
--
To view, visit https://review.coreboot.org/29350
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: I8dab279019fe9ca55522ab3e65f51b9f459529ee
Gerrit-Change-Number: 29350
Gerrit-PatchSet: 1
Gerrit-Owner: Richard Spiegel <richard.spiegel(a)silverbackltd.com>
Gerrit-Reviewer: Richard Spiegel <richard.spiegel(a)silverbackltd.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Gerrit-Comment-Date: Tue, 30 Oct 2018 20:27:11 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No
Martin Roth has submitted this change and it was merged. ( https://review.coreboot.org/29342 )
Change subject: mb/google/kahlee: Disable IOMMU
......................................................................
mb/google/kahlee: Disable IOMMU
Unfortunately Stoney has an issue where enabling the IOMMU causes
a 10%-50% decrease in the integrated graphics performance. It is
also disabled by default on other stoney platforms.
BUG=b:118612241
TEST=Verify that IOMMU is disabled.
Change-Id: Ia396c7227cb21461ec8afbdf746721d4fb28083d
Signed-off-by: Martin Roth <martinroth(a)google.com>
Reviewed-on: https://review.coreboot.org/29342
Reviewed-by: Raul Rangel <rrangel(a)chromium.org>
Reviewed-by: Daniel Kurtz <djkurtz(a)google.com>
Tested-by: build bot (Jenkins) <no-reply(a)coreboot.org>
---
M src/mainboard/google/kahlee/variants/aleena/devicetree.cb
M src/mainboard/google/kahlee/variants/careena/devicetree.cb
M src/mainboard/google/kahlee/variants/delan/devicetree.cb
M src/mainboard/google/kahlee/variants/grunt/devicetree.cb
M src/mainboard/google/kahlee/variants/liara/devicetree.cb
5 files changed, 5 insertions(+), 5 deletions(-)
Approvals:
build bot (Jenkins): Verified
Daniel Kurtz: Looks good to me, approved
Raul Rangel: Looks good to me, approved
diff --git a/src/mainboard/google/kahlee/variants/aleena/devicetree.cb b/src/mainboard/google/kahlee/variants/aleena/devicetree.cb
index 5438e6d..27a8e28 100644
--- a/src/mainboard/google/kahlee/variants/aleena/devicetree.cb
+++ b/src/mainboard/google/kahlee/variants/aleena/devicetree.cb
@@ -58,7 +58,7 @@
end
device domain 0 on
device pci 0.0 on end # Root Complex
- device pci 0.2 on end # IOMMU
+ device pci 0.2 off end # IOMMU (Disabled for performance and battery)
device pci 1.0 on end # Internal Graphics P2P bridge 0x98e4
device pci 1.1 on end # Internal Multimedia
device pci 2.0 on end # PCIe Host Bridge
diff --git a/src/mainboard/google/kahlee/variants/careena/devicetree.cb b/src/mainboard/google/kahlee/variants/careena/devicetree.cb
index b541c03..23253b2 100644
--- a/src/mainboard/google/kahlee/variants/careena/devicetree.cb
+++ b/src/mainboard/google/kahlee/variants/careena/devicetree.cb
@@ -61,7 +61,7 @@
end
device domain 0 on
device pci 0.0 on end # Root Complex
- device pci 0.2 on end # IOMMU
+ device pci 0.2 off end # IOMMU (Disabled for performance and battery)
device pci 1.0 on end # Internal Graphics P2P bridge 0x98e4
device pci 1.1 on end # Internal Multimedia
device pci 2.0 on end # PCIe Host Bridge
diff --git a/src/mainboard/google/kahlee/variants/delan/devicetree.cb b/src/mainboard/google/kahlee/variants/delan/devicetree.cb
index 0dfe204..fd30d5d 100644
--- a/src/mainboard/google/kahlee/variants/delan/devicetree.cb
+++ b/src/mainboard/google/kahlee/variants/delan/devicetree.cb
@@ -55,7 +55,7 @@
end
device domain 0 on
device pci 0.0 on end # Root Complex
- device pci 0.2 on end # IOMMU
+ device pci 0.2 off end # IOMMU (Disabled for performance and battery)
device pci 1.0 on end # Internal Graphics P2P bridge 0x98e4
device pci 1.1 on end # Internal Multimedia
device pci 2.0 on end # PCIe Host Bridge
diff --git a/src/mainboard/google/kahlee/variants/grunt/devicetree.cb b/src/mainboard/google/kahlee/variants/grunt/devicetree.cb
index 5a8906b..b37e1bf 100644
--- a/src/mainboard/google/kahlee/variants/grunt/devicetree.cb
+++ b/src/mainboard/google/kahlee/variants/grunt/devicetree.cb
@@ -61,7 +61,7 @@
end
device domain 0 on
device pci 0.0 on end # Root Complex
- device pci 0.2 on end # IOMMU
+ device pci 0.2 off end # IOMMU (Disabled for performance and battery)
device pci 1.0 on end # Internal Graphics P2P bridge 0x98e4
device pci 1.1 on end # Internal Multimedia
device pci 2.0 on end # PCIe Host Bridge
diff --git a/src/mainboard/google/kahlee/variants/liara/devicetree.cb b/src/mainboard/google/kahlee/variants/liara/devicetree.cb
index cea3425..343fbeb 100644
--- a/src/mainboard/google/kahlee/variants/liara/devicetree.cb
+++ b/src/mainboard/google/kahlee/variants/liara/devicetree.cb
@@ -58,7 +58,7 @@
end
device domain 0 on
device pci 0.0 on end # Root Complex
- device pci 0.2 on end # IOMMU
+ device pci 0.2 off end # IOMMU
device pci 1.0 on end # Internal Graphics P2P bridge 0x98e4
device pci 1.1 on end # Internal Multimedia
device pci 2.0 on end # PCIe Host Bridge
--
To view, visit https://review.coreboot.org/29342
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia396c7227cb21461ec8afbdf746721d4fb28083d
Gerrit-Change-Number: 29342
Gerrit-PatchSet: 3
Gerrit-Owner: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Daniel Kurtz <djkurtz(a)google.com>
Gerrit-Reviewer: Marc Jones <marc(a)marcjonesconsulting.com>
Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Raul Rangel <rrangel(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Nick Vaccaro has posted comments on this change. ( https://review.coreboot.org/29361 )
Change subject: mb/google/poppy/variant/nocturne: adjust RcompTarget to fix DRAM corruption
......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/#/c/29361/1/src/mainboard/google/poppy/variants…
File src/mainboard/google/poppy/variants/nocturne/memory.c:
https://review.coreboot.org/#/c/29361/1/src/mainboard/google/poppy/variants…
PS1, Line 36: 30
> Does this value work for both KBL and AML? Was this change validated on all the different Nocturne S […]
That is a question for Intel. I posted this CL per their request in comment #134 on b:111812662.
I personally haven't performed any verification on this change.
--
To view, visit https://review.coreboot.org/29361
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: Iefc3957f915a39a47ad6018459e65b70d1b34091
Gerrit-Change-Number: 29361
Gerrit-PatchSet: 1
Gerrit-Owner: Nick Vaccaro <nvaccaro(a)google.com>
Gerrit-Reviewer: Caveh Jalali <caveh(a)google.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Nick Vaccaro <nvaccaro(a)google.com>
Gerrit-Reviewer: Shaunak Saha <shaunak.saha(a)intel.corp-partner.google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Tue, 30 Oct 2018 19:30:34 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No
Daniel Kurtz has posted comments on this change. ( https://review.coreboot.org/29343 )
Change subject: soc/amd/stoneyridge: Set IOMMU support to follow device setting
......................................................................
Patch Set 2: Code-Review+2
--
To view, visit https://review.coreboot.org/29343
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: I6cfd6c81f47de23c54a49ec7cf87b219215ced5e
Gerrit-Change-Number: 29343
Gerrit-PatchSet: 2
Gerrit-Owner: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Daniel Kurtz <djkurtz(a)google.com>
Gerrit-Reviewer: Marc Jones <marc(a)marcjonesconsulting.com>
Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Gerrit-Reviewer: Raul Rangel <rrangel(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Tue, 30 Oct 2018 19:18:15 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: Yes