]> git.ipfire.org Git - thirdparty/kernel/stable-queue.git/commitdiff
6.0-stable patches
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 31 Oct 2022 10:54:41 +0000 (11:54 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 31 Oct 2022 10:54:41 +0000 (11:54 +0100)
added patches:
media-ar0521-fix-return-value-check-in-writing-initial-registers.patch

queue-6.0/media-ar0521-fix-return-value-check-in-writing-initial-registers.patch [new file with mode: 0644]
queue-6.0/series

diff --git a/queue-6.0/media-ar0521-fix-return-value-check-in-writing-initial-registers.patch b/queue-6.0/media-ar0521-fix-return-value-check-in-writing-initial-registers.patch
new file mode 100644 (file)
index 0000000..5114884
--- /dev/null
@@ -0,0 +1,36 @@
+From 54bb7671ca6de58929b3994468c330bedb9a3b7e Mon Sep 17 00:00:00 2001
+From: Sakari Ailus <sakari.ailus@linux.intel.com>
+Date: Tue, 30 Aug 2022 12:32:36 +0200
+Subject: media: ar0521: Fix return value check in writing initial registers
+
+From: Sakari Ailus <sakari.ailus@linux.intel.com>
+
+commit 54bb7671ca6de58929b3994468c330bedb9a3b7e upstream.
+
+The return value from register writes is ignored apart from the last
+value. Fix this.
+
+Reported-by: kernel test robot <lkp@intel.com>
+Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
+Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
+Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
+---
+ drivers/media/i2c/ar0521.c |    3 ++-
+ 1 file changed, 2 insertions(+), 1 deletion(-)
+
+--- a/drivers/media/i2c/ar0521.c
++++ b/drivers/media/i2c/ar0521.c
+@@ -756,11 +756,12 @@ static int ar0521_power_on(struct device
+               gpiod_set_value(sensor->reset_gpio, 0);
+       usleep_range(4500, 5000); /* min 45000 clocks */
+-      for (cnt = 0; cnt < ARRAY_SIZE(initial_regs); cnt++)
++      for (cnt = 0; cnt < ARRAY_SIZE(initial_regs); cnt++) {
+               ret = ar0521_write_regs(sensor, initial_regs[cnt].data,
+                                       initial_regs[cnt].count);
+               if (ret)
+                       goto off;
++      }
+       ret = ar0521_write_reg(sensor, AR0521_REG_SERIAL_FORMAT,
+                              AR0521_REG_SERIAL_FORMAT_MIPI |
index a20908ffe30819a92379f5b04fbf059d762388ee..e2d9d59adc683a4a80ef86ceb718e43ff65e4632 100644 (file)
@@ -109,6 +109,7 @@ net-ieee802154-fix-error-return-code-in-dgram_bind.patch
 media-amphion-release-m2m-ctx-when-releasing-vpu-ins.patch
 media-v4l2-fix-v4l2_i2c_subdev_set_name-function-doc.patch
 media-ar0521-fix-error-return-code-in-ar0521_power_o.patch
+media-ar0521-fix-return-value-check-in-writing-initial-registers.patch
 media-ov8865-fix-an-error-handling-path-in-ov8865_pr.patch
 media-sun6i-mipi-csi2-depend-on-phy_sun6i_mipi_dphy.patch
 media-atomisp-prevent-integer-overflow-in-sh_css_set.patch