[coreboot-gerrit] Change in coreboot[master]: soc/intel/apollolake: Move to common dsp driver

Lijian Zhao (Code Review) gerrit at coreboot.org
Mon Oct 30 22:46:55 CET 2017


Lijian Zhao has uploaded this change for review. ( https://review.coreboot.org/22234


Change subject: soc/intel/apollolake: Move to common dsp driver
......................................................................

soc/intel/apollolake: Move to common dsp driver

Move dsp driver implementation to common dsp driver.

TEST=Boot up and check dsp driver loaded or not in OS.

Change-Id: Ia2be1c9f18e0e110600bd56a0b6cb8d40ca5e01f
Signed-off-by: Lijian Zhao <lijian.zhao at intel.com>
---
M src/soc/intel/apollolake/Kconfig
M src/soc/intel/apollolake/Makefile.inc
D src/soc/intel/apollolake/dsp.c
3 files changed, 1 insertion(+), 38 deletions(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/34/22234/1

diff --git a/src/soc/intel/apollolake/Kconfig b/src/soc/intel/apollolake/Kconfig
index e8bfb07..cf2492c 100644
--- a/src/soc/intel/apollolake/Kconfig
+++ b/src/soc/intel/apollolake/Kconfig
@@ -65,6 +65,7 @@
 	select SOC_INTEL_COMMON_BLOCK
 	select SOC_INTEL_COMMON_BLOCK_ACPI
 	select SOC_INTEL_COMMON_BLOCK_CPU
+	select SOC_INTEL_COMMON_BLOCK_DSP
 	select SOC_INTEL_COMMON_BLOCK_FAST_SPI
 	select SOC_INTEL_COMMON_BLOCK_GPIO
 	select SOC_INTEL_COMMON_BLOCK_GPIO_MULTI_ACPI_DEVICES
diff --git a/src/soc/intel/apollolake/Makefile.inc b/src/soc/intel/apollolake/Makefile.inc
index 2848bed..de27000 100644
--- a/src/soc/intel/apollolake/Makefile.inc
+++ b/src/soc/intel/apollolake/Makefile.inc
@@ -48,7 +48,6 @@
 ramstage-y += chip.c
 ramstage-y += cse.c
 ramstage-y += elog.c
-ramstage-y += dsp.c
 ramstage-y += graphics.c
 ramstage-y += heci.c
 ramstage-y += i2c.c
diff --git a/src/soc/intel/apollolake/dsp.c b/src/soc/intel/apollolake/dsp.c
deleted file mode 100644
index 76547ee..0000000
--- a/src/soc/intel/apollolake/dsp.c
+++ /dev/null
@@ -1,37 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright 2016 Google Inc.
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; version 2 of the License.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- */
-
-#include <device/device.h>
-#include <device/pci.h>
-#include <device/pci_ids.h>
-
-static struct device_operations dsp_dev_ops = {
-	.read_resources         = &pci_dev_read_resources,
-	.set_resources          = &pci_dev_set_resources,
-	.enable_resources       = &pci_dev_enable_resources,
-	.scan_bus               = &scan_static_bus,
-};
-
-static const unsigned short pci_device_ids[] = {
-	PCI_DEVICE_ID_INTEL_APL_AUDIO,
-	PCI_DEVICE_ID_INTEL_GLK_AUDIO,
-	0,
-};
-
-static const struct pci_driver apollolake_dsp __pci_driver = {
-	.ops    = &dsp_dev_ops,
-	.vendor = PCI_VENDOR_ID_INTEL,
-	.devices = pci_device_ids,
-};

-- 
To view, visit https://review.coreboot.org/22234
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia2be1c9f18e0e110600bd56a0b6cb8d40ca5e01f
Gerrit-Change-Number: 22234
Gerrit-PatchSet: 1
Gerrit-Owner: Lijian Zhao <lijian.zhao at intel.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20171030/de56b82a/attachment-0001.html>


More information about the coreboot-gerrit mailing list