From ce9d41fe2d8aa764d5970f5509a3b75a79cbed3b Mon Sep 17 00:00:00 2001
From: Felix Singer <felixsinger@posteo.net>
Date: Wed, 22 Sep 2021 22:49:56 +0200
Subject: [PATCH] mb/51nb/x210: 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: I7255ac1ec6c43dd4b21325ae60e117458bea956d
Signed-off-by: Felix Singer <felixsinger@posteo.net>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/57855
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
---
 src/mainboard/51nb/x210/dsdt.asl | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/src/mainboard/51nb/x210/dsdt.asl b/src/mainboard/51nb/x210/dsdt.asl
index a9ed9198c1..398611f6a4 100644
--- a/src/mainboard/51nb/x210/dsdt.asl
+++ b/src/mainboard/51nb/x210/dsdt.asl
@@ -12,11 +12,9 @@ DefinitionBlock(
 )
 {
 	#include <acpi/dsdt_top.asl>
-	#include "acpi/platform.asl"
-
-	#include <soc/intel/skylake/acpi/globalnvs.asl>
-
 	#include <cpu/intel/common/acpi/cpu.asl>
+	#include <soc/intel/skylake/acpi/globalnvs.asl>
+	#include <southbridge/intel/common/acpi/sleepstates.asl>
 
 	Device (\_SB.PCI0)
 	{
@@ -25,8 +23,6 @@ DefinitionBlock(
 		#include "acpi/graphics.asl"
 	}
 
-	#include <southbridge/intel/common/acpi/sleepstates.asl>
-
-	// Mainboard specific
 	#include "acpi/mainboard.asl"
+	#include "acpi/platform.asl"
 }