Rename the power control gpio to reset, since it is the same, and fix the
polarity code since this gpio is active at low and not at high as controlled
before.

Signed-off-by: Rui Miguel Silva <[email protected]>
---
 drivers/media/i2c/ov2680.c | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/drivers/media/i2c/ov2680.c b/drivers/media/i2c/ov2680.c
index cc3d06096d51..8962b397211a 100644
--- a/drivers/media/i2c/ov2680.c
+++ b/drivers/media/i2c/ov2680.c
@@ -98,7 +98,7 @@ struct ov2680_dev {
        struct clk                      *xvclk;
        u32                             xvclk_freq;
 
-       struct gpio_desc                *pwdn_gpio;
+       struct gpio_desc                *reset_gpio;
        struct mutex                    lock; /* protect members */
 
        bool                            mode_pending_changes;
@@ -295,19 +295,19 @@ static int ov2680_load_regs(struct ov2680_dev *sensor,
 
 static void ov2680_power_up(struct ov2680_dev *sensor)
 {
-       if (!sensor->pwdn_gpio)
+       if (!sensor->reset_gpio)
                return;
 
-       gpiod_set_value(sensor->pwdn_gpio, 1);
+       gpiod_set_value(sensor->reset_gpio, 0);
        usleep_range(5000, 10000);
 }
 
 static void ov2680_power_down(struct ov2680_dev *sensor)
 {
-       if (!sensor->pwdn_gpio)
+       if (!sensor->reset_gpio)
                return;
 
-       gpiod_set_value(sensor->pwdn_gpio, 0);
+       gpiod_set_value(sensor->reset_gpio, 1);
        usleep_range(5000, 10000);
 }
 
@@ -535,7 +535,7 @@ static int ov2680_power_on(struct ov2680_dev *sensor)
        if (sensor->is_enabled)
                return 0;
 
-       if (!sensor->pwdn_gpio) {
+       if (!sensor->reset_gpio) {
                ret = ov2680_write_reg(sensor, OV2680_REG_SOFT_RESET, 0x01);
                if (ret != 0) {
                        dev_err(dev, "sensor soft reset failed\n");
@@ -990,11 +990,11 @@ static int ov2860_parse_dt(struct ov2680_dev *sensor)
        struct device *dev = ov2680_to_dev(sensor);
        int ret;
 
-       sensor->pwdn_gpio = devm_gpiod_get_optional(dev, "powerdown",
-                                                   GPIOD_OUT_HIGH);
-       ret = PTR_ERR_OR_ZERO(sensor->pwdn_gpio);
+       sensor->reset_gpio = devm_gpiod_get_optional(dev, "reset",
+                                                    GPIOD_OUT_HIGH);
+       ret = PTR_ERR_OR_ZERO(sensor->reset_gpio);
        if (ret < 0) {
-               dev_dbg(dev, "error while getting powerdown gpio: %d\n", ret);
+               dev_dbg(dev, "error while getting reset gpio: %d\n", ret);
                return ret;
        }
 
-- 
2.17.0

Reply via email to