[coreboot-gerrit] Change in ...coreboot[master]: intel/apollolake: Add IPU to disable_dev function

[Werner Zeh (Code Review)]

Werner Zeh has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/30992

Change subject: intel/apollolake: Add IPU to disable_dev function
......................................................................

intel/apollolake: Add IPU to disable_dev function

The SoC has an Image Processing Unit which is located on PCI 00:03.0.
There is a corresponding parameter for FSP which treats enable/disable
of this functionality (IpuEn). Add this device to the disable_dev()
function of the chip so that if this device is disabled in devicetree
the matching FSP parameter will be disabled as well.

Change-Id: I75444bf483de32ba641f76ca50e9744fdce2e726
Signed-off-by: Werner Zeh 
---
M src/soc/intel/apollolake/chip.c
M src/soc/intel/apollolake/include/soc/pci_devs.h
2 files changed, 7 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/92/30992/1

diff --git a/src/soc/intel/apollolake/chip.c b/src/soc/intel/apollolake/chip.c
index b38265f..102ba34 100644
--- a/src/soc/intel/apollolake/chip.c
+++ b/src/soc/intel/apollolake/chip.c
@@ -482,6 +482,9 @@
    case PCH_DEVFN_SMBUS:
    	silconfig->SmbusEnable = 0;
    	break;
+	case SA_DEVFN_IPU:
+		silconfig->IpuEn = 0;
+		break;
    default:
    	printk(BIOS_WARNING, "PCI:%02x.%01x: Could not disable the device\n",
    		PCI_SLOT(dev->path.pci.devfn),
diff --git a/src/soc/intel/apollolake/include/soc/pci_devs.h b/src/soc/intel/apollolake/include/soc/pci_devs.h
index ad726f8..363301d 100644
--- a/src/soc/intel/apollolake/include/soc/pci_devs.h
+++ b/src/soc/intel/apollolake/include/soc/pci_devs.h
@@ -45,6 +45,10 @@
 #define  SA_DEVFN_IGD		_SA_DEVFN(IGD)
 #define  SA_DEV_IGD		_SA_DEV(IGD)

+#define SA_DEV_SLOT_IPU		0x03
+#define  SA_DEVFN_IPU		_SA_DEVFN(IPU)
+#define  SA_DEV_IPU		_SA_DEV(IPU)
+
 /* PCH Devices */

#define PCH_DEV_SLOT_NPK	0x00