coreboot-gerrit April 2022

coreboot-gerrit@coreboot.org

1 participants
5879 discussions

Change in coreboot[master]: util/cbfstool/linux_trampoline: Use payload calling argument
by Paul Menzel (Code Review) 19 Apr '22

19 Apr '22

Attention is currently required from: Patrick Georgi, Arthur Heymans. Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63611 ) Change subject: util/cbfstool/linux_trampoline: Use payload calling argument ...................................................................... Patch Set 2: Code-Review+1 -- To view, visit https://review.coreboot.org/c/coreboot/+/63611 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ie0e000470299bddde3cae9347a1b18cc5b0cfc77 Gerrit-Change-Number: 63611 Gerrit-PatchSet: 2 Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Patrick Georgi <patrick(a)coreboot.org> Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Attention: Patrick Georgi <patrick(a)coreboot.org> Gerrit-Attention: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Comment-Date: Tue, 19 Apr 2022 13:01:28 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

Change in coreboot[master]: tests: Split Makefile to allow for making host-side test tools
by Felix Held (Code Review) 19 Apr '22

19 Apr '22

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/63637 ) Change subject: tests: Split Makefile to allow for making host-side test tools ...................................................................... tests: Split Makefile to allow for making host-side test tools This patch is based on similar changes [1] done in Depthcharge projects, which aimed to provide unified way to build host-side programs for testing internal code. New test tools might benefit from it by having same base code as unit-tests. [1] https://crrev.com/c/3412108 TEST=make unit-tests TEST=COV=1 make unit-tests coverage-report Signed-off-by: Jakub Czapiga <jacz(a)semihalf.com> Change-Id: Iac4517ab6146fa3f2d2b7a20df54601ab2d04c3d Reviewed-on: https://review.coreboot.org/c/coreboot/+/63637 Tested-by: build bot (Jenkins) <no-reply(a)coreboot.org> Reviewed-by: Arthur Heymans <arthur(a)aheymans.xyz> --- A tests/Makefile.common M tests/Makefile.inc 2 files changed, 170 insertions(+), 161 deletions(-) Approvals: build bot (Jenkins): Verified Arthur Heymans: Looks good to me, approved diff --git a/tests/Makefile.common b/tests/Makefile.common new file mode 100644 index 0000000..03ae7c4 --- /dev/null +++ b/tests/Makefile.common @@ -0,0 +1,169 @@ +# SPDX-License-Identifier: GPL-2.0-only + +# This file contains common definitions, macros and targets for depthcharge +# unit-tests and screenshot utility. It requires list of defined variables: +# - src - source directory +# - testobj - build directory of tests +# - objutil - utility programs/libraries output directory + +testsrc := $(top)/tests + +cmockasrc := 3rdparty/cmocka +cmockaobj := $(objutil)/cmocka +coverage_dir := coverage_reports + +CMOCKA_LIB := $(cmockaobj)/src/libcmocka.so + +CMAKE := cmake +OBJCOPY ?= objcopy +OBJDUMP ?= objdump + +TEST_DEFAULT_CONFIG := $(top)/configs/config.emulation_qemu_x86_i440fx +TEST_DOTCONFIG := $(testobj)/.config +TEST_KCONFIG_AUTOHEADER := $(testobj)/config.src.h +TEST_KCONFIG_AUTOCONFIG := $(testobj)/auto.conf +TEST_KCONFIG_DEPENDENCIES := $(testobj)/auto.conf.cmd +TEST_KCONFIG_SPLITCONFIG := $(testobj)/config/ +TEST_KCONFIG_TRISTATE := $(testobj)/tristate.conf + +TEST_CFLAGS := -include $(src)/include/kconfig.h \ + -include $(src)/commonlib/bsd/include/commonlib/bsd/compiler.h \ + -include $(src)/include/rules.h + +# Include generic test mock headers, before original ones +TEST_CFLAGS += -I$(testsrc)/include/mocks -I$(testsrc)/include + +TEST_CFLAGS += -I$(src) -I$(src)/include -I$(src)/commonlib/include \ + -I$(src)/commonlib/bsd/include -I$(src)/arch/x86/include \ + -I$(top)/3rdparty/vboot/firmware/include + +# Note: This is intentionally just a subset of the warnings in the toplevel +# Makefile.inc. We don't need to be as strict with test code, and things like +# -Wmissing-prototypes just make working with the test framework cumbersome. +# Only put conservative warnings here that really detect code that's obviously +# unintentional. +TEST_CFLAGS += -Wall -Werror -Wundef -Wstrict-prototypes -Wno-inline-asm +TEST_CFLAGS += -Wno-unknown-warning-option -Wno-source-mgr -Wno-main-return-type + +# Path for Kconfig autoheader +TEST_CFLAGS += -I$(dir $(TEST_KCONFIG_AUTOHEADER)) + +TEST_CFLAGS += -std=gnu11 -Os -ffunction-sections -fdata-sections -fno-builtin + +TEST_CFLAGS += -D__TEST__ + +TEST_CFLAGS += -I$(cmockasrc)/include + +ifneq ($(filter-out 0,$(TEST_PRINT)),) +TEST_CFLAGS += -DTEST_PRINT=1 +endif + +# Link against Cmocka +TEST_LDFLAGS := -L$(cmockaobj)/src -lcmocka -Wl,-rpath=$(cmockaobj)/src +TEST_LDFLAGS += -Wl,--gc-sections + +# Some memlayout symbols don't work with userspace relocation -- disable it. +TEST_CFLAGS += -fno-pie -fno-pic +TEST_LDFLAGS += -no-pie + +# Extra attributes for unit tests, declared per test +attributes := srcs cflags config mocks stage + +# Copy attributes of one test to another. +# $1 - input test name +# $2 - output test name +copy-test = $(foreach attr,$(attributes), \ + $(eval $(strip $(2))-$(attr) := $($(strip $(1))-$(attr)))) + +# Create actual targets for unit test binaries +# $1 - test name +define TEST_CC_template + +# Generate custom config.h redefining given config symbols, and declaring mocked +# functions weak. It is important that the compiler already sees that they are +# weak (and they aren't just turned weak at a later stage) to prevent certain +# optimizations that would break if the function gets replaced. (For clang this +# file needs to be marked `system_header` to prevent it from warning about +# #pragma weak entries without a matching function declaration, since there's no +# -Wno-xxx command line option for that.) +$(1)-config-file := $(testobj)/$(1)/config.h +$$($(1)-config-file): $(TEST_KCONFIG_AUTOHEADER) + mkdir -p $$(dir $$@) + printf '// File generated by tests/Makefile.inc\n// Do not change\n' > $$@ + printf '#include <%s>\n\n' "$(notdir $(TEST_KCONFIG_AUTOHEADER))" >> $$@ + for kv in $$($(1)-config); do \ + key="`echo $$$$kv | cut -d '=' -f -1`"; \ + value="`echo $$$$kv | cut -d '=' -f 2-`"; \ + printf '#undef %s\n' "$$$$key" >> $$@; \ + printf '#define %s %s\n\n' "$$$$key" "$$$$value" >> $$@; \ + done + printf '#ifdef __clang__\n' >> $$@; + printf '#pragma clang system_header\n' >> $$@; + printf '#endif\n' >> $$@; + printf '#ifdef __TEST_SRCOBJ__\n' >> $$@; + for m in $$($(1)-mocks); do \ + printf '#pragma weak %s\n' "$$$$m" >> $$@; \ + done + printf '#endif\n' >> $$@; + +$($(1)-objs): TEST_CFLAGS += -I$$(dir $$($(1)-config-file)) \ + -D__$$(shell echo $$($(1)-stage) | tr '[:lower:]' '[:upper:]')__ \ + -D__TEST_NAME__=\"$(subst /,_,$(1))\" \ + -D__TEST_DATA_DIR__=\"$(testsrc)/data\" + +# Give us a way to distinguish between coreboot source files and test files in code. +$($(1)-srcobjs): TEST_CFLAGS += -D__TEST_SRCOBJ__ + +# Compile sources and apply mocking/wrapping of selected symbols. +# For each listed mock add new symbol with prefix `__real_`, +# and pointing to the same section:address. +$($(1)-objs): $(testobj)/$(1)/%.o: $$$$*.c $$($(1)-config-file) + mkdir -p $$(dir $$@) + $(HOSTCC) $(HOSTCFLAGS) $$(TEST_CFLAGS) $($(1)-cflags) -MMD \ + -MF $$(basename $$(a)).d -MT $$@ -c $$< -o $$@.orig + objcopy_wrap_flags=''; \ + for sym in $$($(1)-mocks); do \ + sym_line="$$$$($(OBJDUMP) -t $$@.orig \ + | grep -E "[0-9a-fA-F]+\\s+w\\s+F\\s+.*\\s$$$$sym$$$$")"; \ + if [ ! -z "$$$$sym_line" ] ; then \ + addr="$$$$(echo "$$$$sym_line" | awk '{ print $$$$1 }')"; \ + section="$$$$(echo "$$$$sym_line" | awk '{ print $$$$(NF - 2) }')"; \ + objcopy_wrap_flags="$$$$objcopy_wrap_flags --add-symbol __real_$$$${sym}=$$$${section}:0x$$$${addr},function,global"; \ + fi \ + done ; \ + $(OBJCOPY) $$@.orig $$$$objcopy_wrap_flags $$@ + +$($(1)-bin): $($(1)-objs) $(CMOCKA_LIB) + $(HOSTCC) $$^ $($(1)-cflags) $$(TEST_LDFLAGS) -o $$@ + +endef + +# Build cmocka +$(CMOCKA_LIB): + echo "*** Building CMOCKA ***" + mkdir -p $(cmockaobj) + cd $(cmockaobj) && $(CMAKE) $(abspath $(cmockasrc)) + $(MAKE) -C $(cmockaobj) + +# Kconfig targets +$(TEST_DOTCONFIG): + mkdir -p $(dir $@) + cp $(TEST_DEFAULT_CONFIG) $(TEST_DOTCONFIG) + +# Don't override default Kconfig variables, since this will affect all +# Kconfig targets. Change them only when calling sub-make instead. +$(TEST_KCONFIG_AUTOHEADER): TEST_KCONFIG_FLAGS := DOTCONFIG=$(TEST_DOTCONFIG) \ + KCONFIG_AUTOHEADER=$(TEST_KCONFIG_AUTOHEADER) \ + KCONFIG_AUTOCONFIG=$(TEST_KCONFIG_AUTOCONFIG) \ + KCONFIG_DEPENDENCIES=$(TEST_KCONFIG_DEPENDENCIES) \ + KCONFIG_SPLITCONFIG=$(TEST_KCONFIG_SPLITCONFIG) \ + KCONFIG_TRISTATE=$(TEST_KCONFIG_TRISTATE) \ + KBUILD_DEFCONFIG=$(TEST_DEFAULT_CONFIG) + +$(TEST_KCONFIG_AUTOHEADER): $(TEST_DOTCONFIG) $(objutil)/kconfig/conf + mkdir -p $(dir $@) + $(MAKE) $(TEST_KCONFIG_FLAGS) olddefconfig + $(MAKE) $(TEST_KCONFIG_FLAGS) syncconfig + +$(TEST_KCONFIG_AUTOCONFIG): $(TEST_KCONFIG_AUTOHEADER) + true diff --git a/tests/Makefile.inc b/tests/Makefile.inc index 847beaf..10caf0a 100644 --- a/tests/Makefile.inc +++ b/tests/Makefile.inc @@ -1,7 +1,5 @@ # SPDX-License-Identifier: GPL-2.0-only -testsrc := $(top)/tests - # Place the build output in one of two places depending on COV, so that code # built with code coverage never mixes with code built without code coverage. ifeq ($(COV),1) @@ -10,63 +8,7 @@ testobj := $(obj)/tests endif -cmockasrc := 3rdparty/cmocka -cmockaobj := $(objutil)/cmocka -coverage_dir := coverage_reports - -CMOCKA_LIB := $(cmockaobj)/src/libcmocka.so - -CMAKE := cmake -OBJCOPY ?= objcopy -OBJDUMP ?= objdump - -TEST_DEFAULT_CONFIG := $(top)/configs/config.emulation_qemu_x86_i440fx -TEST_DOTCONFIG := $(testobj)/.config -TEST_KCONFIG_AUTOHEADER := $(testobj)/config.src.h -TEST_KCONFIG_AUTOCONFIG := $(testobj)/auto.conf -TEST_KCONFIG_DEPENDENCIES := $(testobj)/auto.conf.cmd -TEST_KCONFIG_SPLITCONFIG := $(testobj)/config/ -TEST_KCONFIG_TRISTATE := $(testobj)/tristate.conf - -TEST_CFLAGS := -include $(src)/include/kconfig.h \ - -include $(src)/commonlib/bsd/include/commonlib/bsd/compiler.h \ - -include $(src)/include/rules.h - -# Include generic test mock headers, before original ones -TEST_CFLAGS += -I$(testsrc)/include/mocks -I$(testsrc)/include - -TEST_CFLAGS += -I$(src) -I$(src)/include -I$(src)/commonlib/include \ - -I$(src)/commonlib/bsd/include -I$(src)/arch/x86/include \ - -I$(top)/3rdparty/vboot/firmware/include - -# Note: This is intentionally just a subset of the warnings in the toplevel -# Makefile.inc. We don't need to be as strict with test code, and things like -# -Wmissing-prototypes just make working with the test framework cumbersome. -# Only put conservative warnings here that really detect code that's obviously -# unintentional. -TEST_CFLAGS += -Wall -Werror -Wundef -Wstrict-prototypes -Wno-inline-asm -TEST_CFLAGS += -Wno-unknown-warning-option -Wno-source-mgr -Wno-main-return-type - -# Path for Kconfig autoheader -TEST_CFLAGS += -I$(dir $(TEST_KCONFIG_AUTOHEADER)) - -TEST_CFLAGS += -std=gnu11 -Os -ffunction-sections -fdata-sections -fno-builtin - -TEST_CFLAGS += -D__TEST__ - -TEST_CFLAGS += -I$(cmockasrc)/include - -ifneq ($(filter-out 0,$(TEST_PRINT)),) -TEST_CFLAGS += -DTEST_PRINT=1 -endif - -# Link against Cmocka -TEST_LDFLAGS := -L$(cmockaobj)/src -lcmocka -Wl,-rpath=$(cmockaobj)/src -TEST_LDFLAGS += -Wl,--gc-sections - -# Some memlayout symbols don't work with userspace relocation -- disable it. -TEST_CFLAGS += -fno-pie -fno-pic -TEST_LDFLAGS += -no-pie +include $(top)/tests/Makefile.common # Enable code coverage if COV=1 ifeq ($(COV),1) @@ -74,9 +16,6 @@ TEST_LDFLAGS += --coverage endif -# Extra attributes for unit tests, declared per test -attributes := srcs cflags config mocks stage - stages := decompressor bootblock romstage smm verstage stages += ramstage rmodule postcar libagesa @@ -100,78 +39,9 @@ Check your $(dir $(1)$(2))Makefile.inc)) endef -# Copy attributes of one test to another. -# $1 - input test name -# $2 - output test name -copy-test = $(foreach attr,$(attributes), \ - $(eval $(strip $(2))-$(attr) := $($(strip $(1))-$(attr)))) - $(call add-special-class, tests) $(call evaluate_subdirs) -# Create actual targets for unit test binaries -# $1 - test name -define TEST_CC_template - -# Generate custom config.h redefining given config symbols, and declaring mocked -# functions weak. It is important that the compiler already sees that they are -# weak (and they aren't just turned weak at a later stage) to prevent certain -# optimizations that would break if the function gets replaced. (For clang this -# file needs to be marked `system_header` to prevent it from warning about -# #pragma weak entries without a matching function declaration, since there's no -# -Wno-xxx command line option for that.) -$(1)-config-file := $(testobj)/$(1)/config.h -$$($(1)-config-file): $(TEST_KCONFIG_AUTOHEADER) - mkdir -p $$(dir $$@) - printf '// File generated by tests/Makefile.inc\n// Do not change\n' > $$@ - printf '#include <%s>\n\n' "$(notdir $(TEST_KCONFIG_AUTOHEADER))" >> $$@ - for kv in $$($(1)-config); do \ - key="`echo $$$$kv | cut -d '=' -f -1`"; \ - value="`echo $$$$kv | cut -d '=' -f 2-`"; \ - printf '#undef %s\n' "$$$$key" >> $$@; \ - printf '#define %s %s\n\n' "$$$$key" "$$$$value" >> $$@; \ - done - printf '#ifdef __clang__\n' >> $$@; - printf '#pragma clang system_header\n' >> $$@; - printf '#endif\n' >> $$@; - printf '#ifdef __TEST_SRCOBJ__\n' >> $$@; - for m in $$($(1)-mocks); do \ - printf '#pragma weak %s\n' "$$$$m" >> $$@; \ - done - printf '#endif\n' >> $$@; - -$($(1)-objs): TEST_CFLAGS += -I$$(dir $$($(1)-config-file)) \ - -D__$$(shell echo $$($(1)-stage) | tr '[:lower:]' '[:upper:]')__ \ - -D__TEST_NAME__=\"$(subst /,_,$(1))\" \ - -D__TEST_DATA_DIR__=\"$(testsrc)/data\" - -# Give us a way to distinguish between coreboot source files and test files in code. -$($(1)-srcobjs): TEST_CFLAGS += -D__TEST_SRCOBJ__ - -# Compile sources and apply mocking/wrapping of selected symbols. -# For each listed mock add new symbol with prefix `__real_`, -# and pointing to the same section:address. -$($(1)-objs): $(testobj)/$(1)/%.o: $$$$*.c $$($(1)-config-file) - mkdir -p $$(dir $$@) - $(HOSTCC) $(HOSTCFLAGS) $$(TEST_CFLAGS) $($(1)-cflags) -MMD \ - -MF $$(basename $$(a)).d -MT $$@ -c $$< -o $$@.orig - objcopy_wrap_flags=''; \ - for sym in $$($(1)-mocks); do \ - sym_line="$$$$($(OBJDUMP) -t $$@.orig \ - | grep -E "[0-9a-fA-F]+\\s+w\\s+F\\s+.*\\s$$$$sym$$$$")"; \ - if [ ! -z "$$$$sym_line" ] ; then \ - addr="$$$$(echo "$$$$sym_line" | awk '{ print $$$$1 }')"; \ - section="$$$$(echo "$$$$sym_line" | awk '{ print $$$$(NF - 2) }')"; \ - objcopy_wrap_flags="$$$$objcopy_wrap_flags --add-symbol __real_$$$${sym}=$$$${section}:0x$$$${addr},function,global"; \ - fi \ - done ; \ - $(OBJCOPY) $$@.orig $$$$objcopy_wrap_flags $$@ - -$($(1)-bin): $($(1)-objs) $(CMOCKA_LIB) - $(HOSTCC) $$^ $($(1)-cflags) $$(TEST_LDFLAGS) -o $$@ - -endef - $(foreach test, $(alltests), \ $(eval $(test)-srcobjs := $(addprefix $(testobj)/$(test)/, \ $(patsubst %.c,%.o,$(filter src/%,$($(test)-srcs))))) \ @@ -190,36 +60,6 @@ DEPENDENCIES += $(addsuffix .d,$(basename $(all-test-objs))) -include $(DEPENDENCIES) -# Build cmocka -$(CMOCKA_LIB): - echo "*** Building CMOCKA ***" - mkdir -p $(cmockaobj) - cd $(cmockaobj) && $(CMAKE) $(abspath $(cmockasrc)) - $(MAKE) -C $(cmockaobj) - -# Kconfig targets -$(TEST_DOTCONFIG): - mkdir -p $(dir $@) - cp $(TEST_DEFAULT_CONFIG) $(TEST_DOTCONFIG) - -# Don't override default Kconfig variables, since this will affect all -# Kconfig targets. Change them only when calling sub-make instead. -$(TEST_KCONFIG_AUTOHEADER): TEST_KCONFIG_FLAGS := DOTCONFIG=$(TEST_DOTCONFIG) \ - KCONFIG_AUTOHEADER=$(TEST_KCONFIG_AUTOHEADER) \ - KCONFIG_AUTOCONFIG=$(TEST_KCONFIG_AUTOCONFIG) \ - KCONFIG_DEPENDENCIES=$(TEST_KCONFIG_DEPENDENCIES) \ - KCONFIG_SPLITCONFIG=$(TEST_KCONFIG_SPLITCONFIG) \ - KCONFIG_TRISTATE=$(TEST_KCONFIG_TRISTATE) \ - KBUILD_DEFCONFIG=$(TEST_DEFAULT_CONFIG) - -$(TEST_KCONFIG_AUTOHEADER): $(TEST_DOTCONFIG) $(objutil)/kconfig/conf - mkdir -p $(dir $@) - $(MAKE) $(TEST_KCONFIG_FLAGS) olddefconfig - $(MAKE) $(TEST_KCONFIG_FLAGS) syncconfig - -$(TEST_KCONFIG_AUTOCONFIG): $(TEST_KCONFIG_AUTOHEADER) - true - .PHONY: $(alltests) $(addprefix clean-,$(alltests)) .PHONY: unit-tests build-unit-tests run-unit-tests clean-unit-tests -- To view, visit https://review.coreboot.org/c/coreboot/+/63637 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Iac4517ab6146fa3f2d2b7a20df54601ab2d04c3d Gerrit-Change-Number: 63637 Gerrit-PatchSet: 3 Gerrit-Owner: Jakub Czapiga <jacz(a)semihalf.com> Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de> Gerrit-Reviewer: Martin Roth <martinroth(a)google.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Jan Dabros <jsd(a)semihalf.com> Gerrit-MessageType: merged

1 0

Change in coreboot[master]: apple/macbook21: configure the clockgen and add C3 CPU state
by Paul Menzel (Code Review) 19 Apr '22

19 Apr '22

Attention is currently required from: Ferass EL HAFIDI, Arthur Heymans, Evgeny Zinoviev, Felix Held. Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63587 ) Change subject: apple/macbook21: configure the clockgen and add C3 CPU state ...................................................................... Patch Set 6: (3 comments) Commit Message: https://review.coreboot.org/c/coreboot/+/63587/comment/7a8f55c1_d9f8e412 PS6, Line 11: Please extend the commit message, describing why the overridetree is needed for example, and which models have what clockgen, and so on. File src/mainboard/apple/macbook21/cstates.c: https://review.coreboot.org/c/coreboot/+/63587/comment/ec12b9db_fc463a5b PS4, Line 34: .latency = 17, > That latency value works perfectly fine. […] Only because you do not experience any problems with your use cases, this does not have to be the correct value, does it? File src/mainboard/apple/macbook21/cstates.c: https://review.coreboot.org/c/coreboot/+/63587/comment/570d484b_709275de PS6, Line 32: #if !CONFIG(BOARD_APPLE_IMAC52) Preprocessor statement are not indented to my knowledge. -- To view, visit https://review.coreboot.org/c/coreboot/+/63587 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ib97f95599fbdf66aa55e936bdcb9cc0cd69b3824 Gerrit-Change-Number: 63587 Gerrit-PatchSet: 6 Gerrit-Owner: Ferass EL HAFIDI <vitali64pmemail(a)protonmail.com> Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Evgeny Zinoviev <me(a)ch1p.io> Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de> Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Attention: Ferass EL HAFIDI <vitali64pmemail(a)protonmail.com> Gerrit-Attention: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Attention: Evgeny Zinoviev <me(a)ch1p.io> Gerrit-Attention: Felix Held <felix-coreboot(a)felixheld.de> Gerrit-Comment-Date: Tue, 19 Apr 2022 13:00:43 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Ferass EL HAFIDI <vitali64pmemail(a)protonmail.com> Comment-In-Reply-To: Felix Held <felix-coreboot(a)felixheld.de> Gerrit-MessageType: comment

1 0

Change in coreboot[master]: tests: update CMocka to stable-1.1
by Felix Held (Code Review) 19 Apr '22

19 Apr '22

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/63636 ) Change subject: tests: update CMocka to stable-1.1 ...................................................................... tests: update CMocka to stable-1.1 CMocka stable-1.1 has some convenience bugfixes like vprint buffer increase or leftover values log fix (funtion names display correctly now. Signed-off-by: Jakub Czapiga <jacz(a)semihalf.com> Change-Id: I20ebd15324a21c17cccd2976ae9c3f86b040426d Reviewed-on: https://review.coreboot.org/c/coreboot/+/63636 Tested-by: build bot (Jenkins) <no-reply(a)coreboot.org> Reviewed-by: Arthur Heymans <arthur(a)aheymans.xyz> --- M .gitmodules M 3rdparty/cmocka M tests/lib/b64_decode-test.c 3 files changed, 3 insertions(+), 2 deletions(-) Approvals: build bot (Jenkins): Verified Arthur Heymans: Looks good to me, approved diff --git a/.gitmodules b/.gitmodules index 68230b9..69aa470 100644 --- a/.gitmodules +++ b/.gitmodules @@ -48,6 +48,7 @@ path = 3rdparty/cmocka url = ../cmocka.git update = none + branch = stable-1.1 [submodule "3rdparty/qc_blobs"] path = 3rdparty/qc_blobs url = ../qc_blobs.git diff --git a/3rdparty/cmocka b/3rdparty/cmocka index 672c5ce..8931845 160000 --- a/3rdparty/cmocka +++ b/3rdparty/cmocka @@ -1 +1 @@ -Subproject commit 672c5cee79eb412025c3dd8b034e611c1f119055 +Subproject commit 8931845c35e78b5123d73430b071affd537d5935 diff --git a/tests/lib/b64_decode-test.c b/tests/lib/b64_decode-test.c index 631dc15..189ce96 100644 --- a/tests/lib/b64_decode-test.c +++ b/tests/lib/b64_decode-test.c @@ -35,7 +35,7 @@ res = b64_decode((uint8_t *)messages[i].enc, strlen(messages[i].enc), decoded); assert_int_equal(res, (strlen(messages[i].dec))); - assert_string_equal(decoded, messages[i].dec); + assert_string_equal((const char *)decoded, messages[i].dec); free(decoded); } -- To view, visit https://review.coreboot.org/c/coreboot/+/63636 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I20ebd15324a21c17cccd2976ae9c3f86b040426d Gerrit-Change-Number: 63636 Gerrit-PatchSet: 4 Gerrit-Owner: Jakub Czapiga <jacz(a)semihalf.com> Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de> Gerrit-Reviewer: Martin Roth <martinroth(a)google.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Jan Dabros <jsd(a)semihalf.com> Gerrit-MessageType: merged

1 0

Change in coreboot[master]: mb/google/brya/var/brya0: Swap TPM and touchscreen I2C bus
by Eric Lai (Code Review) 19 Apr '22

19 Apr '22

Attention is currently required from: Tim Wawrzynczak, Amanda Hwang. Eric Lai has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63709 ) Change subject: mb/google/brya/var/brya0: Swap TPM and touchscreen I2C bus ...................................................................... Patch Set 1: (1 comment) File src/mainboard/google/brya/variants/brya0/overridetree.cb: https://review.coreboot.org/c/coreboot/+/63709/comment/de0359a1_428f0bb3 PS1, Line 70: .speed = I2C_SPEED_FAST, miss i2c tuning? -- To view, visit https://review.coreboot.org/c/coreboot/+/63709 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ifa6235869f34e0038a8ecad33d59654626cf7815 Gerrit-Change-Number: 63709 Gerrit-PatchSet: 1 Gerrit-Owner: Amanda Hwang <amanda_hwang(a)compal.corp-partner.google.com> Gerrit-Reviewer: Frank Wu <frank_wu(a)compal.corp-partner.google.com> Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Eric Lai <eric_lai(a)quanta.corp-partner.google.com> Gerrit-CC: John Su <john_su(a)compal.corp-partner.google.com> Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-CC: Van Chen <van_chen(a)compal.corp-partner.google.com> Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)chromium.org> Gerrit-Attention: Amanda Hwang <amanda_hwang(a)compal.corp-partner.google.com> Gerrit-Comment-Date: Tue, 19 Apr 2022 12:23:25 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

Change in coreboot[master]: util/cbfstool/linux_trampoline: Use payload calling argument
by Arthur Heymans (Code Review) 19 Apr '22

19 Apr '22

Attention is currently required from: Patrick Georgi. Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63611 ) Change subject: util/cbfstool/linux_trampoline: Use payload calling argument ...................................................................... Patch Set 2: (1 comment) File util/cbfstool/linux_trampoline.S: https://review.coreboot.org/c/coreboot/+/63611/comment/59bee408_61fd1813 PS2, Line 5: * IT SHOULDN'T USE THE STACK this assumption is now broken... -- To view, visit https://review.coreboot.org/c/coreboot/+/63611 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ie0e000470299bddde3cae9347a1b18cc5b0cfc77 Gerrit-Change-Number: 63611 Gerrit-PatchSet: 2 Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Patrick Georgi <patrick(a)coreboot.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Attention: Patrick Georgi <patrick(a)coreboot.org> Gerrit-Comment-Date: Tue, 19 Apr 2022 12:19:26 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

Change in coreboot[master]: mb/google/guybrush/var/dewatt: Update APU STT setting
by Felix Held (Code Review) 19 Apr '22

19 Apr '22

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/63699 ) Change subject: mb/google/guybrush/var/dewatt: Update APU STT setting ...................................................................... mb/google/guybrush/var/dewatt: Update APU STT setting update STT setting for dewatt. BUG=b:228040295 BRANCH=guybrush TEST=build, verify the parameter has been applied to the system by checking the AGT tool. Signed-off-by: Chris.Wang <chris.wang(a)amd.corp-partner.google.com> Change-Id: Id319d42747dd0d5f6a9ca727635d85e6b9bd65af Reviewed-on: https://review.coreboot.org/c/coreboot/+/63699 Tested-by: build bot (Jenkins) <no-reply(a)coreboot.org> Reviewed-by: Felix Held <felix-coreboot(a)felixheld.de> --- M src/mainboard/google/guybrush/variants/dewatt/overridetree.cb 1 file changed, 22 insertions(+), 0 deletions(-) Approvals: build bot (Jenkins): Verified Felix Held: Looks good to me, approved diff --git a/src/mainboard/google/guybrush/variants/dewatt/overridetree.cb b/src/mainboard/google/guybrush/variants/dewatt/overridetree.cb index 866aee5..84bf9da 100644 --- a/src/mainboard/google/guybrush/variants/dewatt/overridetree.cb +++ b/src/mainboard/google/guybrush/variants/dewatt/overridetree.cb @@ -27,6 +27,28 @@ register "telemetry_vddcrsocfull_scale_current_mA" = "29785" #mA register "telemetry_vddcrsocoffset" = "461" + # Enable STT support + register "stt_control" = "1" + register "stt_pcb_sensor_count" = "2" + register "stt_min_limit" = "15000" + register "stt_m1" = "0x022B" + register "stt_m2" = "0x117" + register "stt_m3" = "0" + register "stt_m4" = "0" + register "stt_m5" = "0" + register "stt_m6" = "0" + register "stt_c_apu" = "0xBA4" + register "stt_c_gpu" = "0" + register "stt_c_hs2" = "0" + register "stt_alpha_apu" = "0x199A" + register "stt_alpha_gpu" = "0" + register "stt_alpha_hs2" = "0" + register "stt_skin_temp_apu" = "0x2D00" + register "stt_skin_temp_gpu" = "0" + register "stt_skin_temp_hs2" = "0" + register "stt_error_coeff" = "0x21" + register "stt_error_rate_coefficient" = "0x2666" + #USB 2/3 phy config register "usb_phy" = "{ /* Left USB C0 Port */ -- To view, visit https://review.coreboot.org/c/coreboot/+/63699 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Id319d42747dd0d5f6a9ca727635d85e6b9bd65af Gerrit-Change-Number: 63699 Gerrit-PatchSet: 2 Gerrit-Owner: Chris Wang <chris.wang(a)amd.corp-partner.google.com> Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de> Gerrit-Reviewer: Rob Barnes <robbarnes(a)google.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-MessageType: merged

1 0

Change in coreboot[master]: mb/google/guybrush/var/dewatt: Update APU STT setting
by Felix Held (Code Review) 19 Apr '22

19 Apr '22

Attention is currently required from: Chris Wang, Rob Barnes. Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63699 ) Change subject: mb/google/guybrush/var/dewatt: Update APU STT setting ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://review.coreboot.org/c/coreboot/+/63699 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Id319d42747dd0d5f6a9ca727635d85e6b9bd65af Gerrit-Change-Number: 63699 Gerrit-PatchSet: 1 Gerrit-Owner: Chris Wang <chris.wang(a)amd.corp-partner.google.com> Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de> Gerrit-Reviewer: Rob Barnes <robbarnes(a)google.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Attention: Chris Wang <chris.wang(a)amd.corp-partner.google.com> Gerrit-Attention: Rob Barnes <robbarnes(a)google.com> Gerrit-Comment-Date: Tue, 19 Apr 2022 12:14:36 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

Change in coreboot[master]: soc/intel/common/smbus: Add `finalize` operation for smbus
by Subrata Banik (Code Review) 19 Apr '22

19 Apr '22

Attention is currently required from: Felix Singer, Tim Wawrzynczak, Arthur Heymans, Nick Vaccaro, Eric Lai, Lean Sheng Tan, Werner Zeh. Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63640 ) Change subject: soc/intel/common/smbus: Add `finalize` operation for smbus ...................................................................... Patch Set 4: (1 comment) File src/soc/intel/common/block/smbus/smbus.c: https://review.coreboot.org/c/coreboot/+/63640/comment/4f96f370_d1b209ec PS4, Line 88: smbus_final > Suggestion: […] Ack -- To view, visit https://review.coreboot.org/c/coreboot/+/63640 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ie945680049514e6c5d797790a381a6946e836926 Gerrit-Change-Number: 63640 Gerrit-PatchSet: 4 Gerrit-Owner: Subrata Banik <subratabanik(a)google.com> Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com> Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Eric Lai <eric_lai(a)quanta.corp-partner.google.com> Gerrit-Reviewer: Lean Sheng Tan <sheng.tan(a)9elements.com> Gerrit-Reviewer: Nick Vaccaro <nvaccaro(a)google.com> Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org> Gerrit-Reviewer: Werner Zeh <werner.zeh(a)siemens.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Felix Singer <felixsinger(a)posteo.net> Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Attention: Felix Singer <felixsinger(a)posteo.net> Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)chromium.org> Gerrit-Attention: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Attention: Nick Vaccaro <nvaccaro(a)google.com> Gerrit-Attention: Eric Lai <eric_lai(a)quanta.corp-partner.google.com> Gerrit-Attention: Lean Sheng Tan <sheng.tan(a)9elements.com> Gerrit-Attention: Werner Zeh <werner.zeh(a)siemens.com> Gerrit-Comment-Date: Tue, 19 Apr 2022 12:12:21 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Felix Singer <felixsinger(a)posteo.net> Gerrit-MessageType: comment

1 0

Change in coreboot[master]: soc/intel/common/smbus: Add `finalize` operation for smbus
by Subrata Banik (Code Review) 19 Apr '22

19 Apr '22

Attention is currently required from: Felix Singer, Tim Wawrzynczak, Arthur Heymans, Nick Vaccaro, Eric Lai, Lean Sheng Tan, Werner Zeh. Hello build bot (Jenkins), Tim Wawrzynczak, Angel Pons, Arthur Heymans, Nick Vaccaro, Eric Lai, Lean Sheng Tan, Werner Zeh, I'd like you to reexamine a change. Please visit https://review.coreboot.org/c/coreboot/+/63640 to look at the new patch set (#5). Change subject: soc/intel/common/smbus: Add `finalize` operation for smbus ...................................................................... soc/intel/common/smbus: Add `finalize` operation for smbus This patch implements the required operations to perform prior to booting to OS using coreboot native driver when platform decides to skip FSP notify APIs i.e. Post PCI Enumeration. The smbus `.final` operation ensures locking the TCO register when coreboot decides to skip FspNotifyApi() calls. BUG=b:211954778 TEST=Able to build google/brya with these changes and coreboot log with this code change as below with ADL SoC skip calling into FspNotifyAPIs: [INFO ] Finalize devices... [DEBUG] PCI: 00:1f.4 final > localhost ~ # lspci -xxx | less 00:1f.4 Intel Corporation Alder Lake PCH-P SMBus Host Controller (rev 01) Offset 8, Bit 12 a.k.a TCO Lock bit is set (meaning locked). Signed-off-by: Subrata Banik <subratabanik(a)google.com> Change-Id: Ie945680049514e6c5d797790a381a6946e836926 --- M src/soc/intel/common/block/smbus/smbus.c 1 file changed, 15 insertions(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/40/63640/5 -- To view, visit https://review.coreboot.org/c/coreboot/+/63640 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ie945680049514e6c5d797790a381a6946e836926 Gerrit-Change-Number: 63640 Gerrit-PatchSet: 5 Gerrit-Owner: Subrata Banik <subratabanik(a)google.com> Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com> Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Eric Lai <eric_lai(a)quanta.corp-partner.google.com> Gerrit-Reviewer: Lean Sheng Tan <sheng.tan(a)9elements.com> Gerrit-Reviewer: Nick Vaccaro <nvaccaro(a)google.com> Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org> Gerrit-Reviewer: Werner Zeh <werner.zeh(a)siemens.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Felix Singer <felixsinger(a)posteo.net> Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Attention: Felix Singer <felixsinger(a)posteo.net> Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)chromium.org> Gerrit-Attention: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Attention: Nick Vaccaro <nvaccaro(a)google.com> Gerrit-Attention: Eric Lai <eric_lai(a)quanta.corp-partner.google.com> Gerrit-Attention: Lean Sheng Tan <sheng.tan(a)9elements.com> Gerrit-Attention: Werner Zeh <werner.zeh(a)siemens.com> Gerrit-MessageType: newpatchset

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

coreboot-gerrit April 2022