diff options
| author | Felix Singer <felixsinger@posteo.net> | 2021-09-22 22:24:08 +0200 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2021-09-27 13:23:20 +0000 |
| commit | 35d94009fb53dea63cce7f7c914a26271413be4f (patch) | |
| tree | 7987da7f7ce0563fd8b804eb1df29dbe48143b9c | |
| parent | 0c9bc82d4817497b83aef76935c3efb913ff5b84 (diff) | |
mb/intel/saddlebrook: Clean up dsdt.asl
Move includes using library paths to the top and remove unnecessary
comments. Also, get rid of that unnecessary _SB scope. Use an absolute
path for the PCI0 device instead.
Change-Id: I730cd3eeffff60b3b569bfb748febbdc8ca85990
Signed-off-by: Felix Singer <felixsinger@posteo.net>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/57850
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
| -rw-r--r-- | src/mainboard/intel/saddlebrook/dsdt.asl | 20 |
1 files changed, 6 insertions, 14 deletions
diff --git a/src/mainboard/intel/saddlebrook/dsdt.asl b/src/mainboard/intel/saddlebrook/dsdt.asl index ad20f909c5..e2291da3f9 100644 --- a/src/mainboard/intel/saddlebrook/dsdt.asl +++ b/src/mainboard/intel/saddlebrook/dsdt.asl @@ -11,24 +11,16 @@ DefinitionBlock( ) { #include <acpi/dsdt_top.asl> + #include <cpu/intel/common/acpi/cpu.asl> #include <soc/intel/common/block/acpi/acpi/platform.asl> - - // global NVS and variables #include <soc/intel/skylake/acpi/globalnvs.asl> + #include <southbridge/intel/common/acpi/sleepstates.asl> - // CPU - #include <cpu/intel/common/acpi/cpu.asl> - - Scope (\_SB) { - Device (PCI0) - { - #include <soc/intel/skylake/acpi/systemagent.asl> - #include <soc/intel/skylake/acpi/pch.asl> - } + Device (\_SB.PCI0) + { + #include <soc/intel/skylake/acpi/systemagent.asl> + #include <soc/intel/skylake/acpi/pch.asl> } - #include <southbridge/intel/common/acpi/sleepstates.asl> - - // Mainboard specific #include "acpi/mainboard.asl" } |