From: Hans de Goede <hdegoede@xxxxxxxxxx> ov5693_sw_standby() starts/stops streaming rename it so that it is actually named after what it does. This also removes the weird enable inverting going on before. Signed-off-by: Hans de Goede <hdegoede@xxxxxxxxxx> --- drivers/media/i2c/ov5693.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/media/i2c/ov5693.c b/drivers/media/i2c/ov5693.c index 8ad51f8f88cf..2613bad49f78 100644 --- a/drivers/media/i2c/ov5693.c +++ b/drivers/media/i2c/ov5693.c @@ -742,13 +742,13 @@ static int ov5693_mode_configure(struct ov5693_device *ov5693) return ret; } -static int ov5693_sw_standby(struct ov5693_device *ov5693, bool standby) +static int ov5693_enable_streaming(struct ov5693_device *ov5693, bool enable) { int ret = 0; ov5693_write_reg(ov5693, OV5693_SW_STREAM_REG, - standby ? OV5693_STOP_STREAMING : - OV5693_START_STREAMING, &ret); + enable ? OV5693_STOP_STREAMING : + OV5693_START_STREAMING, &ret); return ret; } @@ -784,9 +784,9 @@ static int ov5693_sensor_init(struct ov5693_device *ov5693) return ret; } - ret = ov5693_sw_standby(ov5693, true); + ret = ov5693_enable_streaming(ov5693, true); if (ret) - dev_err(ov5693->dev, "software standby error\n"); + dev_err(ov5693->dev, "enable streaming error\n"); return ret; } @@ -1119,7 +1119,7 @@ static int ov5693_s_stream(struct v4l2_subdev *sd, int enable) } mutex_lock(&ov5693->lock); - ret = ov5693_sw_standby(ov5693, enable); + ret = ov5693_enable_streaming(ov5693, enable); mutex_unlock(&ov5693->lock); if (ret) -- 2.25.1