diff options
| author | Manivannan Sadhasivam <[email protected]> | 2020-06-12 15:53:54 +0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab <[email protected]> | 2020-06-23 15:18:33 +0200 |
| commit | 6544af9b04b4484867c234ba0be1b5008e4a14ee (patch) | |
| tree | 0ce897ad08a61b01515baa568379fa842966e692 | |
| parent | 3b867fb641d884b714fba390ae866714ba475f29 (diff) | |
media: i2c: imx290: Move the settle time delay out of loop
The 10ms settle time is needed only at the end of all consecutive
register writes. So move the delay to outside of the for loop of
imx290_set_register_array().
Signed-off-by: Manivannan Sadhasivam <[email protected]>
Signed-off-by: Andrey Konovalov <[email protected]>
Signed-off-by: Sakari Ailus <[email protected]>
Signed-off-by: Mauro Carvalho Chehab <[email protected]>
| -rw-r--r-- | drivers/media/i2c/imx290.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/media/i2c/imx290.c b/drivers/media/i2c/imx290.c index fd147fac5ef2..02001c1b0dfc 100644 --- a/drivers/media/i2c/imx290.c +++ b/drivers/media/i2c/imx290.c @@ -404,11 +404,11 @@ static int imx290_set_register_array(struct imx290 *imx290, ret = imx290_write_reg(imx290, settings->reg, settings->val); if (ret < 0) return ret; - - /* Settle time is 10ms for all registers */ - msleep(10); } + /* Provide 10ms settle time */ + msleep(10); + return 0; } |