diff options
| author | Jeremy Compostella <jeremy.compostella@intel.com> | 2024-10-01 11:01:34 -0700 |
|---|---|---|
| committer | Jérémy Compostella <jeremy.compostella@intel.com> | 2024-10-03 16:17:11 +0000 |
| commit | f9e877ea2173696028ded8e82fbe51e13085865b (patch) | |
| tree | f642c7aec20851003aecded41799f8516116cb6e /src/soc/intel/pantherlake/cpu.c | |
| parent | b9c0056b2dc4f1f5658d49edf8d8a43b1bcb9536 (diff) | |
soc/intel/pantherlake: Remove soc_info.[hc] interface
This commit removes the unnecessary layer provided by soc_info.[hc].
It was providing an abstraction which only was resulting in extra
function calls without any added value as the returned constants are
well identified and could be used directly. More importantly, and this
is the actual selling point in my opinion, this extra indirection was
preventing the compiler from detecting array overflows.
BUG=348678529
TEST=Build is successful
Change-Id: Iea26d962748116fa84afdb4afcba1098a64b6986
Signed-off-by: Jeremy Compostella <jeremy.compostella@intel.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/84616
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Subrata Banik <subratabanik@google.com>
Diffstat (limited to 'src/soc/intel/pantherlake/cpu.c')
| -rw-r--r-- | src/soc/intel/pantherlake/cpu.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/soc/intel/pantherlake/cpu.c b/src/soc/intel/pantherlake/cpu.c index 7856383339..1b81856afc 100644 --- a/src/soc/intel/pantherlake/cpu.c +++ b/src/soc/intel/pantherlake/cpu.c @@ -20,7 +20,6 @@ #include <soc/msr.h> #include <soc/pci_devs.h> #include <soc/soc_chip.h> -#include <soc/soc_info.h> bool cpu_soc_is_in_untrusted_mode(void) { |