Hello build bot (Jenkins), Nico Huber, Patrick Georgi, Martin Roth, Furquan Shaikh, Subrata Banik, Aamir Bohra, Patrick Rudolph,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/45951
to look at the new patch set (#14).
Change subject: soc/intel/{skl,cnl,icl,tgl,jsl,ehl}: deduplicate ACPI timer emulation
......................................................................
soc/intel/{skl,cnl,icl,tgl,jsl,ehl}: deduplicate ACPI timer emulation
The code for enabling ACPI timer emulation is the same for the socs
above. Deduplicate it by moving it to common code.
Note: The ACPI timer emulation can only be used by SoCs with ucode
supporting CTC (Common Timer Copy) / ACPI timer emulation.
Change-Id: Ied4b312b6d53e80e71c55f4d1ca78a8cb2799793
Signed-off-by: Michael Niewöhner <foss(a)mniewoehner.de>
---
M src/soc/intel/cannonlake/cpu.c
M src/soc/intel/common/block/cpu/Makefile.inc
A src/soc/intel/common/block/cpu/pm_timer_emulation.c
M src/soc/intel/common/block/include/intelblocks/cpulib.h
M src/soc/intel/elkhartlake/cpu.c
M src/soc/intel/icelake/cpu.c
M src/soc/intel/jasperlake/cpu.c
M src/soc/intel/skylake/cpu.c
M src/soc/intel/tigerlake/cpu.c
9 files changed, 37 insertions(+), 123 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/51/45951/14
--
To view, visit https://review.coreboot.org/c/coreboot/+/45951
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ied4b312b6d53e80e71c55f4d1ca78a8cb2799793
Gerrit-Change-Number: 45951
Gerrit-PatchSet: 14
Gerrit-Owner: Michael Niewöhner <foss(a)mniewoehner.de>
Gerrit-Reviewer: Aamir Bohra <aamir.bohra(a)intel.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-MessageType: newpatchset
Hello build bot (Jenkins), Nico Huber, Paul Menzel, Subrata Banik, Andrey Petrov, Patrick Rudolph,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/46018
to look at the new patch set (#7).
Change subject: soc/intel/*: conversion from constant to Kconfig for XTAL frequency
......................................................................
soc/intel/*: conversion from constant to Kconfig for XTAL frequency
This converts the constant for the XTAL frequency to a Kconfig option.
Change-Id: I1382dd274eeb9cb748f94c34f5d9a83880624c18
Signed-off-by: Michael Niewöhner <foss(a)mniewoehner.de>
---
M src/soc/intel/alderlake/Kconfig
M src/soc/intel/alderlake/cpu.c
M src/soc/intel/alderlake/include/soc/cpu.h
M src/soc/intel/apollolake/Kconfig
M src/soc/intel/apollolake/include/soc/cpu.h
M src/soc/intel/apollolake/pmutil.c
M src/soc/intel/cannonlake/Kconfig
M src/soc/intel/cannonlake/cpu.c
M src/soc/intel/cannonlake/include/soc/cpu.h
M src/soc/intel/common/block/cpu/Kconfig
M src/soc/intel/elkhartlake/Kconfig
M src/soc/intel/elkhartlake/cpu.c
M src/soc/intel/elkhartlake/include/soc/cpu.h
M src/soc/intel/icelake/Kconfig
M src/soc/intel/icelake/cpu.c
M src/soc/intel/icelake/include/soc/cpu.h
M src/soc/intel/jasperlake/Kconfig
M src/soc/intel/jasperlake/cpu.c
M src/soc/intel/jasperlake/include/soc/cpu.h
M src/soc/intel/skylake/Kconfig
M src/soc/intel/skylake/cpu.c
M src/soc/intel/skylake/include/soc/cpu.h
M src/soc/intel/tigerlake/Kconfig
M src/soc/intel/tigerlake/cpu.c
M src/soc/intel/tigerlake/include/soc/cpu.h
25 files changed, 69 insertions(+), 56 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/18/46018/7
--
To view, visit https://review.coreboot.org/c/coreboot/+/46018
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I1382dd274eeb9cb748f94c34f5d9a83880624c18
Gerrit-Change-Number: 46018
Gerrit-PatchSet: 7
Gerrit-Owner: Michael Niewöhner <foss(a)mniewoehner.de>
Gerrit-Reviewer: Andrey Petrov <andrey.petrov(a)gmail.com>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-MessageType: newpatchset
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45953 )
Change subject: soc/intel/skl: guard ACPI PM timer emulation enablement
......................................................................
Patch Set 19:
The BIOS spec seems to suggest that we always enable it. I'd also
prefer that just because it's one `if` less (fewer paths, more test
coverage, less to maintain, ...).
--
To view, visit https://review.coreboot.org/c/coreboot/+/45953
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I529cdbe759e9c9270493bd3fff42ded65c74a6b4
Gerrit-Change-Number: 45953
Gerrit-PatchSet: 19
Gerrit-Owner: Michael Niewöhner <foss(a)mniewoehner.de>
Gerrit-Reviewer: Aamir Bohra <aamir.bohra(a)intel.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Comment-Date: Mon, 05 Oct 2020 15:39:49 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment