]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
media: ov08x40: Extend sleep after reset to 5 ms
authorHans de Goede <hdegoede@redhat.com>
Tue, 11 Mar 2025 11:48:44 +0000 (12:48 +0100)
committerHans Verkuil <hverkuil@xs4all.nl>
Fri, 25 Apr 2025 08:15:14 +0000 (10:15 +0200)
Some users are reporting that ov08x40_identify_module() fails
to identify the chip reading 0x00 as value for OV08X40_REG_CHIP_ID.

Intel's out of tree IPU6 drivers include some ov08x40 changes
including adding support for the reset GPIO for older kernels and
Intel's patch for this uses 5 ms. Extend the sleep to 5 ms following
Intel's example, this fixes the ov08x40_identify_module() problem.

Link: https://github.com/intel/ipu6-drivers/blob/c09e2198d801e1eb701984d2948373123ba92a56/patch/v6.12/0008-media-ov08x40-Add-support-for-2-4-lanes-support-at-1.patch#L4607
Fixes: df1ae2251a50 ("media: ov08x40: Add OF probe support")
Cc: stable@vger.kernel.org
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl>
drivers/media/i2c/ov08x40.c

index cf0e41fc30719d7071187d65f59f2ce8bb0a2855..54575eea3c49069ff0a0af5996473352e1ae6405 100644 (file)
@@ -1341,7 +1341,7 @@ static int ov08x40_power_on(struct device *dev)
        }
 
        gpiod_set_value_cansleep(ov08x->reset_gpio, 0);
-       usleep_range(1500, 1800);
+       usleep_range(5000, 5500);
 
        return 0;