[git:media_stage/master] media: v4l: Don't turn on privacy LED if streamon fails

[Hans Verkuil]

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

Subject: media: v4l: Don't turn on privacy LED if streamon fails
Author:  Sakari Ailus <sakari.ai...@linux.intel.com>
Date:    Wed Apr 10 12:58:44 2024 +0300

Turn on the privacy LED only if streamon succeeds. This can be done after
enabling streaming on the sensor.

Fixes: b6e10ff6c23d ("media: v4l2-core: Make the v4l2-core code enable/disable 
the privacy LED if present")
Signed-off-by: Sakari Ailus <sakari.ai...@linux.intel.com>
Reviewed-by: Hans de Goede <hdego...@redhat.com>
Reviewed-by: Tomi Valkeinen <tomi.valkei...@ideasonboard.com>
Reviewed-by: Umang Jain <umang.j...@ideasonboard.com>
Signed-off-by: Hans Verkuil <hverkuil-ci...@xs4all.nl>

 drivers/media/v4l2-core/v4l2-subdev.c | 22 ++++++++++++----------
 1 file changed, 12 insertions(+), 10 deletions(-)

---

diff --git a/drivers/media/v4l2-core/v4l2-subdev.c 
b/drivers/media/v4l2-core/v4l2-subdev.c
index 0f590c9dc6ca..2d67ec54569a 100644
--- a/drivers/media/v4l2-core/v4l2-subdev.c
+++ b/drivers/media/v4l2-core/v4l2-subdev.c
@@ -442,15 +442,6 @@ static int call_s_stream(struct v4l2_subdev *sd, int 
enable)
        if (WARN_ON(!!sd->enabled_streams == !!enable))
                return 0;
 
-#if IS_REACHABLE(CONFIG_LEDS_CLASS)
-       if (!IS_ERR_OR_NULL(sd->privacy_led)) {
-               if (enable)
-                       led_set_brightness(sd->privacy_led,
-                                          sd->privacy_led->max_brightness);
-               else
-                       led_set_brightness(sd->privacy_led, 0);
-       }
-#endif
        ret = sd->ops->video->s_stream(sd, enable);
 
        if (!enable && ret < 0) {
@@ -458,9 +449,20 @@ static int call_s_stream(struct v4l2_subdev *sd, int 
enable)
                ret = 0;
        }
 
-       if (!ret)
+       if (!ret) {
                sd->enabled_streams = enable ? BIT(0) : 0;
 
+#if IS_REACHABLE(CONFIG_LEDS_CLASS)
+               if (!IS_ERR_OR_NULL(sd->privacy_led)) {
+                       if (enable)
+                               led_set_brightness(sd->privacy_led,
+                                                  
sd->privacy_led->max_brightness);
+                       else
+                               led_set_brightness(sd->privacy_led, 0);
+               }
+#endif
+       }
+
        return ret;
 }