]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
media: i2c: adv748x: Remove PAGE_WAIT
authorKieran Bingham <kieran.bingham+renesas@ideasonboard.com>
Tue, 15 Jan 2019 14:25:09 +0000 (09:25 -0500)
committerMauro Carvalho Chehab <mchehab+samsung@kernel.org>
Mon, 18 Feb 2019 16:25:30 +0000 (11:25 -0500)
The ADV748X_PAGE_WAIT is a fake page to insert arbitrary delays in the
register tables.

Its only usage was removed, so we can remove the handling and simplify
the code.

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
Signed-off-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
drivers/media/i2c/adv748x/adv748x-core.c
drivers/media/i2c/adv748x/adv748x.h

index 252bdb28b18bee080f05a29bd6e2e00fa8debffa..f57cd77a32fac9c741269a02267b1f9b96580e63 100644 (file)
@@ -218,20 +218,13 @@ static int adv748x_write_regs(struct adv748x_state *state,
 {
        int ret;
 
-       while (regs->page != ADV748X_PAGE_EOR) {
-               if (regs->page == ADV748X_PAGE_WAIT) {
-                       msleep(regs->value);
-               } else {
-                       ret = adv748x_write(state, regs->page, regs->reg,
-                                     regs->value);
-                       if (ret < 0) {
-                               adv_err(state,
-                                       "Error regs page: 0x%02x reg: 0x%02x\n",
-                                       regs->page, regs->reg);
-                               return ret;
-                       }
+       for (; regs->page != ADV748X_PAGE_EOR; regs++) {
+               ret = adv748x_write(state, regs->page, regs->reg, regs->value);
+               if (ret < 0) {
+                       adv_err(state, "Error regs page: 0x%02x reg: 0x%02x\n",
+                               regs->page, regs->reg);
+                       return ret;
                }
-               regs++;
        }
 
        return 0;
index 2f8d751cfbb0db1c0fa8d5ea6fc3e85482e0f151..5042f9e94aee2dd75297e131541516bd2b5bc9dc 100644 (file)
@@ -39,7 +39,6 @@ enum adv748x_page {
        ADV748X_PAGE_MAX,
 
        /* Fake pages for register sequences */
-       ADV748X_PAGE_WAIT,              /* Wait x msec */
        ADV748X_PAGE_EOR,               /* End Mark */
 };