| Message ID | 20240313072516.241106-19-sakari.ailus@linux.intel.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [v8,01/38] media: mc: Add INTERNAL pad flag | expand |
Hi Sakari, Thank you for the patch. On Wed, Mar 13, 2024 at 09:24:56AM +0200, Sakari Ailus wrote: > Streaming will have been stopped by the sensor is powered off, and so > sensor->streaming is also false already. Do not set it as part of the > runtime suspend callback. > > Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com> Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> > --- > drivers/media/i2c/ccs/ccs-core.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/media/i2c/ccs/ccs-core.c b/drivers/media/i2c/ccs/ccs-core.c > index e21287d50c15..671540a8ab6a 100644 > --- a/drivers/media/i2c/ccs/ccs-core.c > +++ b/drivers/media/i2c/ccs/ccs-core.c > @@ -1710,7 +1710,6 @@ static int ccs_power_off(struct device *dev) > usleep_range(5000, 5000); > regulator_bulk_disable(ARRAY_SIZE(ccs_regulators), > sensor->regulators); > - sensor->streaming = false; > > return 0; > }
diff --git a/drivers/media/i2c/ccs/ccs-core.c b/drivers/media/i2c/ccs/ccs-core.c index e21287d50c15..671540a8ab6a 100644 --- a/drivers/media/i2c/ccs/ccs-core.c +++ b/drivers/media/i2c/ccs/ccs-core.c @@ -1710,7 +1710,6 @@ static int ccs_power_off(struct device *dev) usleep_range(5000, 5000); regulator_bulk_disable(ARRAY_SIZE(ccs_regulators), sensor->regulators); - sensor->streaming = false; return 0; }
Streaming will have been stopped by the sensor is powered off, and so sensor->streaming is also false already. Do not set it as part of the runtime suspend callback. Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com> --- drivers/media/i2c/ccs/ccs-core.c | 1 - 1 file changed, 1 deletion(-)