diff options
| author | Felix Held <felix-coreboot@felixheld.de> | 2023-01-25 16:44:20 +0100 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2023-01-28 18:51:04 +0000 |
| commit | 5edb51855c2c8885bb1ea4fc3780728b6acd516e (patch) | |
| tree | 425b4a607141253b8a524ec0deafa17f7a5ace15 /src/soc | |
| parent | b57b12f729947c83515b96e45885b69d73027d79 (diff) | |
soc/intel/common/block/acpi/pep: use acpigen_write_processor_namestring
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Change-Id: I43590f0f792fca1c90ee8f8b32e6be47943c59df
Reviewed-on: https://review.coreboot.org/c/coreboot/+/72453
Reviewed-by: Fred Reitberger <reitbergerfred@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/soc')
| -rw-r--r-- | src/soc/intel/common/block/acpi/pep.c | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/src/soc/intel/common/block/acpi/pep.c b/src/soc/intel/common/block/acpi/pep.c index 2dd094908e..0ba38c11af 100644 --- a/src/soc/intel/common/block/acpi/pep.c +++ b/src/soc/intel/common/block/acpi/pep.c @@ -93,7 +93,6 @@ static void read_pmc_lpm_requirements(const struct soc_pmc_lpm *lpm, */ static void acpi_gen_default_lpi_constraints(void) { - char path[16]; printk(BIOS_INFO, "Returning default LPI constraint package\n"); /* @@ -107,8 +106,7 @@ static void acpi_gen_default_lpi_constraints(void) { acpigen_write_package(3); { - snprintf(path, sizeof(path), CONFIG_ACPI_CPU_STRING, 0); - acpigen_emit_namestring(path); + acpigen_write_processor_namestring(0); acpigen_write_integer(0); /* device disabled */ acpigen_write_package(2); { @@ -188,7 +186,6 @@ static void acpi_lpi_get_constraints(void *unused) acpigen_write_package(3); { - char path[32] = { 0 }; /* Emit the device path */ switch (dev->path.type) { case DEVICE_PATH_PCI: @@ -196,9 +193,7 @@ static void acpi_lpi_get_constraints(void *unused) break; case DEVICE_PATH_APIC: - snprintf(path, sizeof(path), CONFIG_ACPI_CPU_STRING, - cpu_index++); - acpigen_emit_namestring(path); + acpigen_write_processor_namestring(cpu_index++); break; default: |