[coreboot-gerrit] Change in coreboot[master]: mb/{google,intel}/{volteer,tglrvp}: Refer to common IPU ASL

[Subrata Banik (Code Review)]

Subrata Banik has submitted this change. ( https://review.coreboot.org/c/coreboot/+/45978 )

Change subject: mb/{google,intel}/{volteer,tglrvp}: Refer to common IPU ASL
......................................................................

mb/{google,intel}/{volteer,tglrvp}: Refer to common IPU ASL

Delete SoC local copy of ipu.asl and refer from common block ipu.asl

TEST=Dump and disassemble DSDT on tglrvp, verify IPU0 device
present there.

Signed-off-by: Subrata Banik 
Change-Id: I6a0f8a919092f7bbcd64d4791746d30fdee33894
Reviewed-on: https://review.coreboot.org/c/coreboot/+/45978
Tested-by: build bot (Jenkins) 
Reviewed-by: Angel Pons 
---
M src/mainboard/google/volteer/dsdt.asl
M src/mainboard/intel/tglrvp/dsdt.asl
D src/soc/intel/tigerlake/acpi/ipu.asl
3 files changed, 4 insertions(+), 12 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Angel Pons: Looks good to me, approved

diff --git a/src/mainboard/google/volteer/dsdt.asl b/src/mainboard/google/volteer/dsdt.asl
index d0bab1a..dc8f7ad 100644
--- a/src/mainboard/google/volteer/dsdt.asl
+++ b/src/mainboard/google/volteer/dsdt.asl
@@ -27,6 +27,9 @@
    		#include <soc/intel/common/block/acpi/acpi/northbridge.asl>
    		#include <soc/intel/tigerlake/acpi/southbridge.asl>
    		#include <soc/intel/tigerlake/acpi/tcss.asl>
+#if CONFIG(VARIANT_HAS_MIPI_CAMERA)
+			#include <soc/intel/common/block/acpi/acpi/ipu.asl>
+#endif
    	}
    	/* Mainboard hooks */
    	#include "mainboard.asl"
@@ -52,7 +55,6 @@

#if CONFIG(VARIANT_HAS_MIPI_CAMERA)
    /* Camera */
-	#include <soc/intel/tigerlake/acpi/ipu.asl>
    #include <variant/acpi/mipi_camera.asl>
 #endif /* VARIANT_HAS_MIPI_CAMERA */
 }
diff --git a/src/mainboard/intel/tglrvp/dsdt.asl b/src/mainboard/intel/tglrvp/dsdt.asl
index e34fd58..e5323ef 100644
--- a/src/mainboard/intel/tglrvp/dsdt.asl
+++ b/src/mainboard/intel/tglrvp/dsdt.asl
@@ -27,6 +27,7 @@
    		#include <soc/intel/common/block/acpi/acpi/northbridge.asl>
    		#include <soc/intel/tigerlake/acpi/southbridge.asl>
    		#include <soc/intel/tigerlake/acpi/tcss.asl>
+			#include <soc/intel/common/block/acpi/acpi/ipu.asl>
    	}
    }

@@ -52,6 +53,5 @@
    #include "acpi/mainboard.asl"

/* Camera */
-        #include <soc/intel/tigerlake/acpi/ipu.asl>
    #include "acpi/mipi_camera.asl"
 }
diff --git a/src/soc/intel/tigerlake/acpi/ipu.asl b/src/soc/intel/tigerlake/acpi/ipu.asl
deleted file mode 100644
index b6d8e8f..0000000
--- a/src/soc/intel/tigerlake/acpi/ipu.asl
+++ /dev/null
@@ -1,10 +0,0 @@
-/* SPDX-License-Identifier: GPL-2.0-only */
-
-Scope (_SB.PCI0)
-{
-	Device (IPU0)
-	{
-		Name (_ADR, 0x00050000)
-		Name (_DDN, "Camera and Imaging Subsystem")
-	}
-}