commit | 14643b33b0f7ea9cde6ce8230793562d4a1b475c | [log] [tgz] |
---|---|---|
author | Angel Pons <th3fanbus@gmail.com> | Sun Oct 17 13:21:05 2021 +0200 |
committer | Felix Held <felix-coreboot@felixheld.de> | Tue Oct 19 14:57:17 2021 +0000 |
tree | 437715af86a62e127f119200b2738d03d6779aab | |
parent | 3b9be63bd32daff1083f38b409beab71f2655b4f [diff] [blame] |
soc/intel/*/acpi.c: Don't copy structs with `memcpy()` A regular assignment works just as well and also allows type-checking. Change-Id: Id772771f000ba3bad5d4af05f5651c0f0ee43d6d Signed-off-by: Angel Pons <th3fanbus@gmail.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/58390 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Raul Rangel <rrangel@chromium.org>
diff --git a/src/soc/intel/cannonlake/acpi.c b/src/soc/intel/cannonlake/acpi.c index d0f3874..5e2d142 100644 --- a/src/soc/intel/cannonlake/acpi.c +++ b/src/soc/intel/cannonlake/acpi.c
@@ -131,7 +131,7 @@ } for (i = 0; i < *entries; i++) { - memcpy(&map[i], &cstate_map[set[i]], sizeof(acpi_cstate_t)); + map[i] = cstate_map[set[i]]; map[i].ctype = i + 1; } return map;