This is an automatic generated email to let you know that the following patch 
were queued:

Subject: media: ov5647: Protect s_stream() with mutex
Author:  Jacopo Mondi <[email protected]>
Date:    Thu Nov 19 17:32:30 2020 +0100

Use the driver mutex to protect s_stream() operations.
This will become more relevant once the sensor will support more formats
and set_format() could be issue concurrently to s_stream().

Signed-off-by: Jacopo Mondi <[email protected]>
Signed-off-by: Sakari Ailus <[email protected]>
Signed-off-by: Mauro Carvalho Chehab <[email protected]>

 drivers/media/i2c/ov5647.c | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

---

diff --git a/drivers/media/i2c/ov5647.c b/drivers/media/i2c/ov5647.c
index 2e7a6cb39689..69a5e25dcd70 100644
--- a/drivers/media/i2c/ov5647.c
+++ b/drivers/media/i2c/ov5647.c
@@ -462,10 +462,17 @@ static const struct v4l2_subdev_core_ops 
ov5647_subdev_core_ops = {
 
 static int ov5647_s_stream(struct v4l2_subdev *sd, int enable)
 {
+       struct ov5647 *sensor = to_state(sd);
+       int ret;
+
+       mutex_lock(&sensor->lock);
        if (enable)
-               return ov5647_stream_on(sd);
+               ret = ov5647_stream_on(sd);
        else
-               return ov5647_stream_off(sd);
+               ret = ov5647_stream_off(sd);
+       mutex_unlock(&sensor->lock);
+
+       return ret;
 }
 
 static const struct v4l2_subdev_video_ops ov5647_subdev_video_ops = {

_______________________________________________
linuxtv-commits mailing list
[email protected]
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to