media: i2c: imx334: Fix runtime PM handling in remove function

[ Upstream commit b493cd3c03641f9bbaa9787e43ca92163cb50051 ]

pm_runtime_suspended() only checks the current runtime PM status and does
not modify it, making it ineffective in this context. This could result in
improper power management if the device remains active when removed.

This patch fixes the issue by introducing a check with
pm_runtime_status_suspended() to determine if the device is already
suspended. If it is not, it calls imx334_power_off() to power down the
device and then uses pm_runtime_set_suspended() to correctly update the
runtime PM status to suspended.

Signed-off-by: Tarang Raval <tarang.raval@siliconsignals.io>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl>
Signed-off-by: Sasha Levin <sashal@kernel.org>

diff --git a/drivers/media/i2c/imx334.c b/drivers/media/i2c/imx334.c
index 57b7416bbfab30ba0bebf653837a1ae47cd4cbc6..af09aafeddf783f496023b6eceebe5eeb7966a48 100644 (file)
--- a/drivers/media/i2c/imx334.c
+++ b/drivers/media/i2c/imx334.c
@@ -1102,7 +1102,10 @@ static int imx334_remove(struct i2c_client *client)
        v4l2_ctrl_handler_free(sd->ctrl_handler);
 
        pm_runtime_disable(&client->dev);
-       pm_runtime_suspended(&client->dev);
+       if (!pm_runtime_status_suspended(&client->dev)) {
+               imx334_power_off(&client->dev);
+               pm_runtime_set_suspended(&client->dev);
+       }
 
        mutex_destroy(&imx334->mutex);