Change in coreboot[master]: src/soc/intel/xeon_sp/acpi.c: Remove unnecessary .h

[Marc Jones (Code Review)]

Marc Jones has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/48826 )

Change subject: src/soc/intel/xeon_sp/acpi.c: Remove unnecessary .h
......................................................................

src/soc/intel/xeon_sp/acpi.c: Remove unnecessary .h

Remove the unnecessary header file includes.

Change-Id: I0d849cb236f304b87332aa64b2f10c73cad2d4dd
Signed-off-by: Marc Jones 
---
M src/soc/intel/xeon_sp/acpi.c
1 file changed, 0 insertions(+), 12 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/26/48826/1

diff --git a/src/soc/intel/xeon_sp/acpi.c b/src/soc/intel/xeon_sp/acpi.c
index b1d5107..b271264 100644
--- a/src/soc/intel/xeon_sp/acpi.c
+++ b/src/soc/intel/xeon_sp/acpi.c
@@ -1,20 +1,8 @@
 /* SPDX-License-Identifier: GPL-2.0-or-later */

#include <assert.h>
-#include <acpi/acpi_gnvs.h>
-#include <acpi/acpigen.h>
-#include <arch/smp/mpspec.h>
-#include <device/pci.h>
-#include <cbmem.h>
-#include <cpu/x86/smm.h>
 #include <intelblocks/acpi.h>
-#include <intelblocks/lpc_lib.h>
-#include <soc/acpi.h>
-#include <soc/cpu.h>
-#include <soc/intel/common/acpi.h>
 #include <soc/pci_devs.h>
-#include <soc/pm.h>
-#include <soc/soc_util.h>
 #include <soc/util.h>
 #include <string.h>

[Kyösti Mälkki (Code Review)]

Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48826 )

Change subject: src/soc/intel/xeon_sp/acpi.c: Remove unnecessary .h
......................................................................

Patch Set 1: Code-Review+1

[Jay Talbott (Code Review)]

Jay Talbott has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48826 )

Change subject: src/soc/intel/xeon_sp/acpi.c: Remove unnecessary .h
......................................................................

Patch Set 1: Code-Review+1

[Jonathan Zhang (Code Review)]

Jonathan Zhang has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48826 )

Change subject: src/soc/intel/xeon_sp/acpi.c: Remove unnecessary .h
......................................................................

Patch Set 1: Code-Review+2

[Marc Jones (Code Review)]

Marc Jones has submitted this change. ( https://review.coreboot.org/c/coreboot/+/48826 )

Change subject: src/soc/intel/xeon_sp/acpi.c: Remove unnecessary .h
......................................................................

src/soc/intel/xeon_sp/acpi.c: Remove unnecessary .h

Remove the unnecessary header file includes.

Change-Id: I0d849cb236f304b87332aa64b2f10c73cad2d4dd
Signed-off-by: Marc Jones 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/48826
Tested-by: build bot (Jenkins) 
Reviewed-by: Kyösti Mälkki 
Reviewed-by: Jay Talbott 
Reviewed-by: Jonathan Zhang 
---
M src/soc/intel/xeon_sp/acpi.c
1 file changed, 0 insertions(+), 12 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Kyösti Mälkki: Looks good to me, but someone else must approve
  Jay Talbott: Looks good to me, but someone else must approve
  Jonathan Zhang: Looks good to me, approved

diff --git a/src/soc/intel/xeon_sp/acpi.c b/src/soc/intel/xeon_sp/acpi.c
index b1d5107..b271264 100644
--- a/src/soc/intel/xeon_sp/acpi.c
+++ b/src/soc/intel/xeon_sp/acpi.c
@@ -1,20 +1,8 @@
 /* SPDX-License-Identifier: GPL-2.0-or-later */

#include <assert.h>
-#include <acpi/acpi_gnvs.h>
-#include <acpi/acpigen.h>
-#include <arch/smp/mpspec.h>
-#include <device/pci.h>
-#include <cbmem.h>
-#include <cpu/x86/smm.h>
 #include <intelblocks/acpi.h>
-#include <intelblocks/lpc_lib.h>
-#include <soc/acpi.h>
-#include <soc/cpu.h>
-#include <soc/intel/common/acpi.h>
 #include <soc/pci_devs.h>
-#include <soc/pm.h>
-#include <soc/soc_util.h>
 #include <soc/util.h>
 #include <string.h>