Previously, the sensor was not streaming after resume from suspend,
i.e. on S0->S3->S0 transition. Due to this, camera app preview appeared
as stuck.

Now, handle streaming state correctly in case of suspend-resume.

Signed-off-by: Chiranjeevi Rapolu <chiranjeevi.rap...@intel.com>
---
 drivers/media/i2c/ov5670.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/drivers/media/i2c/ov5670.c b/drivers/media/i2c/ov5670.c
index 6f7a1d6..bdfb5b9 100644
--- a/drivers/media/i2c/ov5670.c
+++ b/drivers/media/i2c/ov5670.c
@@ -2323,8 +2323,6 @@ static int ov5670_start_streaming(struct ov5670 *ov5670)
                return ret;
        }
 
-       ov5670->streaming = true;
-
        return 0;
 }
 
@@ -2338,8 +2336,6 @@ static int ov5670_stop_streaming(struct ov5670 *ov5670)
        if (ret)
                dev_err(&client->dev, "%s failed to set stream\n", __func__);
 
-       ov5670->streaming = false;
-
        /* Return success even if it was an error, as there is nothing the
         * caller can do about it.
         */
@@ -2370,6 +2366,7 @@ static int ov5670_set_stream(struct v4l2_subdev *sd, int 
enable)
                ret = ov5670_stop_streaming(ov5670);
                pm_runtime_put(&client->dev);
        }
+       ov5670->streaming = enable;
        goto unlock_and_return;
 
 error:
-- 
1.9.1

Reply via email to