summaryrefslogtreecommitdiff
path: root/src/drivers
diff options
context:
space:
mode:
authorjimlai <jim.lai@intel.com>2022-11-23 16:47:40 +0800
committerEric Lai <eric_lai@quanta.corp-partner.google.com>2022-11-25 01:03:21 +0000
commitbbc8d912e827b229f81027b804414fecd999590c (patch)
tree0821fa78b1fe342e1f82b74fe1ba8841f83c213a /src/drivers
parent8aebfd34d9432d66d41a4b21a2957cd4aaa702ef (diff)
drivers/intel/mipi_camera: Remove IPU ES support entry for RPL
The current IPU ES entry value is always set to true for RPL and kernel picks the ES version of the main IPU FW even for the production bootloader but loading is not successful due to the authentication failure. On Raptor Lake silicon, the production binaries are backward compatible with ES parts. This change removes the IPU ES support ACPI entry since the kernel needs to load the production IPU main firmware on both the ES/QS parts. BUG=b:258125833 TEST=Verify the Camera functionality by enabling the IPU secure mode on RPL variants with both ES/QS silicon. Signed-off-by: Jim Lai <jim.lai@intel.com> Change-Id: I9fd8ea0dd6ffdb16961bb017ba4388bf99e4d5bc Reviewed-on: https://review.coreboot.org/c/coreboot/+/69929 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Eric Lai <eric_lai@quanta.corp-partner.google.com>
Diffstat (limited to 'src/drivers')
-rw-r--r--src/drivers/intel/mipi_camera/camera.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/src/drivers/intel/mipi_camera/camera.c b/src/drivers/intel/mipi_camera/camera.c
index 5b69c01aba..ec89971cbd 100644
--- a/src/drivers/intel/mipi_camera/camera.c
+++ b/src/drivers/intel/mipi_camera/camera.c
@@ -138,9 +138,7 @@ static void camera_fill_cio2(const struct device *dev)
port_name[i] = strdup(name);
if (CONFIG(ACPI_ADL_IPU_ES_SUPPORT)) {
u32 cpu_id = cpu_get_cpuid();
- if (cpu_id == CPUID_ALDERLAKE_J0 || cpu_id == CPUID_ALDERLAKE_Q0 ||
- cpu_id == CPUID_RAPTORLAKE_P_J0 ||
- cpu_id == CPUID_RAPTORLAKE_P_Q0)
+ if (cpu_id == CPUID_ALDERLAKE_J0 || cpu_id == CPUID_ALDERLAKE_Q0)
acpi_dp_add_integer(dsd, "is_es", 1);
else
acpi_dp_add_integer(dsd, "is_es", 0);