Marc Jones has posted comments on this change. ( https://review.coreboot.org/28652 )
Change subject: mainboard/google/kahlee: Don't set global subsystem IDs
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit https://review.coreboot.org/28652
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: I3c56534b094ede8d8200b72f4433a891d0094064
Gerrit-Change-Number: 28652
Gerrit-PatchSet: 1
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: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Raul Rangel <rrangel(a)chromium.org>
Gerrit-Reviewer: Simon Glass <sjg(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Mon, 17 Sep 2018 21:15:09 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: Yes
Raul Rangel has posted comments on this change. ( https://review.coreboot.org/28652 )
Change subject: mainboard/google/kahlee: Don't set global subsystem IDs
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit https://review.coreboot.org/28652
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: I3c56534b094ede8d8200b72f4433a891d0094064
Gerrit-Change-Number: 28652
Gerrit-PatchSet: 1
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: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Raul Rangel <rrangel(a)chromium.org>
Gerrit-Reviewer: Simon Glass <sjg(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Mon, 17 Sep 2018 21:05:53 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: Yes
Raul Rangel has posted comments on this change. ( https://review.coreboot.org/28651 )
Change subject: mainboard/google/kahlee: allow oem.bin file to update smbios
......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/#/c/28651/1/src/mainboard/google/kahlee/variant…
File src/mainboard/google/kahlee/variants/baseboard/mainboard.c:
https://review.coreboot.org/#/c/28651/1/src/mainboard/google/kahlee/variant…
PS1, Line 93: const char *smbios_mainboard_manufacturer(void)
ah, it looks like nami already does this. Should we extract the struct and method into a common file?
https://review.coreboot.org/#/c/28651/1/src/mainboard/google/kahlee/variant…
PS1, Line 111:
It might be easier to read if you calculated the number of oem_entries and just did a for loop over the count.
--
To view, visit https://review.coreboot.org/28651
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: I997dc39c7f36f70cf4320ef335831245889eb475
Gerrit-Change-Number: 28651
Gerrit-PatchSet: 1
Gerrit-Owner: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Daniel Kurtz <djkurtz(a)google.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Raul Rangel <rrangel(a)chromium.org>
Gerrit-Reviewer: Simon Glass <sjg(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Mon, 17 Sep 2018 21:05:35 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No
Martin Roth has posted comments on this change. ( https://review.coreboot.org/28651 )
Change subject: mainboard/google/kahlee: allow oem.bin file to update smbios
......................................................................
Patch Set 1:
This just sets up coreboot so it can be used. The change for the .bin is in the internal private overlay repo, and the saved .config files will need to be updated for the variants that need this.
--
To view, visit https://review.coreboot.org/28651
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: I997dc39c7f36f70cf4320ef335831245889eb475
Gerrit-Change-Number: 28651
Gerrit-PatchSet: 1
Gerrit-Owner: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Daniel Kurtz <djkurtz(a)google.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Raul Rangel <rrangel(a)chromium.org>
Gerrit-Reviewer: Simon Glass <sjg(a)chromium.org>
Gerrit-CC: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Mon, 17 Sep 2018 20:52:33 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No
Martin Roth has uploaded this change for review. ( https://review.coreboot.org/28652
Change subject: mainboard/google/kahlee: Don't set global subsystem IDs
......................................................................
mainboard/google/kahlee: Don't set global subsystem IDs
These values override the default subsystem IDs, and aren't needed.
Change-Id: I3c56534b094ede8d8200b72f4433a891d0094064
Signed-off-by: Martin Roth <martinroth(a)chromium.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/grunt/devicetree.cb
M src/mainboard/google/kahlee/variants/liara/devicetree.cb
4 files changed, 0 insertions(+), 4 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/52/28652/1
diff --git a/src/mainboard/google/kahlee/variants/aleena/devicetree.cb b/src/mainboard/google/kahlee/variants/aleena/devicetree.cb
index 61476a2..e33ed2c 100644
--- a/src/mainboard/google/kahlee/variants/aleena/devicetree.cb
+++ b/src/mainboard/google/kahlee/variants/aleena/devicetree.cb
@@ -54,7 +54,6 @@
device lapic 10 on end
end
device domain 0 on
- subsystemid 0x1022 0x1410 inherit
device pci 0.0 on end # Root Complex
device pci 1.0 on end # Internal Graphics P2P bridge 0x98e4
device pci 1.1 on end # Internal Multimedia
diff --git a/src/mainboard/google/kahlee/variants/careena/devicetree.cb b/src/mainboard/google/kahlee/variants/careena/devicetree.cb
index 513bd4e..1677cdf 100644
--- a/src/mainboard/google/kahlee/variants/careena/devicetree.cb
+++ b/src/mainboard/google/kahlee/variants/careena/devicetree.cb
@@ -54,7 +54,6 @@
device lapic 10 on end
end
device domain 0 on
- subsystemid 0x1022 0x1410 inherit
device pci 0.0 on end # Root Complex
device pci 1.0 on end # Internal Graphics P2P bridge 0x98e4
device pci 1.1 on end # Internal Multimedia
diff --git a/src/mainboard/google/kahlee/variants/grunt/devicetree.cb b/src/mainboard/google/kahlee/variants/grunt/devicetree.cb
index b749a94..b140939 100644
--- a/src/mainboard/google/kahlee/variants/grunt/devicetree.cb
+++ b/src/mainboard/google/kahlee/variants/grunt/devicetree.cb
@@ -54,7 +54,6 @@
device lapic 10 on end
end
device domain 0 on
- subsystemid 0x1022 0x1410 inherit
device pci 0.0 on end # Root Complex
device pci 1.0 on end # Internal Graphics P2P bridge 0x98e4
device pci 1.1 on end # Internal Multimedia
diff --git a/src/mainboard/google/kahlee/variants/liara/devicetree.cb b/src/mainboard/google/kahlee/variants/liara/devicetree.cb
index 9b39b78..636f0c6 100644
--- a/src/mainboard/google/kahlee/variants/liara/devicetree.cb
+++ b/src/mainboard/google/kahlee/variants/liara/devicetree.cb
@@ -54,7 +54,6 @@
device lapic 10 on end
end
device domain 0 on
- subsystemid 0x1022 0x1410 inherit
device pci 0.0 on end # Root Complex
device pci 1.0 on end # Internal Graphics P2P bridge 0x98e4
device pci 1.1 on end # Internal Multimedia
--
To view, visit https://review.coreboot.org/28652
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: I3c56534b094ede8d8200b72f4433a891d0094064
Gerrit-Change-Number: 28652
Gerrit-PatchSet: 1
Gerrit-Owner: Martin Roth <martinroth(a)google.com>