From 3574c894bbaafcb457620b3eb339bc7f189fcd90 Mon Sep 17 00:00:00 2001 From: Jan Baisch Date: Wed, 12 Aug 2026 20:49:15 +0200 Subject: [PATCH] media: intel/ipu-bridge: fix OVTID858 link frequencies The OVTID858 configuration currently sets nr_link_freqs to 4 while providing only a single 540 MHz value. The second IPU_SENSOR_CONFIG argument is the number of link frequencies, not the CSI lane count; the lane count is obtained separately from firmware SSDB. The OV13858 driver exposes two link frequencies, 540 MHz and 270 MHz, and uses both across its supported modes. Publish those two frequencies in the OVTID858 bridge configuration instead. The entry was originally annotated for Surface Pro 9, but OVTID858 is also used by later Surface Pro generations, so make the sensor comment device-independent. The 540/270 MHz configuration was independently tested successfully with the OV13858 rear camera on Surface Pro 12 Intel. Signed-off-by: Jan Baisch Link: https://github.com/linux-surface/kernel/pull/174 Patchset: cameras --- drivers/media/pci/intel/ipu-bridge.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/pci/intel/ipu-bridge.c b/drivers/media/pci/intel/ipu-bridge.c index 3418ff4751985..c046a8ffa977c 100644 --- a/drivers/media/pci/intel/ipu-bridge.c +++ b/drivers/media/pci/intel/ipu-bridge.c @@ -63,8 +63,8 @@ static const struct ipu_sensor_config ipu_supported_sensors[] = { IPU_SENSOR_CONFIG("INT33F0", 1, 384000000), /* Omnivision OV5693 - Surface Pro 9 */ IPU_SENSOR_CONFIG("OVTI5693", 1, 419200000), - /* Omnivision OV13858 - Surface Pro 9 */ - IPU_SENSOR_CONFIG("OVTID858", 4, 540000000), + /* Omnivision OV13858 */ + IPU_SENSOR_CONFIG("OVTID858", 2, 540000000, 270000000), /* Omnivision OV2740 */ IPU_SENSOR_CONFIG("INT3474", 1, 180000000), /* Omnivision OV5670 */