diff options
| author | Arthur Heymans <arthur@aheymans.xyz> | 2022-05-14 16:16:36 +0200 |
|---|---|---|
| committer | Martin L Roth <gaumless@tutanota.com> | 2022-05-15 23:58:14 +0000 |
| commit | 026978bce588f9d26d443fbacbf8dcc90957e331 (patch) | |
| tree | 67a348360fdb02ebe35592f6b2ae16b0bb2d16df /src/soc/intel/alderlake/acpi.c | |
| parent | 7e0af339ee1e8c4aa3a4ddd65e02070dea5debac (diff) | |
soc/intel/alderlake: Move array declaration
Clang does not like array declarations inside plain switch cases. There
are 2 options to fix this: use a block inside the switch statement, or
declare it outside the switch statement. This does the latter.
Change-Id: I9a02136fd63ac171b2bec4647c30c7eece930246
Signed-off-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/64349
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Felix Singer <felixsinger@posteo.net>
Diffstat (limited to 'src/soc/intel/alderlake/acpi.c')
| -rw-r--r-- | src/soc/intel/alderlake/acpi.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/soc/intel/alderlake/acpi.c b/src/soc/intel/alderlake/acpi.c index e588f803c7..39897e7abd 100644 --- a/src/soc/intel/alderlake/acpi.c +++ b/src/soc/intel/alderlake/acpi.c @@ -286,6 +286,7 @@ void soc_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: @@ -293,8 +294,6 @@ void soc_lpi_get_constraints(void *unused) break; case DEVICE_PATH_APIC: - char path[32] = {0}; - /* Lookup CPU id */ for (size_t i = 0; i < CONFIG_MAX_CPUS; i++) { if (cpu_get_apic_id(i) == dev->path.apic.apic_id) { |