2019-09-19 14:43:19 +00:00
|
|
|
From 3be30ee1b3aafd7c6cc45bcea77f25c9613732f4 Mon Sep 17 00:00:00 2001
|
2019-07-09 18:32:28 +00:00
|
|
|
From: Dave Stevenson <dave.stevenson@raspberrypi.org>
|
|
|
|
Date: Tue, 18 Sep 2018 11:08:51 +0100
|
2019-09-19 14:43:19 +00:00
|
|
|
Subject: [PATCH 323/806] media: ov5647: Use gpiod_set_value_cansleep
|
2019-07-09 18:32:28 +00:00
|
|
|
|
|
|
|
All calls to the gpio library are in contexts that can sleep,
|
|
|
|
therefore there is no issue with having those GPIOs controlled
|
|
|
|
by controllers which require sleeping (eg I2C GPIO expanders).
|
|
|
|
|
|
|
|
Switch to using gpiod_set_value_cansleep instead of gpiod_set_value
|
|
|
|
to avoid triggering the warning in gpiolib should the GPIO
|
|
|
|
controller need to sleep.
|
|
|
|
|
|
|
|
Signed-off-by: Dave Stevenson <dave.stevenson@raspberrypi.org>
|
|
|
|
---
|
|
|
|
drivers/media/i2c/ov5647.c | 8 ++++----
|
|
|
|
1 file changed, 4 insertions(+), 4 deletions(-)
|
|
|
|
|
|
|
|
--- a/drivers/media/i2c/ov5647.c
|
|
|
|
+++ b/drivers/media/i2c/ov5647.c
|
|
|
|
@@ -373,7 +373,7 @@ static int ov5647_sensor_power(struct v4
|
|
|
|
dev_dbg(&client->dev, "OV5647 power on\n");
|
|
|
|
|
|
|
|
if (ov5647->pwdn) {
|
|
|
|
- gpiod_set_value(ov5647->pwdn, 0);
|
|
|
|
+ gpiod_set_value_cansleep(ov5647->pwdn, 0);
|
|
|
|
msleep(PWDN_ACTIVE_DELAY_MS);
|
|
|
|
}
|
|
|
|
|
|
|
|
@@ -415,7 +415,7 @@ static int ov5647_sensor_power(struct v4
|
|
|
|
|
|
|
|
clk_disable_unprepare(ov5647->xclk);
|
|
|
|
|
|
|
|
- gpiod_set_value(ov5647->pwdn, 1);
|
|
|
|
+ gpiod_set_value_cansleep(ov5647->pwdn, 1);
|
|
|
|
}
|
|
|
|
|
|
|
|
/* Update the power count. */
|
|
|
|
@@ -649,13 +649,13 @@ static int ov5647_probe(struct i2c_clien
|
|
|
|
goto mutex_remove;
|
|
|
|
|
|
|
|
if (sensor->pwdn) {
|
|
|
|
- gpiod_set_value(sensor->pwdn, 0);
|
|
|
|
+ gpiod_set_value_cansleep(sensor->pwdn, 0);
|
|
|
|
msleep(PWDN_ACTIVE_DELAY_MS);
|
|
|
|
}
|
|
|
|
|
|
|
|
ret = ov5647_detect(sd);
|
|
|
|
|
|
|
|
- gpiod_set_value(sensor->pwdn, 1);
|
|
|
|
+ gpiod_set_value_cansleep(sensor->pwdn, 1);
|
|
|
|
|
|
|
|
if (ret < 0)
|
|
|
|
goto error;
|