intel/apollolake: Add IPU to disable_dev function
commitde3ace06290d8625b5f54cae25deca682f773698
authorWerner Zeh <werner.zeh@siemens.com>
Tue, 15 Jan 2019 07:03:43 +0000 (15 08:03 +0100)
committerPatrick Georgi <pgeorgi@google.com>
Mon, 28 Jan 2019 13:44:10 +0000 (28 13:44 +0000)
tree1e578eb1afe459edb4f0dba97dcaf6ed322ef699
parentcfa435a0fffad43d18b7251cb8d73c61a938eb2f
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 handles
enabling/disabling 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. As this parameter is only valid for Apollo Lake, use the config
switch CONFIG_SOC_INTEL_GLK to disable this code if compiled not for
Apollo Lake. As this issue is regarding a missing structure member,
this check needs to be done on preprocessor level and not at runtime.

Test=Verified this function on mc_apl2.

Change-Id: I75444bf483de32ba641f76ca50e9744fdce2e726
Signed-off-by: Werner Zeh <werner.zeh@siemens.com>
Reviewed-on: https://review.coreboot.org/c/30992
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
src/soc/intel/apollolake/chip.c
src/soc/intel/apollolake/include/soc/pci_devs.h