Jakub Czapiga has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/70359 )
Change subject: tests/Makefile.inc: Build utilities in separate directory
......................................................................
tests/Makefile.inc: Build utilities in separate directory
Utilities like kconfig/conf now will be built inside tests build tree.
It will eliminate possible colisions of target names when using test
framework in more than one place (see CB:70110)
Signed-off-by: Jakub Czapiga <jacz(a)semihalf.com>
Change-Id: I4c1eb901c921f4ec6ee8985b154362153c5fd0e7
---
M tests/Makefile.inc
1 file changed, 16 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/59/70359/1
diff --git a/tests/Makefile.inc b/tests/Makefile.inc
index 04792c1..edeea8e 100644
--- a/tests/Makefile.inc
+++ b/tests/Makefile.inc
@@ -8,6 +8,8 @@
testobj := $(obj)/tests
endif
+objutil := $(testobj)/util
+
include $(top)/tests/Makefile.common
# Enable GDB debug build if requested
--
To view, visit https://review.coreboot.org/c/coreboot/+/70359
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I4c1eb901c921f4ec6ee8985b154362153c5fd0e7
Gerrit-Change-Number: 70359
Gerrit-PatchSet: 1
Gerrit-Owner: Jakub Czapiga <jacz(a)semihalf.com>
Gerrit-MessageType: newchange
Jakub Czapiga has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/70298 )
Change subject: tests/Makefile.common: Remove unnecesary dependency to kconfig/conf
......................................................................
tests/Makefile.common: Remove unnecesary dependency to kconfig/conf
Utility will be built while executing kconfig targets so it is not
necessary to keep hard dependency on kconfig here.
Signed-off-by: Jakub Czapiga <jacz(a)semihalf.com>
Change-Id: I578f3e1d9de63e91ded44746539265bcd55bf579
---
M tests/Makefile.common
1 file changed, 14 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/98/70298/1
diff --git a/tests/Makefile.common b/tests/Makefile.common
index cda7bf5..b568e08 100644
--- a/tests/Makefile.common
+++ b/tests/Makefile.common
@@ -182,7 +182,7 @@
KCONFIG_TRISTATE=$(TEST_KCONFIG_TRISTATE) \
KBUILD_DEFCONFIG=$(TEST_DEFAULT_CONFIG)
-$(TEST_KCONFIG_AUTOHEADER): $(TEST_DOTCONFIG) $(objutil)/kconfig/conf
+$(TEST_KCONFIG_AUTOHEADER): $(TEST_DOTCONFIG)
mkdir -p $(dir $@)
$(MAKE) $(TEST_KCONFIG_FLAGS) olddefconfig
$(MAKE) $(TEST_KCONFIG_FLAGS) syncconfig
--
To view, visit https://review.coreboot.org/c/coreboot/+/70298
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I578f3e1d9de63e91ded44746539265bcd55bf579
Gerrit-Change-Number: 70298
Gerrit-PatchSet: 1
Gerrit-Owner: Jakub Czapiga <jacz(a)semihalf.com>
Gerrit-MessageType: newchange
Attention is currently required from: Martin L Roth.
Hello build bot (Jenkins), Martin L Roth,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/70110
to look at the new patch set (#3).
Change subject: [WIP][DO NOT MERGE] ssdtgen: ACPI SSDT generator
......................................................................
[WIP][DO NOT MERGE] ssdtgen: ACPI SSDT generator
Work in progress.
TODO:
- Documentation
- Dynamic/config-based mocks
- Modular devicetree (multi-override in sconfig?)
- Cleanup
BUG=b:161391394
Signed-off-by: Jakub Czapiga <jacz(a)semihalf.com>
Change-Id: I78766cd1f47e24b2cd7f9b5cf1cca066441ead06
---
M Makefile
A ssdtgen/Makefile.inc
A ssdtgen/devicetree.cb
A ssdtgen/fw_config.c
A ssdtgen/mock/fsp/util.h
A ssdtgen/mock/gpio.c
A ssdtgen/mock/soc/gpio.h
A ssdtgen/mock/soc/nvs.h
A ssdtgen/ssdtgen.h
A ssdtgen/ssdtgen_main.c
A ssdtgen/ssdtgen_sys.c
11 files changed, 458 insertions(+), 4 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/10/70110/3
--
To view, visit https://review.coreboot.org/c/coreboot/+/70110
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I78766cd1f47e24b2cd7f9b5cf1cca066441ead06
Gerrit-Change-Number: 70110
Gerrit-PatchSet: 3
Gerrit-Owner: Jakub Czapiga <jacz(a)semihalf.com>
Gerrit-Reviewer: Martin L Roth <gaumless(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Jan Dabros <jsd(a)semihalf.com>
Gerrit-CC: Noga Smith <nogasmith(a)google.com>
Gerrit-Attention: Martin L Roth <gaumless(a)gmail.com>
Gerrit-MessageType: newpatchset
Attention is currently required from: Stefan Ott, Angel Pons, Arthur Heymans, Alexander Couzens.
Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/69298 )
Change subject: nb/intel/x4x: Remove apic 0 from devicetree
......................................................................
Patch Set 18: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/69298
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I7716f8a972e2280179aa6aee00488b22413c0c73
Gerrit-Change-Number: 69298
Gerrit-PatchSet: 18
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Alexander Couzens <lynxis(a)fe80.eu>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Gerrit-Reviewer: Stefan Ott <coreboot(a)desire.ch>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Stefan Ott <coreboot(a)desire.ch>
Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Attention: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Attention: Alexander Couzens <lynxis(a)fe80.eu>
Gerrit-Comment-Date: Mon, 05 Dec 2022 14:22:35 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment