This is an automatic generated email to let you know that the following patch
were queued:
Subject: media: ccs: Partially revert "media: i2c: Use
pm_runtime_resume_and_get()"
Author: Sakari Ailus <sakari.ai...@linux.intel.com>
Date: Fri Sep 29 12:00:42 2023 +0300
ccs_pm_get_init() depends on the return values > 0 of
pm_runtime_get_sync(), thus it can't use pm_runtime_resume_and_get().
There's even a comment in the driver on this, a few lines above the code.
Fixes: aa0adb399d09 ("media: i2c: Use pm_runtime_resume_and_get()")
Signed-off-by: Sakari Ailus <sakari.ai...@linux.intel.com>
Signed-off-by: Hans Verkuil <hverkuil-ci...@xs4all.nl>
drivers/media/i2c/ccs/ccs-core.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
---
diff --git a/drivers/media/i2c/ccs/ccs-core.c b/drivers/media/i2c/ccs/ccs-core.c
index c6853622946b..34f4f62a9523 100644
--- a/drivers/media/i2c/ccs/ccs-core.c
+++ b/drivers/media/i2c/ccs/ccs-core.c
@@ -1873,9 +1873,9 @@ static int ccs_pm_get_init(struct ccs_sensor *sensor)
* relies at the returned value to detect if the device was already
* active or not.
*/
- rval = pm_runtime_resume_and_get(&client->dev);
- if (rval)
- return rval;
+ rval = pm_runtime_get_sync(&client->dev);
+ if (rval < 0)
+ goto error;
/* Device was already active, so don't set controls */
if (rval == 1)
_______________________________________________
linuxtv-commits mailing list
linuxtv-commits@linuxtv.org
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits
